Kadai2 shinta

[ShintaNakama]

課題2

io.Readerとio.Writerについて調べてみよう

標準パッケージでどのように使われているか

• os(os.Open, os.Create, os.Stdin, os.Stdout, os.Stderr)
  • *os.File型、ファイルを開いたりするときに使う
• bytes.Buffer (struct), bytes.Reader (struct)
  • ファイルではなくメモリへデータを書き込むのに使う。*bytes.Buffer が io.Writer として利用可能。
  • *bytes.Reader が io.Reader として利用可能。
• bufio.Scanner
  • ファイルや標準入力から作られた io.Reader から１行ずつ文字列を読み込む。
• io/ioutil.ReadAll, io/ioutil.ReadFile, io/ioutil.WriteFile, io.Copy
  • io/ioutil.ReadAll: io.Reader から全てデータを読み込んで[]byte を作成する。
  • io/ioutil.ReadFile: 指定されたファイル名から全てのデータを読み込んで[]byte を作成する。
  • io/ioutil.WriteFile: 指定されたファイル名に[]byte を書き込む。os.Create に合わせるなら第三引数permは0666を渡す。
  • io.Copy: io.Reader から io.Writer にデータを全てコピーする便利関数

io.Readerとio.Writerがあることでどういう利点があるのか具体例を挙げて考えてみる

• io.Reader io.Writerを持っている関数であれば、抽象的にIOしていると考えて良い
• 呼び出し側はI/O処理がどんなことをしているのかを理解する必要が無い
• DIPにできる。

test

// 通常のtest
go test ./imageconversion

// カバレッジ
go test -cover ./imageconversion
go test -coverprofile=cover.out ./imageconversion
go tool cover -html=cover.out -o cover.html

• エラーケースのテストがまだできていない。
• test しやすい設計についてもっと考えないといけない。

[tenntenn]

これで良さそうです。

allowExtMap := map[string]bool{
    "jpg": true,
    "jpeg": true,
    "png": true,
    "gif": true,
}

[tenntenn]

何回も呼ぶと何回も.がつきますね。

[tenntenn]

型で分かるので名前付き戻り値にしない。
特に理由がない場合は、戻り値もポインタにする。

[tenntenn]

ファイルパスの結合にはfilepath.Joinを使う

[tenntenn]

上記と同じ

[tenntenn]

下の代入でかき消される

[tenntenn]

t.Runを呼び出すのはヘルパーとは言えない

[tenntenn]

ヘルパーではない

[tenntenn]

テストデータは極力テスト関数の中で定義する

[tenntenn]

こうするなら、flag.StringVarでdirとかのポインタを渡した方が良さそう