【编程好习惯】 引入中间变量使程序更易读

今天查看自己为《专业嵌入式软件开发》一书所写的代码时发现,个别函数由于没有引入中间变量,使代码行既长又不易读。重构前后的代码如下所示。
 
重构前:
 
 
  1. if (TIMER_STARTED == _handle->state_) { 
  2.     timer_handle_t next; 
  3.      
  4.     if (g_timer_next == _handle) { 
  5.         g_timer_next = (timer_handle_t) dll_next (&g_bucket_firing->dll_,  
  6.             &_handle->node_); 
  7.     } 
  8.     next = (timer_handle_t)dll_next  
  9.         (&g_buckets [_handle->bucket_index_].dll_, &_handle->node_); 
  10.     if (0 != next) { 
  11.         next->round_ += _handle->round_; 
  12.     } 
  13.     dll_remove (&g_buckets [_handle->bucket_index_].dll_, &_handle->node_); 
  14.     if (g_buckets [_handle->bucket_index_].reentrance_ > 0) { 
  15.         g_bucket_firing->level_ ++; 
  16.     } 
 
重构后:
 
 
  1. if (TIMER_STARTED == _handle->state_) { 
  2.     timer_handle_t next; 
  3.     bucket_t *p_bucket = &g_buckets [_handle->bucket_index_]; 
  4.      
  5.     if (g_timer_next == _handle) { 
  6.         g_timer_next = (timer_handle_t) dll_next (&g_bucket_firing->dll_,  
  7.             &_handle->node_); 
  8.     } 
  9.     next = (timer_handle_t)dll_next (&p_bucket->dll_, &_handle->node_); 
  10.     if (0 != next) { 
  11.         next->round_ += _handle->round_; 
  12.     } 
  13.     dll_remove (&p_bucket->dll_, &_handle->node_); 
  14.     if (p_bucket->reentrance_ > 0) { 
  15.         g_bucket_firing->level_ ++; 
  16.     } 
 
本文出自李云的博客,请务必保留此出处:http://blog.csdn.net/hzliyun/article/details/7553846。
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值