手順
- 対象の特定: リファクタリング対象のファイル/関数を特定
- 既存テストの確認:
- 対応する
*_test.go を確認
go test ./... を実行してベースライン確立（全テストがパスすることを確認）
- リファクタリング計画: 以下の観点で改善機会を特定
- 命名改善（変数名、関数名の明確化）
- 関数分割（長すぎる関数、複数の責任を持つ関数）
- 重複コード削除（DRY原則）
- 複雑な条件式の簡素化
- マジックナンバーの定数化
- エラーハンドリングの改善
- リファクタリング実施: 小さなステップで段階的に変更
- テスト実行: 各ステップ後に
go test ./... で機能不変を確認
- コード品質確認:
go fmt ./... でフォーマットgo vet ./... で静的解析
- コミット: 変更が成功したら、適切なメッセージでコミット（
refactor: ...形式）
リファクタリングパターン
1. 命名改善
go
1// Before
2func f(n []int) int { ... }
3
4// After
5func Sum(numbers []int) int { ... }
2. 関数分割
go
1// Before: 1つの関数が複数の責任
2func ProcessData(data []byte) error {
3 // バリデーション
4 // パース
5 // 変換
6 // 保存
7}
8
9// After: 単一責任に分割
10func ValidateData(data []byte) error { ... }
11func ParseData(data []byte) (*Data, error) { ... }
12func TransformData(data *Data) *Result { ... }
13func SaveResult(result *Result) error { ... }
3. 重複削除
go
1// Before: 重複コード
2func AddUser(u User) { db.Insert("users", u) }
3func AddPost(p Post) { db.Insert("posts", p) }
4
5// After: 汎用化
6func Add[T any](table string, entity T) { db.Insert(table, entity) }
4. マジックナンバーの定数化
go
1// Before
2if len(items) > 100 { ... }
3
4// After
5const MaxItems = 100
6if len(items) > MaxItems { ... }
重要な原則
- ✅ 機能不変: 外部から見た動作は一切変更しない
- ✅ テストファースト: リファクタ前に必ずテストを実行
- ✅ 小さなステップ: 一度に多くを変更しない
- ✅ 継続的な検証: 各ステップ後にテスト実行
- ❌ 新機能追加: リファクタリングと機能追加は分離
リファクタリング対象の優先順位
- High: 複雑度が高く、バグを生みやすい箇所
- Medium: 頻繁に変更される箇所
- Low: 安定していて変更頻度が低い箇所
出力例
markdown
1# Refactoring Report
2
3## Target
4- File: `pkg/calc/sum.go`
5
6## Changes Applied
71. ✅ Improved function naming clarity
82. ✅ Added nil check for defensive programming
93. ✅ Simplified loop condition
10
11## Test Results
12✅ All tests pass (before and after)
13- Tests run: 5
14- Coverage: 85% (unchanged)
15
16## Performance Impact
17No performance regression detected
ベストプラクティス
- リファクタリング前後でベンチマークを取る（パフォーマンス改善の場合）
- コミットメッセージに「何を」「なぜ」を明記
- レビューしやすいように、変更を論理的に分割
