Ausgabe mit Datein

Skeptar

Hier das Beispiel aus dem Buch..
Ich wollte daraus einfach nur eine einfache Ausgabe ableiten:

/* cppbuch/k2/datcopy.cpp
   Beispiel zum Buch von Ulrich Breymann: Der C++ Programmierer; Hanser Verlag
   Diese Software ist freie Software. Website zum Buch: http://www.cppbuch.de/ 
*/
#include<string>
#include<cstdlib> // für exit( )
#include<fstream>
#include<iostream>

using namespace std;

int main( ) {
    // Definieren der Eingangsdatei
    ifstream quelle;            // Datentyp für Eingabestrom 
    string quelldateiname;
    cout << "Quelldatei? ";
    cin >> quelldateiname;

    // Datei öffnen:
    // nicht portabel: quelle.open(quelldateiname.c_str());
    quelle.open(quelldateiname.c_str(), ios::binary|ios::in);
    //quelle.open(quelldateiname, ios::binary|ios::in); // neuer Standard
    // Ohne {\tt ios::binary} sind nur Textdateien kopierbar
    // (gilt nur für Windows)

    if (!quelle) {   // Fehlerabfrage
        cerr << quelldateiname
             << " kann nicht geöffnet werden!\n";
        exit(-1);
    }

    string zieldateiname;
    cout << "Zieldatei? ";
    cin >> zieldateiname;
    // Definieren und Öffnen der Ausgabedatei hier in einem Schritt
    // nicht portabel: ofstream ziel(zieldateiname.c_str());
    ofstream ziel(zieldateiname.c_str(),ios::binary|ios::out);
    if (!ziel) {     // Fehlerabfrage
        cerr << zieldateiname
             << " kann nicht geöffnet werden!\n";
        exit(-1);
    }

    char ch;
    while (quelle.get(ch))
           ziel.put(ch); // zeichenweise kopieren
}  // Dateien werden am Programmende automatisch geschlossen.

Was ist jetzt die Fehlermeldung, wo liegt das Problem?

Wo ist dein Code, der nicht macht was du willst? Hab bis jetzt nur kleine Ausschnitte gesehen und die waren ziemlich Fehlerhaft

was für einen sinn macht denn bitte
ofstream und ios::in ?
Was ist bitte ofstream os(test.text.c_str())?

Hast du eine class test mit einem std::string Member text, welches den Dateipfad angibt, oder wie?

LG

Skeptar

Ich habe nichts.
Ich bin gerade dabei C++ zu lernen, und ich habe ein Buch ich bin gerade beim zweiten Kapitel angelangt. (Ein- und Ausgabe mit Datein).
Und ich probiere das dann immer aus. Und jedzt möchte ich etwas in eine Datei schreiben, aber ich bekomme das nicht hin deswegen bin ich ja jedzt hier :p
Und wie du mir das geschrieben hast wird das wohl richtig sein, Aber ich möchte das aus dem Buch ja auch verstehen, und nicht komplett andere Wörter dahin schreiben.

Nathan

Wo ist dein Problem bei dem gezeigten Code?

Skeptar

...........
Der Code ist ein Beispiel aus dem Buch.
Aus dem Code wollte ich eigendlich eine einfache Ausgabe ableiten das bekomme ich aber nicht hin also frage ich euch.

Skeptar

Habe das nun so:

#include <iostream>
#include <fstream>

using namespace std;

int main(){

    ofstream ausgabe("test.txt");
    ausgabe << "Haus" << endl;
    ausgabe.close();
    return 0;
}

Ich bekomme auch keinen Fehler mehr angezeigt, es klappt auch.
Ist das denn so richtig?

Ein char ist im Prinzip 1 Byte, d.h. du liest aus der einen Datei jedes byte und schreibst es in die andere... kenne Leute, die da gerne schreiben würden

for(char ch; quelle.get(ch);)
    ziel.put(ch);

^^

hardware schrieb:

