Question

我正在制作一个简单的计算器作为我的第一个由我制作的应用程序，我有两个EditTexts如果我按下4个计算按钮中的任何一个应用程序崩溃，我设置了一个Toast消息显示如果EditText1＆amp; ; EditText2为空，如果没有根据按下的按钮进行计算...如何防止它崩溃？这是我的MainActivity.java：

public class MainActivity extends Activity {
    EditText number1text;
    EditText number2text;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        number1text = (EditText) findViewById(R.id.num1text);
        number2text = (EditText) findViewById(R.id.num2text);
    }

    public void calcadd(View v) { // here
        if (number1text.getText().toString().isEmpty() || number2text.getText().toString().isEmpty()) {
            Toast msg = Toast.makeText(getBaseContext(),
            "You Should Put Numbers To Do Calculation",
            Toast.LENGTH_LONG);
            msg.show();
        }
        else{

        }
        Integer num1text = Integer.parseInt(number1text.getText().toString());
        Integer num2text = Integer.parseInt(number2text.getText().toString());
        Integer ans = num1text + num2text;

        TextView answer = (TextView) findViewById(R.id.ans);
        answer.setText("Answer:" + ans.toString());
    }

    public void calcminus(View v) { // here 
        if (number1text.getText().toString().isEmpty() || number2text.getText().toString().isEmpty()) {
            Toast msg = Toast.makeText(getBaseContext(),
            "You Should Put Numbers To Do Calculation",
            Toast.LENGTH_LONG);
            msg.show();
        }
        else{

        }
        Integer num1text = Integer.parseInt(number1text.getText().toString());
        Integer num2text = Integer.parseInt(number2text.getText().toString());
        Integer ans = num1text - num2text;

        TextView answer = (TextView) findViewById(R.id.ans);
        answer.setText("Answer:" + ans.toString());
    }

    public void calcdivide(View v) { // here
        if (number1text.getText().toString().isEmpty() || number2text.getText().toString().isEmpty()) {
            Toast msg = Toast.makeText(getBaseContext(),
            "You Should Put Numbers To Do Calculation",
            Toast.LENGTH_LONG);
            msg.show();
        }
        else{

        }
        Integer num1text = Integer.parseInt(number1text.getText().toString());
        Integer num2text = Integer.parseInt(number2text.getText().toString());
        Integer ans = num1text / num2text;

        TextView answer = (TextView) findViewById(R.id.ans);
        answer.setText("Answer:" + ans.toString());
    }

    public void calcmultiply(View v) { // here
        if (number1text.getText().toString().isEmpty() || number2text.getText().toString().isEmpty()) {
            Toast msg = Toast.makeText(getBaseContext(),
            "You Should Put Numbers To Do Calculation",
            Toast.LENGTH_LONG);
            msg.show();
        }
        else{

        }
        Integer num1text = Integer.parseInt(number1text.getText().toString());
        Integer num2text = Integer.parseInt(number2text.getText().toString());
        Integer ans = num1text * num2text;

        TextView answer = (TextView) findViewById(R.id.ans);
        answer.setText("Answer:" + ans.toString());
     }  




    @Override
    public boolean onCreateOptionsMenu(Menu menu) { // here
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.main, menu);
        return true;
    }
}

这是我的activity_main.xml：

