Benötige Hilfe wegen Problem

[Maxi.1]

Hallo liebe Community,

Ich habe mal versucht einen BMI-Tester zu programmieren. Jetzt zu meinem Problem:

Hier zunächst der Code:

package bmi_Test;
import javax.swing.*;
public class BMI {

       boolean BMI1 () {	
    	   String eingabeGewicht = JOptionPane.showInputDialog("Geben sie bitte ihr Gewicht ein (in kg).");
    	   int eingegebenesGewicht = Integer.parseInt (eingabeGewicht);
    	   System.out.println("Sie wiegen "  + eingegebenesGewicht +  "kg.");
    	   String eingabeGroesse = JOptionPane.showInputDialog("Geben sie bitte ihre Größe ein (in cm).");
    	   int eingegebeneGroesse = Integer.parseInt (eingabeGroesse);
    	   System.out.println("Und Sie sind "  + eingegebeneGroesse +  "cm groß.");

    	   
    	   double bmi = eingegebenesGewicht/((eingegebeneGroesse/100)*(eingegebeneGroesse/100));
    		if (bmi<20) {
    			System.out.println("Sie sind untergewichtig!");
    			return true;
    			
    		}else if (bmi>=20 && bmi<25){
    		System.out.println("Sie sind normalgewichtig!");
    		    return true;
    		    
    		}else if (bmi>=25)
    		System.out.println("Sie sind übergewichtig!");
    		    return true;
    			
    			   
    	   
    
       }
	
}

Wenn ich nun Gewicht + Größe eingebe, die eigentlich im normalen Bereich liegt, wird trotzdem "übergewichtig" angezeigt ;(

Danke schonmal :applaus:

 

[BRoll]

Gib doch mal den errechneten BMI Wert zum Debug aus.
Dann wirst sehen dass es an dem Variablentyp liegt aus dem er berechnet wird.
Die eingegebene Größe und Gewicht sind Integer, also Ganzzahlen.
Und du willst mit denen ein Double berechnen, also ne Kommazahl.
Vorher die Integer in Double umrechnen dann sollte es gehen,
ansonsten ist deine Formel oder die Abfragen falsch?

 

[Andy17]

Hallo Maxi 1.

Was der BRoll sagt hat Hand und Fuß. Dank dem Integer kommt bei der Berechnung Gewicht / Groesse eine 1 raus. Sodass das Gewicht immer durch 1 dividiert wird und das bei 75 Kg einen BMI von 75 ergibt,.. und dass das übergewichtig ist steht außer Frage.

Richtig wäre es so:

package BMI;

public class bmiRechner
{
    
    static double eingegebenesGewicht = 75;
    static double eingegebeneGroesse = 185;
    
    static void groessenBesorgen() {
        
        /*
         * Hier besorgst du dir deine Groessen.
         * Ich spare mir das an dieser Stelle, soll schließlich nur ein Beispiel sein.
         * */
        
        eingegebeneGroesse = eingegebeneGroesse / 100;
        System.out.println(eingegebeneGroesse);
        
    }
    
    static double bmiBerechnen(double groesse, double gewicht) {
        
        double bmi = eingegebenesGewicht / (eingegebeneGroesse * eingegebeneGroesse);
        
        return bmi;
        
    }
    
    public static void main(String[] args)
    {
        groessenBesorgen();
        System.out.println("Der ermittelte BMI liegt bei: " + bmiBerechnen(eingegebeneGroesse, eingegebenesGewicht));
    }

}

Ich hoffe, das Beispiel hat dir geholfen.
Einfach etwas mehr auf die Typen aufpassen. Gott sei Dank ist Java schließlich eine so streng typisierte Sprache.

--> Java cares about Type.

LG

 

[Maxi.1]

Erstmal danke ihr zwei :toll:

Ich habe es jetzt hinbekommen, habe einfach nur aus dem "int" ein "double" gemacht, wie du es im Beispiel getan hattest und es hat funktioniert!

Kann man sich hier iwie bedanken?

 

[Andy17]

Hau auf den Danke Button.

Ansonsten ist es Freude genug, Dir bei Deinem Problem geholfen zu haben