kalkulator bmi

Question

Witam, robie kalkulator bmi a tu nagle problem, za kazdym razem otrzumuje wynik zero oto kod:

 

#include <iostream>

using namespace std;
int waga, wzrost, masa, bmi;
int main()
{
    cout << "Oblicz swoje bmi"<< endl;
	cout<<"Podaj swoj wzrost (cm): ";
	cin>>wzrost;
	cout<<"Podaj swoja wage (kg): ";
	cin>>waga;
	masa=waga/100;
	bmi= masa/(wzrost*wzrost);
	
    cout<<bmi;
    
    return 0;
}

nie mam pojecia w czym blad.

Comments

bmi nie może być liczbą całkowitą i po za tym to po co liczysz tą masę skoro jeśli dobrze sprawdziłem to wzór wygląda tak: waga/(wzrost*wzrost)

---

#include <iostream>

using namespace std;
int waga, wzrost, wzrost1;
float bmi;
int main()
{
    cout << "Oblicz swoje bmi"<< endl;
	cout<<"Podaj swoj wzrost (cm): ";
	cin>>wzrost;
	cout<<"Podaj swoja wage (kg): ";
	cin>>waga;
	wzrost1=wzrost/100;
	bmi= waga/(wzrost1*wzrost1);

    cout<<bmi;

    return 0;
}

rzeczywiscie moj błąd, z tym ze gdy zmienilem typ zmiennej na float rownież pokazuje 0.
Z ta masa to pomyłka, ale zamiast masy zrobilem wzrost1 a to z tego powody że użytkownik podaje wzrost w centymetrach a we wzorze są metry.

 

---

Chciałbym zaznaczyć że kod który podałem również nie działa. Przy Podaniu wzrostu w cm: 188 i wagi 88kg oblicza bmi na 88 xD

Answer 1

Dałem rade!

 

#include <iostream>

using namespace std;
int waga, wzrost;
float bmi, wzrost1;
int main()
{
    cout << "Oblicz swoje bmi"<< endl;
	cout<<"Podaj swoj wzrost (cm): ";
	cin>>wzrost;
	cout<<"Podaj swoja wage (kg): ";
	cin>>waga;
	wzrost1=wzrost*0.01;
	bmi= waga/(wzrost1*wzrost1);

    cout<<bmi;

    return 0;
}

 

Comments

Nie jestem ekspertem, ale często można spotkać się z poradą, żeby nie używać zmiennych globalnych. Wagę, wzorst, bmi, wzrost1 możesz jak najbardziej zadeklarować wewnątrzn main. W przypadku tego programu nie widać korzyści tego działania, ale unikanie zmiennych globalnych może się okazać koniecznością w przypadku większych projektów. Myślę, że warto stosować taką praktykę nawet na początku nauki. Przyznam szczerze, że sam nie zawsze słuchałem tego typu zaleceń i nadal czasami tego nie robię, a efekt może być taki, że w kodzie później łatwiej się pogubić, albo jest tak napisany, że tylko dla autora jest czytelny (być może niedługo opublikuję tu pewien mały projekcik i spodziewam się, że jeśli ktoś przeczyta kod to nie będzie zadowolony z jakości kodu).

Answer 2

Uruchom sobie ten kod:

std::cout << (5/2) << std::endl;

Answer 3

Zmienna bmi musi być typu double,gdyż nie jest ona liczbą całkowitą i nie potrzebnie pytasz o masę bo wzór wygląda inaczej : masa w kg/wzrost^2.Usuń zmienną masa i linijkę gdzie przypisujesz jej wartość oraz zastąp typ zmiennej bmi z int na double i wszystko powinno działać ;).

Answer 4

Też  niedawno robiłem swój kalkulator bmi, haha.

Comments

to moze pomożesz ;]