<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:paddingBottom="@dimen/activity_vertical_margin"
    android:paddingLeft="@dimen/activity_horizontal_margin"
    android:paddingRight="@dimen/activity_horizontal_margin"
    android:paddingTop="@dimen/activity_vertical_margin"
    tools:context=".MainActivity" >

    <TextView
        android:id="@+id/num1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignParentLeft="true"
        android:layout_alignParentTop="true"
        android:text="@string/num1"
        android:textAppearance="?android:attr/textAppearanceMedium" 
        />

    <EditText
        android:id="@+id/num1text"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignLeft="@+id/num1"
        android:layout_below="@+id/num1"
        android:ems="10"
        android:inputType="number" 
        >

        <requestFocus />
    </EditText>

    <TextView
        android:id="@+id/num2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignLeft="@+id/num1text"
        android:layout_below="@+id/num1text"
        android:text="@string/num2"
        android:textAppearance="?android:attr/textAppearanceMedium" 
        />

    <EditText
        android:id="@+id/num2text"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignLeft="@+id/num2"
        android:layout_below="@+id/num2"
        android:ems="10"
        android:inputType="number" 
        />

    <TextView
        android:id="@+id/ans"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignLeft="@+id/num2text"
        android:layout_below="@+id/num2text"
        android:layout_marginTop="52dp"
        android:text="@string/anstxt"
        android:textAppearance="?android:attr/textAppearanceMedium" 
        />

    <Button
        android:id="@+id/button1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignLeft="@+id/num2text"
        android:layout_below="@+id/num2text"
        android:text="@string/btnadd" 
        android:onClick="calcadd"/>

    <Button
        android:id="@+id/Button01"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignBaseline="@+id/button1"
        android:layout_alignBottom="@+id/button1"
        android:layout_toRightOf="@+id/button1"
        android:text="@string/btnminus" 
        android:onClick="calcminus"/>

    <Button
        android:id="@+id/button2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignBaseline="@+id/Button01"
        android:layout_alignBottom="@+id/Button01"
        android:layout_toRightOf="@+id/Button01"
        android:text="@string/btnmultiply" 
        android:onClick="calcmultiply"/>

    <Button
        android:id="@+id/Button02"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignBaseline="@+id/button2"
        android:layout_alignBottom="@+id/button2"
        android:layout_toRightOf="@+id/button2"
        android:text="@string/btndivide" 
        android:onClick="calcdivide"/>

</RelativeLayout>

Answer 1

else{

    }
    Integer num1text = Integer.parseInt(number1text.getText().toString());
    Integer num2text = Integer.parseInt(number2text.getText().toString());
    Integer ans = num1text + num2text;

    TextView answer = (TextView) findViewById(R.id.ans);
    answer.setText("Answer:" + ans.toString());

你在else {}语句之外做你的工作，你正在检查它们是否为空是好的，但是如果它们是空的“”，你仍然试图通过在其他括号。你应该改为我在下面显示的内容......

else{
       Integer num1text = Integer.parseInt(number1text.getText().toString());
       Integer num2text = Integer.parseInt(number2text.getText().toString());
       Integer ans = num1text + num2text;

       TextView answer = (TextView) findViewById(R.id.ans);
       answer.setText("Answer:" + ans.toString());

      }

Answer 2

即使这不是您当前的问题

还要确保检查是否没有除以0 你的分工功能：
Integer ans = num1text / num2text;
您可以简化代码，创建一个执行当前操作的函数在每个功能中重复（干燥;不要重复自己）

@ 1：例如在你的除法函数中添加另一个if / else：

    public void calcdivide(View v) { // here
        if (number1text.getText().toString().isEmpty() || number2text.getText().toString().isEmpty()) {
            Toast msg = Toast.makeText(getBaseContext(),
            "You Should Put Numbers To Do Calculation",
            Toast.LENGTH_LONG);
            msg.show();
        }
        else{
            Integer num1text = Integer.parseInt(number1text.getText().toString());
            Integer num2text = Integer.parseInt(number2text.getText().toString());
            if (num2text>0)
            {
             //only do the division if divisor > 0
             Integer ans = num1text / num2text;
             TextView answer = (TextView) findViewById(R.id.ans);
             answer.setText("Answer:" + ans.toString());
            }
            else
            {
             //e.g. show a toastmessage here
            }
        }
    }

Answer 3

在测试EditTexts为空的所有四个if结构中，请更改：

else{

}
Integer num1text = Integer.parseInt(number1text.getText().toString());
Integer num2text = Integer.parseInt(number2text.getText().toString());
Integer ans = num1text + num2text; // +, -, * or / depending

TextView answer = (TextView) findViewById(R.id.ans);
answer.setText("Answer:" + ans.toString());

到此（代码需要在 else块中）：

else{
    Integer num1text = Integer.parseInt(number1text.getText().toString());
    Integer num2text = Integer.parseInt(number2text.getText().toString());
    Integer ans = num1text + num2text;

    TextView answer = (TextView) findViewById(R.id.ans);
    answer.setText("Answer:" + ans.toString());
}

另外，在每个Toast中，将getBaseContext()更改为getApplicationContext()。

当我按下一个按钮并且EditTexts为空时它会崩溃

3 个答案: