BMI-Berechnung, Ausgabe mit Alertdialog

Question

Ich habe ein Problem mit meinem aktuellen Quellcode. Ich bin damit ziemlich neu. Es geht um die BMI-Berechnung und zeigt die Ausgabe mithilfe von alertdialog an. Ich habe die Bedingung bereits gestellt, aber die Ausgabe ist immer "Sie sind übergewichtig" auf sonst Bedingung, obwohl meine Eingabe legitim sein sollte IDEAL Ausgabe.

Die Aktivität hat keinen Fehler beim Laufen, aber ich habe das Gefühl, dass es das Berechnungsproblem ist, das ich nicht kenne. Es tut mir wirklich leid, wenn das eine doppelte Frage ist.

public class BMIcalcu extends Activity{ 
    EditText weight, height; 
    final Context context = this; 
    Button calculate; 
    float result, height1, weight1; 

    @Override 
    protected void onCreate(Bundle savedInstanceState){ 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.bmi_calcu); 

     height = (EditText)findViewById(R.id.editText1); 
     weight = (EditText)findViewById(R.id.editText2); 
     calculate = (Button)findViewbyId(R.id.btn_bmi); 

     String str1 = height.getText().toString(); 
     String str2 = weight.getText().toString(); 

     try{ 
     height1 = Float.parseFloat(str1)/100; 
     weight1 = Float.parseFloat(str2); 
     }catch(NumberFormatException nfe){ 
      } 

     calculate.setOnClickListener(new View.OnClickListener(){ 

            @Override 
            public void onClick(View v){ 
     result = weight1/(height1*height1); 
     if(result<18.5){ 

      AlertDialog.Builder alertDialogBuilder = new AlertDialog.Builder(context); 

      alertDialogBuilder.setTitle("Your BMI"); 

      alertDialogBuilder 
        .setMessage("You are UNDERWEIGHT! START EATING!") 
        .setCancelable(false) 
        .setPositiveButton("List", new DialogInterface.OnClickLister(){ 
             public void onClick(DialogInterface dialog, int id){ 
      Intent intent = new Intent(BMIcalcu.this, list.class); 
      startActivity(intent); 
      } 
      }) 
        .setNegativeButton("Back", new DialogInterface.OnClickListener(){ 
             public void onClick(DialogInterface dialog, int id){ 

         dialog.cancel(); 
         } 
     }); 
     AlertDialog alertDialog = alertDialogBuilder.create(); 
     alertDialog.show(); 

     } 
else if(result<25){ 

      AlertDialog.Builder alertDialogBuilder = new AlertDialog.Builder(context); 

      alertDialogBuilder.setTitle("Your BMI"); 

      alertDialogBuilder 
        .setMessage("You are on your IDEAL!") 
        .setCancelable(false) 
        .setPositiveButton("List", new DialogInterface.OnClickLister(){ 
             public void onClick(DialogInterface dialog, int id){ 
      Intent intent = new Intent(BMIcalcu.this, list.class); 
      startActivity(intent); 
      } 
      }) 
        .setNegativeButton("Back", new DialogInterface.OnClickListener(){ 
             public void onClick(DialogInterface dialog, int id){ 

         dialog.cancel(); 
         } 
     }); 
     AlertDialog alertDialog = alertDialogBuilder.create(); 
     alertDialog.show(); 
} 
else{ 


      AlertDialog.Builder alertDialogBuilder = new AlertDialog.Builder(context); 

      alertDialogBuilder.setTitle("Your BMI"); 

      alertDialogBuilder 
        .setMessage("You are OVERWEIGHT! START EXERCISING!") 
        .setCancelable(false) 
        .setPositiveButton("List", new DialogInterface.OnClickLister(){ 
             public void onClick(DialogInterface dialog, int id){ 
      Intent intent = new Intent(BMIcalcu.this, list.class); 
      startActivity(intent); 
      } 
      }) 
        .setNegativeButton("Back", new DialogInterface.OnClickListener(){ 
             public void onClick(DialogInterface dialog, int id){ 

         dialog.cancel(); 
         } 
     }); 
     AlertDialog alertDialog = alertDialogBuilder.create(); 
     alertDialog.show(); 
} 
} 
}); 
} 
} 

