┏─────────────────┓
.やる夫で学ぶTDD 第9日目 スコア計算
┗─────────────────┛
／￣￣＼
／ _ノ ＼ もともと、
| （ ●）（●） ┏───────────────────┓
. | （__人__） bowling.bowl(4);
| ｀ ⌒´ﾉ assertEquals(1, bowling.getCurrentFrame());
. | } ... ┗───────────────────┛
. ヽ }
ヽ ノ ＼ という形で、現在のフレームだけ求めていたものに・・・
/ く ＼ ＼
| ＼ ＼ ＼
| |ヽ、二⌒)､ ＼
／￣￣＼
／ ノ ＼ ＼ ┏───────────────────┓
| （●）（●） | assertEquals(4, bowling.getFrameScore(1));
. | （__人__） | ..┗───────────────────┛
| ｀ ⌒´ ﾉ というようにフレームの番号を指定したものと
. | } ┏──────────────┓
. ヽ } assertEquals(4, bowling.socre());
ヽ ノ ＼ .┗──────────────┛
/ く ＼ ＼ 現在のスコアを取得するものを追加する
| ＼ ＼ ＼
| |ヽ、二⌒)､ ＼
＿＿＿
／）／ノ ' ヽ､＼ 最初の
／ .ｲ '（●） .（●）＼ ┏───────────────────┓
. /,'才.ﾐ). （__人__） ＼ assertEquals(4, bowling.getFrameScore(1)); は仕様にはないけど
. | ≧ｼ' ´ ⌒｀ | .┗───────────────────┛
. ＼ ヽ ／ getCurrentFrameと同じく、ヘルパーメソッドかお？
／￣￣＼
／ _ノ .ヽ、＼
| （●）（●） | そうだ。最初から、「 score 」だけpublicにしてテストするとちとしんどいので
. | （__人__） .|
| . '´ｎ｀' .ﾉ ヘルパーメソッドもpublicにして、テストしていくようにしただろ
.ヽ . | | }
ヽ.. ﾉ .ュノ 最終的にはpackage privateにしてもいいかな
/ { ..ﾆj､
|. | "ﾂ ＼
|. | .l |ヽ、二⌒)
／￣￣＼
／ _ノ ＼
| （ ●）（●） 同じようにして、通常の投球
| （__人__）
| ﾉ つまり、ストライクでもスペアでもないテストケースについて
| ∩ノ ⊃ }
/ヽ ／ ＿ノ } assertをこんな風に追加していく
( ヽ / / ノ
ヽ “ /＿| |
＼＿_/＿＿ /
＿＿＿_
／⌒ ⌒＼
／（ ●） （●）＼
／::::::⌒（__人__）⌒::::: ＼ もちろん、最初は固定値を返すところからやるんだお
| |r┬-| |
＼ `ー'´ (⌒) テストケースを付け足しながら、汎用的な実装に変えていくんだお！
＞ ﾉ ~.ﾚ-r┐､
/ ノ__ | .| | |
| 〈￣ ｀-Lλ_ﾚﾚ
| ￣｀ー┬--‐‐´
∩＿
〈〈〈 ヽ
＿＿＿_ 〈⊃ }
／⌒ ⌒＼ | | ストライクもスペアも考慮しなくていいなら、クラスScoreは簡単だお！
／（ ●） （●）＼ ! ! ┏──────────────┓
／ :::::⌒（__人__）⌒:::::＼| l private int[] pins = new int[21];
| |r┬-| | ／ ...┗──────────────┛
＼ ｀ ー'´ ／／
/ ＿＿ / 投球ごとのピンの数を格納する配列を用意してあげて
(＿＿＿） / これはMax21投球で固定にするお
(ヽ三/) ））
＿＿ ( i))) 現在のピンの位置を表す『.private int pinIndex = 0;.』を用意して
／⌒ ⌒＼ ＼
／（ ●） （●）＼ ） ┏─────────────────────┓
.／:::::: ⌒（__人__）⌒::::＼ ... public void add(int pin) { pins[pinIndex++] = pin; }
| (⌒)|r┬-| | ┗─────────────────────┛
,┌､-､!.~〈`ー´/ ＿/
| | | | __ヽ、 / コレを作って、投げる毎に『.Bowling.』が呼び出すようにするお！
ﾚレ'､ﾉ‐´ ￣〉 |
｀ｰ---‐一'￣
（（ (ヽ三/) (ヽ三/) ））
(((i )＿＿ ( i)))
／ ／⌒ ⌒＼ ＼
( ／（ ●） （●）＼ ） そして、『.public int getFrameScore(int frameIndex) 』で
／::::::⌒（__人__）⌒::::: ＼
| | 指定したフレームのスコアを求めるようにするお
＼ ／
『 for (int currentPin = 0, currentFrame = 1; currentFrame <= frameIndex; currentFrame++) { 』
ﾌｩｰﾅｶﾞｶｯﾀ ＿＿＿_
／ ＼
／ ─ ─＼
／ （●） （●） ＼ 上記のfor文でぐるぐるフレームごとにループを回すようにして、ループ内は
| （__人__） | ┏───────────────────────────────┓
＼ ｀ ⌒´ ,／ frameScore += pins[currentPin] + pins[currentPin + 1]; currentPin += 2;
/⌒ヽ ー‐ ｨヽ＿ ┗───────────────────────────────┛
/ ､ ､ 〆ヽﾞ￣ ｀i
/ / ﾞ lヾ_ ,|ヽ─ヘ とすればさっきのテストには全合格だお！
/ /l `､´/ ヽ
/ / ヽ ,＿＿__ ｀ ,ヽ ヽ
-─ヽ ｲ_ ヽ / ￣￣￣￣`ヽ Ｙ ゝ‐-､
( _i _i＿ﾄ､__) ヽ､＿＿_⊥＿＿＿＿ﾉ 「＿＿＿ﾉ
／￣￣＼
／ _ノ ＼
| （ ●）（●）
. | （__人__） そして、その後、ストライクとスペアのケースについても
| ｀ ⌒´ﾉ
.ｌ^l^lｎ } 『 score 』のassert文を追加していって、合格するような実装を
.ヽ L }
ゝ ﾉ ノ テストを追加しながら書いていくだろ
／ / ＼
／ / ＼
. / / -一'''''''ー-､.
人＿＿ﾉ (⌒＿(⌒)⌒)⌒))
＿＿＿_
／ ＼
／ ─ ─＼ これまた既に課題2でほとんど作っているから、ロジックを流用できるお
／ （●） （●） ＼
| （__人__） | ..ただし、課題2では、ストライクの場合、「 10、0 」のようにデータを与えていたけど
/ ∩ノ ⊃ ／/ ∩ノ ⊃
( ＼ ／ ＿ノ ＼／ ＿ノ 課題3では powl(10); powl(5);のように
.＼ “ ／ . ＼ “ ／
＼ ／ /＼／ ストライクの場合は明示的に0を入れないようにするから
＼ ＼
＼ ＼ ＼ そのデータ構造の違いには注意する必要があるお
> > >
／ ／ ／
＿＿＿_
／ ＼
／ ヽ、 _ノ ＼ 逆に、「 10、0 」と課題3のデータ構造で成る場合は
／ （●） （●） ＼ ｎ
| U （__人__） l^l.| | /） ストライクを取ったフレームの後のフレームで
＼ |i||||||i| ／| U ﾚ'/／)
γ⌒ ｀ ー'´ ﾉ ／ ガーターになってしまった時を意味するお！
i j ｒニ ﾉ
ヽ､ l !ヽ､_,__／
_＿＿_
／⌒ ⌒＼
／（ ●） （●）＼
／::::::⌒（__人__）⌒:::::＼
| |r┬-| | データ構造と仕様をシンプルにしたから
＼ `ー'´ ／
r´ / ＼ ）） 実装ももっとシンプルになってこんな感じになったお
| ﾊ |＼二つ
ヽ_つ | これまた全ケース合格だお！！
（（ ＼ /
／⌒, ／ ）
＿/ r ＿/ ﾉ
(＿＿.）(＿＿.）
／￣￣＼
／ _ノ ＼
| （ ●）（●） テストケースを見てもらえばわかる通り、現在のスコアについては
. | （__人__）
| ｀ ⌒´ﾉ 、『.10フレーム投げ終わった場合以外はその後連続してガーター(つまり0）が続く.』
.ｌ^l^lｎ }
.ヽ L } という最悪パターンでの値を返すようにしている
ゝ ﾉ ノ
／ / ＼ ここが実装がシンプルにできた理由だろ
／ / ＼
. / / -一'''''''ー-､.
人＿＿ﾉ (⌒＿(⌒)⌒)⌒))
＿＿＿
／ ⌒ ⌒＼
／ （⌒） （⌒）＼
／ ///（__人__）///＼
| u. `Y─┴──┴──┐
＼,,. | |
/⌒ヽ(＾う 完成だお！！...|
`ｧー─イ. |
/ |＿＿＿＿___________|
/ /
／ ⌒ヽ
___／ ／￣￣｀) ﾉ
(__r＿__ノ (.＿＿つ
／￣￣＼
／ _ノ ＼
| （ ●）（●） 待て！
. | （__人__）
| ｀ ⌒´ﾉ 境界値として危ない10フレーム目のテストケースについても
. | }
. ヽ } スコアに対するaasertを追加して、その上でテストケースが
ヽ ノ ﾋﾞｼｯ
/ く＿＿,-ｭ__ て 全合格したら、その時が無事実装成功といえるだろ・・・
| ＿＿＿ 三） （
| | ￣ ´
／￣￣＼
/ ヽ､_ ＼
. （ （● ） | 今回の実装では、特に10フレーム目を特殊扱いしなくても
. （人__） |
r-ヽ | 問題なく動作すると思うが、念のためにテストケースを拡張する
（三） | |
> ノ / 10フレーム目の考慮はBowlingクラスだけがするようにしているだろ
/ / ヽ /
/ / へ> <
|＿__ヽ ＼／ )
|＼ ／|
| ＼_／ |
○
┌─────────│
│ │
│ もちろん │
│ 全合格だおー！. │
│ │
└─────────│
／⌒ ⌒＼ │
／（ ●） （●）＼ │
／::::::⌒（__人__）⌒::::: ＼/⌒）
| |r┬-| | ﾉ
_＿＼ `ー'´ ／ /
（⌒ |_,,,ノ
""''''''ヽ_ |
| |
／￣￣＼
／ _ノ ヽ､.＼
| （●）（●） .| 実は境界値とエラー推測の観点からは
! （__人__） |
, っ ｀⌒´ | テストケースと実装の両方がまだ不足しているのだが
/ ミ） /
./ ノゝ / それはデブサミ当日にぜひ参加者の皆さんと作っていきたいんだな
i ﾚ'´ ヽ
| |／| | |
／￣￣＼
／ _ノ ＼
| （ ●）（●） 今回はテストケースの作成に直交表を使うべきは
. | u. （__人__）
| ｀ ⌒´ﾉ ちょっと迷うところではあったのだがボウリングの場合
. | }
. ヽ } どちらかというと10フレーム目の境界値が鬼門となるので
ヽ ノ
i⌒＼ ,__(‐- ､ それ以外の組み合わせは多少アドホックでも問題ないと考えて、使わないことにしただろ
l ＼ 巛ー─;＼
| ｀ヽ-‐ーく＿）
. | l
＿＿＿_
／─ ─＼
／ （●） (●）.＼ 確かに、直交表とかそういうのに抜きして
／ （__人__） ＼
| ｀ ⌒´ | エラーハンドリングはちと甘すぎるような気がするお
＼ ／
ノ ＼
. ／´ ヽ カ
| ｌ l||l 从人 l||l l||l 从人 l||l カ タ
ヽ -一''''''"~~｀`'ー--､ -一'''''''ー-､. タ
ヽ ＿＿＿＿(⌒)(⌒)⌒) ) (⌒＿(⌒)⌒)⌒))
┌ ┌┬┬┐┌┬┬┬┐┌┬┬┬┐┌┬┬┬┐
,. - ''"|￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣ρ￣`ｌ
￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣ノ￣￣
＿＿＿_
／ ＼
／ ＼
／ ─ ─ ＼ 今のままだと21球以上投げると例外で落ちちゃうし
| .（ー） （ー） |
＼ （__人__） ,／ 倒したピンの値を110 とか 4 7とかだと、変な値が計算されてしまうお・・・
ノ ｀ ⌒´ .＼
. ／´ ヽ
| ｌ ＼
ヽ -一''''''"~~｀`'ー--､ -一'''''''ー-､.
ヽ ＿＿＿＿(⌒)(⌒)⌒) ) (⌒＿(⌒)⌒)⌒))
┌ ┌┬┬┐┌┬┬┬┐┌┬┬┬┐┌┬┬┬┐
,. - ''"|￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣ρ￣`ｌ
￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣ノ￣￣
／￣￣＼
／ _ノ ＼ それはこのボウリングのスコア計算のAPIとしてどこまで何を提供させたいか？
｜ （ ●）（ ●）
| （__人__） ..逆にいうと利用側がどこまで何をやってほしいかに依存するからな
｜ ｀⌒ ´ｌ
｜ } まあ、今回の場合はやる夫とやらない夫が
ヽ }
ゝ 丿 APIのユーザーの代わりをしてそれでもいいと判断した事にするだろ
__＿＿（￣￣ /￣￣/´￣/＿
i⌒ゝ、 ヽ ヽ. / / ⊂) i⌒ゝ、
i;;;;;;;;;;|＿＿ Ｌ人 ￣￣｀´￣ ノ_＿i;;;;;;;;;;|
＿＿＿_
／ ＼
／ ─ ─＼
／ （●） （●） ＼ 必ずしもガチガチに暴挙すればいいってもんじゃないってことかお？
| （__人__） |
＼ ｀ ⌒´ ,／
ノ ー‐ ＼
／￣￣＼
／ ヽ､_ ＼
（●）（● ） | エラーハンドリングが多くなると本当にやりたかったコアのビジネスロジックが
（__人__） |
（ | エラーハンドリングに埋もれて分かりにくくなる可能性が高いからな
. { |
⊂ ヽ∩ く 金融関係のシステムなんて大体そんな感じだろ
| '､＿ ＼ ／ )
| |_＼ “ .／
ヽ､ ＿_＼_／
＿＿
|
|
|
|
| ＿＿＿__ 今回のは仕様自体をシンプルにしたから
| ／⌒ ⌒ ＼
| ／（ ●） （●） ＼ クラス『.Bowling.』、クラス『.Score.』とボウリングのスコア計算のロジックそのものを
| ／::::::⌒（__人__）⌒::::: ＼
＿＿| | |r┬-| | そのドメインの言葉で分かり易く記述できているお
ﾆ「￣ ＼ `ー'´ ／
￣￣| /⌒/ .λ／_,,..i_ これなら仕様書じゃなくて単体テストコードや実装コードで
| γ⌒r、/ く」r/( ､ ゝ)ヽ､_￣￣「|
|二ﾆ＝=─-､ム!_ﾉヽﾉｰ'イ』 ﾉi | | ロジックの正しさをユーザーがレビューできそうだお
￣￣￣￣￣￣￣￣￣￣￣￣￣￣￣| | |
／￣￣＼
／ _ノ ＼ Rubyとか使うともっと記述がシンプルになるから
| （ ●）（●
| （__人__） よりドメインの知識と言葉に近くなるな
. | ﾉ
| ∩ ノ ⊃ TDDでテストケースから始めると実装も本質的なビジネスロジックを
／ .／ ＿ノ
(. ＼ ／ .／_ノ │ 分かり易く実装する可能性が高いので、そういう意味でもお勧めだな
＼ “ ／_＿_| |
. ＼／ ＿＿＿ /
＿＿＿＿＿＿＿_
| ＿＿＿＿＿_ ／￣￣￣￣￣￣＼
| | ／ ⌒ ー､ :::::::::::U:＼
| | /（ ○）}liil{（ ○） ::::::::::::::|
| | .|U⌒（__人__） ⌒ ::::::U::::| 後は、課題４のスコア表を作れっていうのが残っているけど・・・
| | | |r┬-| U...:::::::::::::::::::/
| |＿＿＿＿ ヽ ｀⌒´.....:::::::::::::::::::::::<
└＿＿＿／￣￣ :::::::::::::::::::::::::|
|＼ | ::::::::::::::::::::::|
＼ ＼ ＼＿＿＿ ::::::
.r-､ ／￣￣＼
／て ) ／ _ノ ＼
（ _ﾉ ﾌ. | （ ●）（●） 俺、昔はひたすらJavaScriptを６年ほど書いていたのだが
ゝ、 〈 | （__人__）
/ ハ ヽ. | ｀ ⌒´ﾉ ..今はバックエンドとEDDばかりでとんとGUIに弱くなっているだろ
/〃 ﾍ ＼ . | }
ｉ ！ ＼ ｀ ーヽ } だからソコはデブサミの会場でそういうのが得意な技術者を見つけて
丶丶 _ ＞ ヽ ノ､
ゝ'´- ､_ y-､ ＼ 本番でペアプロTDDだろ 実演的に考えて・・・
〈 ￣ う /､ ヽ
`ー― ¬､__ノ | > / まあ、GUIはTDDし難いけどな
| r'＾ヽ'´ _／
| `く__ノ´
＿＿＿
／ ＼
／ _ノ '' 'ー ＼
／ （●） （●） ＼ じゃー、今日でやらない夫とのペアプロTDDは終わりかお？
| （__人__） |
＼ ｀ ⌒´ ／
／￣￣￣＼
／ :::::::::.ヽ
/ :::::::::::::::::::: 〉
| ::::::::::::::::::/
| ::::::::::::::::::::l この９日で学んだのはTDDもテスト技法もほんの入門
| ::::::::::::::::::/
＼__ :::::::::く どちらとも非常に奥が深く、長時間の実践が不可欠だ
. ,ｲ :::::::::ヽ
/ ｀ :::::::::::} これからもTDDとテスト技法の腕を磨いてほしいだろ
,' :::j .:::::::::/
,−´ :::ﾉ .:::::::::{ スーパープログラマー的に考えて・・・
(,. - ―.i_ ::::::::::i
/ ::::::::::: }
. l,. ----ゝ.ー―一´ー--.._
_ ＿｣. ｀l￣￣￣￣￣￣￣!
{._ | | 厂￣￣ｌ￣￣| |
. ｀¨ﾞl__＿＿｣__j ��..___|__j
＿＿＿
／ ⌒ ⌒＼
／ （⌒） （⌒）＼ 鬼畜王ランス14のチームでも阿部さんが主導で
／ ///（__人__）///＼
| u. `Y⌒y'´ | みんなTDDできるところから始めるようなったお
＼ ﾞー ′ ,／
/⌒ヽ ー‐ ｨヽ 今まではスーパープログラマーの阿部さんに頼りっきりだったけど
/ rｰ'ゝ 〆ヽ
/,ﾉヾ ,> ヾ_ノ,| 阿部さんとペアプロTDDすることよってモリモリスキルアップしてきたお！
| ヽ〆 |´ |
／ ） ／￣￣＼
l / ／ ヽ､_ ＼
ｌ l （ ●）（●） | とまあ、先は長いわけだが毎日コツコツと
r―j l , -┐ （__人__） |
〈 l 〔 )' ／ （｀⌒ ´ | ..TDDの実践とテスト技術の実践をがんばってほしい
/ヽﾆｲ 〈 . { | 〈⌒ヽ
｀ T′! / { ﾉ | l では、また、何かやる夫がスーパープログラマーになるために
＼ イ ヽ ノ ＼ j ,
/ , ＿＿＿＿/ヽ' / / r― ､ / / 必要だと思った技術があったら一緒に学ぼうな？
/ l /ﾍ l ム l/ ＿/ / ／ヽ 〉' /ｰ､
l ヽ￣￣ | | // _ ＞ ヽ| / ｜ /、 / ﾄ､
| l ヽヽ /l/／/￣ l | ｲ ／/ ＼ / / /-､
ヽ _ -―┬ﾉﾉ / l ＼／ l( l / / l/ ／ ﾊ 〉
￣￣７ / ／￣〈 ＼ /ｰ{__∠-' /
/ / / ＼＿,＞-､＿ノｰ一'
＿＿＿
／ ＼ やらない夫、ほんとうにありがとうだお
／ ＼
／ ⌒ ⌒ ＼ この頃、阿部さんは関数型言語とか何とかよく言っているので
| /// （__人__） /// |
. (⌒) (⌒) やる夫もそのうちチャレンジしてみようとも思うお
./ i＼ ／i ヽ
＼ ／ ＼ ／
Ｘ ／￣￣￣＼ Ｘ
／ ∩ ／ _ノ ヽ､ ＼ ／ ∩
／ （ ⊂) | （ ●）（● ） | ／ （ ⊂) ＿＿＿_
| | | （__人__） | | | ／⌒ ⌒＼
ﾄニｨ'| ｀⌒´ | ﾄニｨ'. ／( ●） (● ）. ＼
| | } | ／ :::⌒（__人__）⌒::::: ＼
＼ ヽ } ＼ | |r┬-| |
＼ ヽ ノ ＼＼＿ `ー'´ ＿／
ン ゝ ''''''/＞ー､_ ン ゝ ''''''/＞ー､_ ９日間ありがとう（だお！）
/ イ（ ／ / ＼ / イ（ ／ / ＼
/ | Ｙ | / 入 ＼ / | Ｙ | / 入 ＼
（ | ：､ | / / ヽ、 l （ | ：､ | / / ヽ、 l
j | ： | / ｨ | | j | ： | / ｨ | |
くV ヽﾍ_ ヽ ＼ 仁 」 くV ヽﾍ_ ヽ ＼ 仁 」
ー 〕 ＼ 〉（⌒ノ ー 〕 ＼ 〉（⌒ノ
