什么样的代码是坏代码，是有异味（smell）的代码？

1， Duplicatte Code

        如果有重复代码就要考虑重构了。避免以后修改一个逻辑会在两个相同代码处修改。

2， Bad Name

       要让函数名可读易懂。例如：用LightScheduler, 而不要用 lht_sched

3， Spaghetti code

      意面代码，结构，逻辑超复杂，典型的情况就是充斥着goto语句。

4， Long Function

       下面例子是把很多行代码转化成两个标红了的函数，一下子代码的易读性就增强了。Martin Fawler说过Clean Code就是易读代码。

void LightScheduler_WakeUp(void)        { int i; Time time; TimeService_GetTime(&time); for (i = 0; i < MAX_EVENTS; i++) { processEventsDueNow(&time, &eventList[i]); } }

5， Abstraction Distraction

       函数的抽象层度要一致，不能有的太复杂，有的太简单。

6，Bewildering Boolean

      if (!(day == EVERYDAY || day == today ||(day == WEEKEND && (SATURDAY == today || SUNDAY == today)) || (day == WEEKDAY && today >= MONDAY && today <= FRIDAY))) return;

compared to code that is intention-revealing like this: if (!matchesToday(day)) return;

7, Switch Case Disgrace

8，Duplicate Switch Case

9， Nefarious Nesting

     过渡嵌套

10， Feature Envy

11， Long Parameter List

     函数参数过多。如果发现在好几个函数都使用同样较多的参数，那就是该让定义一个结构体的时候了。

12， Willy-Nilly Initialization

     到处都在初始化变量，没有一个统一的地方。

13，Global Free-for-All，

     集中一个地方的Global数据申明，不知道这些数据会被哪些功能调用

14， Comments

     好的代码不需要注释。注释过时就是灾难。

15， Commented-Out Code

     后来者总会担心这些被注释的代码某一天是需要被使用的。（我觉得放在那要配上足够的注释）

16， Conditional Compilation