C++ wyliczanie pkt do liceum

Question

Witam, mam problem z funkcją if. Wkleiłem cały kod dla pewności że gdzieś indziej nie zrobiłem głupiego błędu.

Ten problem pojawia się w wyodrębnionej części którą postarałem się zaznaczyć. A mianowicie chodzi o instrukcję warunkową if else w której zamiast wykonywania się tylko części pod ifem gdy nawias jest prawdziwy wykonuje się cały if włącznie z elsem. Wydaje mi się że może być to błąd w nawiasach tylko za bardzo nie wiem jaki.

    #include <iostream>
#include <windows.h>
#include <cstdlib>
#include <stdio.h>
#include <conio.h>

using namespace std;

float egzamin[5];
int ocena[4];
int punkty=0;
string przedmioty[5];
int ktora_ocena[4];

int main()
{
przedmioty[0]="humanistycznej";
przedmioty[1]="polonistycznej";
przedmioty[2]="przyrodniczej";
przedmioty[3]="matematycznej";
przedmioty[4]="jezykowej";
ktora_ocena[0]=1;
ktora_ocena[1]=2;
ktora_ocena[2]=3;
ktora_ocena[3]=4;


------------------------------------------------------------------------------

    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");
        }
    }

---------------------------------------------------------------------------









    for(int i=0; i<=4; i++)
    {
        punkty=punkty+egzamin[i]*0.2;
    }

    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];
    }

    cout<<"Twoje punkty:"<<punkty;
    return 0;
}

 

Comments

If to nie funkcja, lecz instrukcja warunkowa!

---

Biblioteki brałem z innego programu więc mogą niektóre być niepotrzebne

---

W instrukcji if nie można w taki sposób powtarzać operatora logicznego: OR 3 OR 4 OR 5.

Answer 1

Nie wiem o co Ci konkretnie chodzi, ale:

if(egzamin[z]<0||egzamin[z]>100)
    cout<<endl;
else
    z--;
    cout<<"Wpisz poprawny procent!";
    Sleep(1000);
    system("cls");

Masz najprawdopodobniej błąd w else (domyślam się po wcięciach). A mianowicie wykona się tam wbrew pozorom tylko instrukcja z--. Jeżeli chcesz aby wykonały się wszystkie cztery instrukcje, musisz je umieścić w bloku kodu {}. A najlepiej to w ogóle proponuję nie skracać sobie zapisu i nie pomijać tych bloków, żeby nie narobić sobie problemów.

if(ocena[a]==2||3||4||5||6)
    cout<<endl;

Ten if również działa chyba nie tak, jak ty byś sobie tego życzył. Każda inna wartość niż 0 to true, natomiast w łańcuchu operacji logicznych OR wystarczy jedna wartość true, aby cały wynik też był prawdziwy. Więc na moje oko, ten if będzie się wykonywał zawsze, a będący za nim else - nigdy. Jeśli chcesz poprawić ten kod, to albo porównuj każdą ocenę z osobna, albo lepiej sprawdź zakres w jakim znajduje się ocena.

if (ocena[a] >= 2 && ocena[a] <= 6) {
    std::cout << '\n';
}

Proponuję też używać '\n' zamiast std::endl, nie używać windows.h jeśli nie ma takiej potrzeby, nie używać system("cls")... No i jak wspominałem wcześniej, nie pomijać tych bloczków. W niektórych językach są one opcjonalne, w innych obowiązkowe.

Comments

Dziękuję bardzo za wyłapanie banalnych błędów ponownie. Chyba naprawdę dużo powtórek mnie czeka