golangreview-01jpcvxfxpsvpepv85myk6s6bh.md•1.78 kB
---
description: 掟プロジェクトにおけるGolangコードのレビュー基準とベストプラクティス
ruleId: golangreview-01jpcvxfxpsvpepv85myk6s6bh
tags: [development, golang, review]
aliases: [golang-review]
globs: ["**/*.go"]
---
# Golangコードレビュー
## レビュー時の注意点
- 不要なインタフェースの定義がないか。
- エラー処理が適切に行われているか。
- メモリ効率が考慮されているか。
- 並行処理が適切に実装されているか。
- パッケージ依存関係が適切か。
## コードレビューのチェックリスト
### 設計
- [ ] 単一責任の原則に従っているか。
- [ ] インタフェースは必要最小限か。
- [ ] パッケージ構成は適切か。
- [ ] 依存関係は明確か。
### 実装
- [ ] エラー処理は適切か。
- [ ] ゴルーチンのリークはないか。
- [ ] リソースは適切に解放されているか。
- [ ] 命名規則は一貫しているか。
- [ ] コメントは適切か。
### パフォーマンス
- [ ] 不要なメモリ割り当てはないか。
- [ ] アルゴリズムの選択は適切か。
- [ ] キャッシュは効果的に使用されているか。
- [ ] I/O処理は効率的か。
### テスト
- [ ] ユニットテストは十分か。
- [ ] エッジケースはテストされているか。
- [ ] テストのカバレッジは適切か。
- [ ] テストは読みやすく保守しやすいか。
## コードレビューのベストプラクティス
- 小さな変更を頻繁にレビューする。
- 具体的かつ建設的なフィードバックを提供する。
- コードの良い点も指摘する。
- 個人ではなくコードに焦点を当てる。