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

来源：互联网发布：mac iso镜像编辑：程序博客网时间：2024/04/26 18:40

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

0 0