Problem z if'em.

Question

Witam. :)
Mam pytanie dlaczego if dochodzi tylko do momentu "Sugerowana ocena to: 2!" jeżeli srednia jest wieksza. Mam tez pytanie czy da sie ten warunek zrobić w jakiś bardziej schludny i krotszy sposob?

Mam też prośbę abyście wtrącili swoje uwagi co w tym kodzie jest nie tak jak można go skrócić co poprawić co jest błędne.

Dziękuję bardzo za odpowiedzi. :)

 

Kod:

#include <iostream>

using namespace std;

int rozmiar = 0;
float* p = new float[rozmiar];

float oceny;
float suma, srednia;

int main()
{
	cout <<"Podaj ilosc ocen: ";
	cin >>rozmiar;
	
	
    for (int i=0;i<rozmiar;i++)
    {
        cout << "Podaj " << i+1 <<" ocene: ";
        cin >> oceny;
        suma += oceny;
        p[i] = oceny;
	}
	
	cout << "Twoje oceny to: ";
	for(int i=0; i<rozmiar; i++)
	{
		cout << p[i] <<", ";
	}
    srednia = suma/rozmiar;
    
    cout <<endl <<"Srednia Twoich ocen to: "<< srednia<<endl ;
    
    if(srednia<1.75)
    {
    	cout<<"Sugerowana ocena to: 1!";
	}
	else if(srednia>1.75)
	{
	cout<<"Sugerowana ocena to: 2!";
	}
	else if (srednia> 2,75)
	{
	cout <<"Sugerowana ocena to: 3";
	}
	else if (srednia> 3,75)
	{
	cout <<"Sugerowana ocena to: 4";
	}
	else
	{
	cout <<"Sugerowana ocena to: 5";
	}
	
    
    return 0;
}

 

Comments

(srednia> 2,75)

Przecinek w liczbie w programowaniu pisze się jako kropkę. Tak zapisany jest błędem a kod może ci się nie skompilować.

---

Dziękuje :) zauważyłem po dodaniu postu już poprawione :D

---

float* p = new float[rozmiar];

rozmiar jest równy zeru, twój program ma niezdefiniowane zachowanie

---

Usunąłem to przyrownanie do zera. Co to znaczy ze ma niezdefiniowane zachowanie?

---

Nie miałeś tam żadnego przyrównania do zera. Twoja dynamiczna tablica ma rozmiar zero, bo najpierw ją tworzysz, a dopiero później wczytujesz rozmiar.

"ma niezdefiniowane zachowanie", to znaczy że jest błędny.

Answer 1

Dzieje się tak ponieważ przecież warunek >1.75 jest Twoim ostatnim sprawdzanym warunkiem... każdy następny będzie wyłapany własnie przez ten.

Comments

Dziękuje za odpowiedź. :)

Answer 2

Bo ify musisz robić przedziałami. 5 jest tak samo większe od 1.75 jak 2 czy 3:

else if(srednia>1.75 && srednia <2.75)

I nie ucz sie pracowac na zmiennych globalnych. Deklaruj zmienne w mainie.

Comments

Dziękuje zrobiłem tak jak Pan napisał i działa :)

Answer 3

Nie idzie dalej bo jak masz średnią 5, no to ona jest większa od 1.75, więc nie sprawdza już dalszych warunków.

Comments

Dziękuje za odpowiedź. :)

Answer 4

Sprawdzasz czy średnia > 1.75 więc niezależnie czy średnia = 4 czy średnia = 1.76 warunek jest prawdziwy i nie sprawdza dalej.

proponuję coś takiego:

cout << "Sugerowana ocena to: " << (int)srednia << endl

 

Comments

Oo :)
Czegos takiego szukałem tylko pytanie czy da się jakoś ustalić ,że ocena zostaje zaokraglona jezeli po przecinku znajduje się 0,75? :)
 

I czy (int) ma sluzyć zaokrągleniu sredniej do liczb calkowitych?

---

(typ) to rzutowaniu zmienia aktualny tym zmiennej na tą podaną w nawiasie

---

Ok rozumiem. A jeżeli chciałbym aby było to zaokrąglane po tej wartosci 0,75 to w jaki sposób zrobić to krócej? :)

---

Nie skrócisz

---

Ok dzięki :)