代码Recode

可扩展性

•  是否可以快速的支持新的迭代？

•  SOLID原则

单一职责原则（Single Responsibility） 开闭原则（Open Closed） 里氏代换原则（Liskov Substitution） 接口隔离原原则（Interface Segregation） 依赖倒置原则（Dependency Inversion）

 

可维护性

•  团队认知

如果项目交接，他人能够快速了解代码意图吗？

•  编码规范

PHP 编码规范 https://psr.phphub.org/

•  持续迭代

如果后续功能迭代，能够快速支持且不影响现有功能或者其他模块吗？

•  类库复用

•  设计模式

•  过时用法

 

性能

整体执行耗时

•  空间

按需加载，节省内存

 

•  时间

死循环 无穷递归 外部系统依赖 死锁

 

安全

•  输入输出校检

•  异常处理

•  边界处理（日期、数组等）

•  敏感信息

用户隐私

•  权限设置

日志

•  关键路径埋点

外部系统调用 敏感操作

•  指标纬度

记录时间 日志级别 日志标识 请求体 响应体 用户id 用户ip 服务器ip 会话id 文件行数

•  是否旁路

•  级别开关

监控

•  关键路径埋点

•  指标纬度

耗时 成功/失败率 失败状态码

•  是否旁路

评审方式

 

日常迭代

•  技术负责人评审

•  每日SVN评审变更点

•  合并时变更点周知全员

自动评审

 

评审会议

•  code在线评审

•  随机抽取最近提交

•  非指名提出建议

•  记录到知识库并跟进落地

版本计划

• 出具评审报告

• PM责任制