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