Problem z bmi - polski Spoj.

Question

Robie zadanie na polskim spoju: https://pl.spoj.com/problems/FR_02_06/. Z testów wynika, że wszystkie osoby wrzuca mi do nadwagi i musi być chyba jakiś problem z liczeniem tego bmi, ale go nie widzę

#include <iostream>
#include<vector>
using namespace std;

int main()
{
	string imie;
	unsigned int t, masa, wzrost;
	double bmi;

	vector<string> niedowaga;
	vector<string> waga_prawidlowa;
	vector<string> nadwaga;

	cin >> t;
	while (t--)
	{
		cin >> imie >> masa >> wzrost;
		bmi = masa / ((wzrost/100) * (wzrost/100));
		if (bmi < 18.5) niedowaga.push_back(imie);
		else if (bmi >= 18.5 and bmi < 25) waga_prawidlowa.push_back(imie);
		else if (bmi >= 25) nadwaga.push_back(imie);
	}
	cout << "niedowaga\n";
	for (int i = 0; i < niedowaga.size(); i++) cout << niedowaga[i] << endl;
	cout << "waga prawidlowa\n";
	for (int i = 0; i < waga_prawidlowa.size(); i++) cout << waga_prawidlowa[i] << endl;
	cout << "nadwaga\n";
	for (int i = 0; i < nadwaga.size(); i++) cout << nadwaga[i] << endl;
}

 

Answer 1

Hej błąd jest w tym, że wzrost i masa są typu unsigned int - zamień je na typ double jak to zrobiłeś z bmi - bo nie wystarczy aby zmienna do której przypisujesz wynik była typu zmiennoprzecinkowego - bo przy obliczeniach są int'y i "ucinają" one wynik

Comments

Wydawało mi się, że jeśli to do double wpisujemy iloraz int'ów to nie ma znaczenia, czy będą one int czy double skoro na wejściu są zawsze naturalne, to można dać unsigned int. Jednak poprawiłem na double i jest AC. Mógłbyś wyjaśnić mi dlaczego tak jest, bo chyba coś nie kumam :/ (btw wypisywałem słowo "waga" zamiast "wartosc", co tez byloby problemem).

---

bmi = masa / ((wzrost/100) * (wzrost/100));

Masa i wzrost to jak pisałeś liczby całkowite - gdy np. wzrost będzie 180 to nie wyjdzie 1.8, tylko 1 - bo wzrost to int i ucina wszystko co jest po przecinku. To nawet logiczne, że dla wszystkich wychodziła nadwaga, bo koniec końców było masa / 1

---

Aaa to dlatego wszyscy mieli nadwagę ;D Dzięki wielkie !