结队编程

结队编程作业


结对成员信息:
201521123063 林羽晴
201521123065 洪亚文
码云项目地址:https://gitee.com/jmulyq/CreateFormula.git


1.改进现有的代码

我们选择的是第四个项目
他/她的博客:http://www.cnblogs.com/shizhuangde
git地址:https://coding.net/u/shizhuangde/p/Myapp/git?public=true
原题目要求:


(1)源码分析及修改内容
  • fra_formula
    这个类里面主要是一个output的方法,还有输出格式,我认为只需要将随机生成的式子用字符串的形式打印出来即可,那样一个一个判断条件,有点太繁琐了
  • Fraction
    parseFraction()方法中只考虑了用户输入结果为分数或者整数的情况,在此处添加了小数转为分数的功能,还有遇到负数的情况
    而且在判断结果是否正确的equals()方法中只通过比较分子和分母是否一样,还不够严谨,如果输入的是假分数,那么也应该可以判定为正确的,所以在这个地方只要保证这两个分数转为小数的结果(保留三位小数)相等就可以了
  • int_formula
    这个类是通过setFormula()方法来随机产生
    出现的问题:随机产生的格式比较固定,对两个分数,或一个分数一个整数,或两个整数进行加减乘除,不够灵活
    1109975-20180321184327095-522912591.png
  • main函数,每次循环产生新的题目,题目数量没有更新,导致产生的数目有误
    1109975-20180325100440612-1512918776.png

整体分析:
代码随机产生两个分数,分数的分母为1即为整数,最后,再随机加减乘除,对这两个分数再进行对应的加减乘除,没有任何优先级之分,所以感觉这个代码可以推翻90%重构

(2)生成的类图

修改之后的
1109975-20180325181338090-1162070251.png

(3)单元测试并测试覆盖率
  • 单元测试代码:(部分)
    1109975-20180325192023167-1473903993.png
    1109975-20180325192050514-826671408.png

  • 未修改之前单元测试:
    1109975-20180325132401915-47090897.png
  • 未修改之前的覆盖率:
    1109934-20180325134521925-1287068151.png

  • 修改之后的单元测试:
    1109975-20180325143104178-702859402.png
  • 修改之后的覆盖率:
    1109975-20180325150303406-430872432.png

    通不过的原因果然是因为有小数,以及负数的情况没有考虑充分!
    (4)出现的bug以及修改方法

    1109975-20180325182947682-515309488.png
    1109975-20180321170136425-1593396888.png

  • 添加负数和小数的判断和转化
    1109975-20180325182031884-450484886.png
  • 对结果增加了通分,如果是负数的话,则统一把分子看做负数,分母看做正数
    1109975-20180325182122211-71659365.png
    -将结果全部转为带小数的值(只保留三位)如果出现无限不循环小数,方便和答案统一
  • 增加了init_num的重新赋值
    1109975-20180325191017559-1798849027.png

    (5)修改之后重新测试

    1109975-20180325191814786-2145989810.png

2.功能的改进与扩展

1109975-20180325164106279-753694920.png

(1)增加乘方的功能
   主要根据优先级来计算,具体的详见代码

1109975-20180325193413030-1955329274.png

(2)随机产括号和乘方
    只随机产生一对或者不加括号,考虑到了冗余的情况,去掉类似(1+2)这种的括号

1109975-20180325193220441-976906619.png
1109975-20180325193246007-111638831.png

(3)查重的功能
    这个要用到树的最小表示法,首先要将表达式用二叉树存储,要比较两个表达式是否一样,就是要得出它们的最小表示法,当符号为+或者为*的时候,才比较左右子树,保证左子树的hash值要大于右子树,最后会发现,如果是同一个表达式,它们的最小表示式是一样的,但是这个功能由于时间的关系,没能及时完成,待后续补充

添加功能之后的类图
1109975-20180325164635814-782435627.png

重新单元测试:
1109975-20180325193352010-1177129919.png

覆盖率:
1109975-20180325164106279-753694920.png

回归测试:
能保证原来的计算功能!
1109975-20180325193724772-992045609.png
效能分析:
花费在时间上比较多的是计算的方法,还有事随机式子的产生,在判断有级的时候,暴力给定有限级的效率会比较低一点,觉得可以给符号标记数字,用枚举变量的方式可能会简单一点
逻辑泥球:
1109975-20180325210838083-386538612.png

在输出的格式上编写太复杂了,其实可以直接用一个字符串数组,或者用list来存放这些数和操作符,最后直接遍历输出就可以了
提交的记录:
1109975-20180325202242899-1439734864.png

结对编程照片:
1109975-20180325201307941-1057881104.png

编码规范:

1.类名的首字母大写
Main:main入口函数及执行
Function:含有所有的功能方法
Stack:自定义的栈
Fraction:分数类

2.内部的方法全部以驼峰式命名
ParseDouble:将字符串转为浮点型小数
CreateFormula:随机产生四则运算的式子
Calculator:用来计算结果

3.变量名全部小写
Main当中
number:用来计数
all:生成的题目总数
correct_num:回答正确的题目数
get_answer:用户输入的答案
right_answer:计算机计算的正确答案
option:值为0退出,1继续
在CreateFormula当中:
list:存放式子
number:随机产生数的个数(2-4)
bracket:括号的个数(0-1)
left:表示左括号在第left个数的前面
right:表示右括号在第right个数的后面

PSP表格:

PSP2.1个人开发流程预估耗费时间(分钟)实际耗费时间(分钟)
Planning计划108
· Estimate明确需求和其他相关因素,估计每个阶段的时间成本55
Development开发110135
· Analysis需求分析53
· Design Spec生成设计文档1011
· Design Review设计复审108
· Coding Standard代码规范21
· Design具体设计1012
· Coding具体编码120180
· Code Review代码复审2018
· Test测试(自我测试,修改代码,提交修改)4050
Reporting报告1520
.测试报告1010
.算工作量短文本33
.并提出过程改进计划23

结队编程小结:
本次编程作业,我和我的小伙伴合作完成,期间也有意见不合的时候,也有不知对方所云的时候,友谊的小船说翻就能翻的那种。
但是最后还是比较顺利地完成了任务,给我的感受是1+1=1.5的那种吧,还达不到1+1>2的效果,唯一有点遗憾的是查重部分没有及时完成,根据树的最小表示法原理,一开始查找网页的详解还是一头雾水,看了JLdalao的源代码,发现不是一个上午就能完成的,看来编程能力还要提高,毕竟距离上次敲代码已经有三四个月了吧!

转载于:https://www.cnblogs.com/lyq063/p/8590403.html

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值