Обычный калькулятор

[vlad13reg]

я новичок в android , в принципе как и в программировании.
Дали задание написать калькулятор. У меня не срабатывает кнопка "=", при нажатии на приложение останавливается и вылетает...

https://jsfiddle.net/kqxne9b6/296/ - xml код
И сам код

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

import butterknife.BindView;
import butterknife.ButterKnife;


public class MainActivity extends AppCompatActivity {
    private String display = "";
    private String currentOperator = "";
    @BindView(R.id.buttonEnter)
    TextView buttonEnter;


    @BindView(R.id.textResult)
    TextView textResult;


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

        textResult.setText(display);


    }

    private void updateScreen() {
        textResult.setText(display);
    }

    public void onClickNumber(View view) {
        Button b = (Button) view;
        display += b.getText();
        updateScreen();
    }

    public void onClickOperator(View view) {
        Button b = (Button) view;
        display += ((Button) view).getText();
        currentOperator = b.getText().toString();
        updateScreen();
    }

    protected void clear() {
        display = "";
        currentOperator = "";
    }

    public void onClickClear(View view) {
        clear();
        updateScreen();
    }

    private double operate(String a, String b) {
        double result = 0;
        switch (currentOperator) {
            case "+":
                result = Double.valueOf(a) + Double.valueOf(b);
                break;
            case "-":
                result = Double.valueOf(a) - Double.valueOf(b);
                break;
            case "*":
                result = Double.valueOf(a) * Double.valueOf(b);
                break;
            case "/":
                try {
                    result = Double.valueOf(a) / Double.valueOf(b);
                } catch (Exception e) {
                    System.out.println("Деление на 0");
                }
                break;
        }
        return result;
    }

    public void getResult(View view) {
        Button b = (Button) view;
        double result = 0;
        String[] split = display.split(currentOperator);
        result = operate(split[0], split[1]);
        b.setText(String.valueOf(result));
        textResult.setText(display + "\n" + String.valueOf(result));
    }
}

[pompiduskus]

лучше так не делать. Это плохая практива, когда вешается событие их XML.

Вот так будет лучше. (просто пример)

import android.view.View.OnClickListener;

public class MClass extends ActionBarActivity {

    Button my_btn_a;
    Button my_btn_b;
    Button my_btn_c;
    Button my_btn_d;

    @Override
    public void onCreate(Bundle savedInstanceState) {

        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        my_btn_a = (Button) findViewById( R.id.my_btn_a );
        my_btn_b = (Button) findViewById( R.id.my_btn_b );
        my_btn_c = (Button) findViewById( R.id.my_btn_c );
        my_btn_d = (Button) findViewById( R.id.my_btn_d );

        OnClickListener mOnClickListener = new OnClickListener() {
            @Override
            public void onClick(View view) {

                switch (view.getId()) {

                    case R.id.my_btn_a: func_A(); break;
                    case R.id.my_btn_b: func_B(); break;
                    case R.id.my_btn_c: func_C(); break;
                    case R.id.my_btn_d: func_D(); break;
                }
            }
        }

        my_btn_a.setOnClickListener( mOnClickListener );
        my_btn_b.setOnClickListener( mOnClickListener );
        my_btn_c.setOnClickListener( mOnClickListener );
        my_btn_d.setOnClickListener( mOnClickListener );

    }

}

[vlad13reg]

Спасибо Вам за совет

[pompiduskus]

получилось хоть ?

[vlad13reg]

Да, ещё изменил строку [CODE]String[] split = display.split(currentOperator);
[/CODE

На

String[] split = display.split(Pattern.quote(current operator));