31天重构学习笔记26. 避免双重否定
2013-02-05 10:10
225 查看
概念:本文中的”避免双重否定”是指把代码中的双重否定语句修改成简单的肯定语句,这样即让代码可读,同时也给维护带来了方便。
正文:避免双重否定重构本身非常容易实现,但我们却在太多的代码中见过因为双重否定降低了代码的可读性以致于非常让人容易误解真正意图。存在双重否定的代码具有非常大的危害性,因为这种类型的代码容易引起错误的假设,错误的假设又会导致书写出错误的维护代码,最终会导致bug产生。具体可以看下面的代码:
总结: ”双重否定“很容易让人产生错误的判断,也很难让人理解你的代码,所以这个重构在我们的代码中是很重要的,尤其是在判断条件很多且业务复杂的时候。
正文:避免双重否定重构本身非常容易实现,但我们却在太多的代码中见过因为双重否定降低了代码的可读性以致于非常让人容易误解真正意图。存在双重否定的代码具有非常大的危害性,因为这种类型的代码容易引起错误的假设,错误的假设又会导致书写出错误的维护代码,最终会导致bug产生。具体可以看下面的代码:
using System.Collections.Generic; using LosTechies.DaysOfRefactoring.SampleCode.BreakMethod.After; namespace LosTechies.DaysOfRefactoring.SampleCode.DoubleNegative.Before { public class Order { public void Checkout(IEnumerable<Product> products, Customer customer) { if (!customer.IsNotFlagged) { // the customer account is flagged // log some errors and return return; } // normal order processing } } public class Customer { public decimal Balance { get; private set; } public bool IsNotFlagged { get { return Balance < 30m; } } } }
如上代码中的双重否定可读性非常低,因为我们很难搞明白双重否定的正确值。要重构它也非常容易,如下是重构后的代码:
using System.Collections.Generic; using LosTechies.DaysOfRefactoring.SampleCode.BreakMethod.After; namespace LosTechies.DaysOfRefactoring.SampleCode.DoubleNegative.After { public class Order { public void Checkout(IEnumerable<Product> products, Customer customer) { if (customer.IsFlagged) { // the customer account is flagged // log some errors and return return; } // normal order processing } } public class Customer { public decimal Balance { get; private set; } public bool IsFlagged { get { return Balance >= 30m; } } } }
总结: ”双重否定“很容易让人产生错误的判断,也很难让人理解你的代码,所以这个重构在我们的代码中是很重要的,尤其是在判断条件很多且业务复杂的时候。
相关文章推荐
- 31天重构学习笔记26. 避免双重否定
- 31天重构学习笔记26. 避免双重否定
- 31天重构学习笔记26. 避免双重否定
- 31天重构学习笔记26. 避免双重否定
- 31 天重构学习笔记26. 避免双重否定
- 31天重构学习笔记3. 提升方法
- 31天重构学习笔记19. 提取工厂类
- 31天重构学习笔记24. 分解复杂判断
- 31天重构学习笔记8. 使用委派代替继承
- 31天重构学习笔记25. 引入契约式设计
- 31天重构学习笔记15. 移除重复内容
- 31天重构学习笔记7. 重命名(方法,类,参数)
- 31天重构学习笔记22. 分解方法
- 31天重构学习笔记29. 去除中间人对象
- 31天重构学习笔记4. 降低方法
- 31天重构学习笔记11. 使用策略类
- 31天重构学习笔记16. 封装条件
- 31天重构学习笔记9. 提取接口
- 31天重构学习笔记24. 分解复杂判断
- 31天重构学习笔记16. 封装条件