Obiektowy C++ odc. 2 zadanie domowe.

Question

Witam, właśnie przerabiam kurs Obiektowy C++ odc. 2: Struktura projektu obiektowego i chciałbym zrobić zadanie domowe.

Dodałem coś takiego do kodu

 while(getline(plik, linia))
    {
        if(aktualny_nr==nr_linii) tresc=linia;
        if(aktualny_nr==nr_linii+1) a=linia, A=linia;
        if(aktualny_nr==nr_linii+2) b=linia, b=linia;
        if(aktualny_nr==nr_linii+3) c=linia, C=linia;
        if(aktualny_nr==nr_linii+4) d=linia, D=linia;
        if(aktualny_nr==nr_linii+5) poprawna=linia;

        aktualny_nr++;
    }

 

 string a,A,b,B,C,c,D,d;

Jednak nie działa i nie wiem dlaczego? 

Comments

nie działa

Doprecyzuj, kompilator wywala błąd? Jeśli tak, to jaki? Program się wysypuje po odpaleniu?

---

kompilator  nie wywala błędów, tylko podczas wprowadzania odpowiedzi "A" nie działa przyznawanie punktów, ale kiedy dam literkę "a" to już działa.

---

Jeżeli nie działa przyznawanie punktów tylko dla 'A', to chyba problem leży w innym fragmencie kodu. Mógłbyś podlinkować całość/fragment z przyznawaniem punktów?

Tak swoją drogą, nie lepiej przekonwertować wprowadzane dane od użytkownika na tylko duże/tylko małe zaraz po ich dostaniu? Nie musiałbyś wtedy uwzględniać obu przypadków w if-ach.

---

main.cpp

#include <iostream>
#include "pytanie.h"

using namespace std;

int main()
{
    Pytanie p[5];
    int suma=0;
    for(int i=0; i<=4; i++)
    {
    p[i].nr_pytania=i+1;
    p[i].wczytaj();
    p[i].zadaj();
    p[i].sprawdz();

    suma+=p[i].punkt;
    }
    cout<<"Koniec quizu! Punkty = "<<suma;

    return 0;
}

pytanie.h

#include <iostream>

using namespace std;

class Pytanie
{
public:

    string tresc;
    string a,A,b,B,C,c,D,d;
    int nr_pytania;
    string poprawna;
    string odpowiedz;
    int punkt;// 1 lub 0

    void wczytaj();
    void zadaj();
    void sprawdz();

};

pytanie.cpp

#include <iostream>
#include "pytanie.h"
#include <fstream>
#include <cstdlib>

using namespace std;

void Pytanie::wczytaj()
{
    fstream plik;
    plik.open("quiz.txt", ios::in);

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

    int nr_linii=(nr_pytania-1)*6+1;
    int aktualny_nr=1;
    string linia;

    while(getline(plik, linia))
    {
        if(aktualny_nr==nr_linii) tresc=linia;
        if(aktualny_nr==nr_linii+1) a=linia, A=linia;
        if(aktualny_nr==nr_linii+2) b=linia, b=linia;
        if(aktualny_nr==nr_linii+3) c=linia, C=linia;
        if(aktualny_nr==nr_linii+4) d=linia, D=linia;
        if(aktualny_nr==nr_linii+5) poprawna=linia;

        aktualny_nr++;
    }

    plik.close();

}

void Pytanie::zadaj()
    {
        cout<<endl<<tresc<<endl;
        cout<<a<<endl;
        cout<<b<<endl;
        cout<<c<<endl;
        cout<<d<<endl;
        cout<<"------------------------"<<endl;
        cout<<endl<<"Odpowiedz: ";
        cin>>odpowiedz;
    }


void Pytanie::sprawdz()
    {
        if(odpowiedz==poprawna)
        {
            punkt=1;
        }
        else punkt=0;
    }

 

Answer 1

if(aktualny_nr==nr_linii+2) b=linia, b=linia;

