void fileOutput( string fileName ){
     ofstream myfile;
     myfile.open( fileName );
     myfile << "lalalalalala.";
     myfile.close();
}

Hier gibt er mir bei der Zeile "myfile.open( fileName )" den Fehler aus:

no matching function for call to `std::basic_ofstream<char, std::char_traits<char> >::open(std::string&)'

Das Problem liegt an der string-Variable, denn wenn ich in der Zeile das filename ersetze durch z.b. "blabla.txt", dann gibts keinen Fehler.
Das ergibt für mich aber keinen Sinn, da "blabla.txt" ja auch nur ein string ist, oder?

PS: Mit dem neuen Standard wird alles besser

PPS: Für das open()/close() wird bestimmt noch eine ausführlichere Erklärung kommen, warum es überflüssig ist.

2.) Objekte wie Strings sollte man immer per const-ref nehmen. Also const std::string& oder std::string const&, was dir lieber ist. Der Grund ist, dass bei dir hier ein string kopiert wird, was unter extremen Umständen langsam sein kann.

3.) Vermeide die Funktionen .open() und .close(). C++ Streams können das Wunderbare Prinzip RAII auch. Du kannst den Dateinamen im Konstruktor übergeben, im Destruktor wird der Stream automatisch geschlossen.

Also zusammenfassend so:

void fileOutput(const string& fileName ){
     ofstream myfile(fileName.c_str());
     myfile << "lalalalalala.";
     // .close() passiert hier automatisch
}

PPS: Für das open()/close() wird bestimmt noch eine ausführlichere Erklärung kommen, warum es überflüssig ist.

Also das mit dem const& usw. kann ich noch nicht so richtig "nachvollziehen", da fehlt mir einfach noch das Wissen ^^'

Dann versuchen wir es mal mit der Kurzfassung: Einen String zu kopieren kann verhältnismäßig teuer werden, wegen der Speicherverwaltung und der Umkopiererei des Inhalts. Wenn es nicht notwendig ist, erspart man sich das, indem man nur einen Verweis auf den ursprünglichen String übergibt - und damit die Funktion nicht versehentlich das Original kaputt macht, wird dieser Verweis mit const erweitert. Das Ergebnis ist dann der Parameter const string& fileName .

Danke für die schnellen Antworten!
Also das mit dem const& usw. kann ich noch nicht so richtig "nachvollziehen", da fehlt mir einfach noch das Wissen ^^'
Aber es klappt auf jeden Fall, danke

Mit dem Referenz-Operator & übergibst du quasi die Adresse unter der die Variable abgelegt ist, sprich das Original wird übergeben. Wie 31415.. (komplizierter Name ) ja schon angedeutet hat vermeidest du so sinnloses hin und her kopieren const sagt nur aus, dass die entsprechende variable konstant ist, also nicht abgeändert werden darf.

-edit- war jemand schneller

void fileOutput(const string& fileName ){
     ofstream myfile(fileName.c_str());
     myfile << "lalalalalala.";
     // .close() passiert hier automatisch
}

Aber der Text in der File wird dann gar nicht geändert.
Wenn ichs so mache, also mit dem normalen String funktionierts:

void fileOutput(string fileName ){
     ofstream myfile(fileName.c_str());
     myfile << "lalalalalala.";
     // .close() passiert hier automatisch
}

??

void fileOutput(const string& fileName ){
    ofstream myfile(fileName.c_str());
    if( !myfile)
        cerr << "Fehler beim Öffnen der Datei " << fileName << endl;
    myfile << "lalalalalala.";
}

Wo hätte ich denn die Fehlermeldung lesen sollen? Ich kenn mich ja kein bisschen aus ^^', vielleicht hab ichs nur nicht gesehn

cerr-Ausgaben landen normalerweise im Konsolen-Fenster, aber wenn du keins hast, könnte das etwas schwierig werden.

Naja, ich schau einfach in der Text-datei, ob sich was geändert hat.

Wie kann sich denn am Datei-Inhalt etwas ändern, wenn du immer den selben Text reinschreibst? Achte mal auf das Änderungsdatum - oder ergänze den Stream-Konstruktor zu ofstream myfile(fileName.c_str(),ios::app); .

Wie kann sich denn am Datei-Inhalt etwas ändern, wenn du immer den selben Text reinschreibst?

Im Programm hab ich natürlich schon immer was anderes reinschreiben lassen. Und das Änderungsdatum hat sich nicht geändert, durch myfile(fileName.c_str(),ios::app); ist auch nichts passiert

EDIT:
Aaach, ich hab den Fehler gefunden:
Ich hab vergessen, dass ich auch bei dem Deklarieren der Methode den Konstruktor ändern muss, also zu

void fileOutput( const string& );

Sry, wie gesagt, ich bin noch ein Anfänger in C++ ^^'
In Java gabs das mit dem Methoden deklarieren ja auch gar nicht, da hab ich dann nicht dran gedacht ^^'
Jetzt gehts, danke nochmal