Wieso wird die Schleife 2x aufgerufen?

kernel64

Hallo Leute ich möchte den Inhalt einer Datei aufrufen, in der Datei steht ist nur ein Word in der ersten Zeile.

char buffer[1024];
FILE *f = fopen( argv[1], "r" );
int i = 0;
while( !feof( f ) )
{
	i++;
	fgets( buffer, sizeof( buffer ), f ) ;
	cout << buffer << " i:" << i;
	str = buffer;

}

Bei diesem Code habe ich absichtlich int i eingebaut zum Test und es kommt für i=2 raus, also wieso wird die Schleife 2x mal ausgeführt?

Beim erstem mal ließt er das Wort, beim zweiten Mal das EOF.
==> Das solltest du in deiner Schleife überprüfen ...

Konrad Rudolph

Wieso verwendest Du C-Streams? Verwende doch die C++-Funktionen und beachte den folgenden Hinweis, dann sollte alles fein funktionieren:

http://fara.cs.uni-potsdam.de/~kaufmann/?page=GenCppFaqs&faq=eof#Answ

kernel64

Ist das besser?

string str;
ifstream ifs("c:\\test.txt");
int n = 0;
while( !ifs.eof() )
{
	n++;
	ifs >> str;
	cout << str << " " << n << endl;
}

Krux

kernel64 schrieb:

Ist das besser?

jepp es war absolut lebensnotwendig das i in n umzubenennen.

while((ifs >> str) && !ifs.eof())
{
   ++n;
   cout << ....;
}

Edit->Tags vergessen

kernel64

Mit diesem Code wird die letzte zeile nicht ausgegeben.

int n = 0;

while((ifs >> str) && !ifs.eof())
{ 
	++n;
	cout << str << " " << n << endl;

}

In meiner Datei steht folgendes:

info 
help
lock
lock

In der Konsole werden nur alle drei zeilen ausgegeben.

Konrad Rudolph

EInfach das '&& !ifs.eof()' weglassen, dann sollte es funktionieren:

while (ifs >> str)
    cout << str << " " << ++n << endl;

Anscheinend steht das EOF direkt an deinem letzten Wort.
Ein einfaches

while(ifs >> str) { ... }

Sollte alle Probleme lösen ...

Edit->ToSlow()

kernel64

Hallo das mit dem weglassen von '&& !ifs.eof()' klappt prima.
Noch eine Frage, wie kann ich eine ganze zeile auslesen und nicht einzelne strings

Beispiel:

hallo auto test
blabla
help me

Also wie bekomme ich in der ersten und letzten Zeile alles aufeinmal in str gespeichert. Denn mit dem obigen Code wird jedes Word einzelnd behandelt.

getline in Verbindung mit std::string ist dein Freund.

kernel64

getline ist für eine Benutzereingabe und nicht zum auslesen von Dateien.

Also ich hoffe ihr könnt mir helfen folgendes:

hallo auto test
blabla
help me

so auszulesen:

erste zeile:

hallo auto test
...

kernel64

Ups mit getline gehts doch

Wenn ich folgenden Code benutze wird das erste wort nicht angezeigt;

while(ifs >> str)
{	
	getline(ifs,str);
	cout << str << endl;
}

nucknuck

while(ifs >> str)

Damit wird das erste Wort schon eingelesen.

string line;
while( getline( ifs, line ) )
{
    cout << line << "\n";
}

Binarys

Das geht mit 'stream.getline'.

Edit: upps da war ja noch eine seite ^^