这是一些假设的代码示例:
if (e.KeyCode == Keys.Enter)
{
if (this.CurrentElement == null) {
return false;}
if (this.CurrentElement == this.MasterElement) {
return false;}
if (!Validator.Exist (this.CurrentElement)) {
return false;}
if (!Identifier.IsPictureElement (this.CurrentElement)) {
return false;}
this.FlattenObjects(this.CurrentElement);
}
VS
if (e.KeyCode == Keys.Enter)
{
if (this.CurrentElement != null) {
if (this.CurrentElement != this.MasterElement) {
if (Validator.Exist (this.CurrentElement)) {
if (Identifier.IsPictureElement (this.CurrentElement)) {
this.FlattenObjects(this.CurrentElement);}}}}}}
}
您认为哪一个在可读性、可维护性等方面更好?
另外,第二个示例可以通过不同使用括号来进行不同的格式化。
早期回报更具可读性。
每当一个方法内的嵌套超过四层或五层时,就该重构该方法了。
单个if
与||
子句有时会更具可读性:
if (this.CurrentElement == null
|| this.CurrentElement == this.MasterElement
|| !Validator.Exist(this.CurrentElement)
|| !Identifier.IsPictureElement(this.CurrentElement))
return false;
本文内容由网友自发贡献,版权归原作者所有,本站不承担相应法律责任。如您发现有涉嫌抄袭侵权的内容,请联系:hwhale#tublm.com(使用前将#替换为@)