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

Gopher道場 #2 #41

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Gopher道場 #2 #41

wants to merge 1 commit into from

Conversation

po3rin
Copy link

@po3rin po3rin commented Jul 5, 2018

プルリクをおくり忘れていました。。
ご確認よろしくお願いいたします。

感想と課題

全パターンは Table Driven Tests を実装した。convert する全6パターンが非常に見やすくなった。しかし、mapにfor文を適用すると、書いた順番で繰り返されない仕様にハマった。for文で順番を担保するにはどうしたらいいのだろう。。

カバレッジは80%いくが他の20%はエラー処理。
エラー処理もカバーする必要がある?

サブテストとテーブル駆動テストのパターンを採用した場合、テストヘルパーを使っても呼び出し行が変わらないのであまり意味がない??そもそも使い方が間違っているかもしれない。

宿題 io.Writer と io.Reader について

io.Writer io.Readerはそれぞれインターフェースであり、抽象化されている。故に振る舞いを与えられているだけなのでFile Buffer等の様々なものに対して実装されている。
ゆえに実装が使い回せるので、結果的にGo言語のシンプルさに貢献している。

@po3rin po3rin added the kadai2 label Jul 5, 2018
@po3rin po3rin self-assigned this Jul 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant