Zitat:

---

Original von Holger:
Damit hast Du ja noch nicht viel gewonnen. Du schreibst ja bisher nur in die Struktur und mußt ja immer noch den code hinzufügen, der die Änderungen herbeiführt.
Ich nehme mal an, Du hast bisher nur code, der diese Attribute beim Öffnen des Fensters in eine TagList verpackt und übergibt. Aber wenn man schon ein Fenster in einem Objekt kapselt, sollte man als Nutzer der Klasse für das Setzen eines Attributs nicht mehr darüber nachdenken müssen, ob das Fenster gerade offen ist oder nicht.

---

code:

---

#ifndef WINDOWDATA_H
#define WINDOWDATA_H

#include <string>
#include <intuition/intuition.h>
 
class WindowDataC
{
    private:
		ULONG windowLeft, windowTop, windowWidth, windowHeight, flagsIDCMP;
		BOOL dragBar, borderless, reportMouse, newLookMenus, sizeGadget, depthGadget, closeGadget, activate,
			 noCareRefresh, backdrop, trapRMB, gimmeZeroZero, autoAdjust, menuHelp, notifyDepth;
		string windowTitle;
	
		inline void setIDCMPFlag(ULONG flag) { flagsIDCMP = flagsIDCMP | flag; }
	
	public:
		WindowDataC() : flagsIDCMP(0) {}
		~WindowDataC() {}
					   
		void setLeft(const ULONG left = 0) { windowLeft = left; }
		void setTop(const ULONG top = 0) { windowTop = top; }
        void setWidth(const ULONG width = STDSCREENWIDTH) { windowWidth = width; }
		void setHeight(const ULONG height = STDSCREENHEIGHT) { windowHeight = height; }
		void setTitle(const string title = "") { windowTitle = title; }
		void setDragBar(const BOOL setDragBar = TRUE) { dragBar = setDragBar; }
		void setBorderless(const BOOL setBorderless = FALSE) { borderless = setBorderless; }
		void setReportMouse(const BOOL setReportMouse = TRUE) { reportMouse = setReportMouse; }
		void setNewLookMenus(const BOOL setNewLookMenus = TRUE) { newLookMenus = setNewLookMenus; }
		void setSizeGadget(const BOOL setSizeGadget = TRUE) { sizeGadget = setSizeGadget; }
		void setDepthGadget(const BOOL setDepthGadget = TRUE) { depthGadget = setDepthGadget; }
		void setCloseGadget(const BOOL setCloseGadget = TRUE) { closeGadget = setCloseGadget; }
		void setActivate(const BOOL setActivate = TRUE) { activate = setActivate; }
		void setNoCareRefresh(const BOOL setNoCareRefresh = TRUE) { noCareRefresh = setNoCareRefresh; }
		void setBackdrop(const BOOL setBackdrop = FALSE) { backdrop = setBackdrop; }
		void setRMBTrap(const BOOL setRMBTrap = FALSE) { trapRMB = setRMBTrap; }
		void setGimmeZeroZero(const BOOL setGimmeZeroZero = FALSE) { gimmeZeroZero = setGimmeZeroZero; }
		void setAutoAdjust(const BOOL setAutoAdjust = TRUE) { autoAdjust = setAutoAdjust; }
		void setMenuHelp(const BOOL setMenuHelp = TRUE) { menuHelp = setMenuHelp; }
		void setNotifyDepth(const BOOL setNotifyDepth = TRUE) { notifyDepth = setNotifyDepth; }
		void notifyNewSize() { setIDCMPFlag(IDCMP_NEWSIZE); }
		void notifyRefreshWindow() { setIDCMPFLag(IDCMP_REFRESHWINDOW); }
		void notifyMouseButtons() { setIDCMPFLag(IDCMP_MOUSEBUTTONS); }
		void notifyMouseMove() { setIDCMPFLag(IDCMP_MOUSEMOVE); }	// nur wenn ReportMouse gesetzt wurde
		void notifyGadgetDown() { setIDCMPFLag(IDCMP_GADGETDOWN); }
		void notifyGadgetUp() { setIDCMPFLag(IDCMP_GADGETUP); }
		void notifyMenuPick() { setIDCMPFLag(IDCMP_MENUPICK); }
		void notifyCloseWindow() { setIDCMPFLag(IDCMP_CLOSEWINDOW); }
		void notifyIntuiTicks() { setIDCMPFLag(IDCMP_INTUITICKS); }
		void notifyActiveWindow() { setIDCMPFLag(IDCMP_ACTIVEWINDOW); }
		void notifyInactiveWindow() { setIDCMPFLag(IDCMP_INACTIVEWINDOW); }
		void notifyChangeWindow() { setIDCMPFLag(IDCMP_CHANGEWINDOW); }
		void notifyMenuHelp() { setIDCMPFLag(IDCMP_MENUHELP); }
		void notifiyIDMCP(ULONG flags = 0) { setIDCMPFlag(flags); }

		ULONG getLeft() const { return windowLeft; }
		ULONG getTop()const { return windowTop; }
		ULONG getWidth() const { return windowWidth; }
		ULONG getHeight()const { return windowHeight; }
		string getTitle() const { return windowTitle; }
		BOOL hasDragBar() const { return dragBar; }
		BOOL isBorderless() const { return borderless; }
		BOOL hasReportMouse() const { return reportMouse; }
		BOOL hasNewLookMenus() const { return newLookMenus; }
		BOOL hasSizeGadget() const { return sizeGadget; }
		BOOL hasDepthGadget() const { return depthGadget; }
		BOOL hasCloseGadget() const { return closeGadget; }
		BOOL isActivate() const { return activate; }
		BOOL hasNoCareRefresh() const { return noCareRefresh; }
		BOOL isBackdrop() const { return backdrop; }
		BOOL hasRMBTrap() const { return trapRMB; }
		BOOL isGimmeZeroZero() const { return gimmeZeroZero; }
		BOOL isAutoAdjust() const { return autoAdjust; }
		BOOL hasMenuHelp() const { return menuHelp; }
		BOOL hasNotifyDepth() const { return notifyDepth; }
};
#endif

---

Zitat:

---

vergiß die NewWindow-Struktur. Ich bin davon ausgegangen, daß Du sowieso nur mit TagList arbeitest. Wenn Du alle Attribute unterstützen willst, mußt Du sowieso TagLists unterstützen, also ist es nur konsequent, dafür dann auf die NewWindow-Struktur zu verzichten. Für alle Attribute der NewWindow-Struktur gibt es auch ein gleichwertiges TAG, umgekehrt gilt das nicht.
Also laß die NewWindow Struktur weg. Und implementiere alle set... Methode so, daß sie ein TagItem erzeugen/verändern. Dazu kannst Du wiederverwendbare Hilfsmethoden schreiben und sparst eine Menge Code.

mfg

--
Good coders do not comment. What was hard to write should be hard to read too.

---

Oh, da hab ich wohl ein wenig fehlerhaft gepostet. Ich verwende ja Tags, bezog mich auf die WA_... Sachen und hab das in nen Zusammenhang mit der NewWindow-Struktur gesetzt. Sorry!

Durch die Setter sieht das Ganze bei mir sehr starr aus. Habe bei OpenWindowTags und OpenScreenTags für jeden Setter ein Tag gesetzt.
Durch die Defaultwerte kann man Objekte der Klasse ohne Setter-Aufrufe verwenden, wenn man damit auskommt.

Durch diese Konstruktion verliert man leider die Flexibilität. Wenn man Deinen Ansatz mit den Konstanten mit ner Liste o.ä. kombiniert, kann man sehr flexibel die TagItems für ein OpenWindowTags zusammenstellen, oder nich?!

Überlege mir gerad ne halbwegs vernünftige Lösung für das Problem, dass ich einmal ne Methode habe, die ein Fenster anhand eines solchen Datenobjektes öffnet ohne PubScreen und einmal mit. Beide Methoden erhalten ein Objekt, welches diese Werte enthält und über Getter zur Verfügung stelllt. Die 2. Methode erhält zusätzlich noch die notwendige screen-Struktur (bzw. nen Zeiger drauf gekapselt in nem Objekt).
Hab mir überlegt, dass ich ne private Methode mach, die TagItem-Strukturen zusammenstellt, wobei die Methode mit dem PubScreen schon nen TagItem-Eintrag erhält (ist die Reihenfolge der TagItems von
Bedeuting?).
Da ich in C++ sehr unerfahren bin, fehlen mir hier die Fertigkeiten die Möglichkeiten effizient auszunutzen.
Hast Du nen guten Vorschlag!?
Ne Lösung wäre zwar schön, ne Idee oder nen Link, wo man erfährt, wie man die C++ Möglichkeiten clever einsetzt (so wie Du mit Deiner Attr-Klasse) aber noch viel besser!
Warte gerade auf die Neuauflage von "Thinking in C++"!

Ciao

