错误类型汇总:
- 需对结构体成员属性添加解释说明
- 日志输出存在中文
- 正式代码中尽量不适用context.TODO()(这个是说要从父ctx派生)
- 日志输出量较少
- 日志输出缺少上下文
- 日志描述语句不规范
- 日志不规范,缺少必要的提示日志
- 根据代码规范,函数return前不需要加空行
- 日志描述语句不规范
- 程序在退出时为清理掉k8s中产生的临时资源
- 日志中存储多余’\n’
- 不涉及对调用对象的改变,用普通类型
- 创建工作流函数过长
- 将job对象的结构体打包暴露出来,看能否省略掉权限转化
- 删除无效代码
- 代码中存在魔数
附:代码评审-哥
代码审核.docx
- 变量名建议 —go项目中 Error的变量为err的比较多,errWrite等等改成err。
- 使用空行间隔不同逻辑代码块 不同逻辑代码块之间使用空行进行分割,这样代码读起来比较轻松
- 日志输出避免使用中文
- 代码优化建议:分情况讨论(写if-else语句)
- 注释的代码块应该去掉
- 去掉多余的空行