Obliczenie bmi java

Question

Cześć, dopiero zaczynam progrmowanie z java i mam problem. Chciałem stworzyć program liczący bmi. W ogóle nie w Public void podziel, nie dzieli mi z reszta. Program wyświetla

Wzrost=180
Waga=89
89.0
Wychudzenie

Wklejam niżej kod z góry dziękuje za pomoc

import java.util.Scanner;

public class Kaukulator {
	float wzrost;
	float waga;
	
	

public Kaukulator(){
	wzrost=0;
	waga=0;
}

public Kaukulator(float wzrost, float waga) {
	this.wzrost = wzrost;
	this.waga = waga;

}
public void podaj_dane() {
	System.out.println("Wzrost=");
	Scanner in = new Scanner(System.in);
	wzrost= in.nextInt();
	System.out.println("Waga=");
	waga= in.nextInt();
}

public void podziel() {
	float cel;
	cel = waga % wzrost;
	System.out.println(cel);
	if(cel<16) {
		System.out.println("Wyglodzenie");
	}
	else if(cel>16 || cel<= 17)
	{
		System.out.println("Wychudzenie");
	}
	else if(cel>17 || cel<= 19)
	{ 
		System.out.println("Niedowaga");
	}
	else if(cel>19 || cel<= 25)
	{ 
		System.out.println("Wartosc Prawidlowa");
	}
	else if(cel>25 || cel<= 30)
	{ 
		System.out.println("Nadwaga");
	}
	else if(cel>30 || cel<= 35)
	{ 
		System.out.println("I stopien otylosci");
	}
	else if(cel>35 || cel<= 40)
	{ 
		System.out.println("II stopien otylosci");
	}
	else if (cel>40)
	{ 
		System.out.println("Otylosc skrajna");
	}
	
}


}

public class WynikKaukulator {

	public static void main(String[] args) {
		Kaukulator kaukulator = new Kaukulator();
		kaukulator.podaj_dane();
		kaukulator.podziel();
	}
	

}

 

Answer 1

cel = waga % wzrost;

to nie jest dzielenie, tylko to jest operacja modulo- czyli reszta z dzielenia, zwraca Ci wartość która jest resztą z tego dzielenia stąd cel = 89 % 189 -> reszta z dzielenia to 89... 

Jeśli używasz cm- jako wzrost, a nie metrów - to obliczanie bmi jest takie 
 

double cel = (100*100*waga)/(wzrost*wzrost);

lepiej trzymać to jako double. 

Wzrost= 180
Waga= 89
27.469135284423828
Wychudzenie

Dodatkowo masz operator || alternatywy stąd wypiszę błędne informacje np 
 

 else if(cel>16 || cel<= 17)
        {
            System.out.println("Wychudzenie");
        }

 tutaj w warunku masz że jesli CEL JEST większy niż 16 lub mniejszy badź równy ... to wychudzenie,  zauważ żę ta alternatywa jest zawsze prawdziwa :) dla celu > 16 ... bo wystarczy że jeden warunek bedzie prawdziwy żeby if się wykonał, zamień ten warunek na "i" skoro oba mają zajść jednocześnie powinno być 
 

else if(cel>16  && cel<= 17) // tylko gdy dwa warunki spełnione

analogicznie przeanalizuj reszte 

Answer 2

Dzięki wielkie teraz działa