[Sanju2]

Hey Leute,

ich bin zur zeit dran mit C+ das Programmieren zu lernen...
Ich habe versucht mir einen BMI Rechner zu programmieren und bin dabei auf folgendes Problem gestoßen...

Meine eigentlich Simple Formel berechnet nicht den erwartenden Wert...

Formel: (10000*Gewicht)/(Größe*Größe) // Gewicht und Größe sind Variablen

z.b. gebe ich Werte ein für; Gewicht= 80 und Größe = 180

Mit dem Taschenrechner komme ich auf das Ergebnis 24,7 aber mein Programm rechnet 281,25 aus!

Ich könnte mir vorstellen, dass das Problem wegen der Division der float-variablen zustande kommt... Ich habe diesbezüglich auch schon etwas recherchiert... Leider ohne Erfolg.

Ich hoffe jemand kann mir helfen. Quellcode ist im Spoiler.


Mfg Sanju

# include <stdio.h>

float gewicht, groese;
float bmi (float gewicht, float groese);
int geschlecht;
float ergebnis;


char text1[]="Body Mass Index Rechner\n\n";
char text2[]="Ihr Gewicht in kg:";
char text3[]="Ihre Gr\x94se in cm:";
char text4[]="Ihr Geschlecht (0=Frau, 1=Mann):";

int main (void)
{
printf(text1);
printf(text2); scanf_s ("%f", &gewicht);
printf(text3); scanf_s ("%f", &groese);
printf(text4); scanf_s ("%d", &geschlecht);

ergebnis= bmi (groese, gewicht);
printf("\nIhr BMI betraegt: %f", ergebnis);

if (geschlecht == 0 && ergebnis<19)
{
printf("\nBMI-Gruppe: Untergewicht!");
}
else
{
if (geschlecht == 0 && ergebnis>=19 && ergebnis<=24)
{
printf("\nBMI-Gruppe: Normalgewicht!");
}
else
{
if (geschlecht == 0 && ergebnis>=25 && ergebnis<=30)
{
printf("\nBMI-Gruppe: Uebergewicht!");
}
else
{
if (geschlecht == 0 && ergebnis>30)
{
printf("\nBMI-Gruppe: Behandlungsbeduerftiges Uebergewicht!");
}
else
{
if (geschlecht == 1 && ergebnis<20)
{
printf("\nBMI-Gruppe: Untergewicht!");
}
else
{
if (geschlecht == 1 && ergebnis>=20 && ergebnis<=25)
{
printf("\nBMI-Gruppe: Normalgewicht!");
}
else
{
if (geschlecht == 1 && ergebnis>=26 && ergebnis<=30)
{
printf("\nBMI-Gruppe: Uebergewicht!");
}
else
{
if (geschlecht == 1 && ergebnis>30)
{
printf("\nBMI-Gruppe: Behandlungsbeduerftiges Uebergewicht!");
}

}
}
}
}
}
}
}

return 0;
}


float bmi (float gewicht, float groese)
{
return ((10000*gewicht)/(groese*groese));
}

[urga]

zum einen lassen sich im spoiler auch code bzw. php-tags verschachteln:

PHP-Code:

# include <stdio.h>

float bmi (float gewicht, float groese) {
    return ((10000*gewicht)/(groese*groese));
}

int main (void)
{
    float gewicht, groese;
    int geschlecht;
    float ergebnis;
    char text1[]="Body Mass Index Rechner\n\n";
    char text2[]="Ihr Gewicht in kg:";
    char text3[]="Ihre Gr\x94se in cm:";
    char text4[]="Ihr Geschlecht (0=Frau, 1=Mann):";


    printf(text1);
    printf(text2); scanf_s ("%f", &gewicht);
    printf(text3); scanf_s ("%f", &groese);
    printf(text4); scanf_s ("%d", &geschlecht);

    ergebnis= bmi (groese, gewicht);
    printf("\nIhr BMI betraegt: %f", ergebnis);

    if (geschlecht == 0 && ergebnis<19)
    {
        printf("\nBMI-Gruppe: Untergewicht!");
    }
    else
    {
        if (geschlecht == 0 && ergebnis>=19 && ergebnis<=24)
        {
            printf("\nBMI-Gruppe: Normalgewicht!");
        }
        else
        {
            if (geschlecht == 0 && ergebnis>=25 && ergebnis<=30)
            {
                printf("\nBMI-Gruppe: Uebergewicht!");
            }
            else
            {
                if (geschlecht == 0 && ergebnis>30)
                {
                    printf("\nBMI-Gruppe: Behandlungsbeduerftiges Uebergewicht!");
                }
                else
                {
                    if (geschlecht == 1 && ergebnis<20)
                    {
                        printf("\nBMI-Gruppe: Untergewicht!");
                    }
                    else
                    {
                        if (geschlecht == 1 && ergebnis>=20 && ergebnis<=25)
                        {
                            printf("\nBMI-Gruppe: Normalgewicht!");
                        }
                        else
                        {
                            if (geschlecht == 1 && ergebnis>=26 && ergebnis<=30)
                            {
                                printf("\nBMI-Gruppe: Uebergewicht!");
                            }
                            else
                            {
                                if (geschlecht == 1 && ergebnis>30)
                                {
                                    printf("\nBMI-Gruppe: Behandlungsbeduerftiges Uebergewicht!");
                                }

                            }
                        }
                    }
                }
            }
        }
    }

    return 0;
} 


das fördert ungemein die lesbarkeit deines prgrammcodes. beachte, ich habe (u.a.) deine globalen variablen als lokale variablen von main() deklariert. gewöhne dir bitte so früh wie möglich ab globale variablen zu verwenden!

zum andern ist dein code standard C. kein c++ sprachmerkmal für mich erkennbar. aber egal, c++ ist ja abwärtskompatibel.

da mein cygwin-gcc gerade buggy ist, und ich jetzt keinen bock habe deinetwegen linux zu starten, allgemeine ratschläge:

a) per cout (c++) oder eben printf in bmi() die eingabe-parameter ausgeben, um zu schaun, ob auch die erwarteten parameter an die funktion übergeben werden.
b) die ausgabe der bmi-auswertung auch in eine funktion auslagern, und da per switch
m/w zu unsterscheiden. so brauchst du kein einziges else. 9-fach geschachtelte if-cascaden sind sehr schwer les- bzw. wartbar.

PHP-Code:

void auswertung (int mw, float bmi) {
 switch (mw) {
  case 0:
   if (bmi < 19) {
     cout << "\nBMI-Gruppe: Untergewicht!"; 
   }
   if (bmi >= 19 && bmi <=24) { 
     cout << "\nBMI-Gruppe: Normalgewicht!";
   }
   .....
   break;
  case 1:
   .....
   break;
  default:
    cout << "aliens unerwünscht!\n";
    exit (-1);
 }
} 


viel erfolg!
und nicht vergessen: globale variablen sind baba

[HappyMike34]

Du hast beim Aufruf von bmi() Größe und Gewicht vertauscht!

[Sanju2]

Du hast beim Aufruf von bmi() Größe und Gewicht vertauscht!

Danke