Mógłby mi ktoś powiedzieć co jest źle w tym kodzie?

Question

Program ma za zadanie obliczać czy osoba z danymi ocenami dostała czy nie dostała pasek na świadectwie.

#include <iostream>
using namespace std;
int main()
{
cout<<"Podaj swoje oceny; ";
int a,b,c;
float d = (a+b+c)/3;
cin>>a;
cin>>b;
cin>>c;
if(d>=1.00||d<=4.74)
    {
    cout<<"Nie masz paska";
    }

else if(d>=4.75||d<=6.00)
    {
    cout<<"Masz pasek!";
    }

else if(d>6.00||d<1.00)
    {
    cout<<"Błędne dane!";
    }

return 0;


}

 

Answer 1

W tych dwóch pierwszych warunkach powinno być && a nie ||.

float d = (a+b+c)/3;
cin>>a;
cin>>b;
cin>>c;

Błąd logiczny. Najpierw liczysz d z wartości zmiennych a, b i c, a potem pobierasz te wartości. Nie sądzisz, że powinno być na odwrót?

 

Comments

Dzięki za szybką odpowiedź jestem nowy i przeoczyłem to pisząc kod. Jeszcze raz dziekuj.