# 任务8：反转条件逻辑

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

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

```javascript
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）


---

# Agent Instructions: Querying This Documentation

If you need additional information that is not directly available in this page, you can query the documentation dynamically by asking a question.

Perform an HTTP GET request on the current page URL with the `ask` query parameter:

```
GET https://ethan-lin.gitbook.io/refactoring/xiang-jin-mei-gui/11-task-8.md?ask=<question>
```

The question should be specific, self-contained, and written in natural language.
The response will contain a direct answer to the question and relevant excerpts and sources from the documentation.

Use this mechanism when the answer is not explicitly present in the current page, you need clarification or additional context, or you want to retrieve related documentation sections.