0

Прошу помочь. Правильно ли я применил try-catch для чисел вне диапазона [1...10]? Как обработать исключения:

  1. ввод не целых чисел(калькулятор принимает только целые),
  2. работает либо с арабскими, либо с римскими
  3. неправильная арифметическая операция (правильные a + b, a - b, a * b, a / b) ?
import java.util.Scanner;

class calc {
    private static int value(char r) {...}

    private static int romanToDecimal(String str) {....}

    private static String intToRoman(int num) {....}

    private static int calculate(int number1, int number2, char operation){...}

    public static void main(String[] args) {

        Integer[] arabic = {10, 9, 8, 7, 6, 5, 4, 3, 2, 1};
        String[] roman = {"X", "IX", "VIII", "VII", "VI", "V", "IV", "III", "II", "I"};
        Scanner scanner = new Scanner(System.in);
        String text = scanner.nextLine();
        String[] blocks = text.split(" ");
        int number1 = 0, number2 = 0;
        String operation = " ";
        operation = blocks[1];
        char operation2 = operation.charAt(0);
        boolean flag = false;

        for (int i = 0; i < roman.length; i++) {
            if (roman[i].equals(blocks[0]) || roman[i].equals(blocks[2])) {
                flag = true;
            }
        }    
        try {
            if (flag) {
                number1 = romanToDecimal(blocks[0]);
                number2 = romanToDecimal(blocks[2]);
                System.out.println(intToRoman(calculate(number1, number2, operation2)));

            } else {
                number1 = Integer.parseInt(blocks[0]);
                number2 = Integer.parseInt(blocks[2]);
                System.out.println(calculate(number1, number2, operation2));
            }
        }
        catch(IllegalArgumentException e) {
            if ((number1 > 10 || number1 < 1) || (number2 > 10 || number2 < 1)) {
                throw new IllegalArgumentException();
            }
        }
    }
}
7
  • 1
    Я что-то не понял маневра с flag. Если хотя бы один элемент - римская запись, то мы берем за истину, что оба элемента - римские записи.
    – Miron
    24 ноя 2019 в 11:43
  • здесь недаработка, хотя бы два элемента - римская запись
    – plaza2009
    24 ноя 2019 в 11:47
  • Необходимо ввести переменные isRoman1, isRoman2. Далее если оба истинны - производим парс, если одна истинна и одна ложна - то запись неверна, если обе ложны, то пробуем перевести обе в int(try, дальше пишем перевод). Зашли в catch - значит запись неверна.
    – Miron
    24 ноя 2019 в 11:47
  • у меня ничего не работает, я не понимаю, могу я с вами договориться? ваше время на показ как сделать нужно правильно в обмен на что-то
    – plaza2009
    24 ноя 2019 в 13:17
  • Что вы можете предложить? Сделать могу.
    – Miron
    24 ноя 2019 в 13:33

1 ответ 1

2
  1. Если вы попытаетесь выполнить, например это - Integer.parseInt("1.22"), то вам выкинет java.lang.NumberFormatException. Оберните ваши парсы в соответствующий try catch
  2. Создайте try catch для Integer.parseInt. Переводите в int оба элемента в блоке try. Если они оба не числовые, то зайдет в catch. Далее вы получаете либо два элемента в римской записи, либо два элемента, один из которых точно записан неправильно. Если вы пробежались по массиву констант, и обнаружили элемент, не входящий в этот массив, значит элемент записан неверно. Кидайте ошибку.
  3. Если операция не равна ни одному из строчек константного массива new String[]{"+", "-", "*", "/"}, то данная арифметическая операция неправильна.
2
  • пойду разбирать, а для 3-го эта ошибка нужна IllegalArgumentException?
    – plaza2009
    24 ноя 2019 в 11:38
  • 1
    Нужна, ведь это нелегальный аргумент
    – Miron
    24 ноя 2019 в 11:39

Ваш ответ

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

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