Рекомендации по обработке исключений в C#

Question

У меня есть вопрос об обработке исключений избыточно в вызывающем методе и вызванном методе. Рассмотрим следующий пример:

protected void method1(string action) 
{ 
    if (string.IsNullOrWhiteSpace(action)) 
    { 
    throw new ArgumentException("action cannot be null or empty or white space."); 
    } 
    //do something 
} 

public void method2(string name, string action) 
{ 
    if (string.IsNullOrWhiteSpace(name)) 
    { 
    throw new ArgumentException("name cannot be null or empty or whitespace."); 
    } 
    method1(action); 
    //do something 
} 

В приведенном выше примере следует также проверить, чтобы действие было null в методе2? Я чувствую, что, поскольку он обрабатывается в методе 1, мне не нужно иметь дело с ним, так как это может привести к избыточному коду.

Любые предложения будут полезны при изучении передовых методов обработки исключений. Спасибо

Answer 1

Я всегда проверяю. Я знаю, как работает код , но я не знаю, что он будет делать в будущем. Код изменится; моей рукой или чужим. Код, который не проверяется, - это бомба замедленного действия, ожидающая взрыва.

Answer 2

Если method1 является реализация method2private/internal и вызываться только method2, то вам не нужно поместить аргумент охранника на обоих методов.