Tu zamiast jednego "b" powinno być jedno duże "B".

if(aktualny_nr==nr_linii+5) poprawna=linia;

if(odpowiedz==poprawna)
        {
            punkt=1;
        }
        else punkt=0;

Sprawdzasz tutaj jedynie czy odpowiedz podana przez użytkownika jest równa tej z pliku - jeżeli ktoś poda "A", a poprawną odpowiedzią w pliku będzie "a", to wtedy nie zaliczy poprawnej odpowiedzi. Moim zdaniem należałoby to przebudować, ustalić jakie literki są w pliku, i przy wejściu konwertować wprowadzoną wartość, by zgadzała się z formatem przyjętym w pliku (np.: funkcja "tolower()").

Comments

P.S.: Jeżeli używasz zmiennej i przyjąłeś, że chcesz do niej zapisywać tylko 2 stany, np.: 0 i 1, to rozważ użycie zmiennej typu "bool". Przechowuje wartości TRUE i FALSE, w warunkach sprawdzasz czy są równe tym wartościom logicznym. Zajmuje to znacznie mniej pamięci operacyjnej, i nie wpiszesz tam przez przypadek czegoś innego, np 144, co może ułatwić szukanie błędów w przyszłości.

---

@Vellur, Poprawiłem, ale nadal nie działa.

---

W sensie dalej nie liczy tego "A"?

---

Tak, nie przydziela punktów z duże "A". Przydziela punkty tylko za małe "a".

---

Na pewno wszystko zrobiłeś dobrze? Wrzuć tu może ten poprawiony kod.

---

pytanie.cpp

#include <iostream>
#include "pytanie.h"
#include <fstream>
#include <cstdlib>

using namespace std;

void Pytanie::wczytaj()
{
    fstream plik;
    plik.open("quiz.txt", ios::in);

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

    int nr_linii=(nr_pytania-1)*6+1;
    int aktualny_nr=1;
    string linia;

    while(getline(plik, linia))
    {
        if(aktualny_nr==nr_linii) tresc=linia;
        if(aktualny_nr==nr_linii+1) a=linia, A=linia;
        if(aktualny_nr==nr_linii+2) b=linia, B=linia;
        if(aktualny_nr==nr_linii+3) c=linia, C=linia;
        if(aktualny_nr==nr_linii+4) d=linia, D=linia;
        if(aktualny_nr==nr_linii+5) poprawna=linia;

        aktualny_nr++;
    }

    plik.close();

}

void Pytanie::zadaj()
    {
        cout<<endl<<tresc<<endl;
        cout<<a<<endl;
        cout<<b<<endl;
        cout<<c<<endl;
        cout<<d<<endl;
        cout<<"------------------------"<<endl;
        cout<<endl<<"Odpowiedz: ";
        cin>>odpowiedz;
    }


void Pytanie::sprawdz()
    {

        if(odpowiedz==poprawna)
        {
            punkt=1;
        }
        else punkt=0;
    }

 

Answer 2

Myślę, że najprościej po prostu dodać linijkę z funkcją transform po wczytaniu odpowiedzi od użytkownika w metodzie zadaj():

#include <string>
#include <algorithm>

using namespace std;

cin >> odpUzytkownika;
transform(odpUzytkownika.begin(),odpUzytkownika.end(),odpUzytkownika.begin(),::tolower);

 

Answer 3

Należy dodać w pliku pytanie.cpp do metody sprawdzającej 2 linijki z funkcją transform, najlepiej dla obu atrybutów "odpowiedz" i "poprawna", oraz 2 biblioteki wymagane do działania funkcji transform.

 

#include <string>
#include <algorithm>

void Pytanie::sprawdz()
{
    transform(odpowiedz.begin(), odpowiedz.end(), odpowiedz.begin(), ::tolower);
    transform(poprawna.begin(), poprawna.end(), poprawna.begin(), ::tolower);

    if (odpowiedz==poprawna) punkt=1;
    else punkt=0;
}