c++ pętla for

Question

Tworzę prosty program do liczenia punktów do liceum. Napotkałem problem ponieważ program zwraca mi błędny wynik i nie mam pojęcia co jest nie tak.Wydaje mi się że jest problem w tej części ale mogę się mylić.

for(int x=0; x<=3; x++)
    {
    if(ocena[x]=3)
        ocena[x]=8;
    else if (ocena[x]=4)
        ocena[x]=14;
    else if (ocena[x]=5)
        ocena[x]=17;
    else if (ocena[x]=6)
        ocena[x]=18;

    punkty=punkty+ocena[x];
    }

  Dopiero zaczynam z programowaniem więc jak robię podstawowy błąd to wybaczcie.

Comments

Rozważ użycie pętli switch-case. Dużo bardziej tu pasuje zwłaszcza jeśli będzie miał zamiar dokładać tych warunków.

---

@Paweł Dymek,

No ja bym raczej switcha pętlą nie nazwał. 

---

Przejęzyczenie, oczywiście miało być "instrukcji warunkowej"

---

@pawel33335555,

Mam kolejny problem. Program działał super, tylko postanowiłem go uprościć i zrobić pętle zamiast wypisywać 10 razy prawie identycznego couta i cina. Ale jak to w programowaniu nie obyło się bez błędów.(Wstawiam tylko kawałek, bo reszta pozostała tak jak na początku)

    for(int z=0;z<=4;z++)
    {
        cout<<"Podaj % zdobytych punktow z czesci "<<przedmioty[z]<<": ";
        cin>>egzamin[z];
        if(egzamin[z]<0||egzamin[z]>100)
            cout<<endl;
        else
            z--;
            cout<<"Wpisz poprawny procent!";
            Sleep(1000);
            system("cls");
    }

    for(int a=0;a<=3;a++)
    {
        cout<<"Podaj "<<ktora_ocena[a]<<"ocene: ";
        cin>>ocena[a];
        if(ocena[a]==2||3||4||5||6)
            cout<<endl;
        else
        {
            a--;
            cout<<"Wpisz poprawna ocene!";
            Sleep(1000);
            system("cls");
        }
    }

A mianowicie w ifach wykonuje się zarówno if jak i else.

 Wydaje mi się że jest coś nie tak w nawiasach, tylko że nie wiem co.

---

Przepraszam jednak a-- działa.

Answer 1

Nie przeczytałem dokładnie kodu, ale chyba wiem w czym problem. Do przypisywania wartości do zmiennych, owszem, służy operator =. Natomiast do porównywania dwóch wartości służy operator ==. A więc w warunkach instrukcji warunkowych if musisz pozamieniać pojedynczy znak = na podwójny ==.

Kod po poprawieniu powinien wyglądać więc tak:

for (int x = 0; x <= 3; x++)
{
    if(ocena[x] == 3)
        ocena[x] = 8;
    else if (ocena[x] == 4)
        ocena[x] = 14;
    else if (ocena[x] == 5)
        ocena[x] = 17;
    else if (ocena[x] == 6)
        ocena[x] = 18;
 
    punkty += ocena[x];
}

Zwróć jeszcze uwagę, że instrukcję:

punkty=punkty+ocena[x];

Można zapisać krócej, tak jak ja to zrobiłem, przy pomocy operatora +=

punkty += ocena[x];

Do tego, zamiast tych wszystkich ifów, możesz użyć konstrukcji switch.

Jeszcze jedna sprawa. Nie widziałem całego kodu, ale:

for(int x=0; x<=3; x++)

Jeżeli tablica ocena jest tylko trzyelementowa, to ten warunek pętli for spowoduje problemy. Zapewne miało tam być x < 3.

Comments

Znam fajną definicję eksperta:

Ekspert to osoba, która popełniła już wszystkie możliwe błędy w swojej dziedzinie

---

Eryku, czyzbys byl ekspertem w BrainFucku? :D

---

Też dodam coś podobnego:

Nauczyć się coś robić dobrze, to poznać doświadczalnie wszystkie możliwe sposoby na zepsucie tego :)

---

@DragonCoder, a skąd mnie to wiedzieć? Jak mogę wiedzieć czy jakiś błąd może zaistnieć, jeśli jeszcze go nie popełniłem? Jakbym wiedział o jego istnieniu, to raczej bym go nie popełnił (no chyba że z rozpędu). 

---

Nie jest tak źle! Ten akurat błąd zdarza się na początku prawie każdemu. Wszystko przez to, że w C++ indeksy tablicy zaczynają się od zera. W niektórych językach można wybierać - czy mają zaczynać się od 0, czy od 1.