Приложение вылетает каждый раз, когда я нажимаю кнопку результата

avatar
Tin Tin
17 июня 2018 в 18:06
89
2
1

E/AndroidRuntime: НЕИСПРАВНОЕ ИСКЛЮЧЕНИЕ: main Процесс: com.vineet.calculator, PID: 29435 java.lang.NullPointerException: попытка вызвать виртуальный метод 'boolean java.lang.Boolean.enter code herebooleanValue()' для нулевой ссылки на объект в com.vineet.calculator.MainActivity$16.onClick(MainActivity.java:177) в android.view.View.performClick(View.java:6294) в android.view.View$PerformClick.run(View.java:24770) в android.os.Handler.handleCallback(Handler.java:790) в android.os.Handler.dispatchMessage(Handler.java:99) на android.os.Looper.loop(Looper.java:164) в android.app.ActivityThread.main(ActivityThread.java:6494) в java.lang.reflect.Method.invoke (собственный метод) в com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438) на com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807) Приложение прекращено.

**Activity.java**

package com.vineet.calculator;

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.Button;
import android.widget.EditText;
import android.widget.TextView;

public class MainActivity extends AppCompatActivity {


    Button btn0;
    Button btn1;
    Button btn2;
    Button btn3;
    Button btn4;
    Button btn5;
    Button btn6;
    Button btn7;
    Button btn8;
    Button btn9;
    Button btnResult;
    Button btnClear;
    Button btnExit;

    Button btnAdd;
    Button btnSub;
    Button btnMul;
    Button btnDiv;

    EditText etNumber;
    TextView tvAnswer;

    int val1, val2;
    Boolean boolMul, boolAdd, boolSub, boolDiv;



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

        btn0 = findViewById(R.id.btn0);
        btn1 = findViewById(R.id.btn1);
        btn2 = findViewById(R.id.btn2);
        btn3 = findViewById(R.id.btn3);
        btn4 = findViewById(R.id.btn4);
        btn5 = findViewById(R.id.btn5);
        btn6 = findViewById(R.id.btn6);
        btn7 = findViewById(R.id.btn7);
        btn8 = findViewById(R.id.btn8);
        btn9 = findViewById(R.id.btn9);
        btnAdd = findViewById(R.id.btnAdd);
        btnSub = findViewById(R.id.btnSub);
        btnMul = findViewById(R.id.btnMul);
        btnDiv = findViewById(R.id.btnDiv);
        btnResult = findViewById(R.id.btnResult);
        btnClear = findViewById(R.id.btnClear);
        btnExit = findViewById(R.id.btnExit);
        etNumber = findViewById(R.id.etNumber);
        tvAnswer = findViewById(R.id.tvAnswer);


        btn0.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"0");
            }
        });
        btn1.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"1");
            }
        });
        btn2.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"2");
            }
        });
        btn3.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"3");
            }
        });
        btn4.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"4");
            }
        });
        btn5.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"5");
            }
        });
        btn6.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"6");
            }
        });
        btn7.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"7");
            }
        });
        btn8.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"8");
            }
        });
        btn9.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText(etNumber.getText()+"9");
            }
        });
        btnClear.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                etNumber.setText("");
                tvAnswer.setText("");
            }
        });


        btnAdd.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                val1 = Integer.valueOf(etNumber.getText().toString());
                boolAdd = true;
                etNumber.setText(null);
            }
        });

        btnSub.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                val1 = Integer.valueOf(etNumber.getText().toString());
                boolSub = true;
                etNumber.setText(null);
            }
        });

        btnMul.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                val1 = Integer.valueOf(etNumber.getText().toString());
                boolMul = true;
                etNumber.setText("");
            }
        });

        btnDiv.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                val1 = Integer.valueOf(etNumber.getText().toString());
                boolDiv = true;
                etNumber.setText("");
            }
        });

        btnResult.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                val2 = Integer.valueOf(etNumber.getText().toString());

                if(boolAdd == true){
                    tvAnswer.setText(val1 + val2+"");
                    boolAdd = false;
                }
                if(boolSub == true){
                    tvAnswer.setText(val1 - val2 + "");
                    boolSub = false;
                }

            }
        });

    }
}
Источник
Vladyslav Matviienko
17 июня 2018 в 18:43
2

почему if(boolAdd == true)? Почему не if(boolAdd)?

Ответы (2)

avatar
Dhiraj Sharma
17 июня 2018 в 18:13
5

Вы используете Boolean(не примитивный тип данных) вместо boolean.

Вы должны инициализировать логическое значение для использования :

Boolean boolMul = new Boolean(false);

Аналогичным образом инициализируйте все ваши логические переменные или используйте логические значения.

avatar
rmdroid
18 июня 2018 в 11:36
1

Используйте "boolean" вместо "boolean". Boolean — это класс, и он оборачивает значение логического примитивного типа в объект. Избегайте его использования, так как он занимает больше памяти по сравнению с примитивным типом.

Кроме того, вместо того, чтобы писать слишком много setOnclickListeners, реализуйте View.OnClickListener в MainActivity и используйте регистр Switch для обработки нажатий кнопок.