Чи не є хорошою практикою обробляти винятки з виконання коду в коді?

Я працюю над додатком Java, і я бачу, що винятки з виконання часу обробляються в багатьох місцях. Наприклад,

try {
    // do something
} catch(NullPointerException e) {
    return null;
}

Мої запитання полягають у тому, коли корисно обробляти винятки під час виконання програми? Коли винятки слід залишати без змін?

Це залежить.

Наприклад, Integer#parseIntкидки NumberFormatException(що є RTE), якщо поданий рядок неможливо проаналізувати. Але напевно ви не хочете, щоб ваш додаток вийшов з ладу лише тому, що користувач написав "х" у текстове поле, яке було для цілих чисел? І звідки ви знаєте, чи можна проаналізувати рядок, якщо ви не спробуєте розібрати її спочатку? Тож у цьому випадку RTE - це лише сигнал про помилку, який повинен викликати якесь повідомлення про помилку. Можна стверджувати, що це повинен бути перевірений виняток, але що ви можете зробити - це не так.

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

Але іноді доцільно обробляти RunTimeExceptions. Наприклад, коли ви не можете змінити код, щоб додати нульову перевірку у відповідному місці, або коли виняток є чимось іншим, ніж NullPointerException.

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

Я обробляю очікувані винятки там, де я їх очікую. (Як і помилки читання / запису БД). Несподівані винятки я піднімаю. Десь ще можна очікувати винятку і мати логіку для цього.

Винятки мають бути лише такими .. винятками. Найкращою практикою при використанні винятків є використання їх для висвітлення ситуації, в якій трапляється щось, що суперечить тому, що ви б очікували, що трапиться. Класичний приклад - FileNotFoundException, який викидається, коли файлу просто немає. Якщо ви тестуєте існування файлу, тоді ви використовуєте File.exists (), оскільки ви просто продаєтеся 10-футовою паличкою, щоб побачити, чи вдарили ви щось.

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

Є багато ситуацій, в яких я напишу метод, який отримує якесь значення з бази даних. Тисяча речей може піти не так, і, дивлячись, як мені потрібна лише одна невелика інформація, незручно оточувати виклик списком спробу вловлювання, який містить 5 різних винятків. Тож я виберу винятки в методі отримання. Якщо щось піде не так, я вживаю будь-яких відповідних дій, щоб закрити підключення до бази даних чи що-небудь в остаточному пункті та повернути null. Це хороша практика не лише тому, що вона спрощує ваш код, але й тому, що "null" надсилає те саме повідомлення, яке ви могли отримати від винятку .. що щось не пішло, як було заплановано. Керуйте специфікою виключень у методі отримання, але керуйте, що робити, коли речі не роблять '

Наприклад:

Integer getUserCount() {
   Integer result = null;
   try {
      // Attempt to open database and retrieve data
   } catch (TimeoutException e) {
      logger.error("Got a watch?");
   } catch (MissingDatabaseException e) {
      logger.error("What are you smoking?");
   } catch (PermissionsToReadException e) {
      logger.error("Did you *really* think you were getting away with that?");
   } catch (PressedSendButtonToHardException e) {
      logger.error("Seriously.. just back away from the computer... slowly..");
   } catch (WTFException e) {
      logger.error("You're on your own with this one.. I don't even know what happened..");
   } finally {
      // Close connections and whatnot
   }
   return result;
}

void doStuff() {
   Integer result = getUserCount();
   if(result != null) {
       // Went as planned..
   }
}

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