什么样的代码是坏代码,是有异味(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


  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值