代码审查就是在排大便——你懂的!
豆瓣数据API和后台数据同步已经完成,我大概两天没去管这部分代码,主要是因为产品、老大都不是很积极。因为这个毕竟不是非常重要的。
今天我对代码进行了审查,也进行了一些修复,比如命名规范,代码逻辑,还有就是让整个代码更加符合整个框架的规范(如文件分类,指定文件做它该做的事情)。于是有了想写一篇“代码审查”博文的冲动,故有了此文。
盯哨
作为程序员应该知道Code Review的重要性。我相信很多程序员都不喜欢Code Review,那你喜欢它吗?
在公司两年多,一直是在原有的代码基础上进行代码修复和功能添加。这样势必会经常接触其他人代码,自然而然就养成Code Review的习惯。
在Code Review的时候,我经常感觉我后脑勺发凉,老是觉得有人(呵呵,一般是老大)在盯着我。
我有没有偷懒?
我有没有完全弄懂逻辑?
我写的代码罗不罗嗦?
我有没有按照命名规范进行编码?
我有没有写出漂亮而完美的代码?...

过程
现在,每次写完代码,我会间隔一段时间翻看旧代码,如果发现不好的地方,我就会去进行修复(这应该就是Code Review了)。
我是一名phper,我不太喜欢使用工具去做这个Review,我喜欢用眼睛看、用脑去思考。
有人说了,你这个不标准,效率不高。——呵呵,毕竟我工作量我自认为不大,所以很多时间我一直在Code Review。
每次看我的代码就像看一件艺术品,我时常问自己,我写的代码我满意吗?还缺点什么?我还能写的更好吗?

感觉
Code Review是个美妙的过程,让我的脑袋从混沌逐渐清晰。刚开始,我会头疼,但是随着你不断的Review,整个思路越来越明朗,代码越来越规范。
人脑是个非常奇特的东西,混沌到清晰是一个痛苦到愉悦的过程。如果长期不经过这个过程,你脑袋很容易锈到。
那么混沌是一个什么感觉?我比较喜欢我外甥的一句话,“脑袋瓜子里就像进入一堆浆糊似的”,他把这个混沌比喻成浆糊。我相信等你把浆糊弄干净了,你脑袋也就清晰了。
再举一个不好听的比喻,就像你把大便从肚子里排除去后的那种感觉。大便越多,最后排出去之后,感觉越爽。所以,你代码中,越有很多理不清的东西,等你真正理清完之后,你的脑子会非常舒服,整个身体都会非常轻松。

方法
我一般如何Code Review呢?
初期:

我首先找到一个切入点(也许是一个页面,也许是一个定时执行的脚本main()方法),然后逐个去找类,找方法,直到最后输出。——不停的去理这个过程。
然后再找下一个切入点。
后期:
我会挑选几个比较常见的业务,没有太明白的业务,然后寻找切入点,在进行以上循环。

推荐

posted on 2012-11-14 14:10 川山甲 阅读(...) 评论(...) 编辑 收藏

转载于:https://www.cnblogs.com/baochuan/archive/2012/11/14/2769549.html

