是隐式地使用空参数作为标志良好的代码风格

本文关键字：标志 风格 代码 参数 | 更新日期: 2023-09-27 18:11:09

下面哪个是更好的编码风格，为什么?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;
}

是隐式地使用空参数作为标志良好的代码风格

在Clean Code书中不建议使用标志。你可以更好地创建两个不同的方法，一个名为GetRequiredValue，另一个名为GetOptionalValue。错误字符串对我来说似乎是多余的，使用自定义的异常("GetSettingFromSomewhereException")并传递设置值。

IsNullOrEmpty函数是Light，它返回一个布尔值，所以我想对于这种情况，这并不重要。重要的是您认为如何使您的函数对可能使用函数

的人更具可读性
相关文章：