Code Review即代码审查,审查包括:
1、风格问题(比如不符合团队的代码规范)
如果项目很庞大、代码质量要求很高,那么,制定和执行代码规范所花费的时间,将大大减少后期因为不规范开发带来的种种调试和维护成本。
没有任何一种代码规范是完美的,团队的代码规范,一般由领导和大佬们制定后(需要独裁),大家都统一实行。
如果代码规范限制太松,那么等于没有规范;如果太严,大大影响开发效率。这里面的尺度,需要根据项目需要、团队成员特点全面考量,进行取舍。
举例:
if else级联问题
2、安全性问题(请求处理外部数据时,XSS攻击等过滤)
是否使用PHP框架中规定的请求函数来接收用户提交数据
3、优雅性问题(大量冗余代码、可复用性低、耦合度高)
举例:
在一个
1、风格问题(比如不符合团队的代码规范)
如果项目很庞大、代码质量要求很高,那么,制定和执行代码规范所花费的时间,将大大减少后期因为不规范开发带来的种种调试和维护成本。
没有任何一种代码规范是完美的,团队的代码规范,一般由领导和大佬们制定后(需要独裁),大家都统一实行。
如果代码规范限制太松,那么等于没有规范;如果太严,大大影响开发效率。这里面的尺度,需要根据项目需要、团队成员特点全面考量,进行取舍。
举例:
if else级联问题
2、安全性问题(请求处理外部数据时,XSS攻击等过滤)
是否使用PHP框架中规定的请求函数来接收用户提交数据
3、优雅性问题(大量冗余代码、可复用性低、耦合度高)
举例:
在一个
4、正确性问题(算法设计错误)
未完待续。。。