kenne Leute, die da gerne schreiben würden
for(char ch; quelle.get(ch); ziel.put(ch))

Skeptar schrieb:

Habe das nun so:
#include <iostream>
#include <fstream>

using namespace std;

int main(){

    ofstream ausgabe("test.txt");
    ausgabe << "Haus" << endl;
    ausgabe.close();
    return 0;
}
Ich bekomme auch keinen Fehler mehr angezeigt, es klappt auch.
Ist das denn so richtig?

Wenn du einen Zeilenumbruch willst, nutze '\n' statt endl
ausgabe.close() kannst du weglassen, passiert automatisch, wenn ausgabe "out of scope" geht
LG

P.S.: Sorry für meine zu vielen Posts, sollte mich mal einloggen, damit ich editieren kann^^

Nathan

Skeptar schrieb:

Habe das nun so:
#include <iostream>
#include <fstream>

using namespace std;

int main(){

    ofstream ausgabe("test.txt");
    ausgabe << "Haus" << endl;
    ausgabe.close();
    return 0;
}
Ich bekomme auch keinen Fehler mehr angezeigt, es klappt auch.
Ist das denn so richtig?

Rein formal ja.
Allerdings zwei Dinge:
- endl ist nicht einfach nur eine Ausgabe von '\n', es wird auch geflusht. Meistens willst du das nicht, deswegen schreib einfach nur '\n'
- du musst den Stream nicht closen, das erledigt der Destruktor von ofstream automatisch

Edit: Verdammt, zu spät.

Skeptar

Okey danke.
In der Schule hat der Lehrer uns das mit << endl; beigebracht. Ich versuche jedzt immer \n zu schreiben aber klappt nicht immer wie man sieht. ABer es ist nicht falsch mit

ausgabe.close();

?

Nathan

Skeptar schrieb:

Okey danke.
In der Schule hat der Lehrer uns das mit << endl; beigebracht. Ich versuche jedzt immer \n zu schreiben aber klappt nicht immer wie man sieht. ABer es ist nicht falsch mit
ausgabe.close();
?

Es ist überflüssig.

Das Problem ist ganz offensichtlich das Buch. Was ließste denn da?

out schrieb:

Das Problem ist ganz offensichtlich das Buch. Was ließste denn da?

AH, da oben stehts ja, du ließt den Breymann. Der ist gut. Aber steht da echt open und close drin, und exit? Muss ich mal nachschauen.

Skeptar

Ja da steht halt das man das generell auch nicht machen muss
Danke für die Hilfe :p

Lymogry

Aha .... das wollt ich schon immer wissen:

Nathan schrieb:

Allerdings zwei Dinge:
- endl ist nicht einfach nur eine Ausgabe von '\n', es wird auch geflusht. Meistens willst du das nicht, deswegen schreib einfach nur '\n'

Ich beobachte, dass ein "\n" nicht reicht, um in einer DATEI in die nächste Zeile zu kommen, dafür klappts aber mit "\r\n" ...
Den Unterschied zwischen \r und \n hab ich nicht verstanden ...

Auch nicht, warum es schlecht sein soll, den AUSGABEpuffer zu leeren.
Ich meine, wenn es der Eingabepuffer wäre, kann ich das nachvollziehen, dass man den haben will und auch dass es gut ist, den mal zu leeren, man könnte ja womöglich noch seltsame Zeichen dadrin haben. Aber wozu braucht man den Ausgabepuffer noch, wenn man was ausgegeben hat ....?
Bitte um Erleuchtung!

Ich häng mich mit der Frage mal an. (Wenn ihr das nicht wollt, mach ich eigenen Thread auf, aber ich denke, das ist grad im Sinne des Threaderstellers)

Lymogry schrieb:

Auch nicht, warum es schlecht sein soll, den AUSGABEpuffer zu leeren.

