Es ist ja immer wieder verwunderlich, an welch scheinbar leichten Aufgaben man scheitern kann. Ich zumindest verstehe nicht, wieso bei diesem Code trotzdem jede Zeile in die neue Datei geschrieben wird... Testweise Ausgabe von i ergab, dass korrekt gezählt wird, daher dürfe i%2 == 0 nicht immer wahr sein...

int main()
{
    std::vector<std::string> temp_lines;
    int i = 0;

    std::fstream file( "<Pfad>/Log.txt" );
    while( file.is_open() )
    {
        if( i%2 == 0 )
        {
            std::string line;
            std::getline( file, line, '\n' );
            temp_lines.push_back( line );
        }
        if( file.eof() ) break;
        ++i;      
    }

    std::fstream write( "<Pfad>/Log_Edit.txt" );
    if( write.is_open() )
        for( std::size_t i = 0; i < temp_lines.size(); ++i )
        {
            write << temp_lines[i] << '\n';
        }
}

Kann mir bitte jemand auf die Sprünge helfen?

gewollte Zeile
ungewollte Zeile
gewollte Zweile
ungewollte Zweile
...

Möchte nur die gewollten Zeilen !

vielen Dank im <edit: Voraus >, ich grübel schonmal weiter...

mfg
HarteWare

Warum schreibst du nicht zwei Mal getline(); , verwendest aber nur jeweils die, welche du haben möchstest?
Also in diesem Fall immer die erste?

mfg
HarteWare

Warum schreibst du nicht zwei Mal getline(); , verwendest aber nur jeweils die, welche du haben möchstest?
Also in diesem Fall immer die erste?

Oder besser noch, einmal getline, einmal ignore bis zum nächsten Zeilenumbruch. getline macht intern durchaus komplexe Dinge (zumindest die string-Version), die man sich lieber sparen will, wenn man sie nicht braucht.

Du liest nur bei jedem zweiten Durchgang eine Zeile. Das ist nicht das, was du willst.

Aaach *facepalm*, klar! Immer diese Denkfehler.

Noch eine frage ...
std::cin.ignore( <was muss denn hier rein?>, '\n'); <- so gemeint?

Aber sonst, das macht prinzipiell in meinem Fall keinen Unterschied...

Vielen Dank für die schnellen Antworten!

mfg
HarteWare

std::cin.ignore(256,'\n');   // ignore until new line

Wobei 256 natürlich willkürlich gewählt ist.

std::cin.ignore( <was muss denn hier rein?>, '\n'); <- so gemeint?

Ja, so ist das gemeint. Als Wert für wie viele Zeichen übersprungen werden sollen, könntest du eine sehr große Zahl nehmen, aber es gibt noch einen Sonderwert, nämlich das Maximum des Typen streamsize . Das heißt dann "unendlich". Das Maximum kannst du aus dem limits-Header erfahren:

std::cin.ignore( std::numeric_limits<std::streamsize>::max(), '\n');

Ja, ist ein bisschen umständlich zu schreiben :p . Glücklicherweise braucht man das nicht so oft. Oder wenn doch, dann definiert man sich eben eine Konstante als Kurzschreibweise.

istream& ignore_until( istream& is, char ch = '\n' )
{
    return is.ignore( std::numeric_limits<std::streamsize>::max(), ch );
}

(ungetestet)

Übrigens, was soll denn diese Abbruchbedingung:

while( file.is_open() )

Da schaudert es mir.

std::string line;
    while( std::getline(file,line) )
    {
        temp_lines.push_back( std::move(line) ); // move ist optional
        file.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );    
    }

Dankesehr, funktioniert einwandfrei. Irgendwas mit numeric_limits hatte ich noch im Kopf, schonmal in anderen Threads gelesen.

In dem Fall wäre die Sache so weit geklärt. Man lernt nie aus...

mfg
HarteWare

EDIT: Ein Problem hätte ich noch:

Zeile beginnt mit string

a
a
b
a
b
a
a
etc. (kein festes Muster)

ich will nur Zeilen haben mit Anfangsstring a. Die anderen ist nicht wichtig.

Nun habe ich Sachen versucht wie

Nur wenn: string[4] == "char, welchen nur string a an fünfter Stelle hat"
Dann: schreib in die Datei

Jedoch klappt es nicht. Es werden weiterhin alle strings aus dem vector in die Datei geschrieben... Wie würde man korrekter Weise sowas angehen?

while (std::getline(file, line))
{
    if (line[0] == 'a')
        lines.push_back(std::move(line));
}

Dann liest du auch nur die Zeilen ein, die du haben willst:

while (std::getline(file, line))
{
    if (line[0] == 'a')
        lines.push_back(std::move(line));
}

std::string line;
while( std::getline( file, line ))
{
    if( line[4] == 'n' )
        temp_lines.push_back( line );
    file.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
}

wobei gewollter "Startstring Teil" (ST) -> "Login" und ungewollter ST -> "Log Out"

