Resharper (或者是Visual -它是一个黄色灯泡,里面有一个小小的红色灯丝)在这个代码上提供了“倒转”的功能:
if (tItems >= 0)
{
return tItems;
}
SetTotalItemsAndTotalAmt();如果允许的话,...and会变成这样:
if (tItems < 0)
{
SetTotalItemsAndTotalAmt();
}
return tItems;然后,...but提出“倒置如果”,如果我轻信上钩,它会让它回到以前的辉煌。
如果一种方法被认为比另一种方法更好(我猜,更容易摸索),为什么它也允许反向反转呢?如果这是一个“六人半打”的案子,为什么还要费心去改变一下呢?
实际上,想想看,其中一种方法甚至是无效的,因为它说,“WorkFiles.getAmount():不是所有代码路径都返回一个值”,除非它在if条件之外有返回语句。
我猜这不是Resharper做的;那些乌尔克尔动力的猫不会做这样的事,我的。
发布于 2013-08-30 00:03:53
是ReSharper提供了这个,它提供了它,因为它可以。有很多很好的理由让它这么做。在重构代码时,我经常使用这种方法,如:
if (cond1) {
if (cond2) {
if (cond3) {
DoSomething();
} else {
throw "Error 3";
}
} else {
throw "Error 2";
}
} else {
throw "Error 1";
}
return;转入:
if (!cond1) throw "Error 1";
if (!cond2) throw "Error 2";
if (!cond3) throw "Error 3";
DoSomething();
return;当那些ifs运行几个屏幕的代码时,它特别有用。是的,我经常看到这样的代码。
https://softwareengineering.stackexchange.com/questions/209879
复制相似问题