4. Speicherbereich von Element wird überschrieben

Speicherbereich von Element wird überschrieben

  • C

    nein, es findet keine Vererbung statt, ich erstelle nur eine Instanz in der CWinApp-Klasse von meiner Klasse_B und übertrage die Adresse des Zeigers an m_pMainWnd, also so:

    CMenue dlg;
	m_pMainWnd = &dlg;

    ...in der Klasse B lege ich ne Instanz der Klasse A an...

    class Klasse_B: public CDialog
{
...
Klasse_A container;
...
}

    ...um dann in den anderen Klassen C,D,E auf das Objekt der Klasse A zugreifen zu können:

    Klasse_B* Klasse_B_objekt= (Klasse_B*)AfxGetMainWnd();
      m_system=  Klasse_B->container.GetSystem();

    ...oder muss ich die Klasse A am Besten in der CWinApp Klasse schon mal anlegen und dann über den MainWnd aus den Kalssen C,D,E auf die CWinApp-Klasse zugreifen?

    Gruss,
    chullain

    0
  • C

    Das MainWnd ist, wenn ich jetzt deinen Code richtig deute, ein CMenu-Objekt (schon diese Zuweisung sollte imho nicht funktionieren) - das wandelst du um in ein Klasse_B-Objekt, das damit überhaupt nicht verwandt ist. Also interpretierst du irgendwelche (möglicherweise unbelegten) Speicherbereiche als Klasse_B, da kann nur Müll herauskommen.

    (Und nebenbei wird das lokal angelegte Objekt, das du als Hauptfenster angemeldet hast, am Ende der Init-Funktion wieder pulverisiert)

    0
  • C

    wie spreche ich dann das Objekt der Klasse A aus den Klassen C,D,E an, wenn ich das Objekt der Klasse A in der Klasse B erstellt habe. Ich meine das Objekt der Klasse B ist die ganze Zeit aktiv, auch wenn ich andere Dialogfenster der Klassen C,D,E auf habe...

    ...über referenzierung klappt das irgendwie auch nicht...

    Gruss,
    chullain

    0
  • C

    Solange du das Klasse_B-Objekt hast, kannst du über . (bei Zeigern ->) auch auf dessen Member zugreifen. Aber aus deinem Code kann man nicht erkennen, wo das Objekt angelegt wurde (und das ist wichtig um zu wissen, ob es noch exisitert).

    0
  • C

    ..also in der Klasse B werden alle anderen Objekte (Dialoge) erzeugt, jedoch schliesse ich nicht das Objekt der Klasse_B

    void Klasse B::OnRepositoryOperation() 
{
...
    Klasse_C repoOperation(this);
...
}

    BOOL Klasse_C::OnSetActive() 
{
...
Klasse_B* Klasse_B_objekt= (Klasse_B*)AfxGetMainWnd();
...
}

    ...irgendwann überschreibe ich nur den Speicherbereich einer Variable, denn ich eigentlich überhaupt nicht angefasst habe...und das ist ärgerlich...

    Gruss,
    chullain

    0
  • C

    chullain schrieb:

    ..also in der Klasse B werden alle anderen Objekte (Dialoge) erzeugt, jedoch schliesse ich nicht das Objekt der Klasse_B

    Ja, sehr schön - aber WO LEGST DU DEIN 'Klasse_B' OBJEKT AN? Bevor du diese Frage nicht beantwortet hast, kommen wir nicht weiter.

    0
  • C

    ja, hab ich doch schon erwähnt in meiner CWinApp-Klasse, also der Hauptklasse der Anwendung...

    Header:

    class CAPIApp : public CWinApp
{
public:
	CAPIApp();
...
}

    *.cpp

    BOOL CAPIApp::InitInstance()
{
...
	Klasse_B dlg;
	m_pMainWnd = &dlg;
...
}
    0
  • C

    chullain schrieb:

    BOOL CAPIApp::InitInstance()
{
...
	Klasse_B dlg;
	m_pMainWnd = &dlg;
...
}

    Und da haben wir auch schon das Problem. Dein Objekt 'dlg' liegt lokal in der InitInstance()-Methode. Das bedeutet, am Ende der Methode wird es wieder demontiert und sein Speicherplatz kann von der nächsten verwendeten Funktion neu überschrieben werden. Leg das Objekt also besser an einem Ort an, wo es bis zum Programmende überleben kann, z.B. als static Variable in der InitInstance(), als globale Variable oder als Member der CAPIApp-Klasse (ich würde letzteres bevorzugen).

    0
  • Mod

    @CStoll:
    Aber wenn es das normale Design einer Dialog-Applkation ist, dann ist dieses Objekt so lange gültig, wie auch DoModal läuft.

    @chullain:
    Aber das eigentliche Problem sehe ich jetzt:

    // Deklaration
CString arrayT1[30][8];

// Code
void Klasse_A::DeleteArray()
{
    for (int i=0;i<=30;i++) {
    //             ^ HIER IST DER BUG!
        for (int j=0;j<=7;j++) {
            arrayT1[i][j] = "";    
        }
    }
}

    Array überlauf! Du darfst nur bis <30 gehen!

    Gewöhne dir immer an mit < zu testen und den die Größe des Arrays als Vergleichswert zu nutzen.

    Noch besser Du verwendest _countof oder einen ähnlichen Makro:

    void Klasse_A::DeleteArray()
{
    for (int i=0; i<_countof(arrayT1); ++i) 
    {
        for (int j=0; j<_countof(arrayT1[0]); ++j) 
            arrayT1[i][j].Empty();    
    }
}

    BTW: Zum Löschen von CString verwende Empty()!

    0
  • C

    das wars gewesen, vielen lieben Dank.

    Gruss,
    chullain

    0
Anmelden zum Antworten