2018-04-27 最简单的代码重构

一个 Code Review 的心得

最简单的代码重构，就是把重复代码提取成函数。

TJ

重构前

其实这个功能超级简单，就是判断一下当前环境是否是 dcs 环境。

dcs: [DreamArts Clound Service](https://www.dreamarts.co.jp/dcs/) 是我司提供的云服务平台。

重构后

// envUtil.js
export function isClound() {
  if (window.config !== undefined) {
    return window.config.dcs === 1
  } else {
    return false
  }
}

// use
import { isClound } from './envUtil'

if ( isClound() ) {
  ...
}

所有判断 dcs 环境的功能，只要调用 isClound() 即可。

当然这段代码还有问题，window.config.dcs === 1 的 1 应该算是魔法数字，也应该被重构。

将来即使 dcs 判断条件变成 dcs === 'dcs'，我们只需要修改一行代码即可。不需要对业务代码进行任何修改。