代码审查就是在排大便——你懂的!相关推荐

  1. 如何训练孩子上厕所(初级篇)

    开始前准备 在开始训练自闭症孩子之前,家长首先思考以下几个问题,可能有助于为你的孩子找到不同的解决办法. (1)健康问题:孩子可能由于身体或疾病因素导致上厕所困难,家长需要向儿科医生咨询. (2)语言 ...

  2. 低代码会让程序员更加内卷吗?

    作者| Mr.K   编辑| Emma 来源| 技术领导力(ID:jishulingdaoli) 今天的主题呢主要是来源于最近的一个很火的话题--『低代码』. 低代码会不会让程序员更加内卷? 要回答这 ...

  3. java多重继承和多继承_Java不支持多重继承,但可以通过 类实现多继承。类的继承具有 性。 (10.0分)_学小易找答案...

    [填空题]Java中每个类的定义由类头和类体两部分组成,其中类体部分包括 和 . (5.0分) [单选题]病人男性,64岁,手术后第三天,护士通过评估认为目前存在以下问题属于首有问题的是 [判断题]J ...

  4. 十二时辰和人体经络图文

    中医将一天分成十二个时辰,并用十二地支代表,子午流注学说.子午流注学说是中医学的主要组成部分,是研究人体气血运行的时刻表.中医认为,自然界与人是统一的整体,自然界的年.季.日.时周期变化,影响着人们的 ...

  5. 高血压患者的饮食宜忌等

    有益于降压的食物 大致概括起来,有这么几类: ① 叶菜类:芹菜.茼蒿.苋莱.汕菜.韭菜.黄花菜.荠莱.菠菜等: ② 根茎类:茭白.芦笋.萝卜.胡萝卜.荸荠.马蹄: ③ 瓜果.水果类:西瓜.冬瓜.西红柿 ...

  6. 一个测人的眼力的贴子!

    zz<<<<<<怎样让大排不硬?>>>>>> 楼主 http://www.douban.com/group/topic/1462 ...

  7. 痤疮(又叫青春痘、粉刺、毛囊炎)

    个人皮肤本身呈油性,如不注意会经常反复. 原因: 由内分泌失调引起,皮脂分泌过盛,毛囊及皮脂腺阻塞,发炎所引发的一种皮肤病. 青春期时,体内的荷尔蒙会刺激毛发生长,促进皮脂腺分泌更多油脂,毛发和皮脂腺 ...

  8. 关于治疗便秘的方法 - 生活至上,美容至尚!

    蜂蜜加黑芝麻粉泡水,这个方法是很有效果的. 以下告诉你更多的关于治疗便秘的方法,你可以按照你能接受的方法,和按你的喜好,选择性的治疗便秘吧! 以下的这些方法,都是相当有效的治便秘的方法. 便秘应多吃这 ...

  9. 20150911新华医院听课内容

    20150911新华医院听课内容 Friday, September 11, 2015 8:56 PM 视频讲课内容: 破水后 平躺送入医院 初产妇: 减轻疼痛:侧卧,面对椅背坐下,来回倾斜骨盆,注意 ...

最新文章

  1. 使用CKEditor编辑器进行文本编辑
  2. 在线课程学习、科研科技视频网站
  3. spotify 数据分析_我的Spotify流历史分析
  4. python arcgis批量绘图_ARCGIS中Python实现批量裁剪
  5. 按home键退出的activity可以不进入stop模式码_用了几年的iPhone,竟然不知道苹果手机还有“游戏模式”?...
  6. 【属性对比】defer 与 async
  7. python设计模式5-原型模式
  8. css揭秘实战技巧 - 形状 [二]
  9. mysql导入查询失败_mysql数据库中的查询失败
  10. 利用百度搜索结果爬取邮箱
  11. 汽车VIN码超详细解析规则
  12. Emmet For Faster HTML CSS Workflow by Edwin Diaz
  13. 维盟无线ap服务器地址,让WiFi飞起来!维盟室外无线AP桥接与中继配置方案
  14. (github附源码)毕设微信小程序二手书交易后台PHP微擎
  15. ​微信小程序 获取地理位置(显示地图并显示经纬度)​
  16. 线性代数计算器C语言(1)——计算行列式值
  17. Say Hello to Integers (Java)
  18. HTML5 canvas 拖尾效果(或尾巴 或方向标 或留痕。。。)
  19. F5 Networks数据中心资源整合解决方案
  20. 群晖Docker安装网心云容器魔方

热门文章

  1. docker--在centos镜像安装mysql
  2. 79. Word Search
  3. Java基础系列--Executor框架(一)
  4. thinkphp整合系列之phpqrcode生成二维码
  5. 字节流 system.in
  6. dropbear环境搭建
  7. getchar getche getch的区别
  8. Flex,Fms3相关文章索引(2009-3-25更新)
  9. .net2.0下数据库连接字符串的使用
  10. Android开发之核心特性SearchView的开发使用(源代码分享)