#include <iostream>
#include <istream> 
#include <fstream> 
#include <limits>  
#include <string>

struct MD  // string für datum und doubles für  die werte in zahlen
{ 
    std::string datum; 
    double hours_worked; 
    double filescreated; 
};

namespace 
{ 
    const std::streamsize ALL = std::numeric_limits< std::streamsize >::max(); 
}; 
// filter :P
std::istream& operator>>( std::istream& in, MD& md ) 
{ 
    in.ignore( ALL, '[' ) >> md.datum; 
    std::string token; 
    if( getline( in.ignore( ALL, '|' ), token, '=' ) ) 
    { 
        md.hours_worked = 0.;  
        md.filescreated = 0.; 
        if( token == "hours_worked" ) 
            in >> md.hours_worked; 
        else if( token == "filescreated" ) 
            in >> md.filescreated; 
        else 
            in.setstate( std::ios_base::failbit ); 
    } 
    return in; 
}
struct Komma : public std::numpunct< char > 
{ 
protected: 
    virtual char_type do_decimal_point() const 
    { 
        return ','; 
    } 
};

int main() 
{ 
    using namespace std; 
    ifstream fin("C:/Ruby193/1.txt"); 
   fin.imbue( locale( fin.getloc(), new Komma ) );
    if( !fin.is_open() ) 
    { 
        cerr << "Fehler beim Oeffnen der Datei" << endl; 

        return -2; 
    } 

    for( MD md; fin >> md; ) 
    {  
           // md's   auswerten
          cout << md.datum << endl ;    
          cout <<  md.hours_worked << endl ;
          cout << md.filescreated << endl ;

          ofstream myFile;  
       myFile.open ("C:/Ruby193/fast.csv");

       if (myFile)
   {
      // in datei schreiben 
       myFile << md.datum << endl ;
       myFile << md.hours_worked << endl ;
       myFile << md.filescreated << endl ;
       myFile.close();

       }

         //   md's   summieren 
    } 
    if( fin.eof() ) 
    { 

    } 
    getchar () ;
    return 0; 
}

Hallo erstmal und entschuldigt das neue Thema
ich habe folgendes Problem in meinem Programm wenn ich die md's filescreated und hours_worked in der Console ausgebe zeigt er mir die richtigen Daten lasse ich sie aber in ein neues Textdokument speicher hab ich nur ein Teil meiner Daten.
Wieso ?

Vorschlag: verallgemeinere deine Serialisierung, in dem du den operator<< überlädst/eine Funktion schreibst, die in einen übergebenen Stream schreibt.

Dein Problem ist übrigens, dass std::ofstream den Datei-Inhalt in der Schleife immer wieder überschreibt, weil der Dateizeiger beim Öffnen standardmäßig am Anfang platziert wird. Definiere den Stream also außerhalb der Schleife, und öffne auch die Datei außerhalb der Schleife, dann dürfte das laufen.

(Eine andere Möglichkeit wäre, das Flag ios_base::ate anzugeben, aber trotzdem ist das vielfache Öffnen und Schließen unperformant).

Okay danke erstmal ich habe es aus der schleife schon versucht aber dann kommt immer eine Fehlermeldung

Wäre schön, wenn du die zitierst.
Dann kann man dir helfen, weißt du.

80 C:\Dev-Cpp\valueextractor.cpp name lookup of `md' changed for new ISO `for' scoping

Das spuckt er mir aus wenn ich es außerhalb der Schleife mache

jetzt nicht in zeile 80 sonder ich glaube in zeile 71 ... weil ich den code ja geändert hatte

