异常构造函数是否应该抛出异常？ [复制]

Question

应该允许异常的构造函数抛出异常吗？

例子：

public class MyException : Exception
{
   public List<string> Errors { get; private set; }

   public MyException(List<string> errors)
   {
      if(errors == null)
      {
         throw new ArgumentNullException();
      }
      else if(errors.Count == 0)
      {
         throw new ArgumentException("At least one error must be present");
      }
      Errors = errors;
   }
}

在构造函数中抛出的那些异常（ArgumentNullException 和 ArgumentException）在使用 MyException 时可能会造成伤害？

这是此异常的用例：

class Program
{
    private static void ErrorHandling()
    {
        List<string> lst = new List<string>();

        // ... here do some checks and insert the errors in the list 'lst'

        // This check prevent the ArgumentNullException and
        // the ArgumentException to be thrown
        if (lst.Count > 0) 
        {
            throw new MyException(lst);
        }
    }

    static void Main(string[] args)
    {
        try
        {
            ErrorHandling();
        }
        catch(MyException e)
        {
            foreach(string s in e.Errors)
            {
                Console.WriteLine(s);
            }
        }
    }
}

我所说的危害是：如果由于某些原因使用MyException 的程序员没有检查输入列表（if (lst.Count > 0)），它可能会导致不需要的ArgumentNullException/ ArgumentException.

我认为这可能会导致程序员试图用错误的参数抛出 MyException 却抛出 ArgumentNullException/ArgumentException 的错误。

我应该：

• 不要在 MyException 的构造函数中进行检查，而将 Errors 属性的管理完全由用户管理
• 做检查并抛出ArgumentNullException/ArgumentException，知道这会导致错误

Answer 1

这些异常在构建 MyException 时可能会造成伤害

我根本没有看到您正在构建异常，而且您的 MyException 类几乎没有意义。您正在做的事情可以简单地使用另一个private 方法来完成。那为什么要为此不必要地构建应用程序异常呢？

public List<string> CheckErrors(List<string> errors)
{
      if(errors == null)
      {
         throw new ArgumentNullException()
      }
      else if(errors.Count == 0)
      {
         throw new ArgumentException("At least one error must be present");
      }

 return errors;
}