TDD重构

Gilded Rose 需求描述

https://github.com/emilybache/GildedRose-Refactoring-Kata

欢迎来到”镀金玫瑰”！这是一家魔兽世界里的小商店，但地段超好。老板叫艾利森，是个友善的人。我们出售的商品也都是高质量的。但不妙的是，随着商品逐渐接近保质期，它们的质量也不断下滑。我们用一个IT系统来更新库存信息。开发这个系统的程序员叫勒罗伊，他已经不在我们公司了。现在，你的任务就是要在这个系统中添加新的特性，这样我们可以销售新的商品。

首先，简单介绍一下我们的系统：

• 所有商品都有一个”SellIn”值，这是商品的保质期，最好在这么多天之内卖掉
• 所有商品都有一个”Quality”值，代表商品的价值
• 每过一天，所有商品的”SellIn”值和”Quality”值都减1

很简单对吧？别着急，有意思的来了……

• 一旦过了保质期，”Quality”就以双倍的速度下滑。
• 陈年干酪（Aged Brie）是一种特殊的商品，放得越久，价值反而越高。
• 商品的品质永远不会小于0，也永远不会超过50。
• 萨弗拉斯（Sulfuras）是一种传奇商品，没有保质期的概念，质量也不会下滑。它的品质是80且永远不变。
• 后台通行证（Backstage pass）和陈年干酪有相似之处：越是接近演出日，随着”SellIn”值的推移，商品价值”Quality”值反而上升。在演出前10天内，价值每天上升2点；演出前5天内，价值每天上升3点。但一旦过了演出日，价值就马上变成0。

我们最近还签了一个供应商，给我们供应魔法召唤物品。这又需要升级我们的系统：

• 魔法召唤（Conjured）物品的贬值速度是普通物品的两倍。

现在，请开始你的表演~

请随意对updateQuality()函数进行修改和添加新代码，只要系统还能正常工作。然而，不要修改Item类或其属性，因为那属于角落里的地精，他会非常愤怒地爆你头，因为他不相信代码共享所有制（如果你愿意，你可以将UpdateQuality方法和Items属性改为静态的，我们会掩护你的）。

再次澄清，每种物品的品质不会超过50，然而"Sulfuras"是一个传奇物品，因此它的品质是80且永远不变。

重构步骤

第一步:理解需求，为已有代码增加注释

通过分析代码，我们发现现在的代码是实现了已有的功能的，除了本身的代码质量问题之外，我们还发现以下的问题：

1. 要增加"Conjured"物品，改动量有点大，典型的意大利面式的代码；
2. Sulfuras的品质永远是80这个需求没体现在代码中，只存在于调用者的代码中，这点明显不合理。

第二步：增加测试代码，分需求逐个测试

原有的代码只有一个未通过的测试用例。所以为了验证已有的需求功能的正确性，增加测试。

原有的代码除了不满足Conjured物品的品质值的变化需求，其他功能都满足。

第三步:开始重构

通过测试加深了对需求的理解，我们发现不同物品间的差异在于保质期和品质值的更新策略不同。

1. 增加一个更新策略的接口；
2. 扩展接口，实现一般物品、Aged Brie、Backstage pass、Conjured、Sulfuras5个不同的物品类，实现更新策略的接口；
3. 重构updateQuality()函数,根据传入的物品的名称，通过简单工厂模式，返回对应的处理类，调用处理类的更新策略接口；统一处理品质值的上下限值，如果超过极值，则返回极值。

参考

源码地址：
https://github.com/hongzhenglin/GildedRose-Refactoring-Kata