Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

第11章 不要になったら消す #15

Merged
merged 9 commits into from
Nov 8, 2017
Merged

第11章 不要になったら消す #15

merged 9 commits into from
Nov 8, 2017

Conversation

at-grandpa
Copy link
Owner

@at-grandpa at-grandpa commented Nov 8, 2017

TODO

  • $5 + 10 CHF = $10(レートが2:1に場合)
  • $5 * 2 = $10
  • amountをprivateにする
  • Dollarの副作用どうする?
  • Moneyの丸め処理をどうする?
  • equals
  • hashCode
  • nullとの等価性比較
  • 他のオブジェクトとの等価性比較
  • 5CHF*2=10CHF
  • DollarとFrancの重複
  • equalsの一般化
  • timesの一般化
  • FrancとDollarを比較する
  • 通貨の概念
  • testFrancMultiplicationを削除する?

@at-grandpa
Copy link
Owner Author

不要なところをどんどん消そう。

@at-grandpa
Copy link
Owner Author

テスト通る。

@at-grandpa
Copy link
Owner Author

Francはまだテスト側で参照しているので消せない。

@at-grandpa
Copy link
Owner Author

等価性比較のテストは十分になされているので、良さそう。不要な部分を消そう。

@at-grandpa
Copy link
Owner Author

パッと見、消してよいのか不安になるが、「等価性の比較のテスト」はできているので大丈夫。

@at-grandpa
Copy link
Owner Author

複数のクラスがある場合にテストしたい部分だったので、今クラスが一つしかない場合は、この部分のテストは不要だった。

@at-grandpa
Copy link
Owner Author

Francのテストは不要なので消そう。

@at-grandpa
Copy link
Owner Author

通った。

 crystal spec' <

....

Finished in 788 microseconds
4 examples, 0 failures, 0 errors, 0 pending

@at-grandpa
Copy link
Owner Author

掛け算のテストもDollarとFrancがある。結局、差異は以前に消してしまっているので、テストが重複している。

@at-grandpa
Copy link
Owner Author

けした。

@at-grandpa
Copy link
Owner Author

すっきりした。足し算に進む準備ができた。

@at-grandpa
Copy link
Owner Author

振り返り。

  • サブクラスの仕事を減らし続け、とうとう消すところまでたどり着いた
  • サブクラス削除前の構造では意味があるものの、削除後は冗長になってしまうテストたちを消した

@at-grandpa
Copy link
Owner Author

at-grandpa commented Nov 8, 2017

TODO

  • $5 + 10 CHF = $10(レートが2:1に場合)
  • $5 * 2 = $10
  • amountをprivateにする
  • Dollarの副作用どうする?
  • Moneyの丸め処理をどうする?
  • equals
  • hashCode
  • nullとの等価性比較
  • 他のオブジェクトとの等価性比較
  • 5CHF*2=10CHF
  • DollarとFrancの重複
  • equalsの一般化
  • timesの一般化
  • FrancとDollarを比較する
  • 通貨の概念
  • testFrancMultiplicationを削除する?

@at-grandpa
Copy link
Owner Author

11章終了。

@at-grandpa at-grandpa merged commit 696e999 into master Nov 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants