Как обрабатывать исключения?

Question

Это вопрос о лучшей практике .net. Я всегда пойман и хранить информацию об исключениях во всех моих веб и оконных формах приложений следующим образом:

1. окруженных каждый метод с попыткой поймать (Exception исключения)
2. Для любого слоя, за исключением переднего слоя, выбросил исключение в слой выше
3. На переднем слое зарегистрировано исключение из файла журнала (обычно с помощью log4config.dll) и предоставлено пользователю удобное для пользователя сообщение.

Это лучший способ справиться с исключениями? Или я должен делать что-то другое в моих следующих проектах?

Answer 1

Следующий код является проблематичным, поскольку он перезаписывает исходную трассировку стеки для e, что делает проблемы труднее диагностировать:

public void Foo() { 
    try { 
     Bar(); 
    } catch(Exception e) { 
     throw e; // re-throw; overwrites original stacktrace in 'e' 
    } 
} 

Следующий код не имеет выше StackTrace проблемы перезаписи, но по-прежнему излишне многословные :

public void Foo() { 
    try { 
     Bar(); 
    } catch(Exception e) { 
     throw; // re-throw; preserves original stacktrace in 'e' 
    } 
} 

Оба будут лучше написаны, как показано ниже. Это происходит потому, что если единственное, что вы делаете в вашем поймать блок повторно бросает исключение, нет необходимости писать крылатую блок на все:

public void Foo() { 
    Bar(); 
} 

Answer 2

Лучший ответ вы можете получить на Best Practices for Handling Exceptions

Answer 3

Я бы не добавить 1 & 2, если у меня не было какой-то конкретной причины; например, чтобы изменить (обернуть) сообщение; нет необходимости, поскольку в любом случае исключения будут подниматься вверх. И сделано неправильно, вы можете случайно удалить все важные трассировки стека (throw; vs throw ex; - первый из них предпочтителен).

Answer 4

Вот как НЕ в обрабатывать исключения.

public void method1(){ 
    try{ 
     .... 
     // all the code goes here 
     // declare any helper methods throw throwable 
     // eg: private boolean check1() throws Throwable{ ...} 
     .... 
    }catch(Throwable t){ 
     System.out.println("oops...error...good luck debugging. "); 
    } 
}