51Testing软件测试论坛

 找回密码
 (注-册)加入51Testing

QQ登录

只需一步,快速开始

微信登录,快人一步

手机号码,快捷登录

查看: 1723|回复: 0
打印 上一主题 下一主题

[原创] iOS Code Review

[复制链接]
  • TA的每日心情
    无聊
    4 天前
  • 签到天数: 11 天

    连续签到: 2 天

    [LV.3]测试连长

    跳转到指定楼层
    1#
    发表于 2019-3-1 14:57:42 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式
    Code Review的意识
    • 作为一个 Developer,不仅要提交可工作的代码(Deliver working code),更要提交可维护的代码(Deliver maintainable code)
    • 必要时进行重构,随着项目的迭代,在计划新增功能的同时,开发要主动计划重构的工作项
    • 开放的心态,虚心接受大家的评审建议(Review Comments)

    代码评审的方式
    • 开 Code Review 会议
    • 团队内部会整理 Check List
    • 团队内部成员交换代码
    • 找出可优化方案
    • 多问问题,例如:“这块儿是怎么工作的?”、“如果有XXX 情况,你这个怎么处理?”
    • 区分重点,优先抓住设计,可读性,健壮性等重点问题
    • 整理好的编码实践,用来作为 Code Review的参考

    评审内容[1]架构/设计/常规
    • 1.单一职责原则
      • 这是经常被违背的原则。一个类只能干一个事情,一个方法最好也只干一件事情。比较常见的违背是一个类既干UI的事情,又干逻辑的事情,这个在低质量的客户端代码里很常见
    • 2.行为是否统一,例如:
      • 1.缓存是否统一
      • 2.错误处理是否统一
      • 3.错误提示是否统一
      • 4.弹出框是否统一
      • ……
    • 3.代码污染
      • 代码有没有对其他模块强耦合
    • 4.重复代码-->应该抽取
    • 5.开闭原则
    • 6.面向接口编程
    • 7.健壮性
      • 是否考虑线程安全
      • 数据访问是否一致性
      • 边界处理是否完整
      • 逻辑是否健壮
      • 是否有内存泄漏
      • 有没有循环依赖
      • 有没有野指针
      • 是否检查了数组的“越界“错误
      • ……
    • 8.错误处理
    • 9.改动是不是对代码的提升
      • 新的改动是打补丁,让代码质量继续恶化,还是对代码质量做了修复
    • 10.效率/性能
      • 1)关键算法的时间复杂度多少?有没有可能有潜在的性能瓶颈
      • 2)客户端程序对频繁消息和较大数据等耗时操作是否处理得当


    [2]代码风格
    • 1.可读性
      • 衡量可读性的可以有很好实践的标准,就是 Reviewer 能否非常容易的理解这个代码。如果不是,那意味着代码的可读性要进行改进
    • 2.命名
      • 命名对可读性非常重要
      • 是否跟系统属性命名造成冲突
      • 英语用词尽量准确一点,必要时可以查字典
    • 3.函数长度/类长度
      • 函数太长的不好阅读
      • 类太长了,检查是否违反的 单一职责原则
    • 4.注释
      • 恰到好处的注释,不是注释越多越好
    • 5.参数个数
      • 不要太多,一般不要超过 3 个


    Review Your Own Code First
    • 每次提交前整体把自己的代码过一遍非常有帮助,尤其是看看有没有犯低级错误

    分享到:  QQ好友和群QQ好友和群 QQ空间QQ空间 腾讯微博腾讯微博 腾讯朋友腾讯朋友
    收藏收藏
    回复

    使用道具 举报

    本版积分规则

    关闭

    站长推荐上一条 /1 下一条

    小黑屋|手机版|Archiver|51Testing软件测试网 ( 沪ICP备05003035号 关于我们

    GMT+8, 2024-9-23 22:30 , Processed in 0.078536 second(s), 26 queries .

    Powered by Discuz! X3.2

    © 2001-2024 Comsenz Inc.

    快速回复 返回顶部 返回列表