hamuhamu/README.md

## README.md

      
    Raw
  

              README.md
            
          
    TDD読書会vol2 資料

概要

「テスト駆動開発」読書会 Vol.2の資料です。

前回のTDD読書会vol1 資料
1章辺り、各自5分黙読。

その後、10分ディスカッションタイムをとった。
第5章 原則をあえて破るとき

Francクラスとテストを追加した

Francクラスがコピペになったとしても、テストを通すことを優先。
テスト実行結果
[hamuhamu@hamuhamu-no-MacBook-Air] % make test                                  
phpunit --configuration phpunit.xml test/
PHPUnit 6.3.0 by Sebastian Bergmann and contributors.

...                                                                 3 / 3 (100%)

Time: 157 ms, Memory: 10.00MB

OK (3 tests, 6 assertions)
第6章 テスト不足に気づいたら

DollarクラスをMoneyクラスから継承させた

DollarクラスのequalsメソッドをMoenyクラスに引き上げた

テストが十分にない又は全くないコードに対してTDDを行うことも今後あるだろう。

十分な量のテストがない場合、テストによって守られていないコードのリファクタリングを行わざるをえない。  
テストがないまま、リファクタリングを続けてしまうとコードを破壊してしまう。  
テストがない場合は、リファクタリングのためにテストコードを書く。
足りてなかったFrancのtestEqualityテストを追加した

FrancクラスをMoneyクラスから継承させた

第6章 ディスカッションタイム

リファクタリングの前にテストを書く。  

レガシーコード改善ガイド

試行リファクタリング

理解を深める
参考資料: レガシーコードの触り方
コード中にassertを埋め込む
第7章 疑念をテストに翻訳する

DollerとFrancを比較するテストを追加した

[hamuhamu@hamuhamu-no-MacBook-Air] % make test                                  
phpunit --configuration phpunit.xml test/
PHPUnit 6.3.0 by Sebastian Bergmann and contributors.

.F.                                                                 3 / 3 (100%)

Time: 233 ms, Memory: 10.00MB

There was 1 failure:

1) App\MoneyTest::testEquality
Failed asserting that true is false.

/Users/hamuhamu/Test_Driven_Development_By_Example/test/MoneyTest.php:29

FAILURES!
Tests: 3, Assertions: 9, Failures: 1.
make: *** [test] Error 1
不安に思ったことをテストにするのは大事。
FrancとDollarを比較できるようにした

[hamuhamu@hamuhamu-no-MacBook-Air] % make test                                  
phpunit --configuration phpunit.xml test/
PHPUnit 6.3.0 by Sebastian Bergmann and contributors.

...                                                                 3 / 3 (100%)

Time: 205 ms, Memory: 10.00MB

OK (3 tests, 9 assertions)

悩みをテストで表現
完璧ではないが、まずまずのやり方でテストを通す
さらなる設計は、本当に必要なときまで先延ばしにした

第7章 ディスカッションタイム

TDDをやっていてもバグを防げない。

バグの混入率は、TDDをやっていれば多少減る。
テストを書く人と、コードを書く人が同じなので、TDDのテストはバグがないことを保証するテストではない。
テストを後回しにしたい場合は　 @disabled を使用する。

PHPUnitだと、markIncomplete。

TODOは、もっと具体的に書いたほうがよい。(通貨の概念)　　
第8章 実装を隠す

DollarとFrancのtimesメソッドの戻り値の型をMoneyにした

MoneyクラスにDollarのファクトリメソッドを追加した

利用者のコード(テストコード)から、サブクラスのDollarを使っている箇所が消えた。
MoneyにFrancのファクトリメソッドを追加した

第8章 ディスカッションタイム

対応通貨が増えるほど、Moneyにファクトリが増えていってデカくなりそう。  
Moneyクラスの責務が大きい。
Moneyがファクトリメソッドを持っているので、DollarがFrancを生成できてしまう。  
それでいいのか？
Money::dollar(5)::franc(1);
親が子を知っている状態なので、よくなさそう。
ふりかえり、今後の進め方

・今回と同じスタイル同じペースで第３回も行う

・写経してきてコードを見せ合うといったことをしてもいい(希望者がいれば時間をとります)
次回のTDD読書会vol3 資料