重构第26天移除双重否定(Remove Double Negative)

理解：”移除双重否定”是指把代码中的双重否定语句修改成简单的肯定语句，这样即让代码可读，同时也给维护带来了方便。

详解：避免双重否定重构本身非常容易实现，但我们却在太多的代码中见过因为双重否定降低了代码的可读性以致于非常让人容易误解真正意图。存在双重否定的代码具有非常大的危害性，因为这种类型的代码容易引起错误的假设，错误的假设又会导致书写出错误的维护代码，最终会导致bug产生。具体可以看下面的代码：

 1 public class  Order
 2     {
 3         public void  Checkout(IEnumerable<Product> products, Customer  customer)
 4         {
 5             if  (!customer.IsNotFlagged)
 6             {
 7                 // the customer account is flagged
 8                 // log some errors and return
 9                 return;
10             }
11
12             // normal order processing
13         }
14     }
15
16     public class  Customer
17     {
18         public decimal  Balance { get; private set; }
19
20         public bool  IsNotFlagged
21         {
22             get  { return  Balance < 30m; }
23         }
24     }

如上代码中的双重否定可读性非常低，因为我们很难搞明白双重否定的正确值。要重构它也非常容易，如下是重构后的代码：

 1 public class Order
 2     {
 3         public void Checkout(IEnumerable<Product> products, Customer customer)
 4         {
 5             if (customer.IsFlagged)
 6             {
 7                 // the customer account is flagged
 8                 // log some errors and return
 9                 return;
10             }
11
12             // normal order processing
13         }
14     }
15
16     public class Customer
17     {
18         public decimal Balance { get; private set; }
19
20         public bool IsFlagged
21         {
22             get { return Balance >= 30m; }
23         }
24     }

”双重否定“很容易让人产生错误的判断，也很难让人理解你的代码，所以这个重构在我们的代码中是很重要的，尤其是在判断条件很多且业务复杂的时候。

重构第26天移除双重否定(Remove Double Negative)相关推荐

Remove Double Negative（去除双重否定）
WHAT 双重否定的条件语句,可以使用一个肯定的条件语句代替记住:双重否定 = 肯定本次重构涉及到的基本重构手法: Extract Method(提炼方法) Inline Method(内联方法) ...
R语言可视化包ggplot2移除（remove）可视化结果的图例（legend）实战
R语言可视化包ggplot2移除(remove)可视化结果的图例(legend)实战目录 R语言可视化包ggplot2移除(remove)可视化结果的图例(legend)实
重构第15天移除重复的代码(Remove Duplication)
理解:移除重复的代码,顾名思义就是把多处重复的代码搬移到一个公共的地方,来减少代码量,提高代码可维护性. 详解:看下面的例子就很容易理解重构前code 1 using System; 2 using ...
jQuery 元素移除empty() remove()与detach()的区别？
@1.empty() 删除匹配元素集合中所有的后代字节点元素: <p>hello<span>world</span></p> $("p&quo ...
重构26-Remove Double Negative（去掉双重否定）
尽管我在很多代码中发现了这种严重降低可读性并往往传达错误意图的坏味道,但这种重构本身还是很容易实现的.这种毁灭性的代码所基于的假设导致了错误的代码编写习惯,并最终导致bug.如下例所示: public ...
macOS下malware移除实战之搜索引擎Google劫持为trovi的移除（Remove Google hijacking for trovi removal under macOS）
声明: Declaration: 由于网络中的病毒virus/malware等存在随时变异或者对应多种感染方式等情况,本文所针对的处理方法仅针对本次样本负责,个人如有误操作,后果自负.如需帮助,请在W ...
LeetcCode 27：移除元素 Remove Element（python、java）
公众号:爱写bug 给定一个数组 nums 和一个值 val,你需要原地移除所有数值等于 val 的元素,返回移除后数组的新长度. 不要使用额外的数组空间,你必须在原地修改输入数组并在使用 O(1) ...
linux svn e170001 认证失败,jenkins - svn: E170001报错的原因以及解决方案
1. 什么问题What? 使用Jenkins配置的svn拉取项目,Jenkins报错:svn: E170001; Your credentials to connect to the reposito ...
【JAVA】代码重构技巧
简介重构是持续改进代码的基础.抵制重构将带来技术麻烦:忘记代码片段的功能.创建无法测试的代码等等. 而有了重构,使用单元测试.共享代码以及更可靠的无bug 的代码这些最佳实践就显得简单多了. 鉴于重 ...

重构第26天移除双重否定(Remove Double Negative)

重构第26天移除双重否定(Remove Double Negative)相关推荐

最新文章

热门文章

重构第26天 移除双重否定(Remove Double Negative)

重构第26天 移除双重否定(Remove Double Negative)相关推荐

最新文章

热门文章

重构第26天移除双重否定(Remove Double Negative)

重构第26天移除双重否定(Remove Double Negative)相关推荐