C# Try/Catch best practices

Question

Я просматривал какой-то производственный код, и кажется, что есть много Try/Catches, с некоторой логикой в ​​Try, но пустой Catch.

Это обычная практика?

Не могли бы вы просто использовать инструкцию If?

Кажется странным использовать Try/Catch, если вы не собираетесь что-либо поймать.

Answer 1

Я нашел ответ здесь:

Try-catch: is this acceptable practice?

Это действительно плохая практика.

Answer 2

неоднородная утверждение, что делает ничего не является, к сожалению, общее плохое практика программирования:

try 
{ 
    DoSomething(); 
} 
catch (Exception e) 
{ 
    //empty 
} 

Он насмешливо называли Pokémon Exception Handling (#2), потому что «вы просто Готта поймать«Em All».

Answer 3

Вы частично правы.

Если нет, это не замена для попытки. Если ваш код генерирует исключение (добровольно или неохотно), и вы хотите, чтобы либо код продолжал обрабатывать, либо просто отбрасывал другое исключение вверх для вызывающего абонента, попытка catch - это путь. Я не вижу Ваш код, чтобы оценить правильность реализации try catch. Может быть полезно показать это.

Answer 4

У меня есть история о парне, который был лучшим программистом в команде разработчиков. Никто не мог победить его, потому что все его коды никогда не бросали никаких исключений на протяжении многих лет.

Когда он ушел, а другие начали пересматривать свои коды, началась история ужасов.

Он использовал попытку и пустую уловку для каждой логики в своем коде.

Команда разработчиков должна была переписать всю свою базу кода.