可扩展性
-
是否可以快速的支持新的迭代？
-
SOLID原则
单一职责原则（Single Responsibility） 开闭原则（Open Closed） 里氏代换原则（Liskov Substitution） 接口隔离原原则（Interface Segregation） 依赖倒置原则（Dependency Inversion）
可维护性
-
团队认知
如果项目交接，他人能够快速了解代码意图吗？
-
编码规范
PHP 编码规范 https://psr.phphub.org/
-
持续迭代
如果后续功能迭代，能够快速支持且不影响现有功能或者其他模块吗？
-
类库复用
-
设计模式
-
过时用法
性能
整体执行耗时
-
空间
按需加载，节省内存
-
时间
死循环 无穷递归 外部系统依赖 死锁
安全
-
输入输出校检
-
异常处理
-
边界处理（日期、数组等）
-
敏感信息
用户隐私
-
权限设置
日志
-
关键路径埋点
外部系统调用 敏感操作
-
指标纬度
记录时间 日志级别 日志标识 请求体 响应体 用户id 用户ip 服务器ip 会话id 文件行数
-
是否旁路
-
级别开关
监控
-
关键路径埋点
-
指标纬度
耗时 成功/失败率 失败状态码
-
是否旁路
评审方式
日常迭代
-
技术负责人评审
-
每日SVN评审变更点
-
合并时变更点周知全员
自动评审
评审会议
-
code在线评审
-
随机抽取最近提交
-
非指名提出建议
-
记录到知识库并跟进落地
版本计划
-
出具评审报告
-
PM责任制