Probleme mit statischer Member_funktion

[mstank]

Hallo!

Den ganzen Nachmittag habe ich damit verbracht, daß ich via Serialize Zeichen einlese, eine andere Funktion aufrufe, die diese verarbeitet und dann an View::OnDraw weitergibt.

Und was ist das Ergebnis? Ich werfe den Schrott gleich aus dem Fenster!

Meine Codes sehen so aus:

c++:

void CLadenSpeichernDoc::Serialize(CArchive& ar)
{
if (ar.IsStoring())
{
// ZU ERLEDIGEN: Hier Code zum Speichern einfügen
}
else
{
// ZU ERLEDIGEN: Hier Code zum Laden einfügen
ar >> m_c_Zeichen;
CDateiZugriff::Lesen();
}
}
--------------------------------------------
void CDateiZugriff::Lesen()
{
//AfxMessageBox("Tach!");
m_s_Zeile += m_c_Zeichen;
}


Fehlermeldung:
My Projects\LadenSpeichern\DateiZugriff.cpp(32) : error C2597: Ungueltige Referenz auf ein Datenelement 'CDateiZugriff::m_s_Zeile' in einer statischen Member-Funktion


Die Variablen sind:

char m_c_Zeichen und CString m_s_Zeile in beiden Klassen als public deklariert

Wer hat eine Ahnung, was ich falsch mache, weil eigendlich will meinen Compi nicht aus den Fensterwerfen, aber wer nicht höhren will, der muß fühlen!

THX Markus

 

[RedWing]

So wie es ausschaut versuchst du in einer statischen Funktion auf eine Objektvariable zuzgreifen. Da statische Funktionen unabhängig von Objekten
existieren und nicht statische Memebrvariablen von der existenz eines
Objektes abhängig sind geht das natürlich nicht und der Compiler meldet
dir einen Fehler.

Ich gehe mal von deinem Code aus das void CDateiZugriff::Lesen() als statische
Memberfunktion deiner Klasse deklariert ist
Wenn void CDateiZugriff::Lesen() unbedingt statisch sein muss wirsrt du wohl nicht drumrumkommen, die Membervariablen m_s_Zeile, m_c_Zeichen auch als statisch
zu deklarieren, wenn du dies beabsichtigst...


Gruß

RedWing

 

[mstank]

Hi Redwing!

Vielen Dank für deine schnelle Antwort!

Ich werden mal schauen, ob die Fkt. statisch ist.

 

[mstank]

So!


Die Fkt. war als static deklariert, welches ich entfernt habe.

Aber nun taucht die fehlermeldung auf:

:\My Projects\LadenSpeichern\LadenSpeichernDoc.cpp(68) : error C2352: 'CDateiZugriff::Lesen' : Unzulaessiger Aufruf einer nichtstatischen Member-Funktion
h:\my projects\ladenspeichern\dateizugriff.h(17) : Siehe Deklaration von 'Lesen'

Und hier kommt sie her:

void CLadenSpeichernDoc::Serialize(CArchive& ar)
{
if (ar.IsStoring())
{
// ZU ERLEDIGEN: Hier Code zum Speichern einfügen
}
else
{
// ZU ERLEDIGEN: Hier Code zum Laden einfügen
ar >> m_c_Zeichen;
CDateiZugriff::Lesen(); <<<<<----FEHLER!
}
}
Ich will hier nur die Fkt. CDateiZugriff aufrufen!

 

[RedWing]

Ja das liegt daran das du jetzt um die Funktion aufzurufen
ein Objekt vom Typ der Funktion brauchst, und nicht mehr den Klassennamen
für den Aufruf verwenden kannst....
Es gibt zwei Möglichkeiten:

1.) Du machst die Funktion lese nicht statisch und übergibst der
Funktion CLadenSpeichernDoc::Serialize(CArchive& ar) ein Objekt vom
Typ CDateiZugriff damit du mit Hilfe dieses Objekts auf die Funktion lese
draufzugreifen kannst oder:
2.) Du lässt die Funktion lese statisch und machst zusätzlich die Membervariablen
die die Funktion benutzt auch statisch...


Gruß

RedWing

 

[mstank]

zu1.


Meinst du etwas wie: CDateiZugriff::Serialize(ar);

Ich hab nicht so die Ahnung:-(

 

[mstank]

nein das funktioniert nicht!

 

[mstank]

Ich habe jetzt mal die beiden Varialblen auf static gesetzt. Aber da kamen nur neue Fehlermeldungen mit vielen @'s.

 

[CodeFatal]

Hallo!

Falls noch nichts aus dem Fenster geflogen ist schlag ich mal fogendes vor:

1) alle static's aus dem Code entfernen (machen gerade bei "Nicht Berufs Progra's" Probleme)

2) Die Klassen sind unabhängig voneinander und wissen erstmal nichts voneinander.
Dann werden bei der lese Funktion nicht die Daten in der Klasse CLadenSpeichernDoc geändert sondern nur in CDateiZugriff. Zumindest wenn vorher ein Objekt angelegt wurde (Ursprünglich nicht gemacht -> Fehler da die Variablen noch nicht exestieren - Speicherzugriff)

3) Bei mir würde sowas folgend aussehen:
Headerfiles

LadenSpeichernDoc.h
"
"mach was noch so soll
"
#include "DateiZugriff.h"
"
"mach was noch so soll
"

class CLadenSpeichernDoc
{
public:
char m_c_Zeichen;
CString m_s_Zeile;
void Serialize(CArchive& ar) ;
}

DateiZugriff.h
"
"mach was noch so soll
"
class CDateiZugriffDoc
{
public:

CString Zeile;//umbenannt damit der Unterschied klar wird
virtual BOOL PreTranslateMessage(MSG* pMsg);//vom Assi hinzufügen lassen

}

extern CDateizugriffDoc cdd;// irgendwo auf der Welt gibt es ein globales Objekt


Änderungen in den CPP's
DateiZugriff.cpp
#include "DateiZugriff.h"

1) alle statics weg
2) statt Lese()
BOOL CDateiZugriff:reTranslateMessage(MSG* pMsg)
{
// TODO: Speziellen Code hier einfügen und/oder Basisklasse aufrufen
//pMsg auslesen und in Zeile speichern;
//MSDN hilft
return CDialog:reTranslateMessage(pMsg);
}
3) unten folgendes anlegen:
CDateizugriffDoc cdd; //global!

LadenSpeichernDoc.cpp

#include "LadenSpeichernDoc.h"

void CLadenSpeichernDoc::Serialize(CArchive& ar)
{
if (ar.IsStoring())
{
// ZU ERLEDIGEN: Hier Code zum Speichern einfügen
}
else
{
// ZU ERLEDIGEN: Hier Code zum Laden einfügen
ar >> m_c_Zeichen; //komm eher aus der C-Welt und bin mir hier nicht ganz sicher m_c_Zeichen mit Inhaltvon ar füllen;
m_s_Zeile += cdd.Zeile;//aktuelles von globalem Objekt auslesen und in eigener Variablen speichern
cdd.Zeile.Empty();//Inhalt löschen, damit nichts doppelt gelesen wird

}
}
Das sollte jetzt eigentlich weiter helfen und die Warnungen beseitigen... Zumindest wenn keine Schreibfehler mehr drin sind.

Gruß Michael

 

[mstank]

Hi Michale!


Autsch

Das muß ich mir erst einmal auf der Zunge zergehen lassen!

Ich melde mich, sobald ich alles ausprobiert habe.

Soweit allerbesten Dank!

Markus