4

我实际上正在寻找一个指导方针，如何在 C#-DLL 内的公共方法中记录多个异常。

例子：

/// <summary>
/// This method does something
/// </summary>
/// <param name="p_Parameter1">First parameter</param>
/// <param name="p_Parameter2">Second parameter</param>
/// <param name="p_Number">A number</param>
/// <exception cref="ArgumentNullException">
/// Thrown if p_Parameter1 is null</exception>
/// <exception cref="ArgumentNullException">
/// Thrown if p_Parameter2 is null</exception>
/// <exception cref="ArgumentNullException">
/// Thrown if any element of p_Parameter2 is null</exception>
/// <exception cref="ArgumentOutOfRangeException">
/// Thrown if p_Number is below or equal 0</exception>
/// <returns>A object</returns>

public static object DoSomething(
    object p_Parameter1, IList<object> p_Parameter2, 
    object p_Parameter3, int p_Number)
{
    if(p_Parameter1 == null)
        throw new ArgumentNullException(
            paramName:"p_Parameter1", 
            message:"Parameter is needed");
    if (p_Parameter2 == null)
        throw new ArgumentNullException(
            paramName: "p_Parameter2", 
            message: "Parameter is needed");

    for (int i = 0; i < p_Parameter2.Count; i++)
    {
        if(p_Parameter2[i] == null)
            throw new ArgumentNullException(
                paramName: String.Format("p_Parameter2[{0}]", i),
                message: "All elements have to be initialized");
    }

    if(p_Number < 0)
        throw new ArgumentOutOfRangeException(
            paramName: "p_Number", 
            message: "Parameter should be bigger then zero");

    var returnValue = new object();

    // do something where p_Parameter3 == null is allowed

    return returnValue;
}

记录这些例外情况是否正确？我应该为每种情况添加一个异常标签，还是应该只为所有不允许使用空值的参数添加一个？

/// <exception cref="ArgumentNullException">
/// Thrown if p_Parameter1, p_Parameter2
/// or any element of p_Parameter2 are null</exception>
4

2 回答 2

5

我肯定会按类型对异常进行分组，即Thrown if p_Parameter1, p_Parameter2 or any element of p_Parameter2 are null.

作为参考，请查看 MSDN 上的文档。一个例子：

ArgumentNullException   |   Either path, contents, or encoding is null.
于 2012-10-24T12:59:16.297 回答
2

在这种情况下，MSDN 是一个很好的模拟来源，并且对那里的一些 函数 进行了调查，看起来它们倾向于使用一个块并枚举其中的不同参数。exception

这样做可以使代码的使用者更容易知道可能捕获哪些异常，因为查看不同异常的列表比包含重复的列表更容易理解。

于 2012-10-24T12:59:25.903 回答