任务8：反转条件逻辑

现在，代码已经有了基本的结构。接下来的重构，整体的想法应该是把每种商品特定的行为搬移到各自的子类中去，怎么做到这一点呢？需要一点小技巧，那就是微调一下if条件逻辑。

这里有两种思路：一种是分别处理updateSellIn()、updateQualityAfterExpiration()、updateQuality()里的条件逻辑；一种是分别针对isSulfuras()、isAgedBrie()、isBackstagePass()几个分支进行移除。无论选择哪一种，我依然建议你尝试从最简单的地方入手。

export class Item {
  ...
  passOneDay() {
    this.updateQuality()
    this.updateSellIn()
    if (this.isExpired()) {
      this.updateQualityAfterExpiration()
    }
  }

  updateSellIn() {
    if (!this.isSulfuras()) {
      this.sellIn = this.sellIn - 1
    }
  }

  ...
}

你的任务

Java：

git checkout task-8-if-conditions
./gradlew clean build

JavaScript：

git checkout task-8-if-conditions
npm test

1. 要求小步前进，每一步修改之后马上运行所有测试，确保软件行为不变

具体消除路径，你可以任选一种尝试：

1. 消除updateSellIn()方法中的if条件，把商品相关的行为搬移到特定的子类中

2. 消除updateQualityAfterExpiration()方法中的if条件，把商品相关的行为搬移到特定的子类中

或者：

1. 消除isSulfuras()条件判断，将相关的行为逻辑搬移到特定的子类中

2. 消除isBackstagePass()条件判断，将相关的行为逻辑搬移到特定的子类中

思考

• 反转条件逻辑的过程有没有遇到困难？

• 重构过程，有没有细心检验重构中的分支是否被测试完整覆盖？

• 体会每一步重构过程中先挑简单任务做的意义

• 选择不同的路径对于重构过程或重构结果是否会有所区别？

参考资料

• 《重构 2》10.3 以卫语句取代嵌套条件表达式（Replace Nested Conditional with Guard Clauses）

• 《重构 2》10.4 以多态取代条件表达式（Replace Conditional with Polymorphism）