ich habe eine Frage bzgl. diesen zwei folgenden Methoden:

Headerfile (file.h)

...
char* m_File;
...

CPP-File (file.cpp)

void method1
{
   char fileName[100]

   ...
   //Ermittlung des Filenamen
   ...

   m_File = fileName;
}

void method2
{  
   ...
   logFile = fopen(m_File,"a");
   ...

}

Nun wird in method1 der Filename ermittelt und der Membervariablen vom Typ char* uebergeben. In method2 ist in dieser Membervariable nicht der gewuenschte Filename, der in method1 ermittelt wurde, sondern irgend etwas. Wo ist der Denkfehler passiert? Was muss ich anders machen?

Danke euch schon im Voraus.

Gruss Fabian

Sollte jeder zwar mal gemacht haben (mE), aber wer sich die Mühe ersparen will...

ich habe eine Frage bzgl. diesen zwei folgenden Methoden:

Bitte, auch wenn man Code sinnvoll kürzen sollte, schreib zumindestens soviel das man sieht was dein Problem ist. Du schreibst zwar Methoden, was für mein Verständnis Funktionen von Klassen sind, aber zum einen sieht man weder den Klassenbezug noch einen vollständigen Funktionskopf, sondern nur ungültigen Code.

FabianR schrieb:

CPP-File (file.cpp)

void method1
{

Das ist weder eine Funktion- noch ein Methode. Gib mindestens soviel Code mit an wie für das Problem nötig ist. Und ist deine Variable im Header nun ein Member einer Klasse, eine globale Variable oder was? (zumindest den Klassenrumpf sollte man schreiben).

Sobald du einen Minimalcode (das ist deiner nicht, der ist noch nicht mal Minimal sondern garnichts) präsentierst, kann man dir auch mehr helfen

Grundsätzlich aber eins: Einem Zeiger kannst du nicht einfach einen lokalen Speicherbereich zuweisen (Stichwort Allozierung[new]/Dealozierung[delete]). Und wie schon erwähnt: Verwende wenn möglich std::string statt char-Zeigern und char-Arrays...

cu André
P.S: Zudem benutze bitte statt die Code-Tags die C/C++-Tags...

m_File = new char[count]
// zuweisung

oder spaeter irgendwo

delete [] m_File;
m_File = NULL;

also entweder gleich allokieren und damit arbeiten, oder mit fileName arbeiten und dann allokieren und zuweisen

a) Du machst aus m_File einen char-Array, musst dafür aber halt die maximale Größe zu Kompilierzeit wissen.

b) Du forderst den nötigen Speicher an zur Laufzeit. Und natürlich später wieder mit delete[] freigeben.

void method1(...)
{
   char fileName[100];

   ...
   //Ermittlung des Filenamen
   ...

   m_File = new char[strlen(fileName)+1];
   strncpy(m_File, fileName, strlen(fileName)+1);
}

Ich moechte diese String-Variante aber umgehen, soll heissen ich moechte die Loesung in char*-Form, oder ist das in diesem Fall viel schwieriger?

Vorgabe von einen Lehrer, oder einfach Masochistisch veranlagt? Sorry, aber warum muss man sich eigentlich immer wieder mit Zeigerarithmetik rumschlagen, wo es wirklich nicht sein muss (std::string gehört zum C++ Standard).

cu André

Danke fuer eure Hilfe.

Vorgabe von einen Lehrer, oder einfach Masochistisch veranlagt? Sorry, aber warum muss man sich eigentlich immer wieder mit Zeigerarithmetik rumschlagen, wo es wirklich nicht sein muss (std::string gehört zum C++ Standard).

Weil Lehrer, Profs und Buchautoren doof sind? Ich bin der letzte der sagen würde, man muß sich nicht mit Array und Pointer beschäftigen, aber es gibt anscheinend immer noch Menschen, die meinen einen Anfänger damit zuschmeissen zu müssen.

Wenn du siehst wie überlaufen Informatikstudiengänge heutzutage sind, da ist das eine gute und beliebte Möglichkeit um auszusieben.

Wo sind die überlaufen? Sorry, aber ich höre immer etwas von einem Mangel...

cu André
P.S: Und das entschuldigt nicht das die Dozenten auf einen Stand vor 10+ Jahren arbeiten. Mit Zeigern kann man die Schüler/Studenten noch immer anderweitig zur Verzweiflung bringen.