Ich finde das Wort flushen besser als leeren. Es ist eben sinnvoll, eine gewisse Menge an Zeichen zuerst in einem Puffer zu schreiben, und dann den gesamten Puffer in einem Rutsch rausschreiben, z.B. in eine Datei. Das ist viel effizienter, als jedes Zeichen einzeln rauszuschreiben. Wann es sinnvoll ist, den Puffer zu flushen, weiß dein Betriebssystem schon, da brauchst du kein endl das da rumpfuscht.

Fytch

Lymogry schrieb:

Aha .... das wollt ich schon immer wissen:

Nathan schrieb:

Allerdings zwei Dinge:
- endl ist nicht einfach nur eine Ausgabe von '\n', es wird auch geflusht. Meistens willst du das nicht, deswegen schreib einfach nur '\n'

Ich beobachte, dass ein "\n" nicht reicht, um in einer DATEI in die nächste Zeile zu kommen, dafür klappts aber mit "\r\n" ...
Den Unterschied zwischen \r und \n hab ich nicht verstanden ...

dann hast du die datei möglicherweise mit std::ios::binary geöffnet. wenn das der fall ist, dann werden '\n' nicht in "\r\n" (auf windows z.b.) konvertiert, damit man in einer bilddatei z.b. ein pixel so färben kann wie der entsprechende wert von '\n' hoch ist, ohne dass uns ofstream da ein '\r' reinpfuscht.

Lymogry schrieb:

Auch nicht, warum es schlecht sein soll, den AUSGABEpuffer zu leeren.
Ich meine, wenn es der Eingabepuffer wäre, kann ich das nachvollziehen, dass man den haben will und auch dass es gut ist, den mal zu leeren, man könnte ja womöglich noch seltsame Zeichen dadrin haben. Aber wozu braucht man den Ausgabepuffer noch, wenn man was ausgegeben hat ....?
Bitte um Erleuchtung!

Ich häng mich mit der Frage mal an. (Wenn ihr das nicht wollt, mach ich eigenen Thread auf, aber ich denke, das ist grad im Sinne des Threaderstellers)

wenn du den ausgabepuffer flushesd, dann kann es z.b. sein dass dein programm dem os den befehl gibt, alles was noch im puffer ist direkt auf die festplatte zu schreiben. das ist sinnlos weil du ja nicht nach jeder zeile wieder den festplattenzugriff brauchst, das kann alles getrost im ram warten bis alles fertig ist.

Lymogry schrieb:

Ich beobachte, dass ein "\n" nicht reicht, um in einer DATEI in die nächste Zeile zu kommen, dafür klappts aber mit "\r\n" ...
Den Unterschied zwischen \r und \n hab ich nicht verstanden ...

Unter Windows besteht ein Zeilenumbruch in Dateien (nicht in stringstream) aus \r und \n. Wenn du eine Datei binär einließt, bekommst du beide Zeichen. Ließt du eine Datei im Textmodus ein, bekommst du nur das \n. Dass du aber für eine Ausgabe \r brauchst, ist mir neu .

SeppJ

Lymogry schrieb:

Ich beobachte, dass ein "\n" nicht reicht, um in einer DATEI in die nächste Zeile zu kommen, dafür klappts aber mit "\r\n" ...
Den Unterschied zwischen \r und \n hab ich nicht verstanden ...

Öffne die Datei nicht binary!

Auch nicht, warum es schlecht sein soll, den AUSGABEpuffer zu leeren.

Weil du damit auslöst, dass die Daten physisch auf das Ausgabegerät geschrieben werden. Das ist langsam. Richtig langsam. Schnell ist es, wenn möglichst viel auf einmal geschrieben wird und dann auch noch am besten dann, wenn das dafür zuständige System meint, dass ein guter Zeitpunkt wäre. Denn die Menge macht nicht so viel aus, es ist der Zugriff an sich. Das machst du kaputt, wenn du nach jeder Zeile flusht. Denn dann sind das viel zu wenige Daten, als das ein Zugriff sich lohnen würde.