@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.
- package/lib/assets/.claude/README.md +2 -0
- package/lib/assets/.claude/commands/dev.md +105 -0
- package/lib/assets/.claude/commands/ops.md +508 -0
- package/lib/assets/.claude/commands/plan.md +7 -2
- package/lib/assets/docs/reference/Java/343/202/242/343/203/227/343/203/252/343/202/261/343/203/274/343/202/267/343/203/247/343/203/263/347/222/260/345/242/203/346/247/213/347/257/211/343/202/254/343/202/244/343/203/211.md +570 -0
- package/lib/assets/docs/reference/TypeScript/343/202/242/343/203/227/343/203/252/343/202/261/343/203/274/343/202/267/343/203/247/343/203/263/347/222/260/345/242/203/346/247/213/347/257/211/343/202/254/343/202/244/343/203/211.md +465 -0
- package/lib/assets/docs/reference//343/202/263/343/203/274/343/203/207/343/202/243/343/203/263/343/202/260/343/201/250/343/203/206/343/202/271/343/203/210/343/202/254/343/202/244/343/203/211.md +704 -0
- package/lib/assets/docs/reference//351/226/213/347/231/272/343/202/254/343/202/244/343/203/211.md +39 -205
- package/lib/assets/docs/template//345/256/214/345/205/250/345/275/242/345/274/217/343/201/256/343/203/246/343/203/274/343/202/271/343/202/261/343/203/274/343/202/271.md +5 -0
- package/lib/assets/scripts/journal.js +17 -7
- package/package.json +1 -1
|
@@ -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
|
+
これらの実践により、「変更を楽に安全にできて役に立つソフトウェア」の実現を目指します。
|