https://github.com/mwumli/book-refactoring2-code
《重构 改善既有代码的设计第二版》中示例代码
https://github.com/mwumli/book-refactoring2-code
Last synced: 3 months ago
JSON representation
《重构 改善既有代码的设计第二版》中示例代码
- Host: GitHub
- URL: https://github.com/mwumli/book-refactoring2-code
- Owner: MwumLi
- Created: 2021-02-15T14:56:29.000Z (over 4 years ago)
- Default Branch: main
- Last Pushed: 2021-02-24T09:26:58.000Z (over 4 years ago)
- Last Synced: 2025-01-14T16:44:36.547Z (5 months ago)
- Language: JavaScript
- Size: 42 KB
- Stars: 4
- Watchers: 2
- Forks: 0
- Open Issues: 0
-
Metadata Files:
- Readme: README.md
Awesome Lists containing this project
README
# book-refactoring2-code
存储 《重构: 改善既有代码的设计第二版》的一些示例代码
## 单元测试
使用 [mochajs](https://mochajs.org/) 作为我们单元测试套件
运行所有单元测试:
```
npm run test
```
运行某个单元测试:
```
npx mocha ./src/chapter1/*.test.js
```
## 变化
1. 提炼函数 `amountFor`
> 提炼成内联函数的意义不需要再把外部作用域中的数据传给新提炼的函数, 少了一件操心的事情, 尽管当前实例意义不大
2. 变量改名: `thisAmount` to `result`
> 永远将函数的返回值命名为 “result”
3. 变量改名: `perf` to `aPerformance`
> 动态类型语言中变量命名带上类型很有意义, 这里使用不定冠词 `a` 修饰它强调一个场表演
4. 提炼函数: `playFor`
> play 变量可以由 performance 变量计算得到, 以查询取代临时变量
5. 内联变量: 使用 `playFor(perf)` 替换 `statement` 函数中的 `play` 变量
6. 移除 `amountFor` 的 `play` 函数参数
1. 使用 `playFor` 代替 `play` 参数
2. 移除 `amountFor` 的 `play` 参数以及调用 `amountFor` 传参
7. 内联变量: 使用 `amountFor(perf)` 替代 perf
8. 提炼函数: 提炼观众量积分逻辑到函数 `volumeCreditsFor`
> 由于之前把 `play` 替换成 `platFor`, 这样提炼函数就少了一个局部作用域的变量
9. 变量改名: `perf` to `aPerformance`, `volumeCredits` to `result`
10. 函数变量 `format` 改变成函数声明 `format`
11. 函数改名: `format` to `usd`
> `format -> formatAsUSD -> usd`, 表意, 因为需要在模板中使用, 所以太长使用起来不太方便, 因此最后改成 `usd`
12. 移除观众量积分总和
1. `volumeCredits` 被在循环迭代过程中累加, `volumeCredits` 累加过程可以分离出来