[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

Zitat:

---

Original von thomas:
Der erste Ahaltspunkt ist immer der Inhalt der Alerts. Eine Erklärung der Alert-Nummer findest du in exec/alerts.h.

---

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

Zitat:

---

Original von Reth:
@Thore:
Hab kein Alloc/Free, "nur" new/delete…

---

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

C++ code:

---

{
        // zunächst wird eine "Aktivität" eingerichtet, die beim Eintreten von INTUITICKS ausgelöst werden soll. Die Ticks werden über einen entsprechenden Empfänger am Fenster entgegen genommen
	RefreshActivityC refrAct(win);
	inTickMsgRec.addIntuiTickActivity(&refrAct);
	win.setIntuiTickMessageReceiver(&inTickMsgRec);

    // nun werden 2 Gadgets mit Bildern erzeugt und im Fenster dargestellt
    ImageButtonC towerButton(&tower_button_normal, &tower_button_pressed, BUILDTOWER);
	ImageButtonC lightningButton(&lightning_button_normal, &lightning_button_pressed, ATTACK);

	win.addGadgetAt(towerButton, 829, 573, 29, 52);
	win.addGadgetAt(lightningButton, 874, 573, 29, 52);

	win.getRastPort().drawImageAt(&statusimage, 808, 15);
	win.getRastPort().drawImageAt(&buttonimage, 808, 552);

	win.refreshAllGadgets();

	win.getRastPort().setPrimaryPen(1);
	win.setTextFont((char *)"EstroFont.font", 18, 0, 0);

    srand(time(NULL));

        // Spielfeld usw. erstellen 
	setupWindowComponents(win);

      // Weitere benötigte Empfänger erzeugen
      // A C H T U N G ! ! !
      // sobald hier GadgetMessageReceiver und MouseMessageReceiver gemeinsam erzeugt werden bekomme ich die Alerts mit doppelter Speicherfreigeba beim Beenden, auch wenn danach in dieser Methode alles auskommentiert ist!!!
     // Andererseits läuft alles glatt durch, sobald ich im Konstruktor nur die beiden Zeilen mit addGadgetAt auskommentiere!
     // Daher vermute ich, dass es an meinen Gadgets bzw. deren Behandlung und dem ganzen Freigabegedöns von Intuition, Gadgets etc. liegt (ist für mich alles noch viel zu undurchsichtig!)! Allerdings macht dann die Konstellation der aktivierten Zeilen im Zshg. mit der Tatsache, wann der Fehler erscheint und wann nicht absolut keinen Sinn!!! Entweder die Gadgets nicht ins Fenster übernehmen, dann kann ich beide Receiver erstellen, oder die Gadgets übernehmen, dafür nur einen der beiden Receiver anlegen (wohlgemerkt: Nur anlegen, nicht ins Fenster hängen und egal welchen der beiden)! Das ist doch komplett undurchsichtig, kann ich nicht nachvollziehen! Ihr vielleicht???
//    GadgetMessageReceiverC gadMsgRec;
//    MouseMessageReceiverC mouseMsgRec(*cursor);
...

---

C++ code:

---

#ifndef GADGETMESSREC_H
#define GADGETMESSREC_H

#include <intuition/intuition.h>
#include "IntuiMessageReceiverC.h"

class GadgetMessageReceiverC : public IntuiMessageReceiverC
{
	public:
        GadgetMessageReceiverC();
        ~GadgetMessageReceiverC();

        void receive(IntuiMessageC &intuiMessage);
};
#endif


#ifndef MOUSEMESSREC_H
#define MOUSEMESSREC_H

#include <intuition/intuition.h>

#include "PointC.h"
#include "CursorC.h"
#include "AnimObjectC.h"
#include "RefreshActivityC.h"
#include "IntuiMessageReceiverC.h"
#include "HexManagerC.h"

class MouseMessageReceiverC : public IntuiMessageReceiverC
{
	public:
        MouseMessageReceiverC(CursorC& cursor);
        ~MouseMessageReceiverC();
        
        void receive(IntuiMessageC &intuiMessage);
        void mouseMoved(const PointC& newPos);
        void mouseLeftClicked();
        
    private:
    	CursorC& cursr;
};
#endif

---

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

C++ code:

---

#ifndef INTUIMESSREC_H
#define INTUIMESSREC_H

#include "IntuiMessageC.h"

class IntuiMessageReceiverC
{
	public:
        IntuiMessageReceiverC();
        virtual ~IntuiMessageReceiverC() = 0;

	protected:
        virtual void receive(IntuiMessageC &intuiMessage) = 0;
};
#endif

...

#include "IntuiMessageReceiverC.h"

IntuiMessageReceiverC::IntuiMessageReceiverC() {}

IntuiMessageReceiverC::~IntuiMessageReceiverC() {}

---

C++ code:

---

void WindowC::setMenuMessageReceiver(MenuMessageReceiverC *recv)
{
    mnMsgRec = recv;
}

void WindowC::setMouseMessageReceiver(MouseMessageReceiverC *recv)
{
    msMsgRec = recv;
}

void WindowC::setGadgetMessageReceiver(GadgetMessageReceiverC *recv)
{
    gadMsgRec = recv;
}

void WindowC::setIntuiTickMessageReceiver(IntuiTickMessageReceiverC *recv)
{
	intuiTickMsgRec = recv;
}

...

void WindowC::processMessages()
{
	if (getUserPort() == NULL) return;

    while (!closing)
    {
		Wait(1 << getUserPort()->mp_SigBit);

		while (struct IntuiMessage *my_message = (struct IntuiMessage *)GetMsg(getUserPort()))
        {
            IntuiMessageC intuiMessage(my_message);
            ReplyMsg((struct Message *)my_message);

            if (closing) break;

            switch (intuiMessage.getClass())
            {
                case IDCMP_MENUPICK:
                    if (mnMsgRec != NULL) mnMsgRec->receive(intuiMessage);
                    break;

                case IDCMP_GADGETDOWN:
                case IDCMP_GADGETUP:
                	if (gadMsgRec != NULL) gadMsgRec->receive(intuiMessage);
                	break;

                case IDCMP_INTUITICKS:
					if (intuiTickMsgRec != NULL) intuiTickMsgRec->receive(intuiMessage);
                    break;

                case IDCMP_MOUSEMOVE:
                	if (msMsgRec != NULL) msMsgRec->mouseMoved(PointC(intuiMessage.getMouseX(), intuiMessage.getMouseY()));
                	break;
                	
                case IDCMP_MOUSEBUTTONS:
                	if (msMsgRec != NULL) msMsgRec->receive(intuiMessage);
                    break;

                case IDCMP_REFRESHWINDOW:
                	beginRefresh();
                	endRefresh();
					break;
					
                default:
                    break;
            }
        }
    }
}

---

C++ code:

---

WizardGroundsC::WizardGroundsC(WindowC& window) : hexManager(NULL), cursor(NULL), playerManager(NULL), win(window), inTickMsgRec()
{
	RefreshActivityC refrAct(win);
	inTickMsgRec.addIntuiTickActivity(&refrAct);
	win.setIntuiTickMessageReceiver(&inTickMsgRec);

	win.getRastPort().drawImageAt(&statusimage, 808, 15);
	win.getRastPort().drawImageAt(&buttonimage, 808, 552);

	win.getRastPort().setPrimaryPen(1);
	win.setTextFont((char *)"EstroFont.font", 18, 0, 0);

       srand(time(NULL));
       setupWindowComponents(win);

       //GadgetMessageReceiverC gadMsgRec;
       MouseMessageReceiverC mouseMsgRec(*cursor);
       MenuMessageReceiverC mnMsgRec(win.getMenu());

       win.setMenuMessageReceiver(&mnMsgRec);
       win.setGadgetMessageReceiver(&gadMsgRec);
       win.setIntuiTickMessageReceiver(&inTickMsgRec);
       win.setMouseMessageReceiver(&mouseMsgRec);

       ImageButtonC towerButton(&tower_button_normal, &tower_button_pressed, BUILDTOWER);
	ImageButtonC lightningButton(&lightning_button_normal, &lightning_button_pressed, ATTACK);

        TowerGadgetActivityC tgActivity(*cursor, *playerManager->getPlayer(HUMAN));
        towerButton.setReleaseActivity(&tgActivity);
        LightningGadgetActivityC lnActivity(*cursor, *playerManager->getPlayer(HUMAN));
        lightningButton.setReleaseActivity(&lnActivity);

	win.addGadgetAt(towerButton, 829, 573, 29, 52);
	win.addGadgetAt(lightningButton, 874, 573, 29, 52);
	win.refreshAllGadgets();

	win.processMessages();
}

---

C++ code:

---

...
       GadgetMessageReceiverC gadMsgRec;
       //MouseMessageReceiverC mouseMsgRec(*cursor);
 ...

---

C++ code:

---

...
       GadgetMessageReceiverC gadMsgRec;
       MouseMessageReceiverC mouseMsgRec(*cursor);
...
	//win.addGadgetAt(towerButton, 829, 573, 29, 52);
	//win.addGadgetAt(lightningButton, 874, 573, 29, 52);
...

---

C++ code:

---

void WindowC::addGadgetAt(GadgetC& gadget, WORD xPos, WORD yPos, WORD width, WORD height)
{
    if (window)
    {
        gadget.gadget.LeftEdge = xPos;
        gadget.gadget.TopEdge = yPos;
        gadget.gadget.Width = width;
        gadget.gadget.Height = height;

        UWORD pos = AddGadget(window, &(gadget.gadget), -1);

        if (first == NULL)
        {
            last = first = &(gadget.gadget);
        } else
        {
            last = &(gadget.gadget);
        }
    }
}

---

[ - Antworten - Zitieren - Direktlink - ]

Eine Vermutung:
Du erstellst Objekte statisch, z.B. mit WindowC win(...);
Dadurch wird es erst freigegeben, wenn die Routine beendet wird.
Deine Libs werden aber schon vorher geschlossen. Wird die Lib noch benutzt, obwohl sie geschlossen ist, crashts. Gilt auch für andere Objekte und Handler, die schon geschlossen sind, aber noch von existierenden Objekten benutzt werden.
Deshalb würde ich die Objekte mit new erzeugen, und die Reihenfolge beim delete umdrehen. Grobes Schema, Pseudocode:
OpenLibs();
c1 = new Class1C(..);
c2 = new Class2C(..);
WaitForClose();
delete c2;
delete c1;
CloseLibs();

Hab Dir doch schonmal ein Beispiel geschickt, wie es mit C++ möglich ist, ohne daß die Klassen Amok laufen

[ - Antworten - Zitieren - Direktlink - ]

Zitat:

---

Original von Reth:
virtual ~IntuiMessageReceiverC() = 0;

---

C++ code:

---

#ifndef INTUIMESSREC_H
#define INTUIMESSREC_H

#include "IntuiMessageC.h"

class IntuiMessageReceiverC
{
	public:
        IntuiMessageReceiverC() {}
        virtual ~IntuiMessageReceiverC() {}

	protected:
        virtual void receive(IntuiMessageC &intuiMessage) = 0;
};
#endif

---

Zitat:

---

Original von Thore:
Eine Vermutung:
Du erstellst Objekte statisch, z.B. mit WindowC win(...);
Dadurch wird es erst freigegeben, wenn die Routine beendet wird.
Deine Libs werden aber schon vorher geschlossen.

---

Zitat:

---

Deshalb würde ich die Objekte mit new erzeugen, und die Reihenfolge beim delete umdrehen. Grobes Schema, Pseudocode:
OpenLibs();
c1 = new Class1C(..);
c2 = new Class2C(..);
WaitForClose();
delete c2;
delete c1;
CloseLibs();

---

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

[ - Antworten - Zitieren - Direktlink - ]

Zitat:

---

Original von Reth:
In der Lektüre habe ich nachgeschlagen, dass man vollständig abstrakte Klassen mit virtuellen Destruktoren erstellen kann. Daher der virtuelle Konstruktor an dieser Stelle.

---

Zitat:

---

Habt ihr noch ne Idee, wo ich ansetzen könnte um diese komische Konstellation (hinsichtlich funktionieren/nicht funktionieren) zu testen? Für mich ergibt das immer noch keinen Sinn, dass mal der eine, mal der andere Receiver und mal das Hinzufügen der Gadgets weggelassen werden kann und dann alles gut zu sein scheint?

---

Zitat:

---

Ein funktionierender Debugger hätte hier schon etliche Stunden eingespart!

---

[ - Antworten - Zitieren - Direktlink - ]

Zitat:

---

Nicht das "virtual" stört mich, sondern das "=0". Das ist das äquivalent zu "abstract" in Java, nur dass Dich bei C++ der Compiler nicht daran hindert, eine Instanz einer Klasse mit solchen Methoden zu erzeugen. Und da ein Destruktor normalerweise immer den Destruktor der Superklasse aufruft, die Unterklassen also den aus ihrer Sicht nicht existierenden Destruktor aufrufen, klingt das für mich nach einem ziemlich groben Fehler.

Dann hast Du allerdings in einer anderen Datei eine Implementierung IntuiMessageReceiverC::~IntuiMessageReceiverC() {}, was im Widerspruch zur Deklaration der Methode als abstrakt steht (gibt es da keine Warnung?).

---

[ - Antworten - Zitieren - Direktlink - ]