Quiz blad w kodzie?

Question

Witajcie! Zacząłem się uczyc C++ z kursów Pana Mirosława i utknąłem na odcinku 7 z quizem. Obejrzałem już całośc kilka razy i nie jestem w stanie znalezc bledu w moim kodzie. Jedyne co poprawnie sie wyswietla to odpowiedzi na koncu kodu :P plik tekstowy oczywiscie jest utworzony. Z góry dziekuje za jakies podpowiedzi :D

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <algorithm>
#include <string>
using namespace std;

string temat, nick;
string tresc[5];
string odpA[5], odpB[5], odpC[5], odpD[5];
string poprawna[5];
string odpowiedz;
int punkty=0;

int main()
{
    int nr_linii=1;
    string linia;
    int nr_pytania=0;

    fstream plik;
    plik.open("quiz",ios::in);

if (plik.good()!=false)
    {
        cout<<"Nie udao sie otworzyc pliku!";
        exit(0);
    }

    while(getline(plik,linia))
    {
        switch(nr_linii)
        {
            case 1: temat=linia;                  break;
            case 2: nick=linia;                   break;
            case 3: tresc[nr_pytania] = linia;    break;
            case 4: odpA[nr_pytania] = linia;     break;
            case 5: odpB[nr_pytania] = linia;     break;
            case 6: odpC[nr_pytania] = linia;     break;
            case 7: odpD[nr_pytania] = linia;     break;
            case 8: poprawna[nr_pytania] = linia; break;
        }
        if(nr_linii==8) {nr_linii=2; nr_pytania++;}
        nr_linii++;
    }
    plik.close();

    for (int i=0; i<=4; i++)
    {
        cout<<endl<<temat<<endl;
        cout<<endl<<nick<<endl;
        cout<<endl<<tresc[i]<<endl;
        cout<<"A. "<<odpA[i]<<endl;
        cout<<"B. "<<odpB[i]<<endl;
        cout<<"C. "<<odpC[i]<<endl;
        cout<<"D. "<<odpD[i]<<endl;

        cout<<"Twoja odpowiedz: ";
        cin>>odpowiedz;

        transform(odpowiedz.begin(),odpowiedz.end(),odpowiedz.begin(), ::tolower);

        if (odpowiedz==poprawna[i])
        {
            cout<<"Dobrze! Zdobywasz punkt!"<<endl;
            punkty++;
        } else cout<<"Zle!Brak punktu! Poprawna odpowiedz: "<<poprawna[i]<<endl;
    }

    cout<<endl<<
    "Koniec quizu! Zdobyte punkty: "<<punkty<<"/5";
    return 0;
}

 

Comments

Oczywiście linku do lekcji brak, ciekawostka: wersję tekstową kodu masz pod filmem na yt. Wystarczy zrobić diff pomiędzy dwoma filmami aby zobaczyć co jest napisane inaczej

https://forum.pasja-informatyki.pl/126927/cr-c-%237-pliki-tekstowe-zapis-i-odczyt-z-pliku-tekstowego

Answer 1

Nie podoba mi się ten fragment:
 

if (plik.good()!=false)
{
    cout<<"Nie udao sie otworzyc pliku!";
    exit(0);
}

Dlaczego wywołujesz funkcję good() zamiast is_open().  Być może w tym przypadku obie funkcje zwrócą ten sam wynik, ale jakoś dziko to wygląda.
Drugi problem: dlaczego sprawdzasz czy wartość jest różna od false?