Berechne alles mit einer Taste

Question

Ich mache eine App, die alles, was ich brauche, mit nur einem Klick auf eine Schaltfläche berechnen kann, aber das erzwungene Schließen, wenn ich die Schaltfläche "Berechnen" drücke, kann mir jemand mit was ich helfen falsch gemacht?

previous = (EditText) findViewById(R.id.editText); 
    present = (EditText) findViewById(R.id.editText2); 

    Button calculate = (Button) findViewById(R.id.calculate); 

    consumption = (TextView) findViewById(R.id.textView1); 
    basic = (TextView) findViewById(R.id.textView2); 
    mmp = (TextView) findViewById(R.id.textView3); 
    stp = (TextView) findViewById(R.id.textView4); 
    before = (TextView) findViewById(R.id.textView5); 
    penalty = (TextView) findViewById(R.id.textView6); 
    after = (TextView) findViewById(R.id.textView7); 

    calculate.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View view) { 
      makeCalculations(); 
     } 
    }); 
} 
private void makeCalculations() { 

    double n1 = Double.valueOf(previous.getText().toString()); 
    double n2 = Double.valueOf(present.getText().toString()); 
    double n3 = Double.valueOf(consumption.getText().toString()); 
    double n4 = Double.valueOf(stp.getText().toString()); 
    double n5 = Double.valueOf(mmp.getText().toString()); 
    double n6 = Double.valueOf(penalty.getText().toString()); 
    double n7 = Double.valueOf(before.getText().toString()); 
    double n8 = Double.valueOf(basic.getText().toString()); 



    consumption.setText("Your Consumption is: " + (n1 + n2)); 
    basic.setText("Your Consumption is: " + (n3 * 280.00)); 
    mmp.setText("Meter Maintenance Fee: " + 10.00); 
    stp.setText("Septage Fee: " + (n3 * 0.068)); 
    before.setText("On/Before Due Date: " + (n3 + n4 + n5)); 
    penalty.setText("Penalty: " + (n3/10.00)); 
    after.setText("After Due Date: " + (n6 + n7)); 

} 

} 

Answer 1

Es gibt keine Validierung. Sie sollten zumindest überprüfen, dass die Zeichenfolge nicht leer ist, aber möglicherweise auch einen Versuch fangen, falls eine Zahl nicht eingegeben wird. Dann könnten Sie den Fehlertext im editText einstellen, um zu erklären, dass eine ungültige Nummer eingegeben wurde.

So könnte man so etwas für jedes Doppel tun:

boolean answerOk = true; 

double n1; 
if (!previous.getText().toString().equals("") { 
    try { 
     n1 = Double.valueOf(previous.getText().toString()); 
    } catch(NumberFormatException e) { 
     answerOk = false; 
     previous.setError("Invalid number"); 
    } 
} else { 
    answerOk = false; 
    previous.setError("Field is empty"); 
} 

Nur den Code unten ausgeführt, nachdem alle Doppel ist in Ordnung sind.

if (answerOk) { 
    // Do the other stuff here 
}