Einzelnen Beitrag anzeigen
Ungelesen 03.11.11, 01:53   #2
urga
Mitglied
 
Benutzerbild von urga
 
Registriert seit: Aug 2009
Ort: void* (*wtf[])(void **);
Beiträge: 453
Bedankt: 137
urga ist noch neu hier! | 0 Respekt Punkte
Standard

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

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 mwfloat 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
__________________
entropie erfordert keine wartung
urga ist offline   Mit Zitat antworten