zoukankan      html  css  js  c++  java
  • 20180320作业2:进行代码复审训练

    一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)

    http://www.cnblogs.com/lixiang66/p/8582869.html

    1、概要部分

    (1)代码符合需求和规格说明么?

    大部分符合。

    (2)代码设计是否考虑周全?

    不够周全,代码中判断出了问题所在,却不输出错误原因,而是统一输出“邮箱格式错误”,多此一举。

    (3)代码可读性如何?

    代码简单明了,可读性良好。

    (4)有冗余的或重复的代码吗?

    if判断语句冗余。

    (5)代码的每一行都执行并检查过了吗?

     检查过,并无问题。

    2、设计规范部分

    (1)设计是否遵从已知的设计模式或项目中常用的模式?

    代码遵循了已知的设计模式和在项目中的常用模式。

    (2)有没有硬编码或字符串存在?

    有字符串和数字的存在。

    (3)代码有没有依赖于某一平台?

    java没有依赖于某一平台,可移植性高。

    (4)有没有无用的代码可以清除?

     没有无用代码可以清除。

    3、代码规范部分

    大部分地方是符合代码标准和风格的

    4、具体代码部分

    (1)数据结构中有没有用不到的元素?

    没有

    (2)对于调用的外部函数,是否检查了返回值?

    检查了返回值

    5、效能

    (1)代码的效能如何?

    达到了具体任务要求的程度

    (2)循环中是否有明显可优化的部分?

    有,if判断语句过多,完全可以合并。

    6、可读性

    可读性良好,虽然没有注释

    7、可测试性

    是否需要更新或创建新的单元测试?

    暂时不需要

    8、代码复审感想

    代码在可复用的基础上,尽量简短。对于邮箱的判断,一般用户注册上,所以我觉得只需要用最简单快捷的代码判断是否正确即可。

  • 相关阅读:
    JavaScript 深入了解对象中的属性
    JavaScript 开发规范
    vue 项目接口管理
    放大镜特效
    多用户ATM机(面向对象编程)
    浏览器检查块代码
    js中innerHTML与innerText的用法与区别
    symbol访问法及symbor注册表
    最常用的15个前端表单验证JS正则表达式
    数组的遍历
  • 原文地址:https://www.cnblogs.com/zyc1246/p/8610777.html
Copyright © 2011-2022 走看看