Anfänger Code

  • Anfänger Code

    Habe ein Übungsskript aus meinem C++ Buch.
    Korrekt abgetippt, trotzdem Fehlöermeldung

    Quellcode

    1. #include<cstdlib>
    2. #include<fstream>
    3. #include<string>
    4. #include<iostream>
    5. using namespace std;
    7. int main () {
    8. ifstream Quelle;
    9. string Quelldateiname;
    10. cout << "Quelldatei? ";
    11. cin >> Quelldateiname;
    12. Quelle.open(Quelldateiname.c_str(), ios::binary|ios::int);
    13. if (!Quelle) {
    14. cerr << Quelldateiname << " kann nicht geöffnet werden!\n";
    15. exit (-1);
    16. }
    17. string Zieldateiname;
    18. cout << "Zieldatei? ";
    19. cin >> Zieldateiname;
    20. ofstream Ziel(Zieldateiname.cstr(), ios::out);
    21. if (!Ziel) {
    22. cerr << Zieldateiname << " kann nicht geöffnet werden!\n";
    23. exit (-1);
    24. }
    25. char ch;
    26. while (Quelle.get(ch)) {
    27. Ziel.put(ch);
    28. }
    29. }
    Alles anzeigen



    Also mein Dev-C++ zeigt Fehler in der Zeile wo
    folgendes steht:

    Code:

    Quelle.open(Quelldateiname.c_str(), ios::binary|ios::int);



    Sollte ein Programm werden, das ein File kopiert, und genau bei
    .c_str() bin ich im Praxisbuch noch nicht. ^^"
    Möchte mir vll trotzdem jemand helfen?

    Edit: Problem gelöst!

  • Neues Problem.

    Quellcode

    1. #include<cstdlib>
    2. #include<fstream>
    3. #include<string>
    4. #include<iostream>
    5. #include<string>
    6. using namespace std;
    8. int main () {
    9. ifstream Quelle;
    10. string Quelldateiname;
    11. cout << "Quelldatei? ";
    12. cin >> Quelldateiname;
    13. Quelle.open(Quelldateiname.c_str(), ios::binary|ios::in);
    14. if (!Quelle) {
    15. cerr << Quelldateiname << " kann nicht geöffnet werden!\n";
    16. exit (-1);
    17. }
    18. string Zieldateiname;
    19. cout << "Zieldatei? ";
    20. cin >> Zieldateiname;
    21. cout << "Wieviele Kopien sollen erstellt werden?";
    22. int Menge;
    23. cin >> Menge;
    24. for (int run = 1; run <= Menge; run++) {
    25. string acName = Zieldateiname+"_"+static_cast<char>(run);
    26. ofstream Ziel(acName.c_str(), ios::out);
    27. if (!Ziel) {
    28. cerr << acName << " kann nicht geöffnet werden!\n";
    29. exit (-1);
    30. }
    31. char ch;
    32. while (Quelle.get(ch)) {
    33. Ziel.put(ch);
    34. }
    35. }
    36. }
    Alles anzeigen


    Man kann eingeben wieviele Kopien das Programm macht, scheint
    nach Eingabe einer Zahl nur abzubrechen.
    Was mache ich falsch?

  • Hallo,

    ich hab mal deinen Code bisschen zurecht gerückt und nun müssts funktionieren:

    Quellcode

    1. #include<cstdlib>
    2. #include<fstream>
    3. #include<string>
    4. #include<sstream>
    5. #include<iostream>
    6. using namespace std;
    8. int main () {
    9. ifstream Quelle;
    10. ofstream Ziel;
    11. string Quelldateiname, Zieldateiname, acName;
    12. stringstream temp;
    13. int Menge;
    14. char ch;
    16. cout << "Quelldatei? ";
    17. cin >> Quelldateiname;
    18. Quelle.open(Quelldateiname.c_str(), ios::binary|ios::in);
    19. if (!Quelle)
    20. {
    21. cerr << Quelldateiname << " kann nicht geöffnet werden!\n";
    22. exit (-1);
    23. }
    24. cout << "Zieldatei? ";
    25. cin >> Zieldateiname;
    26. cout << "Wieviele Kopien sollen erstellt werden? ";
    27. cin >> Menge;
    28. for (int run = 1; run <= Menge; run++)
    29. {
    30. temp << run << ends;
    31. acName = Zieldateiname + "_" + temp.str();
    32. temp.str("");
    33. Ziel.open(acName.c_str(), ios::out);
    34. if (!Ziel.good())
    35. {
    36. cerr << acName << " kann nicht geöffnet werden!\n";
    37. exit (-1);
    38. }
    39. Quelle.clear();
    40. Quelle.seekg(0,ios::beg);
    41. while (Quelle.get(ch))
    42. {
    43. Ziel.put(ch);
    44. }
    45. Ziel.close();
    46. }
    47. }
    Alles anzeigen


    1. Der Cast von int nach string ist nicht so einfach. Um den Dateinamen zu inkrementieren habe ich mal nen stringstream genommen. (4, 12, 30, 31, 32)

    2. Du liest die Datei zeichenweise ein, deswegen wird der Dateipointer jeweils ein Zeichen weitergeschickt. Wenn der Dateipointer am EOF is, kannst du nicht nochmal versuchen die Datei bis zum Ende auszulesen. Also Dateipointer an den Anfang (39, 40)

    3. Achte bitte mehr auf die Lesbarkeit des Codes. Deklarationen gehören in der Regel oben hin (nicht in eine Schleife!), Variablennamen besser wählen, etc.

    4. Ich habe kaum was verbessert. Nur lauffähig gemacht.