79 lines
2.2 KiB
Markdown
79 lines
2.2 KiB
Markdown
# コードレビュー観点 v1.0.11+21
|
||
|
||
## 1. Phase D6設計整合性 ⭐⭐⭐⭐⭐
|
||
|
||
### 1-1. filteredByModeProvider経由の確認
|
||
- [ ] `rawSakeListItemsProvider`が不適切に使用されていない
|
||
- [ ] 例外: 空状態判定で`filteredByModeProvider`を使用
|
||
|
||
### 1-2. Personal/Business Mode判定
|
||
- [ ] `userProfile.isBusinessMode`で判定
|
||
- [ ] または`.select((p) => p.isBusinessMode)`でパフォーマンス最適化
|
||
|
||
---
|
||
|
||
## 2. 非同期処理の安全性 ⭐⭐⭐⭐⭐
|
||
|
||
### 2-1. BuildContextキャプチャパターン
|
||
- [ ] async gap前に`final navigator = Navigator.of(context);`
|
||
- [ ] async gap前に`final messenger = ScaffoldMessenger.of(context);`
|
||
- [ ] async gap後は`navigator.pop()`を使用
|
||
|
||
### 2-2. mountedチェック
|
||
- [ ] すべてのasync gap後に`if (!mounted) return;`
|
||
|
||
### 2-3. Linter警告
|
||
- [ ] `use_build_context_synchronously`警告がゼロ
|
||
|
||
---
|
||
|
||
## 3. パフォーマンス ⭐⭐⭐⭐
|
||
|
||
### 3-1. Provider最適化
|
||
- [ ] `.select()`による部分監視
|
||
- [ ] 不要な`ref.watch()`がない
|
||
|
||
### 3-2. リスト最適化
|
||
- [ ] `ListView.builder`使用（大量データ対応）
|
||
- [ ] 画像キャッシュが適切
|
||
|
||
---
|
||
|
||
## 4. 保守性 ⭐⭐⭐⭐
|
||
|
||
### 4-1. マジックナンバー
|
||
- [ ] `AppConstants`クラスに定義済み
|
||
- [ ] ハードコードされた数値がない
|
||
|
||
### 4-2. コメント
|
||
- [ ] 意図的な設計判断にコメント
|
||
- [ ] 複雑なロジックに説明コメント
|
||
|
||
### 4-3. ドキュメント
|
||
- [ ] CURSOR_HANDOFF_FINAL_v1.0.10+20.md参照
|
||
- [ ] 将来的検討事項が文書化
|
||
|
||
---
|
||
|
||
## 5. 意図的なハードコーディング（保護） ⭐⭐⭐⭐⭐
|
||
|
||
参照: `SAFE_FIXES_REPORT.md`
|
||
|
||
### 5-1. 変更してはいけない箇所
|
||
- [ ] `lib/widgets/home/sake_grid_item.dart`:102 `Colors.black54`
|
||
- [ ] `lib/screens/home_screen.dart`:86-88 `Colors.white`
|
||
- [ ] SpeedDialオーバーレイ
|
||
|
||
### 5-2. 変更理由の確認
|
||
- [ ] 視認性確保のため（画像オーバーレイ）
|
||
- [ ] AppBar常にダーク背景のため（検索フィールド）
|
||
|
||
---
|
||
|
||
## レビュー実施記録
|
||
|
||
**レビュアー**: ___________
|
||
**レビュー日時**: ___________
|
||
**結果**: Approved / Changes Requested
|
||
**コメント**: ___________
|