القائمة
审查Go代码的最佳实践、bug、并发安全和业务逻辑。当用户要求代码审查或完成重要代码更改后调用。
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
| name | code-reviewer |
| description | 审查Go代码的最佳实践、bug、并发安全和业务逻辑。当用户要求代码审查或完成重要代码更改后调用。 |
此技能用于对遵循项目规范和最佳实践的Go项目进行全面的代码审查。
gofmt格式化？if err != nil）？fmt.Errorf("xxx: %v", err)）？panic和recover是否使用得当（避免滥用）？sync.Mutex）或原子操作保护？sync.WaitGroup和context.Context是否正确用于并发控制？+连接字符串而不是strings.Builder）？defer。sync.Pool减少对象分配？t.Fatal而不是panic）？defer？（可能导致资源释放延迟，考虑使用匿名函数）sync.Mutex、sync.WaitGroup？time.After导致内存泄漏？## 代码审查报告
### 概述
代码质量和主要发现的简要概述。
### 严重问题
1. [文件:行号] 问题描述
- 建议修复：...
### 重要问题
1. [文件:行号] 问题描述
- 建议修复：...
### 次要问题
1. [文件:行号] 问题描述
- 建议修复：...
### 建议
- 不是bug但可以提高代码质量的改进建议。
### 积极方面
- 突出代码中发现的好做法。
// 错误
hero := bagHero.GetElem(req.GetId())
hero.Level += 1 // 潜在的空指针解引用
// 正确
hero := bagHero.GetElem(req.GetId())
if hero == nil {
l.Error("hero not found", "heroId", req.GetId())
return nil, errors.New("heroNotFound")
}
hero.Level += 1
// 错误
total := a + b // 可能溢出int32
// 正确
if int64(a) + int64(b) > math.MaxInt32 {
return errors.New("overflow")
}
total := a + b
// 错误
file, _ := os.Open(path)
data, _ := io.ReadAll(file)
// 正确
file, err := os.Open(path)
if err != nil {
return err
}
defer file.Close()
data, err := io.ReadAll(file)
if err != nil {
return err
}
// 错误：先发放，后扣除（可能导致漏洞）
bag.AddItem(item)
bag.DelItems(cost)
// 正确：先扣除，后发放
if !bag.IsEnough(cost) {
return errors.New("notEnough")
}
bag.DelItems(cost)
bag.AddItem(item)
استنادا إلى تصنيف SOC المهني
