Von Bis Befehl

elomaniak

Hi Leute
Ich habe die Aufgabe bekommen ein prog namens Eliza zu programmieren.

so weit so gut
ging alles gut
im Programm soll der BMI einer Person, welche Ihre Daten eingibt, errechnet werden.

sieht bei mir so aus

Code

if (BMI < 20)
aus.println("Du solltest zunehmen.");
else if (BMI > 25)
aus.println("Du solltest unbedingt Sport treiben.");
else
aus.println("Du bist fit.");

nur gibt er mir bei einem BMI von über 20 raus die Person solle Sport machen, also als ob sie einen BMI von über 25 hätte.

Wie lautet die Bedingung damit ich für die Werte zwischen 20 und 25 raubekomme, dass die Person fit sei?

Danke im Vorraus
Mfg
Elomaniak

Ringding

So wie es dasteht, macht es genau das, was es soll.

Christoph R.

Schreib das Programm nochmal mit vernünftiger Klammersetzung ({, }) und Einrückung, dann wird es funktionieren

elomaniak

Zitat von Christoph R.

Schreib das Programm nochmal mit vernünftiger Klammersetzung ({, }) und Einrückung, dann wird es funktionieren

so sieht mein code aus

Java

import java.util.Scanner;
import java.io.PrintStream;
import java.math.*;


public class Eliza
{



    public static void main(String[] args){
        for (int i=0; i<25; i++){ System.out.println(); }


        PrintStream aus = System.out;
        Scanner eingabe = new Scanner(System.in);

        aus.print("Hallo, ich bin Eliza. ");
        aus.println("Ich wurde programmiert um mit Dir ein Gespräch zu führen.");
        aus.println("Wie heisst Du?");
        String name = eingabe.next();
        aus.println("Hallo " + name );
        aus.println("Wie alt bist Du?");
        int alter = eingabe.nextInt();
        aus.println("Wieviel kg wiegst Du?");
        int gewicht = eingabe.nextInt();
        aus.println("Wie gross bist Du? (Bitte in cm angeben)");
        int groesse = eingabe.nextInt();
        aus.println("Dein BMI beträgt: " + ((gewicht) / ((groesse / 100.00)*(groesse / 100.00))));
        int BMI = (gewicht) / ((groesse / 100)*(groesse / 100));
        if (BMI < 20)
        aus.println("Du solltest zunehmen.");
        else if (BMI > 25)
        aus.println("Du solltest unbedingt Sport treiben.");
        else
        aus.println("Du bist fit.");

        System.exit(0);





    }


}

Alles anzeigen

weiss jetzt auch nicht wo der wurm drin ist

Ringding

Abgesehen davon, dass die BMI-Berechnung soundso falsch ist, verwendest du Ganzzahlendivision, die nicht das Ergebnis liefert, das du dir erwartest. Wenn du dir einfach den Wert der Variablen BMI ausgeben würdest, dann könntest du auch erkennen, dass die ifs nicht dein Problem sind.

Plantschkuh!

Zitat von elomaniak

for (int i=0; i<25; i++){ System.out.println(); }

Das ist jetzt nicht das funktionale Problem, das du suchst, aber... pfui! Darf ich bitte selber bestimmen, was auf meinem Bildschirm steht? Vielleicht brauche ich es ja noch? (Im Uebrigen haben meine Terminals 40 Zeilen.)

java-girl

Zitat von elomaniak
Code
System.exit(0);

???

elomaniak

Zitat von Ringding

Abgesehen davon, dass die BMI-Berechnung soundso falsch ist, verwendest du Ganzzahlendivision, die nicht das Ergebnis liefert, das du dir erwartest. Wenn du dir einfach den Wert der Variablen BMI ausgeben würdest, dann könntest du auch erkennen, dass die ifs nicht dein Problem sind.

wenn du wissen würdest dass die rechnung für den BMI Gewicht/(Groesse^2) ist würdest du hier nicht posten

und ich muss die grösse in cm eingeben

Ringding

Wenn du wüsstest, dass du hier bist, weil du Hilfe brauchst, würdest du Leute, die dich auf dein Problem hinweisen, nicht anmachen.

Ich hab gedacht, da muss man 100 von der Größe abziehen. Stimmt aber anscheinend nicht. Aber v.a. hab ich mich verlesen. Sorry.

Trotzdem verwendest du Ganzzahlendivisionen und handelst dir dadurch Probleme ein. Nachdem Größe häufig zwischen 100 und 200 liegen wird, ist "groesse/100" in diesen Fällen immer 1. Das Quadrat davon auch. Wie ich gerade sehe, hast du's eine Zeile weiter oben sogar richtig gemacht...

elomaniak

ok sorry für meine antwort, deine erste war aber auch nicht ganz die feine englische art

also als ergebis für den BMI kommt ein korrekter Wert raus.
die Frage ist nur wie ich es hinbekomme dass er für einen BMI von 20 bis 25 Als Antwort "Du bist fit" rausgibt.

java-girl

Zitat von elomaniak

also als ergebis für den BMI kommt ein korrekter Wert raus.
die Frage ist nur wie ich es hinbekomme dass er für einen BMI von 20 bis 25 Als Antwort "Du bist fit" rausgibt.

ähm, hallo? Nachdem deine if/else if-Kaskade korrekt ist, wie bereits mehrfach festgestellt wurde, muss beim BMI etwas falsches rauskommen. Woher willst du überhaupt wissen, dass der Wert richtig ist?

davide

soll heißen, schreib mal vor den if abfragen ein "aus.out.println("Berechneter BMI: " +BMI);" und vergleich den wert mit der vorigen ausgabe

(ich glaub nämlich nicht dass ich einen bmi von 94 haben sollte die 27 kommen da schon eher hin )

elomaniak

ok jetzt hab ich es verstanden

wenn ich die formel von oben übernehme in die int BMI dann bekomme ich die Meldung "possible loss of precision" als meldung in der Zeile mit der Rechnung

wie komme ich dennoch ans Ergebnis??

Ich danke euch schon mal vielmals für eure Hilfestellungen

EDIT: habs wegbekommen, der Fehler ist beseitigt, hab in den If Anfweisungen einfach das BMI durch die Gleichung ersetzt und es geht

Nochmals Danke für eure Hilfe

davide

Zitat von elomaniak

wenn ich die formel von oben übernehme in die int BMI dann bekomme ich die Meldung "possible loss of precision" als meldung in der Zeile mit der Rechnung

int kann ja nur ganze zahlen speichern, dh da würde gerundet werden, und das geht in dem fall nicht gut aus

mit
double BMI hätts vermutlich geklappt

java-girl

Zitat von elomaniak

habs wegbekommen, der Fehler ist beseitigt, hab in den If Anfweisungen einfach das BMI durch die Gleichung ersetzt und es geht

Nochmals Danke für eure Hilfe

Dir sollte aber klar sein, dass das sehr ineffizient und unschön ist. Weiters solltest du dich besser mit Fließkommazahlen bzw. Datentypen generell beschäftigen.
Und die bereits beanstandeten Sachen würde ich auch mal überarbeiten.

Jetzt mitmachen!