Youtube c++ odcinek 7

Question

Czy mógłby mi któs pomó sprawdzając mój kod? szukam linia po linii bledu ale nic nie widze

konsola pokazuje tylko i wylacznie odp a,b,c,d ale nie co sie w nich znajduje i nie pokazuje tresci

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <algorithm>
#include <string>

using namespace std;

string temat,nick;
string pytania[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.txt", ios::in);
    if(plik.good()==false)
    {
        cout<<"Taki plik nie istnieje";
        exit(0);
    }
    while(getline(plik,linia));
          {
              switch(nr_linii)
              {
                  case 1:temat = linia;                break;
                  case 2:nick= linia;                  break;
                  case 3: pytania[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<<pytania[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<<"Zdobywasz punkt!";

                punkty++;
              }
              else
              {
                  cout<<"Zla odpowiedz! poprawna odpowiedz to: "<<poprawna[i]<<endl;
              }

          }

          cout<<endl<<"koniec quizu! Zdobyte punkty: "<<punkty;
    return 0;
}

 

Comments

Kod się nie kompiluje. Linia 30. w if'ie masz zmienna, która nigdzie wcześniej nie została zdeklarowana.

---

Po co ten if w linii 29?

---

@tkz, Usunalem tego ifa rzeczywiscie nie wiem jak sie tam znalazl

ale to nadal nie naprawia problemu

Answer 1

if(plik.good()==false)

Ten warunek nie da Ci gwarancji, że strumień jest otwarty. Zamiast good() powinno być tam is_open().

Zapewne aplikacja nie otwiera pliku z pytaniami i dlatego wyświetla tylko a. b. c. d.

Comments

Zamienilem zeby sprawdzic lecz niestety nadal bez zmian

---

I co, nie masz komunikatu, że plik nie istnieje?

---

Nie

---

Pokaż kod po poprawce.

Jaki system?

---

windows 10

#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("pytania.txt", ios::in);
    if(plik.is_open()==false)
    {
        cout<<"Taki plik nie istnieje";
        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<<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<<"Zdobywasz punkt!";
                punkty++;
              }
              else
              {
                  cout<<"Zla odpowiedz! poprawna odpowiedz to: "<<poprawna[i]<<endl;
              }

          }


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

 

---

Średnik w linii 30.

---

O boze dziala ajj... taki glupi blad az wstyd ale dzieki !

---

Przenieś też definicje zmiennych z linii 9..14 do wnętrza funkcji main, bo nie wiedzieć czemu autor zrobił je globalne (a to do dobrych praktyk nie należy).

 

W sumie kompilator powinien Cię ostrzec przed tym średnikiem. Czytałeś logi kompilacji?

---

Nie ostrzega przed średnikiem, przynajmniej w tym miejscu, chyba myśli, że wiesz co robisz. Chyba, że są jakieś flagi w gcc do tego typu.

---

gcc z flagą -Wall:

main.cpp: In function ‘int main()’:
main.cpp:38:5: warning: this ‘while’ clause does not guard... [-Wmisleading-indentation]
     while(getline(plik,linia));
     ^~~~~
main.cpp:39:11: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘while’
           {
           ^