Code Review简单总结

概述

程序员都想写出好的代码,在是在真正工作中, 往往不能做到，我自己也不做到。你肯定见过一些，底层技术任务很牛B（其实就是不牛逼），所谓的大牛，但是看他们的代码很烂，逻辑性差,逻辑性差其实业务理解能力差。
代码其实是给人看的，是需要维护的，如果是计算机执行其实无所谓。
在程序员维护其他人的代码时候，第一反应是代码太烂，看不进去了,一坨狗屎，哪个SB写的,其实往往我们自己在别人眼里就是那个SB。但是如果自己写，未必就能写好。无论你的代码写的多好，在别人眼里，也有不好的地方.程序员应该不断总结回自己的代码，是否有在此查看时候，看自己在当时怎么写出一坨狗屎代码。每次结束一个功能的时候，利用半小时或者在1个小时，小步不断重构，让其易懂变好，追求极致。

我个人认为始终贯彻一点代码是给人看的，是需要维护的。如果是计算机其实无论好坏，只要是正常逻辑，满足性能无所谓

推荐几本书

1. 阿里巴巴编程规范指南
2. 重构改善既有代码设计
3. uml和模式应用
4. 软件架构设计

实际项目中总结的一些review点（持续更新）

• 过于随意的代码
• url命名
• 代码格式化
• 类命名
• 类的职责
• 变量作用域
• 方法作用域
• 单一方法行数
• public方法必须判断入参
• 方法注释说明
  -一个方法体中重复get属性
• 代码正常逻辑概览
• 性能,如循环取值
• 日志 无参数语义不能缺的
• sql上查询全部查询必须去掉
  text的字段，如果描述
  描述这种字段不能随着对象一起放在redis里，必须单独获取（如果不常用可以不用放的）
• 如果是批量保存无特殊原因，必须批量保存，不能for循环一条一条insert
• redis缓存不带过期时间，
• redis大对象
• insert或者update操作web层必须携带token