Przerwanie pętli nie działa?

Question

Witam, jestem amatorem C++ i chcę stworzyć kalkulator średniej arytmetycznej z ocen, jednak tu pojawia się pewien problem, otóż dlaczego po wpisaniu liczby innej niż: 1,2,3,4,5,6 pętla nie zamyka się? Chodzi mi o to, że gdy podam ocenę program powinien dodać ją do pozostałych, natomiast gdy np. w środku wpiszę 125 pojawił się napis ocena moze byc tylko liczbami: 1,2,3,4,5 lub 6! i zamknął się. Proszę o pomoc, wklejam kod:

#include <iostream>

using namespace std;

int przedmioty; float liczba; float oceny;

int main()
{
    cout << "Liczba ocen do sredniej:";
    cin >> przedmioty;
    if (przedmioty>0)
    {
        for (int i=0; i<przedmioty; i++)
        {
            cout << "Wpisz ocene:";
            cin >> liczba;
            if ((liczba=1)||(liczba=2)||(liczba=3)||(liczba=4)||(liczba=5)||(liczba=6))
            {
                oceny=liczba+oceny;
            }
            else
            {
                cout << "Ocena moze byc tylko liczbami: 1,2,3,4,5 lub 6!";
                break;
            }
        }

    }
    else if (przedmioty<=0)
    {
        cout << "Liczba przedmiotow musi byc wieksza od 0!";
    }
    return 0;
}

Answer 1

Dużo tu by pisać o tym programie, bo krótko mówiąc jest źle, ale myślę że to o co pytasz to:

jeżeli w C++ napiszesz

liczba = 1

to do zmiennej liczba przypisujesz wartość 1

jeżeli napiszesz

liczba == 1

to porównujesz czy liczba jest równa 1 i nic nie zmieniasz zatem trzeba zmienić

if ((liczba=1)||(liczba=2)||(liczba=3)||(liczba=4)||(liczba=5)||(liczba=6))

na

if ((liczba==1)||(liczba==2)||(liczba==3)||(liczba==4)||(liczba==5)||(liczba==6))

wtedy zamiast przypisywać wartości będzie je porównywać

Comments

Tak faktycznie zapomniałem o tym, bardzo dziękuję za pomoc!  Program już działa tak jak chciałem :)