[C++] ofstream distrutto all'uscita della funzione

[VegetaSSJ5]

Salve a tutti, mi sto cimentando con il C++ e sto facendo alcuni esercizi. Mi trovo a realizzare un piccolo esempio: costruire un vector di ostream e, attraverso una funzione mandare una stringa in tutti gli ostream del vettore. Ho quindi una funzione che costruisce questo vector di (puntatori a) ostream e lo inizializza inserendovi &cout e un &file. Se ciclo il vettore all'interno della funzione, i messaggi vengono correttamente riportati in tutti gli ostream, ma uscito dalla funzione e tornato al chiamante, l'output su cout viene correttamente riportato, ma l'output sull'ofstream su file invece genera un errore a me ancora abbastanza incomprensibile. Come se, all'uscita della funzione, quel puntatore a ostream presente nel vettore non fosse piú valido.
Segue il codice sorgente compilabile e con stesso errore sia su compilatore ms (VS2019) sia su gcc. Potete dirmi per favore cosa c'é di concettualmente sbagliato?

#include <iostream>
#include <fstream>
#include <string>
#include <vector>

using namespace std;

vector<ostream*> createStreams(const string& fileName) {
    vector<ostream*> result;
    result.push_back(&cout);
    ofstream lFile(fileName, ofstream::app);
    result.push_back(&lFile);
    for (ostream* ostr : result) {
        *ostr << "TEST in createStream function" << endl;
    }
    return result;
}

int main() {
    try {
        vector<ostream*> ostreams = createStreams("C:\\Users\\Armando\\Desktop\\testlog.txt");
        for (ostream* ostr : ostreams) {
            *ostr << "TEST in main function" << endl;
        }
    } catch (exception& e) {
        cerr << "Catched exception in main method: " << e.what() << endl;
    }
}

[lorenzo001]

Ti hanno risposto su html.it

[Lampo89]

Originariamente inviato da lorenzo001

Ti hanno risposto su html.it

https://forum.html.it/forum/showthre...readid=2971976

Come hai notato, il tuo vettore contiene un "dangling pointer" poiché l'oggetto ofstream è una variabile locale con "automatic storage duration". Di conseguenza, dato che essa è distrutta all'uscita della funzione CreateStreams, dereferenziando il puntatore otterrai undefined behaviour (se vuoi https://godbolt.org/ qui puoi vedere come varia il comportamento al variare del compilatore e dei flags di compila).

La soluzione che ti hanno dato su html.it risolve questo problema, ma ha un paio di memory leaks (vector e ofstream) da sistemare (come da commento nel codice)

[VegetaSSJ5]

Originariamente inviato da lorenzo001

Ti hanno risposto su html.it

Originariamente inviato da Lampo89

https://forum.html.it/forum/showthre...readid=2971976

Come hai notato, il tuo vettore contiene un "dangling pointer" poiché l'oggetto ofstream è una variabile locale con "automatic storage duration". Di conseguenza, dato che essa è distrutta all'uscita della funzione CreateStreams, dereferenziando il puntatore otterrai undefined behaviour (se vuoi https://godbolt.org/ qui puoi vedere come varia il comportamento al variare del compilatore e dei flags di compila).

La soluzione che ti hanno dato su html.it risolve questo problema, ma ha un paio di memory leaks (vector e ofstream) da sistemare (come da commento nel codice)

Grazie per la risposta e per il suggerimento di godbolt...
C'é poco da fare... C e C++ sono quelli con l'assembly piú stringato possibile