KnowledgeHub
Questions
Tags
Users
Search
Alex Rivera
|
Logout
Edit Question
Title
Body
While trying to get to all green, i got the following suggestion by Resharper. Original code: static public string ToNonNullString(this XmlAttribute attr) { if (attr != null) return attr.Value; else return string.Empty; } Suggestion: remove redundant 'else' resulting in following: static public string ToNonNullString(this XmlAttribute attr) { if (attr != null) return attr.Value; return string.Empty; } To me, the suggested version seems less readable than the original. Does Resharper suggestion reflect the definition of good maintainable code?
Tags (comma-separated)
Save Edits
Cancel