结对编程_队友代码分析

其实,看别人代码这种事情,我的内心向来是拒绝的,毕竟,我的代码风格那么美丽,看别人代码就经常很难受~看之前,我心里很忐忑啊,担心他代码没有注释,过分吝啬空格,tab与回车,还有就是左括号不换行——如果是这样的话,我可能就要靠我丰富的想象力写博客了~不过,幸好,队友代码风格还可以,所以,我决定先说一下优点~

主要有以下几点优点:

  • 总体风格不错,该回车的地方有回车,该缩进的地方有缩进,左大括号也换行了,(如果再在各种符号后面加一个空格就更妙了~),如下:
  • 注释很详细,代码本身也很易懂,理解起来就不难,没有什么晦涩难懂的语句~~

优点说完,现在开始我的吐槽了~

  • ,这个“√”,实际上是中文字符,占两个字节的,而单引号里面只能是单字节字符,所以“√”是不能直接用单引号存的,会有数据损失,可以改为
  • 这个指定文件夹指定的真的够清晰的,就是说这个程序换一个目录就用不了,除非也有这么个目录,然后生成的试卷存入该目录。这个地方应该使用相对路径~
  • 这堆代码一堆重复内容,就比如上面那个pathname的声明,就出现了三次,还有一堆其他重复内容就不一一举例了。。。
  • 代码总共425行,main函数407行,从头到尾只有一个获取本地时间的gettime()函数,额,这代码,扩展是不可能扩展的,这辈子也不可能扩展,也就只有重写才能完成新的需求这样子~~
  • 变量命名,挺容易记的,什么judge1(判断1),judge2(判断2),judge3(判断3),shu(数),shu1(数1),说好的匈牙利呢?说好的驼峰呢?

我注意到的队友代码的优缺点就如上面那么多了,总体来说,风格不错,需求勉强完成,但有一堆槽点~~~

猜你喜欢

转载自www.cnblogs.com/afeida/p/9715778.html