隐式使用null参数作为标志好的代码样式

Question

下面哪个是更好的编码风格，为什么？ 1）

public string GetValue(string setting, string error)
{
    var val = GetSettingFromSomewhere(setting);
    if (val==null && string.IsNullOrEmpty(error)) throw Exception(error);
    return val;
}

或者 2）

public string GetValue(string setting, bool isRequired, string error)
{
    var val = GetSettingFromSomewhere(setting);
    if (val==null && isRequired) throw Exception(error);
    return val;
}

Answer 1

IsNullOrEmpty函数是Light并且它返回一个布尔值，所以我想对于这种情况，特别是它并不重要。重要的是你认为你的功能对于可能使用该功能的人来说更具可读性