Dies ist die XML-

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android" 
android:orientation="vertical" android:layout_width="match_parent" 
android:layout_height="match_parent"> 
<EditText 
    android:layout_gravity="center" 
    android:layout_width="266dp" 
    android:layout_height="wrap_content" 
    android:inputType="text|number" 
    android:hint="Height (Centimeter)" 
    android:ems="10" 
    android:id="@+id/editText1" /> 

<EditText 
    android:layout_gravity="center" 
    android:layout_width="266dp" 
    android:layout_height="wrap_content" 
    android:inputType="text|number" 
    android:hint="Weight (Kilogram)" 
    android:ems="10" 
    android:id="@+id/editText2"/> 

<Button 
    android:id="@+id/btn_bmi" 
    android:layout_width="wrap_content" 
    android:layout_height="wrap_content" 
    android:layout_gravity="center" 
    android:layout_below="@+id/editText2" 
    android:text="Calculate"/> 
</LinearLayout> 

Answer 1

Verschieben Höhe und Gewicht Zuordnung innerhalb onClick().

calculate.setOnClickListener(new View.OnClickListener(){ 

     @Override 
     public void onClick(View v){ 
    String str1 = height.getText().toString(); 
    String str2 = weight.getText().toString(); 

    try{ 
    height1 = Float.parseFloat(str1)/100; 
    weight1 = Float.parseFloat(str2); 
    }catch(NumberFormatException nfe){ 
     } 

      result = weight1/(height1*height1); 
      String msg = ""; 
      if(result<18.5){ 
       msg = "You are UNDERWEIGHT! START EATING!"; 
      } 
      else if(result<25){ 
       msg = "You are on your IDEAL!"; 
      } 
      else{ 
       msg = "You are UNDERWEIGHT! START EATING!"; 
      } 

      AlertDialog.Builder alertDialogBuilder = new AlertDialog.Builder(context); 

      alertDialogBuilder.setTitle("Your BMI"); 

      alertDialogBuilder 
        .setMessage(msg) 
        .setCancelable(false) 
        .setPositiveButton("List", new DialogInterface.OnClickLister(){ 
         public void onClick(DialogInterface dialog, int id){ 
          Intent intent = new Intent(BMIcalcu.this, list.class); 
          startActivity(intent); 
         } 
        }) 
        .setNegativeButton("Back", new DialogInterface.OnClickListener(){ 
         public void onClick(DialogInterface dialog, int id){ 

          dialog.cancel(); 
         } 
        }); 
      AlertDialog alertDialog = alertDialogBuilder.create(); 
      alertDialog.show(); 
     } 
    }); 

Answer 2

Sie Einstellung zur Zeit die Werte von weight und height innerhalb des onCreate(). onCreate wird aufgerufen, wenn die Activity zuerst erstellt wird. onCreate wird nur einmal im Aktivitätslebenszyklus aufgerufen. Weitere Informationen zu Aktivitäten finden Sie hier: https://developer.android.com/guide/components/activities.html.

Um Ihre Frage zu beantworten, sollten Sie die Werte weight und height in onCreate nur einstellen, wenn Sie Standardwerte festlegen. Der Versuch, bewegen Sie sie in Ihre onClick Methode:

calculate.setOnClickListener(new View.OnClickListener(){ 

    @Override 
    public void onClick(View v){ 
     String str1 = height.getText().toString(); 
     String str2 = weight.getText().toString(); 

     try { 
      height1 = Float.parseFloat(str1)/100; 
      weight1 = Float.parseFloat(str2); 
      result = weight1/(height1*height1); 
     } 
     catch(NumberFormatException nfe) { 

     } 
     // Continue Alert Dialog stuff down here... 

Warum sind Sie mit diesem Problem?

Derzeit wird Activity gestartet und onCreate aufgerufen. Sie erstellen Verweise auf Ihre XML-Datei und versuchen dann, den Wert weight und height festzulegen. Wenn jedoch onCreate aufgerufen wird, hat der Benutzer noch nichts in die EditText für weight von height eingegeben. Angenommen, der Benutzer gibt Werte ein. Diese Werte werden nirgendwo aufgezeichnet. Dann, wenn der Benutzer auf Button klickt, um ihren BMI zu sehen, versuchen Sie, das Ergebnis zu berechnen. Das Ergebnis hat nichts darin also dein else wird immer aufgerufen. Aus diesem Grund sehen Sie immer die Ausgabe, die Sie machen.