-2

Хочу обработать деление на ноль, но ошибка в try и catch. Как исправить?

Создал свое исключение:

public class DivisionByZero extends Exception {

public DivisionByZero() {
    super("Деление на ноль!");
}

 }

И кнопку равно:

bt_res.addActionListener(new ActionListener() {
    @Override
    public void actionPerformed(ActionEvent e) {
        try {
            double second_value = Double.valueOf(jtextfield.getText());
            if ("+".equals(operation[0])) {
                jtextfield.setText((first_value[0] + second_value) + "");
            }
            if ("-".equals(operation[0])) {
                jtextfield.setText((first_value[0] - second_value) + "");
            }
            if ("/".equals(operation[0])) {
                if (second_value == 0) {
                } catch (ArithmeticException e1){
                    JOptionPane.showMessageDialog(Calculator.this, "Деление на ноль запрещено!", "Error", JOptionPane.ERROR_MESSAGE);
                }
            } else {
                jtextfield.setText((first_value[0] / second_value) + "");
            }
            if ("*".equals(operation[0])) {
                jtextfield.setText((first_value[0] * second_value) + "");
            }
        } catch (Exception e1) {

        }
    }
});
5
  • У вас со скобками напутано. try и catch должны находится на одном уровне вложенности
    – Barmaley
    28 дек 2017 в 10:14
  • 1
    Не совсем понятно, чего вы хотите добиться... Ошибочные арифметические операции бросают исключение ArithmeticException. Сделайте блок try - catch, внутри которого проводите все вычисления и перехватывайте это исключение. Можно эти вычисления в отдельный метод вызвать, который бросает данное исключение. Все просто. Зачем вам нужен класс DivisionByZero - остается загадкой 28 дек 2017 в 10:23
  • Просто хочу, чтобы это было проверяемое исключение
    – Alex Lip
    28 дек 2017 в 13:10
  • А вот со скобками не получается разобраться.
    – Alex Lip
    28 дек 2017 в 13:19
  • А каким образом нужно сделать, чтобы try и catch находились на одном уровне вложенности?
    – Alex Lip
    28 дек 2017 в 17:24

1 ответ 1

0

возможно лучше будет так (как писали выше) сделать - вынести реализацию в отдельный метод

void method() throws DivisionByZero {
    if ("+".equals(operation[0])) {
        jtextfield.setText((first_value[0] + second_value) + "");
    } else if ("-".equals(operation[0])) {
        jtextfield.setText((first_value[0] - second_value) + "");
    } else if ("/".equals(operation[0])) {
        if (second_value == 0) {
           throw new DivisionByZero();
        }
        jtextfield.setText((first_value[0] / second_value) + "");
    } else if ("*".equals(operation[0])) {
        jtextfield.setText((first_value[0] * second_value) + "");
    }
}

и обрабатывать вызов подобным образом

public void actionPerformed(ActionEvent event) {
    try {
        method();
    } catch (DivisionByZero e) {
        e.printStackTrace();
    }
}
5
  • То есть блок catch (ArithmeticException e1) не нужен?
    – Alex Lip
    28 дек 2017 в 18:24
  • да, верно, он не нужен, т.к. есть проверка значения на 0, которая будет выдавать соответствующее сообщение
    – keekkenen
    29 дек 2017 в 16:30
  • Ну тогда ни какого исключения не будет, этот вариант не подходит.
    – Alex Lip
    29 дек 2017 в 16:58
  • Может это возможно как то по другому сделать?
    – Alex Lip
    30 дек 2017 в 9:00
  • см. выше - я обновил пост с решением
    – keekkenen
    30 дек 2017 в 12:06

Ваш ответ

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

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