科泰北大青鸟java培训专家为你解答！
长沙北大青鸟作者：科泰校区
摘要： 你应该寻找什么?哪种事情对人类来讲是值得的?我们可以在代码审查中发现，我们不能把事情委托给一个工具?这有数量惊人的东西。这肯定不是一个详尽的清单，我们也不会去伟大
你应该寻找什么?
哪种事情对人类来讲是值得的?我们可以在代码审查中发现，我们不能把事情委托给一个工具?
这有数量惊人的东西。这肯定不是一个详尽的清单，我们也不会去伟大的这里详细介绍他们中的任何一个。相反，这应该是你的组织中关于如何在代码审查中寻找东西这样一个对话的开始。
设计
新代码如何适合整体架构?
代码是否遵循扎实的原则，领域驱动的设计和/或其他设计范例团队的青睐?
新代码中使用了哪些设计模式?这些是否合适?
如果代码有一个混合的标准或设计风格，这种新的代码是否遵循目前的做法?代码是否按照正确的方向迁移，或者是否遵循了由于被淘汰的旧代码的示例?
代码在正确的地方?例如，如果代码是与订单相关的，则是以订单服务吗?
新代码可以在已有的代码中重复使用一些东西吗?新的代码是否提供了在现有代码中可以重用的东西?新的代码是否引入重复?如果是这样的话，它应该被重构为一个可重复使用的模式，这是可以接受的在这个阶段?
代码是工程设计的吗?它是否建立了不需要现在的可重用性?团队如何平衡重构和YAGNI的关系?
可读性和可维护性
这些名字(字段，变量，参数，方法和类)实际上反映了他们所代表的东西吗?
我能够读懂这些代码吗?
我能理解测试吗?
测试是否覆盖了一个很好的子集?他们是否包括常规和特殊情况?是否有没考虑到的情况?
异常错误消息可以理解吗?
功能性
代码是否达到了它的期望值?如果有自动测试能确保代码的正确性，那这个测试真的能测试出代码符合要求吗?
代码是否看起来像是一个如使用错误的变量的检查或不小心的细微的bug?
你有没有想过…?
是否存在潜在安全问题的代码?
是否有需要满足的监管要求?
对于不包括自动化性能测试的领域，新的代码是否会引入可避免的性能问题，如不必要的调用数据库或远程服务?
是否需要创建公共文档，或者修改现有的帮助文件?
是否有面向用户的消息被检查正确性?
是否有明显的错误需要停止这一工作的生产?代码要在测试数据库是否不小心，或者是有一个硬编码的存根需要替换成一个真正的服务?