C:\Dev-Cpp\valueextractor.cpp cannot use obsolete binding at `md' because it has a destructor

der fheler kommt aich in zeile 58

jetzt nicht in zeile 80 sonder ich glaube in zeile 71 ... weil ich den code ja geändert hatte

Dann schick uns doch den geänderten Code

gruß
syntax

#include <iostream>
#include <istream> 
#include <fstream> 
#include <limits>  
#include <string>

struct MD  // string für datum und doubles für  die werte in zahlen
{ 
    std::string datum; 
    double hours_worked; 
    double filescreated; 
};

namespace 
{ 
    const std::streamsize ALL = std::numeric_limits< std::streamsize >::max(); 
}; 
// filter :P
std::istream& operator>>( std::istream& in, MD& md ) 
{ 
    in.ignore( ALL, '[' ) >> md.datum; 
    std::string token; 
    if( getline( in.ignore( ALL, '|' ), token, '=' ) ) 
    { 
        md.hours_worked = 0.;  
        md.filescreated = 0.; 
        if( token == "hours_worked" ) 
            in >> md.hours_worked; 
        else if( token == "filescreated" ) 
            in >> md.filescreated; 
        else 
            in.setstate( std::ios_base::failbit ); 
    } 
    return in; 
}
struct Komma : public std::numpunct< char > 
{ 
protected: 
    virtual char_type do_decimal_point() const 
    { 
        return ','; 
    } 
};

int main() 
{ 
    using namespace std; 
    ifstream fin("C:/Ruby193/1.txt"); 
   fin.imbue( locale( fin.getloc(), new Komma ) );
    if( !fin.is_open() ) 
    { 
        cerr << "Fehler beim Oeffnen der Datei" << endl; 

        return -2; 
    } 

    for( MD md; fin >> md;  ) 
    {  
           // md's   auswerten
           cout << "Datum      |   Stunden gearbeitet | Dateien erstellt " << endl ;
          cout << md.datum << "|          " << md.hours_worked << "          |          " << md.filescreated << endl ;    

          ofstream myFile;  
       myFile.open ("C:/Ruby193/fast.txt");

       if (myFile)
   {
      // in datei schreiben 
          myFile << "Datum      |   Stunden gearbeitet | Dateien erstellt " << endl ;
          myFile << md.datum << "|          " << md.hours_worked << "          |          " << md.filescreated << endl ; 
       myFile.close();

       }

         //   md's   summieren 
    } 
    if( fin.eof() ) 
    { 

    } 
    getchar () ;
    return 0; 
}

Hier der Code wo ofstream aus der schleife ist

#include <iostream>
#include <istream> 
#include <fstream> 
#include <limits>  
#include <string>

struct MD  // string für datum und doubles für  die werte in zahlen
{ 
    std::string datum; 
    double hours_worked; 
    double filescreated; 
};

namespace 
{ 
    const std::streamsize ALL = std::numeric_limits< std::streamsize >::max(); 
}; 
// filter :P
std::istream& operator>>( std::istream& in, MD& md ) 
{ 
    in.ignore( ALL, '[' ) >> md.datum; 
    std::string token; 
    if( getline( in.ignore( ALL, '|' ), token, '=' ) ) 
    { 
        md.hours_worked = 0.;  
        md.filescreated = 0.; 
        if( token == "hours_worked" ) 
            in >> md.hours_worked; 
        else if( token == "filescreated" ) 
            in >> md.filescreated; 
        else 
            in.setstate( std::ios_base::failbit ); 
    } 
    return in; 
}
struct Komma : public std::numpunct< char > 
{ 
protected: 
    virtual char_type do_decimal_point() const 
    { 
        return ','; 
    } 
};

int main() 
{ 
    using namespace std; 
    ifstream fin("C:/Ruby193/1.txt"); 
   fin.imbue( locale( fin.getloc(), new Komma ) );
    if( !fin.is_open() ) 
    { 
        cerr << "Fehler beim Oeffnen der Datei" << endl; 

        return -2; 
    } 

    for( MD md; fin >> md;  ) 
    {  
           // md's   auswerten
           cout << "Datum      |   Stunden gearbeitet | Dateien erstellt " << endl ;
          cout << md.datum << "|          " << md.hours_worked << "          |          " << md.filescreated << endl ;    

         //   md's   summieren 
    } 
    if( fin.eof() ) 
    { 

    } 

          ofstream myFile;  
       myFile.open ("C:/Ruby193/fast.txt");

       if (myFile)
   {
      // in datei schreiben 
          myFile << "Datum      |   Stunden gearbeitet | Dateien erstellt " << endl ;
          myFile << md.datum << "|          " << md.hours_worked << "          |          " << md.filescreated << endl ; 
       myFile.close();

       }
    getchar () ;
    return 0; 
}

Fehlermeldungen wen ofstream aus der schleife

81 C:\Dev-Cpp\valueextractor.cpp name lookup of `md' changed for new ISO `for' scoping

58 C:\Dev-Cpp\valueextractor.cpp cannot use obsolete binding at `md' because it has a destructor

Das md in Zeile 58 hat nur innerhalb der Schleife Gültigkeit.

das md in Zeile 8 gehört doch zur schleife ? also zur for-schleife

wie auch immer aber wenn ich den ofstream in der schleife lasse
speichert er nicht alle zeilen von den md's
und außerhalb der schleife gehts es ja nicht !
wie löse ich dies am besten um alle md's komplett zum schreiben der neuen datei zu verwenden ?

und außerhalb der schleife gehts es ja nicht !

Das ist zwar nicht die Lösung, aber warum sollte das nicht gehen? Klar geht das!

wie löse ich dies am besten um alle md's komplett zum schreiben der neuen datei zu verwenden ?

Du musst dir erst einmal klar werden, was du überhaupt willst und welche Objekte wo beteiligt sind. Besonders den Punkt, was du überhaupt willst, scheinst du selber nicht zu wissen. Werner hat dir einen fast kompletten Code vorgesetzt, sogar mit Kommentaren, was wo zu machen wäre. Und du machst irgendwas komplett anderes, was irgendwie überhaupt nicht zu dem Vorhaben passt, was man dir in anderen Threads mit so viel Mühe aus der Nase gezogen hat. Wieso summierst du nicht einfach deine Werte?

MD md;
if (cin >> md){

versteh ich nicht so ganz wegen der "cin" ?
außerdem wenn ich noch nicht in der Lage bin den kompletten Inhalt der md's zu benutzen, was bringt mir das summieren ? wenn das Ergebnis auch nicht korrekt dargestellt wird ?
Gruß Cevin

Hallo um nochmal auf SeppJ antwort zu kommen

Das md in Zeile 58 hat nur innerhalb der Schleife Gültigkeit.

Wie kann ich in diesem code den gültigkeitsbereich erweitern damit der so ausführbar ist ?

Wie kann ich in diesem code den gültigkeitsbereich erweitern damit der so ausführbar ist ?

SeppJ schrieb:

Das ist zwar nicht die Lösung, aber warum sollte das nicht gehen? Klar geht das!

Das ist genau das, was ich befürchtet habe. Du hast keinerlei Ahnung, was dieser Code zu bedeuten hat, wie er funktioniert oder was Werner sich dabei gedacht hat. Technisch korrekte Antwort (die beste Art von korrekt): Zieh das MD md; vor die for-Schleife.

Hilfreiche Antwort ist die gleiche wie bisher: Lern endlich die Grundlagen von C++! Lern die Grundlagen der Programmierung allgemein! Lern an einfacheren Beispielen! So wird das nie etwas werden, außer irgendjemand erbarmt sich endlich mal zur Komplettlösung - die du dann auch nicht verstehen wirst. Wenn du nicht schon eine Woche an diesem Programm verschwendet hättest, wärst du in einem Grundlagenbuch längst bei Gültigkeitsbereichen und grundlegenden Programmstrukturen angelangt.

So ?
Das habe ich schon versucht ...

MD md ;
    for(  fin >>  md ; )

das es so nicht geht weiß ich !

aber als fehlermedlung bekomme ich nur das er noch ein ";" vor dem ")" will also

MD md ;
    for(  fin >>  md ;; )

dann ist das programm fehlerlos aber der linker macht probleme wegen den ganzen md's
Aber wenn es besser so ist lese ich mir gerne noch das kapitel über die schleifen durch gruß cevin

Ich habe 4 bücher von Dirk louis über c++ und das was übder die for schleife oder allgemein über schleifen steht ist nicht viel ..

Wenn da nicht einmal Schleifen drin stehen, dann ist das wohl kein gutes Buch. Das ist wie eine Lesefibel ohne den Buchstaben 'A'. Guck mal den zweiten Link in meiner Signatur an.

MD md ;
    for(  fin >>  md ;; )

Das ist ja noch schlimmer. Das Forum ist kein Ersatz dafür, dass du nicht die Grundlagen lernen willst!

MD md;
for( ; fin >> md ; )
{
  // ...
}

Oder besser gleich

MD md;
while (fin >> md)
{
  // ...
}

Wenn in deinem C++ Buch wirklich nicht drin steht, wie man for in while Schleifen (und umgekehrt) überführen kann, dann würde es mich schon sehr wundern.

Mal ne ganz andere Frage: Wenn du Ruby auf deinem Rechner hast, warum willst du dann C++ für diese Aufgabe benutzen? Ruby ist deutlich geeigneter.

aber er schreibt immeroch nicht die kompltten daten (die er aus der console aber geben kann ) in die csv ? warum ?

Zum dritten und letzten Mal:

SeppJ schrieb:

Das ist zwar nicht die Lösung,

Das war nicht das was du suchtest, weil das Programm ganz was anderes macht, als du denkst. Die Antwort war deher nur technisch korrekt, nicht hilfreich.

Weil du den Code nicht verstehst.

Mal ne ganz andere Frage: Wenn du Ruby auf deinem Rechner hast, warum willst du dann C++ für diese Aufgabe benutzen? Ruby ist deutlich geeigneter.

Ich habe mit c++ schon viele einfache konsolen anwendungen gemacht
da dachte ich währe es vom vorteil wenn ich für diese aufgabe auch c++ nehme da ich von ruby garkeine ahnung habe

das ruby geeigneter ist wusste ich daher habe ich mich mal dran versucht
aber wenn man englisch nicht beherrscht ist das lernen ohne deutsche bücher noch einmal ne nummer schwieriger
und von c++ hab ich mehr als genug lern material ( was sich jetzt blöd anhört weil ich dafür mehr als genug fragen habe )

Das war nicht das was du suchtest, weil das Programm ganz was anderes macht, als du denkst. Die Antwort war deher nur technisch korrekt, nicht hilfreich.

Ok dann wenn ich es nicht weiß ... dann sag du mir ... was macht das programm ?

Ok dann wenn ich es nicht weiß ... dann sag du mir ... was macht das programm ?

Tut mir leid, aber ich habe die Geduld mit dir verloren.

Cevin19994 schrieb:

Ok dann wenn ich es nicht weiß ... dann sag du mir ... was macht das programm ?

Tut mir leid, aber ich habe die Geduld mit dir verloren.

ähm ja ich hab dir schon 2 mal gesagt wenn ich dir es nicht recht machen kann dann lass es mir zu helfen ....