zoukankan      html  css  js  c++  java
  • 重构第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     }

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

  • 相关阅读:
    MySQL执行计划extra中的using index 和 using where using index 的区别
    Python + Apache Kylin 让数据分析更加简单!
    性能测试解读:Kyligence vs Spark SQL
    greenplum 表在各个节点数据的分布情况
    postgresql drop表后空间不释放
    PostgreSQL 查看表、索引等创建时间
    postgresql Kill掉正在执行的SQL语句
    linux ps命令查看最消耗CPU、内存的进程
    Linux shell
    TPC-H 下载参考
  • 原文地址:https://www.cnblogs.com/yplong/p/5381369.html
Copyright © 2011-2022 走看看