本文翻译自"Before You Refactor"，来自于《97 Things Every Programmer Should Know》一书中的某个章节。

在某些时候，每个程序员都需要重构现有的代码。不过在你动手之前，请考虑一下下面的内容，因为这可以节省你和他人的大量时间（以及痛苦）。

· 重构开始的最好方式就是对现有代码库及其测试代码进行总结和评估。

这将帮助你理解现有代码的优点和不足，你也可以确保将优点保留住并避免错误。我们总是认为自己做的可以比现有系统更好…直到最终我们没能拿出更好的系统，甚至做得比系统的前身更糟，因为我们没有从现有系统的错误中汲取教训。

· 抵御重写一切的诱惑

最好尽可能多地重用现有代码。无论这些代码有多么丑陋，它们毕竟是经过评审和测试过的。扔掉旧代码，特别是那些运行在生产环境中的代码，意味着你扔掉了经过数月（或者数年）测试且久经沙场的代码，或许这些代码中还包含了一些你所不知道的特定的应对方案和Bugfix。如果你没有考虑到这些，你写的新代码最终就会出现出同样诡秘的bugs，而这些Bug在旧代码中都已经被Fixed了。这将浪费大量时间、精力以及多年来积累的知识。

· 多次增量改变优于一次巨大改变

增量改变可以让你更容易地通过诸如测试等反馈来评估对系统的影响。做出一次改变就导致上百个测试失败，这可不是什么好玩的事。这可能让你更加沮丧，压力更大，并会相应地导致错误的决策。三两个测试失败容易对付，并且也容易管理。

· 每次迭代后，确保现有的测试通过

如果现有的测试无法覆盖你所做出的改动，那就增加测试。不要不经思考就扔掉针对旧代码的测试。表面上其中的一些测试似乎不适合你新设计了，不过深入分析这个特殊的测试被添加进来原因是非常值得的。

· 个人喜好和自大不应该成为障碍  

如果东西没有损坏，为什么要去修正它呢？ 代码风格或结构不满足你的个人喜好，这不是一个重构的正当理由。认为你会做的比之前的程序员更好同样也不是一个正当理由。

· 新技术不足以成为重构的理由 

因为当前的代码落后于今天所有时髦的新技术，并且相信一门新语言或一个新框架可以使代码更优雅，这些都是最糟糕的重构理由。除非成本效益分析表明一门语言或一个框架将在功能性、可维护性以及生产力方面带来显著的提升，否则最好不要理会它。

· 请记住：人会犯错误 

重构并不总是保证新代码一定就更好，或者与以前的代码一样好。我看过并且亲身参与过许多次失败的重构尝试。漂亮并没有错，犯错的是人。

By Rajith Attapattu

昨天晚饭后，打开本子继续工作，却发现无法连上无线路由器。最初以为路由器忘记打开了，可拿起路由器看了下，不是那么回事儿，路由器工作一切正常。我这才看到发现本子的无线网卡的指示灯不亮了，以前在这台x60本子上还从未出现此类情况，于是开始查找故障原因。

故障查找过程是痛苦的，一次次燃起希望，又一次次被冷水破灭：

* 最初怀疑是我误点击了Fn + F5而把无线网卡关了，于是我又无数次的点击Fn + F5，居然一点反应都没有；
* 我的T400上有无线网卡的硬件开关，我将x60翻转了几周，也没找到无线开关位置；
* Ubuntu上Network Manager面板中，无线网络显示已停用，且菜单项为灰色，无法选择，无法启用；
* N次重启机器，无果；
* 切换到Win7下，Win7设备管理器显示无线网卡设备正常，驱动正常；反复停用、启用无线，都无法使指示灯亮起；
* 重启机器，F1进入BIOS，查看网络设备也是Enabled，遂将BIOS恢复成默认出厂设置；
* 再尝试进入Win7，蓝屏，提示修复，修复若干次依旧无法进入Win7，无线指示灯依旧处于熄灭状态;
* 继续回到Ubuntu下折腾，卸载Network Manager，更换网络管理软件，用T400下载WCID，并用U盘COPY到x60里安装(家里没有备网线)，WCID也没比自带的Network Manager好哪里去，依旧无法找到无线网卡；
* 恢复Network Manager；
* 用系统->系统管理->系统日志查看器查看系统日志，看到如下错误日志：
    dhclient: receive_packet failed on wlan0: Network is down
    wpa_supplicant[824]: Failed to initiate AP scan.
    NetworkManager:   WiFi now disabled by radio killswitch
    NetworkManager:   (wlan0): device state change: 8 -> 2 (reason 0)
    NetworkManager:   (wlan0): deactivating device (reason: 0).
    NetworkManager:   (wlan0): canceled DHCP transaction, dhcp client pid 2816
* 根据网上资料，按如下操作：
  – sudo -i
  – echo 1 > /sys/class/rfkill/rfkill0/state
  – 重启机器
  问题依旧。

* 安装rfkill，rfkill list看到：
  0: phy0: Wireless LAN
 Soft blocked: yes
 Hard blocked: yes
  执行rfkill unblock all，得到：
  0: phy0: Wireless LAN
 Soft blocked: no
 Hard blocked: yes
  依旧无法打开无线网卡

* 被折腾近四个小时后上床睡觉！
* 上班后联系设备维修部门；
* 带着本子到维修部门查找故障原因，说明情况后，维修人员操作我的本子；
* 重启机器，进入BIOS，将Config -> Serial ATA -> SATA Controller的MODE OPTION改为COMPATIBILITY，保存退出；
* 选择Win7，居然不再蓝屏，正常进入Win7；
* 在Win7加载进度条还在闪烁的时候，这位维修人员托起本子看了看，指着本子某个部位对我说：这是不是无线开关？
* 他拨动无线开关，无线信号指示灯亮起；
* 我无语！

不得不承认：我的眼神儿太差了！