0

Доброго времени суток! Пишу простенький калькулятор,но столкнулся с такой проблемой: второе введённое число не парсится и в итоге,если ввести "5+8", то получим "10". Подскажите, как решить данную проблему? Заранее спасибо.

Код:

@Override
public void onClick(View view) {
    inputNum1 = 0;
    inputNum2 = 0;
    result = 0;
    try{
        inputNum1 =  Double.parseDouble(mInputVal.getText().toString());
        inputNum2 =  Double.parseDouble(mInputVal.getText().toString());
    }catch (NumberFormatException e){}
    switch (view.getId()){
        case R.id.btnZero:
        case R.id.btnOne:
        case R.id.btnTwo:
        case R.id.btnThree:
        case R.id.btnFour:
        case R.id.btnFive:
        case R.id.btnSix:
        case R.id.btnSeven:
        case R.id.btnEight:
        case R.id.btnNine:
        case R.id.btnP:
        break;
        case R.id.btnDiv:
            result = inputNum1 / inputNum2;
            break;
        case R.id.btnMult:
            result = inputNum1 * inputNum2;
            break;
        case R.id.btnSub:
            result = inputNum1 - inputNum2;
            break;
        case R.id.btnAdd:
            result = inputNum1 + inputNum2;
            break;
        case R.id.btnExit:
            finish();
            break;
        case R.id.btnClear:
            break;
        default:
            break;
    }
    inDigit = ((Button)view).getText().toString();
    if(strVal.equals("0")) {
        strVal = inDigit;    // no leading zero
    } else {
        strVal += inDigit;   // accumulate input digit
    }
    mInputVal.setText(strVal);
    if(inDigit.equals("C")){
        mInputVal.setText("");
        tvResult.setText("");
        strVal = "";
        result = 0;
    }
    if(inDigit.equals("=")){
        tvResult.setText(strVal + result);
    }
}
5
  • "а в чем же еще? естественно, ты вставил "\ + ", split не нашел попадания и вернул всю строку в val[0], и при попытке обратиться к val[1] выпадает выход за пределы массива." afiki,так а что нужно было вставить?
    – ricky
    25 фев 2013 в 15:51
  • "\\+" тут нет пробелов.
    – afiki
    25 фев 2013 в 16:05
  • юзаю без пробелов,всё равно падает(
    – ricky
    25 фев 2013 в 16:10
  • 1
    ок. вчитался в код. странная логика. очень. получается сначала вводится первое число, потом знак операции и сразу же вычисляется. так то нужно сначала всю строку выражения сформировать и парсить только при нажатии на "=".
    – afiki
    25 фев 2013 в 16:44
  • фуф...всё переделал,всё заработало!)спасибо)
    – ricky
    25 фев 2013 в 17:59

1 ответ 1

2

parseDouble не угадывает какое из чисел тебе нужно. давай ему конкретные строки. для этих четырех операций и двух операндов можно сделать так:

  • нади индекс символа с операцией(+, -, /, *);
  • берешь строку от начала до этого индекса и парсишь в первое число;
  • берешь строку после индекса и до конца и парсишь во второе число.
7
  • попытался для одной операции сделать так: String[] val = mInputVal.getText().toString().split(" + "); try{ if(null!=val && val.length>1){ inputNum1 = Double.parseDouble(val[0]); inputNum2 = Double.parseDouble(val[1]); } }catch (NumberFormatException e){} но ничего не вышло,числа теперь вообще не парсятся
    – ricky
    25 фев 2013 в 13:28
  • .split("\\+"); тут два обратных слеша. один сайтом съедается. в документации по методу написано, что принимает regex. можно было и погуглить как правильно использовать метод.
    – afiki
    25 фев 2013 в 13:56
  • да погуглил уже,спасибо)
    – ricky
    25 фев 2013 в 14:02
  • хотя всё равно,второе введённое значение не парсится
    – ricky
    25 фев 2013 в 14:24
  • ты оставил пробелы в выражении для split. если нет, то опиши подробнее какие входные данные.
    – afiki
    25 фев 2013 в 14:38

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.