博客
关于我
强烈建议你试试无所不能的chatGPT,快点击我
重构第26天 移除双重否定(Remove Double Negative)
阅读量:6945 次
发布时间:2019-06-27

本文共 1536 字,大约阅读时间需要 5 分钟。

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

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

 

1 public class  Order 2     { 3         public void  Checkout(IEnumerable
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 processing13 }14 }15 16 public class Customer17 {18 public decimal Balance { get; private set; }19 20 public bool IsNotFlagged21 {22 get { return Balance < 30m; }23 }24 }

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

1 public class Order 2     { 3         public void Checkout(IEnumerable
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 processing13 }14 }15 16 public class Customer17 {18 public decimal Balance { get; private set; }19 20 public bool IsFlagged21 {22 get { return Balance >= 30m; }23 }24 }

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

 

转载地址:http://eaanl.baihongyu.com/

你可能感兴趣的文章
笔记:ubuntu 12.04 apche2.22与ServerTokens、ServerSignature
查看>>
19. Gradle打印调试日志
查看>>
C 语言-- 原型设计模式
查看>>
将 Outlook 文件夹结构复制到其他计算机
查看>>
设计模式之结构型模式—— 2.4 组合模式
查看>>
应用程序框架实战十:开发环境准备与学习资料清单
查看>>
Citrix VDI-in-a-Box 第二篇:架构篇
查看>>
数据恢复工程师教你用 python解释NTFS runlist的代码
查看>>
如何卸载SQL2005故障转移群集
查看>>
网络工程师实训-RHCA 视频教程
查看>>
linux下配置DRBD
查看>>
企业如何“创新”
查看>>
使用Promise获取用户信息
查看>>
公司员工打开百度上的网页显示不了
查看>>
数组的特殊用法
查看>>
5、elasticsearch与springboot的集成
查看>>
分享企业网络带宽管理的几个技巧
查看>>
CentOS下yum安装LAMP
查看>>
eclipse开发快捷键
查看>>
raid
查看>>