Метання та вилучення виключень у тій же функції / методі

Я написав функцію, яка запитує у користувача введення, поки користувач не введе додатне ціле число (натуральне число). Хтось сказав, що я не повинен кидати і виловлювати винятки у своїй функції, а дозволити абоненту моєї функції керувати ними.

Цікаво, що думають з цього приводу інші розробники. Я також, мабуть, зловживаю винятками у функції. Ось код на Java:

private static int sideInput()
{
    int side = 0;
    String input;
    Scanner scanner = new Scanner(System.in);

    do {
        System.out.print("Side length: ");
        input = scanner.nextLine();
        try {
            side = Integer.parseInt(input);
            if (side <= 0) {
                // probably a misuse of exceptions
                throw new NumberFormatException();
            }
        }
        catch (NumberFormatException numFormExc) {
            System.out.println("Invalid input. Enter a natural number.");
        }
    } while (side <= 0);

    return side;
}

Мене цікавлять дві речі:

1. Чи варто дозволити абоненту перейматися винятками? Сенс функції полягає в тому, що він нудить користувача, поки користувач не введе натуральне число. Чи поганий пункт функції? Я не говорю про інтерфейс користувача (користувач не може вийти з циклу без належного введення), а про циклічне введення з обробкою винятків.
2. Ви б сказали, що заява про кидок (у даному випадку) - це неправильне використання винятків? Я міг легко створити прапор для перевірки дійсності номера та вивести попереджувальне повідомлення на основі цього прапора. Але це додало б більше рядків до коду, і я думаю, що це ідеально читабельно, як є.

Річ у тому, що я часто пишу окрему функцію введення. Якщо користувачеві доводиться вводити число кілька разів, я створюю окрему функцію для введення, яка обробляє всі винятки та обмеження форматування.

Суть винятку полягає в тому, що він дозволяє методу сказати тому, хто викликав стан, який ввійшов у стан, коли він не міг продовжуватись нормально, не змушуючи вас вставляти код помилки у повернене значення.

У вашому випадку ваш метод точно знає, що робити, коли вхід не перевищує 0. Єдина причина, коли ви зберігаєте рядки тут - це те, що ви, здається, кидаєте той самий виняток, який ви отримали, якби введення не було числом. Однак виняток, який ви кидаєте, не відповідає належним чином, чому ваш код не любить введення. Якби хтось інший прийшов і побачив цей код, їм доведеться витратити додатковий час, намагаючись зрозуміти, як саме працюють.

Це погане використання винятків. Для початку непозитивне число не є винятком формату.

Навіщо взагалі використовувати винятки? Якщо ви знаєте, що введення заборонено, просто не виривайте з циклу, поки ви не отримаєте дійсне введення від користувача, щось на зразок наступного:

while (true)
{
   // Get user input.
   String input = scanner.nextLine();

   try
   {
      side = Integer.parseInt(input);

      break;
   }
   catch (NumberFormatException ex)
   {
      // Inform user of invalid input.
      System.out.println("Invalid input. Enter a natural number.");
   }
}

Виловлюйте винятки, лише якщо ви маєте намір зробити щось, що відповідає поточному виклику методу; тобто очищення, логіка відмов і т.д.

Тут можна позбутися від параметра try / catch і просто задокументувати виклик методу:

//Throws NumberFormatException if read input is less than 0
private static int sideInput()

Ще потрібно обробляти цей виняток далі в ланцюзі / стеку викликів!

Ви не повинні обидва кидати і ловити один і той же виняток у методі, я навіть думаю, що блок ловлі схопить той самий виняток, який ви кидали, тому ви насправді не кидаєте його.

Якщо це parseIntбуло успішним, то це не є NumberFormatException.

якщо сторона менше нуля, ви повинні кинути a NegativeSideLengthException;

Створіть виняток на замовлення / бізнес NegativeSideLengthException

public class NegativeSideLengthException extends Exception
{


    public NegativeSideLengthException(Integer i)
    {
        super("Invalid negative side length "+i);        
    }

}

Потім sideInputвикидає NegativeSideLengthException

private static int sideInput() throws NegativeSideLengthException
{
    int side = 0;
    String input;
    Scanner scanner = new Scanner(System.in);

    do {
        System.out.print("Side length: ");
        input = scanner.nextLine();
        try {
            side = Integer.parseInt(input);
            if (side <= 0) {
                throw new NegativeSideLengthException(side);
            }
        }
        catch (NumberFormatException numFormExc) {
            System.out.println("Invalid input. Enter a natural number.");
        }
    } while (side <= 0);

    return side;
}

Ви можете навіть (якщо хочете) додати ще один блок лову для лову, NegativeSideLengthExceptionа не мати спосіб кинути його.

do {
    System.out.print("Side length: ");
    input = scanner.nextLine();
    try {
        side = Integer.parseInt(input);
        if (side <= 0) {
            throw new NegativeSideLengthException(side);
        }
    }
    catch (NumberFormatException numFormExc) {
        System.out.println("Invalid input. Enter a natural number.");
    } catch (NegativeSideLengthException e){
        System.out.println("Invalid input. Enter a non-negative number.");
    }
} while (side <= 0);

Прапори - не гарний спосіб поводження з винятками.

Винятки - це досить кошмарні речі, вони приносять більше складностей, ніж вирішують.

По-перше, якщо ви не виберете свої винятки, абонент може робити лише те on error resume next, що через тиждень навіть ви не знатимете, що може кинути ваша функція та що з нею робити:

{
    ...
}
catch(OutOfMemory, CorruptedMemory, BadData, DanglingPointers, UnfinishedCommit)
{
    Console.WriteLine("Nothing to see here, move on.");
    Console.WriteLine("The app is very stable, see, no crashing!");
}

В основному, якщо ви їх ловите, ви повинні дуже добре розуміти договори та гарантії виключення. Це рідко трапляється в реальному світі. А також ваш код буде важко прочитати.

Крім того, найсмішніше те, що якщо ви справді маєте шанс обробити винятки, вам потрібна справжня мова RAII, яка є якось жартівливою, оскільки Java і .NET - це все про винятки ...

Повторюємо це ще раз, але ...:

http://blogs.msdn.com/b/oldnewthing/archive/2004/04/22/118161.aspx

http://blogs.msdn.com/b/oldnewthing/archive/2005/01/14/352949.aspx

http://www.joelonsoftware.com/items/2003/10/13.html