[wikary]

Czy ktoś może sprawdzić ten kod i powiedzieć co jest nie tak? link.
Nie wychodzą dobre wyniki przy stożku i kuli.
#cpp #programowanie

[k.....r]

@wikary: 1/3 = 0, tak samo 4/3 - wykonujesz dzielenie całkowitoliczbowe, odrzucając część po przecinku.
Np. 1/3 = 0.333 = 0; 4/3 = 1.333 = 1;
Użyj zamiast tego 1.0f/3.0f.

[wikary]

no tak, dzięki

[l.....l]

@wikary: Swoją drogą, zamiast:

while(1)

{

cout<<"Podaj wysokosc bryly:";

cin>>H;

if(H>0) break;

if(H<=0)continue;

}

Nie prościej tak:

float H = 0;

while(H <= 0)

cin >> H;

?
Ten drugi if jest zupełnie bez sensu.

[wikary]

@losiul: Dopiero sie ucze i pewne rzeczy inaczej rozumiem :)

[wikary]

float H = 0;

while(H <= 0)

cin >> H;

@losiul: możesz to jakoś wyjaśnic?

[l.....l]

@wikary: Dlatego zwracam uwagę ; ).

możesz to jakoś wyjaśnic?

Jasne.
W pierwszej linijce inicjalizujesz zmienną H zerem.
W drugiej rozpoczynamy pętlę i następuje pierwsze sprawdzenie warunku: H <= 0 jest prawdą, bo H jest zerem, więc przechodzimy do wykonania pętli - która ma tylko jedną linijkę, więc darowałem sobie nawiasy.
W trzeciej czytujemy do H to co wpisze użytkownik i wracamy do testu pętli: H <= 0. Jeśli test

[sylwke3100]

@wikary: @losiul: Nie zapominajcie o poprawnym porównywaniu liczb zmiennoprzecinkowych.

[l.....l]

@sylwke3100: A co masz dokładnie na myśli?

[k.....r]

@losiul: że równość dla zmiennopozycyjnych trzeba testować jako abs(x - y) < epsilon

[l.....l]

@kuhar: Zgadzam się, ale tutaj, w wypadku zera, chyba to nie jest problem.

[wikary]

@losiul: dziękuję :)