jak przekazać zwróconą wartość do innej bez ponownego jej wywoływania?

Question

witam mam problem otóż mam funkcje powiedzmy

 

string wczytaj_pytanie(string *odp)
{
for(int i=0;i<20;i++)
{
int L;
L=wylosuj_numer_pytania();

string numer;
stringstream strumien;
strumien<<L;
strumien >>numer;
string  napis ="Baza_Pytan/pytanie"+numer+".txt";

    ifstream plik;
    plik.open(napis.c_str());
    if(plik)
    {
        string linia;
        while(getline(plik,linia))
        {
            cout<<linia<<endl;
        }
        cout<<endl;

        cout<<"podaj odpowiedz: ";
        cin>>*odp;
        return *odp;

    }
    else
    {
        cout <<"wystapil blad nie mozna otworzyc pliku";
    }
    plik.close();

    cout <<endl<<endl;

}
}

i chcąc przekazać zwróconą wartość w tym wypadku odpowiedz robię  to tak

 

string odpowiedz;
string wynik=wczytaj_pytanie(&odpowiedz);

wiem że to jest źle bo w ten sposób wywołuje poprzednią funkce jeszcze raz ale nie wiem jak to inaczej moge zrobić tak żeby do zmiennej wynik zapisać samą odpowiedz  

Comments

Tak nawiasem:

int L;
L=wylosuj_numer_pytania();

string numer;
stringstream strumien;
strumien<<L;
strumien >>numer;
string  napis ="Baza_Pytan/pytanie"+numer+".txt";
		
		
ifstream plik;
plik.open(napis.c_str());
...

Można uprościć/zracjonalizować:

ostringstream oss;

oss << "Baza_Pytan/pytanie" << wylosuj_numer_pytania() << ".txt";
		
ifstream plik(oss.str().c_str());
...

 

Answer 1

Po co zwracasz cokolwiek z funkcji skoro odpowiedź pobierasz przez wskaźnik przekazywany do funkcji. Zdecyduj sie na jedno, albo drugie. Jest ok, ale bez sensu. Teraz w `odpowiedz` i `wynik` masz tą samą wartość.

PS: jeśli pliku nie uda się otworzyć, to twoja funkcja nie dojdzie do żadnego `return`. Kompilator powinien cie o tym ostrzegać.