[ Dieser Beitrag wurde von Reth am 04.06.2005 editiert. ]

Zitat:

---

Original von AC-Pseudo:
Versuchs mal unter http://www.dietmar-eilert.net/golded/index.htm




das ging vorhin noch.

---

code:

---

void setLeft(const ULONG left = 0) { windowLeft = left; }
void setTop(const ULONG top = 0) { windowTop = top; }
void setWidth(const ULONG width = STDSCREENWIDTH) { windowWidth = width; }
void setHeight(const ULONG height = STDSCREENHEIGHT) { windowHeight = height; }
void setTitle(const string title = "") { windowTitle = title; }
void setDragBar(const BOOL setDragBar = TRUE) { dragBar = setDragBar; }
void setBorderless(const BOOL setBorderless = FALSE) { borderless = setBorderless; }
void setReportMouse(const BOOL setReportMouse = TRUE) { reportMouse = setReportMouse; }
void setNewLookMenus(const BOOL setNewLookMenus = TRUE) { newLookMenus = setNewLookMenus; }
void setSizeGadget(const BOOL setSizeGadget = TRUE) { sizeGadget = setSizeGadget; }
void setDepthGadget(const BOOL setDepthGadget = TRUE) { depthGadget = setDepthGadget; }
void setCloseGadget(const BOOL setCloseGadget = TRUE) { closeGadget = setCloseGadget; }
void setActivate(const BOOL setActivate = TRUE) { activate = setActivate; }
void setNoCareRefresh(const BOOL setNoCareRefresh = TRUE) { noCareRefresh = setNoCareRefresh; }
void setBackdrop(const BOOL setBackdrop = FALSE) { backdrop = setBackdrop; }
void setRMBTrap(const BOOL setRMBTrap = FALSE) { trapRMB = setRMBTrap; }
void setGimmeZeroZero(const BOOL setGimmeZeroZero = FALSE) { gimmeZeroZero = setGimmeZeroZero; }
void setAutoAdjust(const BOOL setAutoAdjust = TRUE) { autoAdjust = setAutoAdjust; }
void setMenuHelp(const BOOL setMenuHelp = TRUE) { menuHelp = setMenuHelp; }
void setNotifyDepth(const BOOL setNotifyDepth = TRUE) { notifyDepth = setNotifyDepth; }
void notifyNewSize() { setIDCMPFlag(IDCMP_NEWSIZE); }
void notifyRefreshWindow() { setIDCMPFLag(IDCMP_REFRESHWINDOW); }
void notifyMouseButtons() { setIDCMPFLag(IDCMP_MOUSEBUTTONS); }
void notifyMouseMove() { setIDCMPFLag(IDCMP_MOUSEMOVE); } // nur wenn ReportMouse gesetzt wurde
void notifyGadgetDown() { setIDCMPFLag(IDCMP_GADGETDOWN); }
void notifyGadgetUp() { setIDCMPFLag(IDCMP_GADGETUP); }
void notifyMenuPick() { setIDCMPFLag(IDCMP_MENUPICK); }
void notifyCloseWindow() { setIDCMPFLag(IDCMP_CLOSEWINDOW); }
void notifyIntuiTicks() { setIDCMPFLag(IDCMP_INTUITICKS); }
void notifyActiveWindow() { setIDCMPFLag(IDCMP_ACTIVEWINDOW); }
void notifyInactiveWindow() { setIDCMPFLag(IDCMP_INACTIVEWINDOW); }
void notifyChangeWindow() { setIDCMPFLag(IDCMP_CHANGEWINDOW); }
void notifyMenuHelp() { setIDCMPFLag(IDCMP_MENUHELP); }
void notifiyIDMCP(ULONG flags = 0) { setIDCMPFlag(flags); }

---

code:

---

template<class T> void setAttr(Attr<T> key, T value)

---

Zitat:

---

Wenn Du das mit einer Typisierung der Tags verbindest, gewinnst Du auch den Vorteil der Typsicherheit, die ein Kombinieren von TAG-Keys mit falschen Wert-Typen verhindert.
Beispiel:

code:

---

#include <iostream>

template<class T> class Attr
{
  const int tag;
public:
  Attr(const int tagvalue):tag(tagvalue){}
  const int tagValue() const { return tag; }
};

---

---

Den Teil hab ich verstanden!

Zitat:

---

code:

---

