#include <fstream>
#include <iostream>
#include <string>
// Um Schreibarbeit zu sparen
using namespace std;
/*****************************************************************************
* Konsolenprogramm um eine Datei auszulesen und ihren Inhalt auszugeben.
* Der Erste übergebene Parameter (argv[1]) enthält den Pfad zur Datei. Alle
* weiteren Parameter werden ignoriert. Existiert die Datei nicht, wird eine
* Fehlermeldung ausgegeben.
*
* Achtung: Kommentare in der Datei werden wie gewöhnlicher Inhalt behandelt.
*
* Benutzung:
* commando [pfad]
* Eingabdatei:
* test1;test2;test3;
* Ausgabe:
* test1
* test2
* test3
*****************************************************************************/
int main( const int argc, const char *argv[] ) {
// Arguemt handling -> Fehler wenn keine Datei angegen wird
if ( argc<2 ) {
cout << "Bitte Argument angeben!" << endl;
exit(1);
}
// Streamvariable setzten und überprüfen, ob öffnen gelungen
// (Fehlermeldung wenn nicht gelungen)
ifstream inputFile( argv[1] );
if( !inputFile ) {
cout << argv[1] << " Konnte nicht geöffnet werden!" << endl;
exit(1);
}
// Datei in einer Schleife buchstabenweise auslesen
while( !inputFile.eof() ) {
// Aktullen Buchstaben lesen
char currentChar;
inputFile >> currentChar;
switch (currentChar) {
// Wenn Buchstabe Strichpunkt, Leerzeile ausgeben
case ';':
cout << endl;
break;
// Leerzeilen ignorieren
case '\n':
break;
// Leerzeichen ignorieren
case ' ':
break;
// Alle anderen Buchstaben ausgeben
default:
cout << currentChar;
break;
}
}
// Schließende Leerzeile ausgeben und Programm ordnungsgemäß verlassen
cout << endl;
return 0;
}CSV - Comma/Character Separated Value Datei (siehe z.B. Wikipedia).
Naja, ganz richtig ist das nicht. Erstmal braucht man in C++ so gut wie nie die eof() Methode (weil es meistens nämlich falsch ist, so wie bei dir). Dann prüfst du nicht ob überhaupt erfolgreich eingelesen wurde und du liest die Daten falsch ein, da du jegliche Leerzeichen überliest.
string zeile;
ifstream datei("...");
if (datei.is_open()) {
while (getline(datei, zeile)) {
// .. Zeile verarbeiten
}
} else { /* Fehler */ }Das Problem ist, das du die Leerzeichen (Tab, Leerraum und Zeilenende) gar nicht einliest weil der >> Operator diese Zeichen überspringt. Das heißt, du kannst gar nicht sagen wann ein Record endet und ein neuer beginnt. Außerdem wenn zwischen den Trennzeichen Leerzeichen stehen, läßt du die auch einfach unter den Tisch fallen.
Das wird wohl vor allem daran liegen, das du keine Ahnung hast - deswegen kommt es dir wohl so vor. Ich habe nur ungefähr zum 40-zigsten Mal in diesem Forum erklärt wie man mit den iostream umgehen sollte und das man eof() nicht verwendet.
Es ging mir nicht um das getline. Ich habe nur erstmal die gröbsten Fehler berichtigt und eine andere (in meinen Augen) bessere Vorgehensweise vorgeschlagen, denn ich war etwas in Eile - Vorlesung fing in 5 min an.
#include <iostream>
#include <sstream>
#include <string>
using namespace std;
int main()
{
string line;
while (getline(cin, line)) {
istringstream record(line);
string field;
while (getline(record, field, ';')) {
cout << field << endl;
}
}
}Dieter Nuhr hat gesagt.:
