博客
关于我
强烈建议你试试无所不能的chatGPT,快点击我
个人博客作业2
阅读量:5973 次
发布时间:2019-06-19

本文共 1884 字,大约阅读时间需要 6 分钟。

一.是否需要有代码规范

   1.不同意该观点。每个人都有各自的代码风格,当单独工作时不会产生什么影响。但当进行团队项目合作时,正如文章中提到的面对“代码集体所有制”时,多样的风格变成了团队的阻碍。团队间的协作少不了代码之间的分配与合作,此时个人必须也应该服从于集体。统一的代码规范有助于统一的维护,也能减少队内纷争,从而让队伍更加团结有效率;而且也利于另一篇文章中提到的代码复查。所以制定一套合格的代码规范,根本不是一件浪费时间影响效率的事,事实正好相反。

   2.不同意。对于软件工程中的团队合作这件事,个人的艺术天分不应该发挥在代码规范上,更不应该成为阻碍团队协作与成果的借口。代码规范是团队合作的基本保证,是整个团队保持一致性的关键。艺术天分完全可以用在其他方面。

   3.不同意。不强求何来规矩?如果总是有例外的话,规范根本无法维持下去。法律不会给任何人“例外”的权利,代码规范对于一个团队同样如此。

   4.不同意。代码规范是一个团队共同使用的,会对日后的项目工作产生巨大作用。经过团队共同商讨出的代码规范,往往会比个人决策出的代码规范更加合理,更能让大多数人满意,更有利于项目的后续进行。另外,团队合作毕竟是一个协作、磨合、相处的过程,有些事情“集中制”更加有效,有些事以民主的方式往往更好。

 

二.代码复审

参考给出的代码复审各项标准

(一)代码整体

    1.代码是否能够正常工作?是否能够达到预期的功能,逻辑是否正确:未实现全部功能,实现了生成式子并运算的功能。将生成结果及答案写入txt功能为未实现。

       逻辑正确性:(1)对于关于0生成限制不完善。 例:输入n=10 r=2,输出的式子中分子包含0。 0/1+0/1+0/1.

                        (2)计算部分有错误:关于连续减法、除法均有错误。 例:29-29-29=0    

                        (3) 检测负数产生功能不完善。 例子同上

    2.代码是否易于理解:代码结构分明,也有合适的注释,变量命名也规范,可读性强。

    3.是否符合你们的代码规范?通常包括括号的位置,变量和函数的命名,每一行的长度,格式以及注释:在我自己看来符合自己能接受的代码规范,代码清晰。

    4.是否有冗余代码:在Expression.cs中,分情况处理子表达式式时(多个combine方法),各种情况是有很大相通性的,可以将多个方法合并成一个,加控制变量控制即可

                             在SubExpression中也是一样,多个方法有相似性

    5.模块化程度:划分代码时以表达式为划分核心,分为了Expression Expression SubExpression 等多个模块,模块化程度较高

    6.全局变量是否能够被替代?:无全局变量

    7.是否有被注释掉的代码:含有被注释掉的代码

    8.循环是否设置循环长度以及是否可以正确结束:无死循环存在,循环均可正确结束

    9.是否有代码能被库函数代替:对c#不太熟悉,库函数的了解不是很多。。。

    10.可以删除任何日志记录或调试代码吗?:无日志记录和调试代码

(二)安全性

    1.所有的输入/输出数据都被检查(检查格式是否正确,长度,类型以及范围等)并且编码过了吗?:输入参数未检查,当输入参数不合法时,未能正确处理给出错误信息。如输入-r 0,程序会直接不正常结束

           输出方面此项目并未涉及检查。但能正确处理,输出分数对应组成部分

    2.哪些地方使用了第三方程序,返回的错误信息是否全部被捕获了?:未涉及第三方程序

    3.不合法参数:第1条中已提到,输入不合法参数未被处理

(三)文档

    未涉及文档

    1.是否有文档来解释代码?

    2.所有的函数是否有加上规格?

    3.是否对不寻常的行为或边界情况处理进行描述?

    4.是否有记录对第三方函数的使用情况?

    5.对数据结构和度量单位解释了吗?

    6.有不完整的代码吗?如果是这样,应该是删除或标记等一个合适的标记“待办事项”? 

(四)测试

    1.代码是可测试的吗?即是否添加太多隐藏的依赖、是否能够初始化对象、测试框架可以使用方法等:代码可以进行测试

    2.测试是否存在?以及它是否做到了完全覆盖?: 未设计测试代码

    3.单元测试是否测试到了代码能否达到预期的功能?:未设计测试代码

    4.哪些测试代码是可以用已经存在的API来代替的?:未设计测试代码

    5.数组越界检查:并未进行数组越界方面的检查与处理,存在风险。

总结,性能上并不是十分理想,一些边界情况的处理不到位。但在代码规范上做的较好,而关于测试与文档环节,现在可能还并未涉及。

转载于:https://www.cnblogs.com/wcysoftware/p/4841222.html

你可能感兴趣的文章
Linux 内核已支持苹果
查看>>
shell脚本逻辑判断,文件目录属性判断,if,case用法
查看>>
【二叉树系列】二叉树课程大作业
查看>>
Java5的 线程并发库
查看>>
mybatis拦截器
查看>>
App重新启动
查看>>
矩阵乘法
查看>>
得到目标元素距离视口的距离以及元素自身的宽度与高度(用于浮层位置的动态改变)...
查看>>
安装和配置Tomcat
查看>>
实验三
查看>>
第一次实验总结
查看>>
openssh for windows
查看>>
PostgreSQL cheatSheet
查看>>
ASP.NET Core 2 学习笔记(三)中间件
查看>>
转:Mosquitto用户认证配置
查看>>
SpringBoot上传文件到本服务器 目录与jar包同级
查看>>
python开发_difflib字符串比较
查看>>
被解放的姜戈01 初试天涯
查看>>
三极管工作区在Spectre中的表示
查看>>
HT for Web的HTML5树组件延迟加载技术实现
查看>>