Kadai1-fujiokayu

[fujiokayu]

課題１：「画像変換コマンドを作ろう」を実装しました。

Usage、仕様 は kadai1/fujiokayu/README.md に記載しています。
宿題のスライドに記載された仕様は満たしていると思います。

設計/実装方針

• main パッケージから以下の処理をパッケージとして分離する。
  • args
    コマンドライン引数をパースして、ユーザー定義型の構造体に格納する。
  • walker
    引数で指定されたディレクトリを再帰的に走査し、検知したファイルパスをチャネルに格納する。
  • converter
    ファイルパス、および変換対象と変換後のファイル形式を受け取り、画像変換処理を行う。
• エラーが検知された場合は log.Fatal でメッセージを出力して処理を停止する。

レビューしていただきたいポイント

可能であれば、以下の点についてチェックしていただけると嬉しいです。

• パッケージの分離は妥当か
• 他にユーザー定義型を定義することで効率化できたり、可読性が上がるような箇所がないか
  • 今回の課題では、walker、converter 共に関数だけのシンプルなパッケージとなったので、args パッケージで一つ構造体を定義しているのみです。
• Go らしく書けているか。
  • 特にエラー処理周りなどが気になっています。

その他にも気になる点などありましたら、コメントいただけると嬉しいです。

よろしくお願いいたします。