Thema: C++: Bestimmten String aus Datei lesen

[DarkThing]

Ka ob das hier reingehört weil es nichts mit OS Dev zu tun hat sondern eher mit C++ & fstream. Aber man kanns ja mal versuchen.
Also, ich habe eine Datei in der zum Beispiel folgendes steht (nur ein Beispiel):

Code: [Auswählen]


Testtext
#abc Test.
Hallo

Jetzt möchte ich die Datei Byte für Byte lesen und in einen Outputfile schreiben. Wenn das aktuelle Zeichen aber ein # ist möchte ich die nächsten 4 Bytes auch lesen und checken ob da "abc " steht. Wenn nicht soll das #-Zeichen und die gelesenen 4 Bytes unverändert in die Output Datei geschrieben werden und normal weitergemacht werden. Wenn aber in der Datei "#abc " steht und das erkannt wurde möchte ich alle Zeichen bis zum Punkt in einen neuen String schreiben und den ganzen Teil nicht in die Outputdatei schreiben. Ich hab einen Pointer auf die Inputdatei und einen Pointer auf den Outputfile.

Ich weiß die Erklärung ist nicht gerade top, also wenn jemand Fragen dazu hat werd ich die beantworten (eigentlich logisch ^^).

Der Code bisher:

Code: [Auswählen]


char ch;
char tmp[4];
char *str;
while(!FileIn->eof())
{
// Zeichen lesen
ch = FileIn->get();

// Wenn Zeichen kein # ist, einfach speichern
if(ch != '#')
{
FileOut->put(ch);
} else {
// Zeichen ist ein #

// Nächsten 4 Zeichen lesen
FileIn->read(&tmp[0], 4);
tmp[4] = '\0';

// Wurde abc erkannt
if(strcmp(tmp, "abc ") == 0)
{
// String dahinter ermitteln
while(ch != '.')
{
ch = FileIn->get();
*str++ = ch;
}
*str= '\0';
// Zum Debuggen String anzeigen
cout << file << endl;
} else {
// Nein, also alles bisher gelesene speichern...
FileOut->put('#');
FileOut->write(tmp, 4);
// ...und weitermachen
continue;
}
}
}


[DarkThing]

Keiner ne Idee?
Wegen dem Code nochma: Manchmal stürzt das Prog ab, aber ansonsten wird immer der else-Teil vom zweiten if-statement abgearbeitet. Wenn irgendwer nur ne Idee im Ansatz hat oder das ganze völlig anders machen würde, würd mir das auch schon seht helfen.

[Jidder]

hm ich hab gedacht der code sollte so eine art tutorial sein, weil du keine frage gestellt hast.

also was mir auffällt:

tmp ist 4 bytes groß, aber du schreibst auch in tmp[4] (das ist das 5. byte) was rein. -> buffer overflow

str ist ein zeiger auf einen char, aber du initialierst str nicht. deswegen wird ein x-beliebiges stück memory bei *str++ = ch; und *str= '\0';  getrasht -> noch eine art buffer overflow

[DarkThing]

Hab beides eingebaut. Die Abstütze sind logischerweise weg, aber es geht immer noch nicht. In der Zeile
if(strcmp(tmp, "abc ") == 0)
Wird nie erkannt das in tmp abc steht. Das muss aber eigentlich bei der Beispieldatei einmal kommen.

@Tutorial-Code: Der Code hat ja so keine echte verwendbare Funktion. Aber daraus wird noch was.

[stultus]

kann es evtl. sein das der zweite string ("abc ") nicht automatisch nullterminiert wird? sowas in der art könnt ich mir ganz gut vorstellen...

[DarkThing]

Hab ein \0 angehangen, geht aber trotzdem nicht. Ich hab jetzt auch mal versucht das tmp in einen Pointer umzubauen. Half aber auch nicht.

[Svenska]

Muss die Bedingung nicht umgekehrt werden?
*duckundwech* Kann kein C, geschweige denn Cpp

[DarkThing]

