Funktionen funktionieren nur wenn ich sie einzeln aufrufe

Jester schrieb:

Du liest erst alle Tokens aus der Datei und zählst mit, die Datei ist danach sozusagen leer. Anschließend versuchst Du Werte rauszulesen und wunderst Dich, daß nichts kommt.

Aha, ja stimmt, in der version davor hatte ich noch überall getline verwendet, aber mir schien >> einfacher zu sein.

Ich werde es mal testen.

vielen dank.

cu max

Tja, ich scheine zu blöd zu sein das hinzubekommen.
So sieht jetzt meine funktion zum einlesen von Zeilen aus:

//Lies eine Zeile aus einer Textdatei in einen String und gibt diese mit return zurück
string Streaming::m_readLine(int NumberOfLine)
{   
    in.seekg(0,ios::beg);
	string rContent;
	for(int i=0;i<NumberOfLine;i++)
	{
		getline(in,rContent);
	}

	return rContent;
}

Ich bin wahrscheinlich nur zu blöd das zu tuen was du sagst.
Ich habe jetzt jedes >> durch ein getline ersetzt.
Zufallsgenerator und ende such funktion funktionieren gut.

wäre nett, wenn mir noch jemand auf die Socken helfen könnt.

cu max

[EDIT]Die idee ging auch nicht[/EDIT]

cu max

Jester

Hm, das sieht eigentlich korrekt aus. Was funktioniert denn eigentlich genau nicht?

Hi,

Jester schrieb:

Hm, das sieht eigentlich korrekt aus. Was funktioniert denn eigentlich genau nicht

Ich bekomme das Wort nicht angezeigt, der aufruf muss richtig sein, denn wenn ich ihm eine konkrete Zeile gebe dann liest er diese auch aus und zeigt sie mir an. Der aufruf:

string Content=German.m_readLine(RandomValue); 
    cout <<Content<<endl;

Ich kann mir nicht erkären was falsch ist, ich habe auch diese Variante ausprobiert:

string Content;
       Content=German.m_readLine(RandomValue);
cout <<Content<<endl;

Nur um sicher zu gehen.

langsam kommst du in die Credits

cu max

Jester

Mach mal erst das zählen und gib danach mal Zeile 1 aus oder so. Was passiert denn dann?

Hi,

Jester schrieb:

Mach mal erst das zählen und gib danach mal Zeile 1 aus oder so. Was passiert denn dann

Dann sehe ich immer noch nicht das Wort.
Es muss also an der Zähl Funktion liegen, wenn die weg ist dann geht es ja.

[EDIT]Naja, beide benutzen den selben ifstream, ist das Ok so?[/EDIT]

Ich weis nicht mehr weiter, ich kann keinen Fehler finden.

cu max

Ok, jetzt geht es, ich habe einen neuen ifstream gemacht.
Warum konnte m_readLine ihn nicht überschreiben?

vielen dank an alle.

cu max

Tja, hat gestern nur funktioniert weil es nicht in einer Schleife stand.
Jetzt, weis ich allerdings das es an meinem Konstruktor liegen muss:

Streaming::Streaming(const char *In,const char *Out)    
{   
    this->in.open(In, ios::in); //öffnen zum lesen mit Standardoptionen  
    this->out.open(Out, ios::out);  //öffnen zum schreiben mit Standardoptionen  
}

Der Inhalt des Textfiles wird irgendwie immer gelöscht.
In meinme Programm habe ich ein Objekt erstellt:

Streaming German("VocabularyGerman.txt","VocabularyGermanWrite.txt");

In einer do while Schleife greife ich dann auf die methoden der Klasse streaming zu:

German.m_seekEndOfInFile();
German.m_readLine(Lines);

Bei dem ersten Schleifendurchlauf funktioniert alles wunderbar, ab einer wiederholungen ist die Textdatei leer und er kann nichts mehr auslesen.
Beim verlassen des Programms ist die Textdatei wieder voll?
Ich habe es schon einige Sachen probiert:

this->in.open(In,ios::in|ios::ate);

oder:

this->in.open(In,ios::in|ios::app);

Auch in der FAQ bei Hume habe ich nichts finden können.

Ich hoffe es hat noch jemand eine idee, sonst werde ich wohl nicht herumkommen das ganze neu zu schreiben.

danke.

cu max

HumeSikkins

Hallo,
du löschst aber nach dem Lesen schon das eof-Bit, oder?

Hallo,
danke ersteinmal.

HumeSikkins schrieb:

du löschst aber nach dem Lesen schon das eof-Bit, oder?

Ich denke nicht, ich werde mal schauen wie das geht, mit dem eof bit bezeichnest du doch '\n' oder?
Wäre nett wenn du dir mal meine methoden anschaust, ist allerdings recht schlecht gemacht dar ich dateien überschreibe weil ich mich mit vektoren nicht auskenne.
Streaming.h:

#ifndef STREAMING
#define STREAMING

#include <string>
#include <fstream>
using namespace std;

class Streaming
{
private:
	ifstream in;
	ofstream out;
	string content;
public:
	Streaming(const char *In,const char *Out);
	~Streaming();

	int m_seekEndOfInFile(void);
	string m_readLine(int NumberOfLine);
	bool m_writeLine(string Inhalt);

};

#endif //STREAMING

Streaming.cpp:

#include "Streaming.h"
#include <fstream>
#include <iostream>

Streaming::Streaming(const char *In,const char *Out)   
{  
    this->in.open(In,ios::out); //öffnen zum lesen mit Standardoptionen 
    this->out.open(Out,ios::out);  //öffnen zum schreiben mit Standardoptionen 
}

Streaming::~Streaming()
{
	in.close();
	out.close();
}

//Sucht das ende der Textdatei (in Zeilen)
int Streaming::m_seekEndOfInFile(void)
{  
	in.seekg(0,ios::beg);
	int zaehler=0;
	while(getline(in,content))
	{
		++zaehler;
	}

return zaehler;
}

//Lies eine Zeile aus einer Textdatei in einen String und gibt diese mit return zurück
string Streaming::m_readLine(int NumberOfLine)
{   
    in.seekg(0,ios::beg);
	for(int i=0;i<NumberOfLine;i++)
	{
		getline(in,content);
	}

	return content;
}

//Schreibt eine Zeile den im string gespeicherten inhalt in eine Textdatei
bool Streaming::m_writeLine(string Inhalt)
{
	if(!(out<<Inhalt))
	{
		return false;
	}
return true;
}

So wie es jetzt ist, wird der Inhalt der Textdatei zwar nicht gelöscht, aber auch nicht richtig ausgelesen.

cu max

Bashar

Hat das eigentlich einen bestimmten Grund, dass du alle Memberfunktionen mit einem m_ Präfix versiehst?

Ja, ich habe gelesen, das dass algemein üblich ist, ist es denn nicht so?

cu max

Jester

@Bashar:
Vielleicht damit man gleich erkennt, daß es Member sind... bei sowas wie object.doSomething(); kann man sich ja nie ganz sicher sein.

Bashar

Master_Max schrieb:

Ja, ich habe gelesen, das dass algemein üblich ist

Nein, das ist nicht üblich. Manche machen das für (private) Membervariablen.

HumeSikkins

mit dem eof bit bezeichnest du doch '\n' oder?

Nein. Mit eof-bit bezeichne ich das eof-bit. '\n' ist ein Newline-Zeichen. eof steht für end-of-file. Das eof-bit eines Streamobjekts wird gesetzt, wenn man über das Ende einer Datei hinaus gelesen hat. Ist dieses bit einmal gesetzt liefert jeder weitere Lesevorgang solange eof (also keine weiteren Daten) bis das eof-bit wieder gelöscht wird.

Abfragen kannst du das bit mit:

if (in.eof())
// eof-bit gesetzt
else
// eof-bit nicht gesetzt

Wenn du gleich auch noch das bad- bzw. fail-bit abfragen willst:

if (in.good())
// ok. Stram ist zum Lesen bereit
else
// Aus Stream kann nicht gelesen werden

Löschen tust du das eof-bit mit:

in.clear(ios::eofbit)

Oder du löschst alle Statusbits:

in.clear();

Hi,
danke.
Achso ich dachte das Newline Zeichen wird auch ans ende eines Streams gesetzt.
Gut ich werde mal schauen ob das hilft.

[EDIT]Komisch das es in der Version davor noch ging, allerdings ohne eine .eof() abfrage oder ähnliches
Eigentlich liest doch die while Schleife bis .eof und nicht weiter hinaus?
[EDIT]

vielen dank.

cu max

Hi,
danke an alle, es scheint zu gehen, hier nun die neuen methoden:

/Sucht das ende der Textdatei (in Zeilen)
int Streaming::m_seekEndOfInFile(void)
{  
	in.seekg(0,ios::beg);
	int zaehler=0;
	do{
        getline(in,content);
        ++zaehler;
	}while(!in.eof());//Habe ich von Mis2com geklaut :rolling_eyes: 
	in.sync();
	in.clear();

return zaehler;
}

//Lies eine Zeile aus einer Textdatei in einen String und gibt diese mit return zurück
string Streaming::m_readLine(int NumberOfLine)
{   
    in.seekg(0,ios::beg);
	for(int i=0;i<NumberOfLine;i++)
	{
		getline(in,content);	
	}

	in.sync();
	in.clear();

	return content;
}

cu max

HumeSikkins

Master_Max schrieb:

Hi,

/Sucht das ende der Textdatei (in Zeilen)
int Streaming::m_seekEndOfInFile(void)
{  
	in.seekg(0,ios::beg);
	int zaehler=0;
	do{
        getline(in,content);
        ++zaehler;
	}while(!in.eof());//Habe ich von Mis2com geklaut :rolling_eyes: 
	in.sync();
	in.clear();

return zaehler;
}
[...]

Das hättest du besser nicht geklaut, da der Code nicht korrekt ist. Ersetz die falsche do-while-Schleife durch deine ursprüngliche und richtige while-Schleife und gut ist.