Der vector bleibt leider leer, das war auch mein Ansatz (sowohl beim Schreiben probiert, als auch beim Lesen)...

mfg
HarteWare

Der vector bleibt leider leer[..]

Klar, Du ignorierst ja auch nicht die Zeile, die Du geprüft hast.

std::getline( file, line )
 if( line[4] == 'n' )
   // ...

Das schreit ja geradezu nach Ärger. Besser:

std::getline( file, line )
 if( line.length() > 4 && line[4] == 'n' )
   // ...

Das schreit ja geradezu nach Ärger. Besser:

std::getline( file, line )
 if( line.length() > 4 && line[4] == 'n' )
   // ...

Das ist auch nicht richtig, wenn der Stream in einem Fehlerstatus ist.

http://ideone.com/i0d1P6

SeppJ schrieb:

Das schreit ja geradezu nach Ärger. Besser:

std::getline( file, line )
 if( line.length() > 4 && line[4] == 'n' )
   // ...

Das ist auch nicht richtig, wenn der Stream in einem Fehlerstatus ist.

http://ideone.com/i0d1P6

Das ist nur als ungefähre Richtungsangabe zu verstehen, ich wollte jetzt nicht noch eine weitere Ebene an Abfragen hinzu fügen. Aber guter Hinweis an Leute, die das Beispiel zu wörtlich genommen hätten.

Caligulaminus schrieb:

HarteWare schrieb:

Der vector bleibt leider leer[..]

Klar, Du ignorierst ja auch nicht die Zeile, die Du geprüft hast.

Wie ist das zu verstehen? So? Denke ich hab das so falsch verstanden...

std::string line;
    while( std::getline(file,line) )
    {
        if( line.length() > 4 && line[4] == 'n')
        {
            temp_lines.push_back( line ); // move ist optional
            file.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
        }
        file.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
    }

Stimmt, das Programm ist relativ gesehen recht unzuverlässlich und auch nicht wirklich universell verwendbar, aber ich möchte es nur für eine ganz bestimmte Aufgabe. Aber Danke für die Hinweise, nichts spricht dagegen sauber zu arbeiten.

mfg
HarteWare

Gegeben sei folgender Dateiinhalt:

0123m
0123n
0123o

Wo steht der Lesemarker nach dem ersten Schleifendurchlauf? Welchen Inhalt hat line?
Wie sieht es nach dem zweiten Durchlauf aus?

std::string line;
    while( std::getline(file,line) )
    {
        if( line.length() > 4 && line[4] == 'n')
        {
            temp_lines.push_back( line ); // move ist optional
            file.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
        }
        file.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
    }

Fällt Dir was auf?

if(mystring[3] == 'a')
writeToFile(mystring);

std::string mystring;
mystring = "blablabla"

if(mystring[3] == 'a')
writeToFile(mystring);

oh lol hab die zweite Seite nicht gesehen haha einfach ignorieren

Gegeben sei folgender Dateiinhalt:

0123m
0123n
0123o

Wo steht der Lesemarker nach dem ersten Schleifendurchlauf? Welchen Inhalt hat line?
Wie sieht es nach dem zweiten Durchlauf aus?

Fällt Dir was auf?

Hallo,

Also als erstes fällt mir auf, dass dieser Thread theoretisch nach Natans Post schon hätte gelöst sein sollen, jedoch ist dem in der Praxis nicht so...
Na gut:
Nach dem ersten Durchlauf:

0123m
0123n 
0123o <- Lesemarker
Inhalt: "0123m"

Nach dem Zweiten:
nada, std::getline wird fehlschlagen

while( getline( file, line )) // Diese Zeile schlägt fehl. Habs getestet. 

//======================================
while( std::getline( file, line ))
{
    std::cout << line << '\n';
}

Output:

Press <RETURN> to close this window...

Ich finds echt toll, das so durchhaltend versucht wird mir zu helfen

mfg
HarteWare

Also als erstes fällt mir auf, dass dieser Thread theoretisch nach Natans Post schon hätte gelöst sein sollen, jedoch ist dem in der Praxis nicht so...
Na gut:
Nach dem ersten Durchlauf:

0123m
0123n 
0123o <- Lesemarker
Inhalt: "0123m"

Nach dem Zweiten:
nada, std::getline wird fehlschlagen

Sicher?
Hast Du's mal ausprobiert?

HarteWare schrieb:

while( getline( file, line )) // Diese Zeile schlägt fehl. Habs getestet. 

//======================================
while( std::getline( file, line ))
{
    std::cout << line << '\n';
}

Output:

Press <RETURN> to close this window...

Das sollte eigentlich jede Zeile eines Files ausgeben.
Wenn Du jetzt noch die gesuchten Zeilen in einen Vektor einfügst ist die Aufgabe doch gelöst?!

Ich möcht mich nochmals bei allen Helfenden bedanken

mfg
HarteWare