@k2works/claude-code-booster 0.2.1 → 0.4.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -0,0 +1,704 @@
1
+ # コーディングとテストガイド
2
+
3
+ ## 概要
4
+
5
+ このガイドは、開発ガイドのコーディングとテスト部分を詳細化したドキュメントです。TDD(テスト駆動開発)を中核とした実装プロセスの具体的な手順と、品質を維持するための規律を定義します。
6
+
7
+ ## 基本原則
8
+
9
+ ### TDD(Test-Driven Development)の三つの原則
10
+
11
+ 1. **失敗するテストを書くまで、プロダクションコードを書いてはならない**
12
+ 2. **失敗するテストは、失敗するのに十分なだけ書く**
13
+ 3. **現在失敗しているテストを成功させる以上のプロダクションコードを書いてはならない**
14
+
15
+ ### Red-Green-Refactor サイクル
16
+
17
+ ```plantuml
18
+ @startuml
19
+ skinparam state {
20
+ BackgroundColor Red
21
+ BackgroundColor LightGreen
22
+ BackgroundColor LightBlue
23
+ }
24
+
25
+ [*] --> Red
26
+ state Red {
27
+ [*] --> テストを書く
28
+ テストを書く : 失敗する
29
+ }
30
+ Red --> Green : 最小限の実装
31
+
32
+ state Green {
33
+ [*] --> テストが通る
34
+ テストが通る : 最小限のコード
35
+ }
36
+ Green --> Refactor : テスト成功
37
+
38
+ state Refactor {
39
+ [*] --> リファクタリング
40
+ リファクタリング : 設計改善
41
+ }
42
+ Refactor --> Red : 次のテスト
43
+ Refactor --> [*] : 完了
44
+
45
+ note right of Red
46
+ 失敗するテストを
47
+ 最初に書く
48
+ end note
49
+
50
+ note right of Green
51
+ テストを通す
52
+ 最小限のコードを書く
53
+ end note
54
+
55
+ note right of Refactor
56
+ 重複を除去し
57
+ 設計を改善する
58
+ end note
59
+
60
+ @enduml
61
+ ```
62
+
63
+ ## イテレーション開発フロー
64
+
65
+ ### 全体プロセス
66
+
67
+ ```plantuml
68
+ @startuml "イテレーション開発プロセス"
69
+
70
+ start
71
+
72
+ partition "イテレーション開始" {
73
+ }
74
+
75
+ repeat :TODO確認;
76
+ partition "TDD実装サイクル" {
77
+ repeat
78
+ :TODO選択;
79
+
80
+ repeat
81
+ :失敗テスト作成 (Red);
82
+ :最小実装 (Green);
83
+ :リファクタリング (Refactor);
84
+ :品質チェック;
85
+ if (品質OK?) then (yes)
86
+ :コミット;
87
+ else (no)
88
+ :修正;
89
+ endif
90
+ repeat while (TODO完了?)
91
+ partition "コードレビュー" {
92
+ }
93
+ repeat while (全TODO完了?)
94
+ }
95
+
96
+ if (イテレーション完了?) then (yes)
97
+ partition "受け入れ" {
98
+ partition "ユーザーレビュー" {
99
+ }
100
+ if (受け入れOK?) then (yes)
101
+ partition "ふりかえり" {
102
+ }
103
+ else (no)
104
+ partition "修正対応" {
105
+ }
106
+ endif
107
+ }
108
+ else (no)
109
+ partition "設計リファクタリング" {
110
+ partition "アーキテクチャリファクタリング" {
111
+ }
112
+ partition "データモデルリファクタリング" {
113
+ }
114
+ partition "ドメインモデルリファクタリング" {
115
+ }
116
+ partition "UIリファクタリング" {
117
+ }
118
+ }
119
+ endif
120
+ repeat while (次のTODO?)
121
+
122
+ stop
123
+
124
+ @enduml
125
+ ```
126
+
127
+ ## アプローチ戦略
128
+
129
+ ### インサイドアウト vs アウトサイドイン
130
+
131
+ ```plantuml
132
+ @startuml
133
+ title 実装アプローチ選択フロー
134
+
135
+ start
136
+
137
+ :ユーザーストーリー分析;
138
+
139
+ if (基本CRUD実装済み?) then (はい)
140
+ if (ドメインロジックが複雑?) then (はい)
141
+ :ドメインモデル中心;
142
+ #lightgreen:アウトサイドイン推奨;
143
+ note right
144
+ UIからスタート
145
+ ドメインロジックを
146
+ 段階的に実装
147
+ end note
148
+ else (いいえ)
149
+ :シンプルな機能追加;
150
+ #lightblue:どちらでも可;
151
+ endif
152
+ else (いいえ)
153
+ :貧血ドメインモデル;
154
+ #yellow:インサイドアウト推奨;
155
+ note right
156
+ データ層から開始
157
+ 基盤を固めて
158
+ 上位層へ展開
159
+ end note
160
+ endif
161
+
162
+ if (API実装済み?) then (はい)
163
+ #lightblue:インサイドアウト活用;
164
+ note right
165
+ 既存APIを利用し
166
+ 内部実装を改善
167
+ end note
168
+ else (いいえ)
169
+ #lightgreen:アウトサイドイン活用;
170
+ note right
171
+ UIのニーズから
172
+ API設計を導出
173
+ end note
174
+ endif
175
+
176
+ :アプローチ決定;
177
+
178
+ stop
179
+ @enduml
180
+ ```
181
+
182
+ ### インサイドアウトアプローチ
183
+
184
+ ```plantuml
185
+ @startuml
186
+ title インサイドアウトアプローチ
187
+
188
+ participant "テスト" as test
189
+ participant "データベース層" as db
190
+ participant "インフラ層" as infra
191
+ participant "ドメイン層" as domain
192
+ participant "サービス層" as service
193
+ participant "プレゼンテーション層" as ui
194
+
195
+ == Phase 1: データベース ==
196
+ test -> db: テーブル定義テスト
197
+ activate db
198
+ db --> test: スキーマ検証
199
+ deactivate db
200
+
201
+ == Phase 2: インフラストラクチャ ==
202
+ test -> infra: リポジトリテスト
203
+ activate infra
204
+ infra -> db: CRUD操作
205
+ db --> infra: 結果
206
+ infra --> test: 永続化確認
207
+ deactivate infra
208
+
209
+ == Phase 3: ドメイン ==
210
+ test -> domain: ビジネスロジックテスト
211
+ activate domain
212
+ domain -> infra: データ取得
213
+ infra --> domain: エンティティ
214
+ domain --> test: ビジネスルール検証
215
+ deactivate domain
216
+
217
+ == Phase 4: サービス ==
218
+ test -> service: ユースケーステスト
219
+ activate service
220
+ service -> domain: ドメイン操作
221
+ domain --> service: 結果
222
+ service --> test: トランザクション確認
223
+ deactivate service
224
+
225
+ == Phase 5: プレゼンテーション ==
226
+ test -> ui: UIテスト
227
+ activate ui
228
+ ui -> service: API呼び出し
229
+ service --> ui: レスポンス
230
+ ui --> test: 画面表示確認
231
+ deactivate ui
232
+
233
+ @enduml
234
+ ```
235
+
236
+ ### アウトサイドインアプローチ
237
+
238
+ ```plantuml
239
+ @startuml
240
+ title アウトサイドインアプローチ
241
+
242
+ participant "受け入れテスト" as at
243
+ participant "プレゼンテーション層" as ui
244
+ participant "サービス層" as service
245
+ participant "ドメイン層" as domain
246
+ participant "インフラ層" as infra
247
+ participant "データベース層" as db
248
+
249
+ == Phase 1: 受け入れテスト ==
250
+ at -> ui: ユーザーシナリオ
251
+ activate at
252
+ ui --> at: モック応答
253
+ deactivate at
254
+
255
+ == Phase 2: UI実装 ==
256
+ at -> ui: UIテスト
257
+ activate ui
258
+ ui -> service: サービス呼び出し(モック)
259
+ service --> ui: モックレスポンス
260
+ ui --> at: 画面動作確認
261
+ deactivate ui
262
+
263
+ == Phase 3: サービス実装 ==
264
+ at -> service: 統合テスト
265
+ activate service
266
+ service -> domain: ドメイン呼び出し(モック)
267
+ domain --> service: モック結果
268
+ service --> at: ビジネスロジック確認
269
+ deactivate service
270
+
271
+ == Phase 4: ドメイン実装 ==
272
+ at -> domain: ドメインテスト
273
+ activate domain
274
+ domain -> infra: リポジトリ呼び出し(モック)
275
+ infra --> domain: モックデータ
276
+ domain --> at: ビジネスルール確認
277
+ deactivate domain
278
+
279
+ == Phase 5: インフラ実装 ==
280
+ at -> infra: 永続化テスト
281
+ activate infra
282
+ infra -> db: DB操作
283
+ db --> infra: 実データ
284
+ infra --> at: データ永続化確認
285
+ deactivate infra
286
+
287
+ @enduml
288
+ ```
289
+
290
+ ## TDD実装の詳細手順
291
+
292
+ ### 1. TODO作成
293
+
294
+ ```markdown
295
+ ## TODOリストの例
296
+
297
+ - [ ] ユーザー登録機能
298
+ - [ ] Userエンティティの作成
299
+ - [ ] 必須フィールドの検証
300
+ - [ ] メールアドレスの形式検証
301
+ - [ ] パスワードの強度チェック
302
+ - [ ] UserRepositoryの実装
303
+ - [ ] save()メソッド
304
+ - [ ] findByEmail()メソッド
305
+ - [ ] existsByEmail()メソッド
306
+ - [ ] UserServiceの実装
307
+ - [ ] register()メソッド
308
+ - [ ] 重複チェック
309
+ - [ ] パスワードハッシュ化
310
+ - [ ] UserControllerの実装
311
+ - [ ] POST /users エンドポイント
312
+ - [ ] バリデーション
313
+ - [ ] エラーハンドリング
314
+ ```
315
+
316
+ ### 2. Red フェーズ(失敗するテストを書く)
317
+
318
+ ```java
319
+ // UserTest.java
320
+ @Test
321
+ void ユーザー作成時に必須フィールドが検証される() {
322
+ // Given
323
+ String email = "test@example.com";
324
+ String password = "SecurePass123!";
325
+ String name = "山田太郎";
326
+
327
+ // When
328
+ User user = new User(email, password, name);
329
+
330
+ // Then
331
+ assertThat(user.getEmail()).isEqualTo(email);
332
+ assertThat(user.getName()).isEqualTo(name);
333
+ // パスワードはハッシュ化されている
334
+ assertThat(user.getPassword()).isNotEqualTo(password);
335
+ }
336
+
337
+ @Test
338
+ void 無効なメールアドレスで例外が発生する() {
339
+ // Given
340
+ String invalidEmail = "invalid-email";
341
+
342
+ // When/Then
343
+ assertThrows(IllegalArgumentException.class, () -> {
344
+ new User(invalidEmail, "password", "name");
345
+ });
346
+ }
347
+ ```
348
+
349
+ ### 3. Green フェーズ(最小限の実装)
350
+
351
+ ```java
352
+ // User.java
353
+ public class User {
354
+ private String email;
355
+ private String password;
356
+ private String name;
357
+
358
+ public User(String email, String password, String name) {
359
+ if (!isValidEmail(email)) {
360
+ throw new IllegalArgumentException("無効なメールアドレス");
361
+ }
362
+ this.email = email;
363
+ this.password = hashPassword(password);
364
+ this.name = name;
365
+ }
366
+
367
+ private boolean isValidEmail(String email) {
368
+ return email != null && email.contains("@");
369
+ }
370
+
371
+ private String hashPassword(String password) {
372
+ // 簡単なハッシュ化の実装(後で改善)
373
+ return "hashed_" + password;
374
+ }
375
+
376
+ // getters...
377
+ }
378
+ ```
379
+
380
+ ### 4. Refactor フェーズ(設計改善)
381
+
382
+ ```java
383
+ // User.java (リファクタリング後)
384
+ public class User {
385
+ private final Email email;
386
+ private final Password password;
387
+ private final Name name;
388
+
389
+ public User(String email, String password, String name) {
390
+ this.email = new Email(email);
391
+ this.password = new Password(password);
392
+ this.name = new Name(name);
393
+ }
394
+
395
+ // getters...
396
+ }
397
+
398
+ // Email.java (値オブジェクト)
399
+ public class Email {
400
+ private static final Pattern VALID_EMAIL_PATTERN =
401
+ Pattern.compile("^[A-Za-z0-9+_.-]+@[A-Za-z0-9.-]+\\.[A-Za-z]{2,}$");
402
+
403
+ private final String value;
404
+
405
+ public Email(String value) {
406
+ if (value == null || !VALID_EMAIL_PATTERN.matcher(value).matches()) {
407
+ throw new IllegalArgumentException("無効なメールアドレス: " + value);
408
+ }
409
+ this.value = value;
410
+ }
411
+
412
+ public String getValue() {
413
+ return value;
414
+ }
415
+ }
416
+
417
+ // Password.java (値オブジェクト)
418
+ public class Password {
419
+ private static final int MIN_LENGTH = 8;
420
+ private final String hashedValue;
421
+
422
+ public Password(String rawPassword) {
423
+ validate(rawPassword);
424
+ this.hashedValue = BCrypt.hashpw(rawPassword, BCrypt.gensalt());
425
+ }
426
+
427
+ private void validate(String password) {
428
+ if (password == null || password.length() < MIN_LENGTH) {
429
+ throw new IllegalArgumentException(
430
+ "パスワードは" + MIN_LENGTH + "文字以上必要です"
431
+ );
432
+ }
433
+ }
434
+
435
+ public boolean matches(String rawPassword) {
436
+ return BCrypt.checkpw(rawPassword, hashedValue);
437
+ }
438
+ }
439
+ ```
440
+
441
+ ## 品質チェックリスト
442
+
443
+ ### コミット前の必須確認事項
444
+
445
+ ```bash
446
+ # 1. テスト実行
447
+ ./gradlew test # または npm test
448
+
449
+ # 2. コードフォーマット
450
+ ./gradlew spotlessApply # または npm run format
451
+
452
+ # 3. 静的解析
453
+ ./gradlew check # または npm run lint
454
+
455
+ # 4. ビルド確認
456
+ ./gradlew build # または npm run build
457
+
458
+ # 5. カバレッジ確認
459
+ ./gradlew jacocoTestReport # または npm run test:coverage
460
+ ```
461
+
462
+ ### 品質基準
463
+
464
+ | 項目 | 基準 | 必須/推奨 |
465
+ |------|------|-----------|
466
+ | テストカバレッジ | 80%以上 | 必須 |
467
+ | 循環的複雑度 | 10以下 | 必須 |
468
+ | メソッドの行数 | 20行以下 | 推奨 |
469
+ | クラスの行数 | 200行以下 | 推奨 |
470
+ | 重複コード | 0% | 必須 |
471
+ | コンパイラ警告 | 0個 | 必須 |
472
+ | Linter警告 | 0個 | 必須 |
473
+
474
+ ## コミット規約
475
+
476
+ ### コミットメッセージフォーマット
477
+
478
+ ```
479
+ <type>(<scope>): <subject>
480
+
481
+ <body>
482
+
483
+ <footer>
484
+ ```
485
+
486
+ ### タイプ一覧
487
+
488
+ | タイプ | 説明 | 例 |
489
+ |--------|------|-----|
490
+ | feat | 新機能追加 | `feat(auth): ユーザー認証機能を追加` |
491
+ | fix | バグ修正 | `fix(api): NullPointerExceptionを修正` |
492
+ | docs | ドキュメント変更 | `docs(readme): インストール手順を更新` |
493
+ | style | コードスタイル変更 | `style: インデントを修正` |
494
+ | refactor | リファクタリング | `refactor(user): Userクラスを値オブジェクトに分割` |
495
+ | test | テスト追加・修正 | `test(user): ユーザー登録のテストを追加` |
496
+ | chore | ビルド・ツール変更 | `chore(gradle): 依存関係を更新` |
497
+
498
+ ### コミット単位
499
+
500
+ - **1コミット = 1論理的変更**
501
+ - TODOリストの項目単位でコミット
502
+ - Red-Green-Refactorサイクル完了時にコミット
503
+ - ビルドが通る状態でコミット
504
+
505
+ ## テストの種類と戦略
506
+
507
+ ### テストピラミッド
508
+
509
+ ```plantuml
510
+ @startuml
511
+ skinparam defaultTextAlignment center
512
+
513
+ rectangle "E2Eテスト\n(10%)" as e2e #LightCoral
514
+
515
+ rectangle "統合テスト\n(30%)" as integration #LightSalmon
516
+
517
+ rectangle "単体テスト\n(60%)" as unit #LightGreen
518
+
519
+ e2e -[hidden]down-> integration
520
+ integration -[hidden]down-> unit
521
+
522
+ note right of e2e
523
+ ユーザーシナリオ
524
+ システム全体の動作確認
525
+ 実行時間: 遅い
526
+ end note
527
+
528
+ note right of integration
529
+ モジュール間連携
530
+ API・DB接続テスト
531
+ 実行時間: 中程度
532
+ end note
533
+
534
+ note right of unit
535
+ 個別機能のテスト
536
+ ビジネスロジック検証
537
+ 実行時間: 高速
538
+ end note
539
+
540
+ @enduml
541
+ ```
542
+
543
+ ### テスト作成のベストプラクティス
544
+
545
+ #### 1. AAA パターン(Arrange-Act-Assert)
546
+
547
+ ```java
548
+ @Test
549
+ void 商品の在庫が減少する() {
550
+ // Arrange(準備)
551
+ Product product = new Product("商品A", 10);
552
+ int orderQuantity = 3;
553
+
554
+ // Act(実行)
555
+ product.reduceStock(orderQuantity);
556
+
557
+ // Assert(検証)
558
+ assertThat(product.getStock()).isEqualTo(7);
559
+ }
560
+ ```
561
+
562
+ #### 2. テストの命名規則
563
+
564
+ ```java
565
+ // パターン1: 日本語での説明的な名前
566
+ @Test
567
+ void 在庫が不足している場合は注文できない() { }
568
+
569
+ // パターン2: Given-When-Then形式
570
+ @Test
571
+ void given在庫10個_when15個注文_then在庫不足例外() { }
572
+
573
+ // パターン3: メソッド名_条件_期待結果
574
+ @Test
575
+ void reduceStock_在庫不足_IllegalStateException() { }
576
+ ```
577
+
578
+ #### 3. テストデータの準備
579
+
580
+ ```java
581
+ // テストフィクスチャの使用
582
+ public class UserTestFixture {
583
+ public static User createDefaultUser() {
584
+ return new User(
585
+ "test@example.com",
586
+ "password123",
587
+ "テストユーザー"
588
+ );
589
+ }
590
+
591
+ public static User createUserWithEmail(String email) {
592
+ return new User(
593
+ email,
594
+ "password123",
595
+ "テストユーザー"
596
+ );
597
+ }
598
+ }
599
+ ```
600
+
601
+ ## リファクタリングパターン
602
+
603
+ ### よく使うリファクタリング手法
604
+
605
+ | パターン | 適用場面 | 例 |
606
+ |----------|----------|-----|
607
+ | メソッド抽出 | 長いメソッドの分割 | 検証ロジックを別メソッドへ |
608
+ | 変数抽出 | 複雑な式の簡略化 | 計算結果を一時変数へ |
609
+ | クラス抽出 | 責務の分離 | 値オブジェクトの抽出 |
610
+ | メソッド移動 | 適切なクラスへの配置 | ユーティリティメソッドの移動 |
611
+ | 条件記述の分解 | 複雑な条件式の整理 | if文の条件をメソッド化 |
612
+ | ループの分割 | 単一責任の原則適用 | 異なる処理を別ループへ |
613
+
614
+ ### リファクタリング実施のタイミング
615
+
616
+ 1. **Rule of Three(3回ルール)**
617
+ - 1回目: そのまま実装
618
+ - 2回目: 重複に気づくが我慢
619
+ - 3回目: リファクタリング実施
620
+
621
+ 2. **コードの臭い(Code Smells)を検知したとき**
622
+ - 長すぎるメソッド(20行以上)
623
+ - 大きすぎるクラス(200行以上)
624
+ - 長すぎるパラメータリスト(4個以上)
625
+ - データの群れ(同じ引数の組み合わせ)
626
+ - スイッチ文の重複
627
+
628
+ ## 継続的な改善
629
+
630
+ ### ふりかえりでの確認項目
631
+
632
+ ```markdown
633
+ ## イテレーションふりかえりテンプレート
634
+
635
+ ### Keep(継続すること)
636
+ - [ ] TDDサイクルの実践
637
+ - [ ] コードレビューの実施
638
+ - [ ] 品質基準の維持
639
+
640
+ ### Problem(問題点)
641
+ - [ ] テスト作成に時間がかかった箇所
642
+ - [ ] リファクタリングが不十分な箇所
643
+ - [ ] 技術的負債の発生箇所
644
+
645
+ ### Try(次に試すこと)
646
+ - [ ] 新しいテスト手法の導入
647
+ - [ ] リファクタリングの自動化
648
+ - [ ] 品質メトリクスの改善
649
+
650
+ ### 数値指標
651
+ - テストカバレッジ: ___%
652
+ - ビルド成功率: ___%
653
+ - バグ発生率: ___件/イテレーション
654
+ - 平均サイクルタイム: ___時間
655
+ ```
656
+
657
+ ### 技術的負債の管理
658
+
659
+ ```plantuml
660
+ @startuml
661
+ title 技術的負債の対処フロー
662
+
663
+ start
664
+
665
+ :技術的負債の識別;
666
+ note right
667
+ - コードレビューで発見
668
+ - 静的解析ツールの警告
669
+ - パフォーマンス問題
670
+ end note
671
+
672
+ if (緊急度は?) then (高)
673
+ :即座に対処;
674
+ #red:ホットフィックス;
675
+ elseif (影響範囲は?) then (大)
676
+ :次イテレーションで対処;
677
+ #yellow:計画的リファクタリング;
678
+ else (小)
679
+ :バックログに追加;
680
+ #lightgreen:継続的改善;
681
+ endif
682
+
683
+ :対処結果の記録;
684
+ note right
685
+ - 対処内容
686
+ - 所要時間
687
+ - 学習事項
688
+ end note
689
+
690
+ stop
691
+
692
+ @enduml
693
+ ```
694
+
695
+ ## まとめ
696
+
697
+ コーディングとテストは、よいソフトウェアを作るための最も基本的で重要な活動です。TDDを中心としたこれらの規律を守ることで:
698
+
699
+ 1. **品質の作り込み** - バグの早期発見と修正
700
+ 2. **設計の改善** - 継続的なリファクタリング
701
+ 3. **変更容易性** - テストによる安全網
702
+ 4. **ドキュメント性** - テストが仕様書として機能
703
+
704
+ これらの実践により、「変更を楽に安全にできて役に立つソフトウェア」の実現を目指します。