第一次结对编程作业

代码审查表

结对同伴:郭靖
同伴代码地址:https://coding.net/u/q842774912/p/homework1/git/blob/master/main.cpp?public=true

代码评论和心得体会

首先，这是我第一次和同学进行代码审核的任务，这次作业让我对代码审核的作用有了极大的改观。本来我以为，代码审核仅仅就是看一遍对方的代码，然后多想几个测试用例把代码里容易覆盖不到的地方过一遍就可以了，但是看到了老师给的审查表，才发现自己原来的想法有多么大的错误。这张审查表可以说是覆盖了各个方面，函数；内存；表达语句等等，甚至连常量这种我以为根本没有什么可说的内容居然也有考察到，而且难以启齿的是，有一些问题我甚至都不了解他具体在问什么方面，每当遇到那种问题，我只能根据代码凭感觉给出是否，实在是对同伴很不负责。还有一些是根本没有用到的地方，这张表中也有覆盖。在对审查代码的作用产生惊讶的时候，我便去网上看了看审查代码的重要性，真的是受益匪浅。审查代码的重要性主要包括：1.提升代码质量，降低风险，毕竟就算是再好的程序员，也可能会失察，还有一些由于个人习惯产生的问题，也可以被发现和解决；2.大家都能在这个过程中获得进步，被审查的人可以发现自己的错误，审查者也可以通过阅读他人代码来获得提高；3.以后要是一起做项目的话，还可以快速了解项目。进行定期和高效的代码审查对于保持高质量的代码标准来说必不可少，有的同学可能会对请求他人的帮助感到不好意思，认为自己写的代码可能错误百出，怎么能去麻烦别人评价。其实这个真的没有必要不要意思，不仅仅是同学，就算一些出色的大佬前辈也有可能出现问题，毕竟是人总会犯错嘛。
说完了一些心得体会，下面我来说一下同伴代码中的优缺点：在编写这段代码的时候，同伴身为一个大二的学生，写出这个代码还是相当不错的，这段代码有以下几个优点：1.书写规范。在语句中，每一个括号都是对应，每一个语句都单独一行，没有多条语句混在一起的情况，看起来显得十分工整；2.结构清晰。无论是函数的书写和运用，都简单明了，没有乱定义函数名导致根本找不到，不了解这段代码要做什么的情况存在。虽然写的还好但是缺点也同样存在：1.没有相应的注释。其实说这个缺点的时候，我自己也好不到哪去，我的代码也没有注释的习惯，还是看到审查表的那些才意识到有相应的注释是多么的重要，不仅给他人阅读代码的时候带来方便，也是可以让自己在以后看到代码的时候回想起当时的想法和思路，而不会出现自己写的代码自己都看不懂的情况。因为我自己能力的不足，可能暂时找不到更多的优缺点，这个是很可惜的地方，希望能在以后的锻炼中，更加了解代码的优劣，然后优化自己的编程习惯。