class Window
{
  public:
    static const Attr<int> WIDTH;
    static const Attr<int> HEIGHT;
    static const Attr<const char*> TITLE;
    static const Attr<bool> CLOSE_GADGET;
    template<class T> void setAttr(Attr<T> key, T value)
    {
      const int ivalue=(int)value;
      std::cout<<"setting { 0x"<<std::hex<<key.tagValue()<<", "
        <<std::dec<<ivalue<<" }"<<std::endl;
    }
};

---

---

Zitat:

---

code:

---

const Attr<int> Window::WIDTH(0x80001001);
const Attr<int> Window::HEIGHT(0x80001002);
const Attr<const char*> Window::TITLE(0x80001003);
const Attr<bool> Window::CLOSE_GADGET(0x80001004);

---

---

Zitat:

---

code:

---

int main(int x, char**y)
{
  Window win;

  win.setAttr(Window::WIDTH, 200);
  win.setAttr(Window::HEIGHT, 200);
  win.setAttr(Window::TITLE, "Hallo");
  win.setAttr(Window::CLOSE_GADGET, true);
}

---

---

Zitat:

---

Original von Holger:

Das kann ich nicht nachvollziehen. In beiden Fällen mußt Du eine Dokumentation konsultieren. In dem einen Fall ist es die Dokumentation einer Klasse, im anderen Fall die der vordefinierten Konstanten.
Natürlich ist es bei einer Klasse leichter, den Bezug von der Methode zu den möglichen Werten herzustellen, insb. in Verbindung mit assistierenden Entwicklungstools, aber das sollte nicht allein den Ausschlag geben.

---

Zitat:

---

Insbesondere wenn Du durch Verlagerung der Konstanten aus den #define's der OS-includes in statische member Deiner Klassen das gleiche erreichen kannst.

---

Zitat:

---

Interessant wird's allerdings, wenn man den Klassen ein wenig Hierarchie verpaßt. Wenn man einem Screen, Window oder Gadget mit der gleichen set-Methode eine Größe verpassen kann, hat das durchaus seine Vorteile. Und reduziert auch den Nachschlage- und Lernaufwand, wenn man es konsequent auf alle Attribute anwendet.
Warum der Programmierer einmal SA_Width, dann WA_Width und dann wieder GA_Width benutzen muß, ist wohl kaum befriedigend zu erklären...

---

Zitat:

---

Original von aPEX:
dafuer bin ich auch!!! man muss doch als user noch was erreichen
koennen ausser immer ausgefeiltere mordgedanken

---

Zitat:

---

Original von thomas:

Wo ist der Unterschied zwischen bildschirm.setWidth(640) und bildschirm.setAttr(SA_Width,640) oder so ?

---

Zitat:

---

Vielleicht solltest du dir überlegen, deine HashMap intern zu speichern und dem Programmierer erlauben, dir Tag-Listen zu übergeben.

---

Zitat:

---

fr.close();
bs.close(); // eigentlich unnötig, das sollte der Destruktor automatisch machen.

---

Zitat:

---

Original von zacman:
Er hat jetzt zwischenzeitlich eine Antwort von Amiga Inc. (per Email)
erhalten. Sie lautet: "Nein".

---

Zitat:

---

Original von thomas:

Versuchs mal mit MenuC(void){} und ~MenuC(void){}. Immerhin meckert er ja MenuC::MenuC(void) an und nicht MenuC::MenuC().

Gruß Thomas

---

Zitat:

---

Original von thomas:

Versuchs mal mit MenuC(void){} und ~MenuC(void){}. Immerhin meckert er ja MenuC::MenuC(void) an und nicht MenuC::MenuC().

Gruß Thomas

---

Zitat:

---

Original von gni:

Die Librarybasis ist hier also (Klassen-)lokal?

---

Zitat:

---

Dein Programm enthält Aufrufe zu diesen Funktionen und der Linker findet sie in libamiga.a! Alle in libamiga.a enthaltenen Funktionen brauchen eine _globale_ Librarybasis.

---

Zitat:

---

Falls die Funktionen ohne Stub aufgerfufen werden können, gehts auch mit einer lokalen Basis die Du selber mit OpenLibrary() belegen mußt. Für die graphics.library Funktionen sollte das auch klappen. Also am Anfang "#include <proto/graphics.h>, GfxBase sichtbar deklarieren und mit OpenLibrary initialisieren.

---

Zitat:

---

AllocAslRequestTags ist eine Hilfsfunktion um AllocAslRequest(). Die sollte auch ohne Stub aufgelöst werden können, wenn Du <proto/asl.h> einbindest. Die verwendete Methode, um solche varsargs-Funktionen aufzulösen, erzeugt aber bescheidenen Code.

---