您现在的位置:计算机技术学习网 > 技术中心 > 编程开发 > C >

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

来源:未知 责任编辑:责任编辑 发表时间:2013-08-27 15:58 点击:

今天查看自己为《专业嵌入式软件开发》一书所写的代码时发现,个别函数由于没有引入中间变量,使代码行既长又不易读。重构前后的代码如下所示。

重构前:

  1. if (TIMER_STARTED == _handle->state_) {

  2.    timer_handle_t next;

  3. if (g_timer_next == _handle) {

  4.        g_timer_next = (timer_handle_t) dll_next (&g_bucket_firing->dll_,  

  5.            &_handle->node_);

  6.    }

  7.    next = (timer_handle_t)dll_next  

  8.        (&g_buckets [_handle->bucket_index_].dll_, &_handle->node_);

  9. if (0 != next) {

  10.        next->round_ += _handle->round_;

  11.    }

  12.    dll_remove (&g_buckets [_handle->bucket_index_].dll_, &_handle->node_);

  13. if (g_buckets [_handle->bucket_index_].reentrance_ > 0) {

  14.        g_bucket_firing->level_ ++;

  15.    }

  16. }

重构后:

  1. if (TIMER_STARTED == _handle->state_) {

  2.    timer_handle_t next;

  3. bucket_t *p_bucket = &g_buckets [_handle->bucket_index_];

  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 (&p_bucket->dll_, &_handle->node_);

  9. if (0 != next) {

  10.        next->round_ += _handle->round_;

  11.    }

  12.    dll_remove (&p_bucket->dll_, &_handle->node_);

  13. if (p_bucket->reentrance_ > 0) {

  14.        g_bucket_firing->level_ ++;

  15.    }

  16. }

本文出自 “至简李云” 博客,请务必保留此出处http://yunli.blog.51cto.com/831344/859794

    发表评论
    请自觉遵守互联网相关的政策法规,严禁发布色情、暴力、反动的言论。
    用户名: 验证码:点击我更换图片
    最新评论 更多>>
    网站首页 - 友情链接 - 网站地图 - TAG标签 - RSS订阅 - 内容搜索
    Copyright © 2008-2015 计算机技术学习交流网. 版权所有

    豫ICP备11007008号-1