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

Kadai2 phamvanhung2e123 #35

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

phamvanhung2e123
Copy link

課題2

  • テストのしやすさを考えてリファクタリングしてみる
  • テストのカバレッジを取ってみる
go test -coverprofile=coverage.out
go tool cover -func=coverage.out
go tool cover -html=coverage.out
  • テーブル駆動テストを行う
  • テストヘルパーを作ってみる

io.Readerとio.Writerを調べる

  • 標準パッケージでどのように使われているか
    • ファイルの読み書きなど入出力の基本使います
    • バッファリングして読み書きを行うところを使う
    • 多くの標準パッケージがインタフェースを実装していたり、引数として扱える形でサポートしています。
    json
    bytes.Buffer
    bufio.Reader
    os.File
    image
    jpeg
    png
    base64
    
  • io.Readerとio.Writerがあることでどういう利点があるのか具体例を挙げて考えてみる
     - 標準ライブラリに用意されている便利なツールでラップして呼び出すこと
     - モックしやすいでテスト書きやすくなります

@phamvanhung2e123 phamvanhung2e123 added the kadai2 課題2 label Nov 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kadai2 課題2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants