Ich habe vor aus einer Textdatei mehrere Zahlen die untereinanderstehen zu addieren, alle Zahlen ausgeben klappt auch soweit nur wenn ich sie addieren möchte bekomme ich Datenmüll.
Das ist meine Methode Daten_einlesen:

void addition::Daten_einlesen()
{
	string temp;
	double StrToDouble;
	fstream Datei;

	Datei.open(Dateiname.c_str(),ios::in);

	while(!Datei.eof())
	{

		getline(Datei,temp);        //Zeilenweise auslesen

		StrToDouble = atof(temp.data());  //string in dbl wandeln

		ergebnis = StrToDouble+ergebnis;  //addieren

	}

	Datei.close;

        cout<<ergebnis;
}

Lg freeG

Das ist meine Methode Daten_einlesen: ...

Besser:

void addition::Daten_einlesen()
{
	ifstream Datei(Dateiname.c_str()); // (i)fstream steht schon für input
	ergebnis = 0; // initialisieren
	for( double zahl; Datei >> zahl; ) // wenn Du Zahlen willst, so lese Zahlen
	    ergebnis += zahl;
	cout<<ergebnis;
} // Datei.close() ist unnötig, das erledigt der Destruktor beim Verlassen des Scopes

Tipp: eines der wichtigsten Dinge bei OOP implementiert in C++ ist das Paar Konstruktor und Destruktor.

Und eines der wichtigsten Prinzipien der OOP ist das "Single responsibility principle". Das Lesen von Dateien liegt mit Sicherheit nicht im Zuständigkeitsbereich einer Addition (Oder diese Klasse ist schlecht benannt).

ehm ja wieso schlecht benannt?!
es gibt die Klasse "addition" in ihr sind dann daten_einlesen und daten_auslesen
definiert.
einlesen: daten aus datei holen und addieren
auslesen: addierte zahlen auslesen

An CStoll: Danke das ich nicht initialisiert habe war vllt mein fehler.
ergebnis ist eine private variable der klasse

An fr33g: Danke aber Werner Salomon hat den fehler behoben. Das Programm bescheht aus den Dateien main, header und methoden. im header steht die klasse. in methoden sind die methoden der klasse und main ist denke ich klar ;).

Ansonsten solltest du die Aufgaben sauber trennen - eine Klasse liest bzw. schreibt die Zahlen aus Dateien, die andere rechnet mit den Zahlen. Sonst endest du mit einem halben Dutzend Klassen, die alle fast den selben Code besitzen und sich nur in der angewendeten Rechenoperation unterscheiden - und Code-Duplikation ist immer ein schlechtes Omen.

brotbernd schrieb:

Und eines der wichtigsten Prinzipien der OOP ist das "Single responsibility principle". Das Lesen von Dateien liegt mit Sicherheit nicht im Zuständigkeitsbereich einer Addition (Oder diese Klasse ist schlecht benannt).

ehm ja wieso schlecht benannt?!

Hallo runystar,

der Kern des Problems liegt wohl darin, dass die C++-Klasse addition keine(!) Klasse im Sinne von OOP ist. Was eine Klasse im Sinne von OOP ist, kann ich Dir aber leider nicht in drei Sätzen erklären.

runystar schrieb:

es gibt die Klasse "addition" in ihr sind dann daten_einlesen und daten_auslesen
definiert.
einlesen: daten aus datei holen und addieren
auslesen: addierte zahlen auslesen

Ich vermute mal, dass bis auf den Dateinamen beide Methoden nichts miteinander zu tun haben. D.h. beide Methoden könnte auch leicht als frei stehende Funktionen realisiert werden, ohne dass Du in Deinem main irgendetwas an Funktionalität einbüßt.
Das wäre es, was ich mit 'keine Klasse im Sinne von OOP' meine.

Gruß
Werner

PS: Und die Addition würde ich sogar noch kürzer hinbekommen:

ifstream file(Dateiname.c_str());
double ergebnis = accumulate(istream_iterator<double>(file),istream_iterator<double>(),0.0); //Edit: richtigen Startwert angegeben
return ergebnis;

PS: Und die Addition würde ich sogar noch kürzer hinbekommen:

ifstream file(Dateiname.c_str());
double ergebnis = accumulate(istream_iterator<double>(file),istream_iterator<double>(),0);
return ergebnis;

CStoll schrieb:

PS: Und die Addition würde ich sogar noch kürzer hinbekommen:

ifstream file(Dateiname.c_str());
double ergebnis = accumulate(istream_iterator<double>(file),istream_iterator<double>(),0);
return ergebnis;

bei aller Begeisterung für diese Konstruktion, so enthält sie doch einen groben Fehler. Wer es nicht glaubt, probiere es doch mal, damit die Summe von den zwei Zahlen

2.5
3.5

zu bestimmen. Das Ergebnis obigen Programms ist in diesem Fall nicht 6.

Gruß
Werner

CStoll schrieb:

Krümelkacker :p Das kommt davon, wenn ich keinen C++ Compiler greifbar habe.

http://ideone.com/452B5