3. Problem mit dem ifstream::open()

Problem mit dem ifstream::open()

  • X

    Nullterminierung ist bei dem String korrekt. Hab die terminierung eben manuell durchgeführt und es gab keine änderung. Hab auchmal versucht den dateinamen direkt anzugeben also "test.txt" das funktionierte bei Variante 1 einwandfrei bei variante 2 ging es nicht auhc nicht mit (char*)"test.txt", ich finde das wirklich sehr merkwürdig.

    0
  • T

    Schmirgelt Dein Programm beim open ab, oder irgendwann später beim Schreiben?
    Probier außerdem mal den Dateinamen direkt beim open als Literal zu übergeben.
    Ich kann es mir eigentlich nicht vorstellen, dass bei einem Aufruf mit char* die wchar_t* -Variante aufgerufen wird.

    0
  • X

    Es schmiert direkt beim open ab.

    ifstream file("test.txt");//funktioniert
------------------
ifstream file;
file.open("test.txt");//funktioniert nicht
-----------------
ifstream file;
file.open((char*)"test.txt");//funktioniert ebenfalls nicht

    Wenn ich Debugge sagt der Debugegr leider auch nciht viel nur bad ptr, wa smich sehr darauf schließen läst das er aus dem char* ein wchar_t* gemacht hat und der damit total veränderte dateiname ja nicht existiert.

    0
  • T

    Kann es sein, dass vorher irgendwas kaputt geht, weil Du z.B. über Puffergrenzen hinausschreibst?

    Benutzt Du evtl. einen Stream, der vorher schonmal benutzt wurde mehrmals? Also z.B. erst den Stream mit close schließen und dann wieder mit open öffnen?
    Wenn vor dem close irgendwas schiefgegangen ist, werden die Fehlerflags durch das close nicht wieder gelöscht. Also ggf. stream.clear() vor dem open aufrufen.

    Hast Du vielleicht einen exotischen oder komischen Compiler am Werkeln?

    0
  • X

    Der puffer des dateinamens hat noch mehr als genug Platz er ist sehr viel größer als der Inhalt.

    Nein das Programm benutzt den stream nur einmal. Habe eben ein Profilaktisches clear() vor das open gepackt, das brachte nichts.

    Ich nutze VC++2008 Express SP1.

    0
  • T

    Ich meine nicht den Puffer des Filenamens selbst, sondern irgendwelche anderen Sachen, die Du vorher machst. Oder vielleicht auch Dinge, die irgendwelche externen libs vielleicht machen.
    Anders ist das eigentlich nicht zu erklären. Visual Studio 2008 SP1 verhält sich bei Übergabe eines char-Literals an fstream::open auf jeden Fall korrekt.

    0
  • X

    Es funktioniert jetzt, der ifstream war Member einer Klasse, die hat in ihrem Konstruktor zeroMemory() auf sich selbst gemacht hab den ifstream zur globalen Variable gemacht und nun geht open plötzlich.

    0
  • M

    Xebov schrieb:

    der ifstream war Member einer Klasse, die hat in ihrem Konstruktor zeroMemory() auf sich selbst gemacht

    So was macht man ja auch nicht.

    0
  • T

    Oh man... 🙄

    0
  • X

    MFK schrieb:

    Xebov schrieb:

    der ifstream war Member einer Klasse, die hat in ihrem Konstruktor zeroMemory() auf sich selbst gemacht

    So was macht man ja auch nicht.

    Ich verstehe den Grund wieso ehrlich gesagt nicht, die Klasse soll alle ihre variablen mit dem Befehl leeren um Datenmüll vorzubeugen. ist aber gut zu wissen das der ifstream so empfindlich reagiert.

    Aber danke für die Hilfe.

    0
  • M

    Xebov schrieb:

    Ich verstehe den Grund wieso ehrlich gesagt nicht, die Klasse soll alle ihre variablen mit dem Befehl leeren um Datenmüll vorzubeugen. ist aber gut zu wissen das der ifstream so empfindlich reagiert.

    Es geht nicht um ifstream. Das ist grundsätzlich eine Schnapsidee.

    Du kannst nicht davon ausgehen, dass eine Klasse ordentlich initialisiert ist, wenn du ihre binäre Speicherrepräsentation komplett mit Nullen überbügelst. Mit hoher Wahrscheinlichkeit geht dabei eher etwas kaputt.

    Damit zerlegst du dir übrigens auch den VTable-Zeiger bei polymorphen Klassen.

    0
  • T

    Xebov schrieb:

    MFK schrieb:

    Xebov schrieb:

    der ifstream war Member einer Klasse, die hat in ihrem Konstruktor zeroMemory() auf sich selbst gemacht

    So was macht man ja auch nicht.

    Ich verstehe den Grund wieso ehrlich gesagt nicht, die Klasse soll alle ihre variablen mit dem Befehl leeren um Datenmüll vorzubeugen. ist aber gut zu wissen das der ifstream so empfindlich reagiert.

    Aber danke für die Hilfe.

    Alle Klassen, die interne Zustände halten, z.B. Pointer auf Heap-Objekte oder irgendwelche Stati reagieren "empfindlich" auf sowas.

    Instanzen von Standard-Libarary-Klassen dürftest Du damit sicher ins Nirvana der undefinierten Zustände bomben.

    Zum setzen von Klassen auf definierte Zustände gibt es Initialisiererlisten und ggf. den Rumpf der Konstruktoren. Aber ZeroMemory auf die Klasse anzuwenden ist mit Sicherheit eine schlchte Idee. Das kann man nur machen, wenn die Klasse ausschließlich POD-Typen enthält.

    0
  • X

    Also kurz gesagt lieber ein paar Zeilen mehr tippen und die Variablen einzelln Nullen?

    0
  • K

    zeroMemory()

    Damit machst du alles kaputt, was der Konstruktor anlegt, also auch die virtuelle Methodentabelle, interne Informationen, Metainformation. Keine gute Idee, nimm lieber Initialisierungslisten.

    0
  • N

    Xebov schrieb:

    Also kurz gesagt lieber ein paar Zeilen mehr tippen und die Variablen einzelln Nullen?

    Nicht nullen, sondern mit einem gültigen Wert initialisieren! Die Arbeitsweise mit ZeroMemory() oder memset() ist eher C und für die dortigen PODs angebracht. In C++ hat das meistens nichts zu suchen!

    Irgendwelche undefinierten Laufzeitauswirkungen wegen Schreibfaulheit in Kauf zu nehmen hat sich zumindest selten bewährt. Nimm wie gesagt Initialisierungslisten.

    0
Anmelden zum Antworten