我相信你或多或少听说过 DRY 原则，以至于看到标题后嗤之以鼻。但我猜你理解的肯定有所偏差，本文会带你重新回顾一下 DRY 原则，修正多数人的理解误区。

1 引言

DRY，Don’t Repeat Yourself：多个地方表达相同的含义。

1.1 如何理解 DRY 原则

DRY 针对的是知识和意图的复制，强调多个地方表达的东西其实是相同的，只是表达方式不同。

理解误区：有些人将 DRY 固化为编码规范，这是狭隘的。至少，别把它理解为“不要复制粘贴代码”，它和你想的不一样。

1.2 Repeat Yourself 带来了哪些问题

2 DRY 原则描述了哪些重复现象

2.1 代码重复

“代码重复”就是很多人对 DRY 全部的理解，但还不准确，“复制粘贴”代码只是代码重复的一种特例，很多情况下，重复是不明显的：

# 定义账户类
class MyAccount(object):
    __slots__ = ['fee', 'balance']
    def __init__(self, fee, balance):
        self.fee = fee
        self.balance = balance

# 定义打印函数
def printAccount(account):
    if account.fee < 0:
        print('fee:%10.2f' % -account.fee)
    else:
        print('fee:%10.2f' % account.fee)

    if account.balance < 0:
        print('balance:%10.2f' % -account.balance)
    else:
        print('balance:%10.2f' % account.balance)

# 函数调用
myAccount = MyAccount(100, -300)
printAccount(myAccount)

以上代码没有复制粘贴，但仍有两处重复。

修改后：

  def printAccount(account):
    myPrint('fee', account.fee)
    myPrint('balance', account.balance)

  def myPrint(label, amount):
    print('%s: %10.2f' %(label, formatMoney(amount)))

  def formatMoney(amount):
    return abs(amount)

Q：思考一下，所有代码的重复都是知识的重复么？

2.2 文档重复

这里的文档是广义上的，还包括注释等。

比如方法的注释把方法中的逻辑分支都描述了一遍，函数的意图就被描述了两次（注释、代码各一次）。只要经过两次需求变更，代码和注释就会变得不一致。

private static boolean isAllTicketsStatusOk(FlightOrder order, BigOrder bigOrder) {
  /*
    * 三种情况，都能申请邮递：
    * 1. 退票单 || 该机票已经退了
    * 2. 飞后寄 && 已乘机（只作用于新单，新单乘机后，自行递归原单进行处理）
    * 3. 立即寄 && 已出票、已改签、已乘机（由于打印行程单任务可能拖延时间，故加入已乘机）
    */
  Predicate<FlightOrderTicket> isTicketStatusOk = item -> item.getStatus() == Returned
      || bigOrder.getDistInfo().getDistMode() == DistMode.DistAfterRide && item.getStatus() == Ride
      // Changed 表达的是原单
      || bigOrder.getDistInfo().getDistMode() == DistMode.DistImmediately && item.getStatus().in(Changed, TicketSuccess, Ride);

  boolean isAllTicketsStatusOk = order.getOrderTickets().stream()
      .allMatch(isTicketStatusOk);

  LOG.debug("isAllTicketsStatusOk: {}", isAllTicketsStatusOk);

  return isAllTicketsStatusOk;
}

写注释是好习惯，这里绝对不是让你为了规避 DRY 原则，把注释全部删掉。

但是，注释掩盖不了糟糕的代码。

如果是为了掩饰方法中糟糕或者晦涩难懂的代码，这时候应该重构代码。

推荐：

2.3 数据重复

其实就是常说的数据冗余。

class Line {
  Point x;
  Point y;
  double length;
}

x、y 两点即可确定连线的长度，length 字段明显重复了，应该改成方法：

double length() {

}

即使不在同一个类，也可能构成重复。

举个例子，假设一个行程 route，包含多个航段 segment，route 有的距离，segment 上也有距离。

class Route {
  List<Segment> segments;
  int distance;
}

class Segment {
  int distance;
}

route 上的距离是其下所有 segment 距离之和，定义成字段，就重复了，改成方法。

class Route {
  List<Segment> segments;

  int getDistance() {
    return this.segments.stream()
          .map(item -> item.getDistance())
          .sum();
  }
}

class Segment {
  int distance;
}

数据库实体类比较特殊，有时候要考虑性能因素，采取了冗余措施。

比如下面例子的 amount 字段。

@Table
class FlightOrder {
  BigDecimal amount;
  List<FlightOrderPassenger> passengers;
}

@Table
class FlightOrderPassenger {
  BigDecimal amount;
  List<FlightOrderPassengerTicket> tickets;
  FlightOrder belongedOrder;
}

@Table
class FlightOrderPassengerTicket {
  BigDecimal amount;
  FlightOrderPassenger belongedPassenger;
}

将更新冗余字段的逻辑封装在类内部，集中处理。

@Table
class FlightOrderPassengerTicket {
  BigDecimal amount;
  FlightOrderPassenger belongedPassenger;

  void setAmount(BigDecimal amount) {
    this.amount = amount;
    this.resetOrderAmout();
  }

  void addAmount (BigDecimal amount) {
    this.amount = this.amount + amount;
    this.resetOrderAmout();
  }

  private resetOrderAmout() {
    var passenger = this.belongedPassenger;
    passenger.amount = passenger.getTickets().stream()
      .reduce(BigDecimal.ZERO, BigDecimal::add);

    var order = this.belongedPassenger.belongedOrder;
    order.amount = order.getPassengers().stream()
      .reduce(BigDecimal.ZERO, BigDecimal::add);
  }
}

推荐：

2.4 表征重复

主要描述的和外部打交道的时候，不可避免的重复。代码必须持有外部系统已经蕴含的知识（表征）。包括 API、数据 schema，错误码等等。

2.4.1 API 的重复

对于 API，客户端代码、API 定义、服务端代码，两两之间存在重复。

推荐：

以上两种方式，都消除了 API 定义、服务端代码之间的重复，不足是无法消灭客户端重复，但也可以非常便利的手动触发完成重复消除。

2.4.2 数据源的重复

实体类对数据表的定义和数据库实际的表结构存在重复。

推荐：

2.5 开发人员重复

准确的说，是不同服务（开发人员）对同一需求都有自己的实现。它最大的问题是不在同一服务很难实现共用，尤其是前后端，存在语言壁垒。

最容易产生重复的就是校验规则，手机号、邮箱校验等，不同服务不同的开发人员都有自己的实现。

推荐：

但是，无法破除跨语言的壁垒。

3 总结

4 最后的忠告

规则终究是规则，思想终究是思想。实践起来困难重重，忠告：