Also jetz bin ich beim besten willen überfordert. Wenn ich die Zeile in das hier änder: if(strcmp(tmp, "abc ") != 0) , also das ganze umdrehe, komm ich trotzdem nur in den else Teil.

[DDR-RAM]

Wie ist denn jetzt kein Problem?
Und was ist file?

Code: [Auswählen]


// Zum Debuggen String anzeigen
cout << file << endl;


[DarkThing]

file muss natürlich str heißen.  
Das Problem ist, dass es nicht erkannt wird, wenn hinter einem #-Zeichen "abc " steht.

[DDR-RAM]

Code: [Auswählen]


    ch = FileIn->get();


Wenn du auf Fehlermeldungen achten würdest, hättest du bemerkt, das hier etwas nicht stimmt. FileIn->get() gibt kein char zurück

es muss so heißen:

Code: [Auswählen]


FileIn->get(ch);


Außerdem, wenn du str ausgibst gibst du nur dein letztes null-terminierungs zeichen aus. das willst du glaube ich nicht sehen

MfG
DDR-RAM

[DarkThing]

Da kommen keine Fehlermeldungen, nicht mal Warnungen. In der MSDN steht das das so geht wie ich es mache, und das kopieren von den Chars geht ja auch, aber der strcmp-Teil geht nicht. Das anzeigen wird ja auch erst dann interessant, wenn er überhaupt dahin kommt. Ehal, trotzdem thx.

[DDR-RAM]

Also ich hab deinen Code umgeschrieben und bei mir klappts

MfG
DDR-RAM

[DarkThing]

Was klappt? Nur das compilieren oder geht auch die Logik dahinter?
Wenn das Prog alles richtig macht bitte mal den Code posten, vielleicht hab ich ja was übersehen oder so.

[DDR-RAM]

Natürlich auch die Logik

Hier der Code:

Code: [Auswählen]


/////////////////
// main.cpp

#include <iostream>
#include <fstream>

int main()
{
const char strAbc[] = "abc ";
const int nMaxStrAlloc = 100;

std::ifstream FileIn;
std::ofstream FileOut;

FileIn.open("in.txt");
FileOut.open("out.txt");

char ch;
char tmp[sizeof(strAbc)];
char *str = new char[nMaxStrAlloc + 1];

while(!FileIn.eof())
{
// Zeichen lesen
FileIn.get(ch);
       
// Wenn Zeichen kein # ist, einfach speichern
if(ch != '#')
{
FileOut.put(ch);
} else {
// Zeichen ist ein #
// Nächsten 4 Zeichen lesen
FileIn.read(&tmp[0], sizeof(tmp) - 1);
tmp[sizeof(tmp) - 1] = '\0';
// Wurde abc erkannt
if(strcmp(tmp, strAbc) == 0)
{
// String dahinter ermitteln
int i;
for (i = 0; ch != '.' && i < nMaxStrAlloc; i++)
{
FileIn.get(ch);
str[i] = ch;
}
str[i] = '\0';
// Zum Debuggen String anzeigen
std::cout << str << std::endl;
} else {
// Nein, also alles bisher gelesene speichern...
FileOut.put('#');
FileOut.write(tmp, 4);
// ...und weitermachen
continue;
}
}
}

FileOut.close();
FileIn.close();

delete [] str;

return 0;
}


Bei mir sind FileIn und FileOut übrigens keine Zeiger, deshalb nicht verwundern.

MfG
DDR-RAM

[DarkThing]

Thx! Werds bei mir einbauen und testen.

[DarkThing]

Geht perfekt. Thx!! Ich hab aber noch ne Kleinigkeit gefunden: Der Punkt wird bei str mit hinten dran gehangen. Lässt sich aber so beheben: (letzte Zeile):

Code: [Auswählen]


// String dahinter ermitteln
int   i;
for (i = 0; ch != '.' && i < nMaxStrAlloc; i++)
{
    FileIn.get(ch);
    str[i] = ch;
}
str[i-1]   = '\0';