在方法的开头到结尾处错误处理的优缺点是什么

Question

在我的程序员的经验中,我尽可能地混合错误处理......我创造了我的个人风格.

但是,我想听听你认为在开始时和方法结束时错误处理的优缺点.

处理开头:

public String GenerateSomeStringData(String data, int value)
{
    if (data == null)
        throw new ArgumentNullException("data");

    if (value <= 0)
        throw new ArgumentException("value must be greater than zero");

    int dataValue;
    if (!int.TryParse(data, out dataValue))
        throw new InvalidOperationException("data does not contain an integer");

    return dataValue * 4 + value / 12;
}

最后的处理:(同样的例子)

public String GenerateSomeStringData(String data, int value)
{
    if (data != null)
    {
        if (value > 0) 
        {
            int dataValue;
            if (int.TryParse(data, out dataValue))
            {
                return dataValue * 4 + value / 12;
            }
            else 
                throw new InvalidOperationException("data does not contain an integer");
        }
        else 
            throw new ArgumentException("value must be greater than zero");
    }
    else 
        throw new ArgumentNullException("data");
}

在决定如何处理此问题时,您使用什么标准？可读性,可维护性,简洁性？

Answer 1

输入的有效性应该是执行方法的前提条件 - 因此我会(并且确实)首先执行错误处理.

这具有以下优点:

• 解析人类更容易:首先验证先决条件,然后执行逻辑(通常会导致一些后置条件)

• 在方法中明确区分错误处理和执行逻辑之间的关注点
  :验证逻辑不会"嵌入"执行逻辑中

如评论中所述,您必须区分违反前提条件的无效输入和触发错误条件(例如抛出异常)和构成边缘条件的有效输入(即需要一些特殊逻辑来处理).后面的例子我将在你的方法的执行逻辑开始时断言前提条件后单独处理.