ICode9

精准搜索请尝试: 精确搜索
首页 > 编程语言> 文章详细

结对编程代码分析

2022-09-13 20:36:02  阅读:203  来源: 互联网

标签:结对 变量 代码 编程 队友 数组 不写 定义


结构分析:

缺点:

  • 队友的代码只有俩个类,一个用户类和一个菜单类,导致结构混乱,省略后menu中部分如图:

其中含有多个功能的函数,都放在一个模块,一个menu中350左右行,导致每个模块中承担了多个不相关的功能,应该分模块编写。

  • 而且也不需要定义开头的静态loginUser和UerList,这些完全可以在main中定义,定义过多的静态变量浪费资源而且容易产生冲突。
  • 队友不写注释,全篇一个注释没有,让人看到脑袋发晕,而且一堆的临时变量不写注释大大提升了队友对代码的解读难度。

 可以看出,队友完全不写注释,这样的代码难以快速读懂,而且变量名虽然简单却又不能准确代表变量的含义。

  • 变量复用率低,代码写时,没有大局观,定义了一堆明明名可以重用的数组,如:

 

 

可以看出,符号数组应该定义成全局数据,而队友在多个if里面定义了多个数组,这就导致了代码复用率低而且变量明混乱,代码可读性变差

优点:

  • 用户类用文档中读取的数据初始化:

 可以看出,用户初始化这里是提供了一个静态的方法用来初始化一个ArrayList数组,并且使用的是一个默认的txt文档路径

功能分析:

  • 功能实现不符要求

 

 题目要求账户和密码空格分开,而队友是换行输入:

  •  高中题目,按道理是可以有括号的,而他的高中题目却没有括号,功能实现不够完善

 

 其他功能均以按题目要求实现

  点评:

队友有些思想比如读取文件里数据到对象思路值得学校,但队友的问题还是不容小觑。

 

  总结:

  功能基本实现,队友最大的问题,需要规范以及类的合理设计。

标签:结对,变量,代码,编程,队友,数组,不写,定义
来源: https://www.cnblogs.com/yhcboke/p/16690501.html

本站声明: 1. iCode9 技术分享网(下文简称本站)提供的所有内容,仅供技术学习、探讨和分享;
2. 关于本站的所有留言、评论、转载及引用,纯属内容发起人的个人观点,与本站观点和立场无关;
3. 关于本站的所有言论和文字,纯属内容发起人的个人观点,与本站观点和立场无关;
4. 本站文章均是网友提供,不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属;如您发现该文章侵犯了您的权益,可联系我们第一时间进行删除;
5. 本站为非盈利性的个人网站,所有内容不会用来进行牟利,也不会利用任何形式的广告来间接获益,纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。

专注分享技术,共同学习,共同进步。侵权联系[81616952@qq.com]

Copyright (C)ICode9.com, All Rights Reserved.

ICode9版权所有