第二次个人编程作业:代码互改
博客班级 | < https://edu.cnblogs.com/campus/fzzcxy/2018SE2/> |
---|---|
作业要求 | https://edu.cnblogs.com/campus/fzzcxy/2018SE2/homework/11199 |
作业目标 | <查看修改同学的代码,修改自己的代码的不足> |
作业源代码 | https://gitee.com/wang-weihan/personal |
学号 | <211806344> |
“让我看看你的代码!”
- 1.(黄龙骏同学的码云仓库 |https://gitee.com/hljhwc/personal)
在写代码过程中,我一直认为写注释是个很重要的事情。现在我们是写的一百多行的代码是比较少的,但是就算如此,就算过了两三周去看自己的代码可能会发现,自己之前写的是个鸟。所以我觉得写代码写一份好的注释是个很必要的东西,而且许多老师在上课时候的有提到在公司里后期的维护看你的代码也是主要看注释,如果注释写得好那会减少很多的维护成本,所以我也希望自己能够写更好的代码,更好的注释o_o ....- 2.(崔亚明同学的码云仓库 |https://gitee.com/cui-yaming/personal)
看到崔亚明同学的代码一开始我觉得没发现什么,并且跟我一样是使用span标签的,但是在大班课里使用了是否参与的进行判断,在大班课的源文件中并没有是否参与的条件。- 3.(林鑫宇同学的码云仓库 |https://gitee.com/its_fucking_great/personal)
林鑫宇同学跟我相同没有考虑到互评经验的问题- 4.自己的问题(https://gitee.com/wang-weihan)
看了别人的代码也发现自己的代码有问题。“容我再改一改!”
在这里崔亚明同学给我提了个醒,本来自己的命名下意识的按自己的习惯来不够规范也没注意到自己的不规则命名,在崔亚明同学的提醒下希望以后都能够规范的命名。
看了许多同学的代码后我发现自己也存在着没有考虑代码互评的问题,虽然在上课助教教了正则方法,但是还是不够太理解所以在我使用了自己原始的办法来考虑互评问题。
这是一个致命的失误,在之前因为答案数据有误我就使用了几个输出看下数据哪个错误,到后面居然忘记删了。
一开始没注意大班课的html文件中活动没有是否参与,而直接进行运算,但是这次在查看别人代码时候发现了这个问题,所以将他修改为通过颜色判断。
issues模板。
“让我再看一看!”
在提出问题后,林鑫宇同学使用循环解决了互评经验的问题。“Pull Requests”
在对崔亚明同学提出问题后发现自己也存在着相同的问题后fork了她的仓库修改代码
但是在上传的时候遇到了一个问题
这里显示我没有权限上传到远程仓库,也是尝试了新建一个分支也没能够解决。最后我以为是权限问题,在给何一山同学修改了互评经验的问题,修改后上传也遇到了同样的问题
这个问题困扰的我头发掉光。
PR问题更新
一开始因为同名仓库不能fork所以改用了自己的一个小号,在小号上fork了仓库,但是在上传时候没有更改本地连接的信息,所以会显示无权限上传,在修改本地连接信息后便可以上传到fork的仓库,然后PR。
解决问题后向何一山同学和崔亚明同学发起PR。
“小结”
在这次的代码互改中,可以让同学看到我许多自己看不到的错误,并且我可以在同学那里学习一个新的编程思路改正自己的编程坏习惯,而且在码云上可以对issues的模板进行改动这让我觉得十分有趣,但是自己还是存在这一些问题,对git的运用不熟练,对Pull Requests用法不熟练这让会让我之后的团队项目造成困扰,而且在百度上没有找到自己想要的答案,希望下节课助教和老师能够多讲讲git和Pull Requests的用法。