在尝试全绿时,我得到了 Resharper 的以下建议。
原始代码:
static public string ToNonNullString(this XmlAttribute attr)
{
if (attr != null)
return attr.Value;
else
return string.Empty;
}
建议:删除多余的“else”,结果如下:
static public string ToNonNullString(this XmlAttribute attr)
{
if (attr != null)
return attr.Value;
return string.Empty;
}
对我来说,建议的版本似乎比原来的可读性差。Resharper 的建议是否反映了良好可维护代码的定义?