// ❌ 危険: エラー無視
result, _ := userRepo.Find(id)

// ❌ 危険: panic乱用
if err != nil {
    panic(err)  // サーバークラッシュの原因
}

// ✅ 正しい: エラー適切処理
result, err := userRepo.Find(id)
if err != nil {
    return fmt.Errorf("failed to find user: %w", err)
}

2. goroutine リーク

// ❌ 危険: 終了しないgoroutine
func process() {
    ch := make(chan int)
    go func() {
        for v := range ch {  // chがクローズされない
            fmt.Println(v)
        }
    }()
}

// ✅ 正しい: context でキャンセル制御
func process(ctx context.Context) {
    ch := make(chan int)
    go func() {
        defer close(ch)
        for {
            select {
            case <-ctx.Done():
                return
            case v := <-ch:
                fmt.Println(v)
            }
        }
    }()
}

3. インターフェース濫用

// ❌ 危険: 不要なインターフェース
type UserRepositoryInterface interface {
    Find(int) (*User, error)
    Save(*User) error
}

// ✅ 正しい: 必要な場所でのみ定義（Accept interfaces）
// domain/repository.go
type UserRepository interface {
    Find(int) (*User, error)
}

// infrastructure/user_repository.go
type userRepositoryImpl struct { ... }
func (r *userRepositoryImpl) Find(id int) (*User, error) { ... }

🟡 Warning（要改善）

1. context 不使用

// ⚠️ タイムアウト制御がない
func FetchData(url string) ([]byte, error) {
    resp, err := http.Get(url)
    ...
}

// ✅ context でタイムアウト制御
func FetchData(ctx context.Context, url string) ([]byte, error) {
    req, _ := http.NewRequestWithContext(ctx, "GET", url, nil)
    resp, err := http.DefaultClient.Do(req)
    ...
}

2. sync.Mutex の defer 忘れ

// ⚠️ Unlock 忘れのリスク
mu.Lock()
if err := process(); err != nil {
    return err  // Unlock されない
}
mu.Unlock()

// ✅ defer で確実に Unlock
mu.Lock()
defer mu.Unlock()
if err := process(); err != nil {
    return err
}

3. テーブル駆動テスト未使用

// ⚠️ 同じテストコードの繰り返し
func TestAdd(t *testing.T) {
    if Add(1, 2) != 3 { t.Error("failed") }
    if Add(0, 0) != 0 { t.Error("failed") }
}

// ✅ テーブル駆動
func TestAdd(t *testing.T) {
    tests := []struct {
        name string
        a, b int
        want int
    }{
        {"positive", 1, 2, 3},
        {"zero", 0, 0, 0},
    }
    for _, tt := range tests {
        t.Run(tt.name, func(t *testing.T) {
            if got := Add(tt.a, tt.b); got != tt.want {
                t.Errorf("got %d, want %d", got, tt.want)
            }
        })
    }
}

Go イディオムチェック

命名規則

エラーハンドリング

並行処理

設計

テストチェックリスト

出力形式

🔴 Critical: ファイル:行 - エラー処理違反/goroutineリーク - 修正案 🟡 Warning: ファイル:行 - イディオム違反 - 改善案 📊 Summary: Critical X件 / Warning Y件

go-backend

Goバックエンド開発

使用タイミング

開発観点

🔴 Critical（修正必須）

1. エラーハンドリング違反