这几天的工作是评审会，本次评审会按照“阿里巴巴JAVA开发手册”的开发标准进行，评审流程分为计划评审会议、准备评审会议、召开评审会议以及修复缺陷四大环节，评审员对班级活动记录网站的源代码的审查及与作者讨论了问题。
在评审会中，针对几个不同的方面，分别具体提出了一些问题，部分check list如下：
……
• 有没有冗余的代码可以清除？
有一些版本更替时被注释掉但并没有删掉的代码，但是由于项目较小并没有使用版本管理工具。考虑到这样注释掉的代码数量并不多，问题不是很大。为了避免未来用到，还是暂时保留了这些代码。
3.具体代码问题
• 边界条件是如何处理的？循环有没有可能出现死循环？
代码在发布公告等功能部分，对日期时间这一易引发边界值问题的变量采用读取系统时间方式，基本不会导致边界值问题。但在查询等功能部分，存在边界值出错的问题。代码基本采用if循环方式，过程中没有对循环控制变量进行重复赋值，不会导致死循环。
……
可以看到，虽然代码已经基本完成了设计时的需求与规范，但在确定了问题并进行详细的观察测试和讨论中看，仍然可以发现不少实际潜在的问题。
对这些问题的提出和回答也是一个过程，需要对整个代码有所了解，并且借助一些工具进行测试，而能否完成这些，又可能牵涉到代码的可读性等规范问题。因此也表明，这些问题之间本身就是存在内在的联系的。