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

kadai1 nagaa052 #10

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

Conversation

nagaa052
Copy link

@nagaa052 nagaa052 commented Jun 4, 2019

課題

画像変換コマンドを作ろう

要件

  • ディレクトリを指定する
  • 指定したディレクトリ以下のJPGファイルをPNGに変換(デフォルト)
  • ディレクトリ以下は再帰的に処理する
  • 変換前と変換後の画像形式を指定できる(オプション)

実装について

  • ディレクトリ構成はgolang-standardsを参考
    • ただしmainパッケージはcurrentに配置
  • テストは外部パッケージとして実装
  • 対象のディレクトリから再帰的にフィルターし、一致したものから並列に画像処理を行う

レビュー頂きたい点

  • GOの作法に準拠しているか
  • Production Readyな実装に足りてない箇所はどこか

ご相談したい点

  • ディレクトリ構成は他にもルート直下にフラットに配置する形や、clean architectureなど様々あると思いますが、使い分けのデファクト的なものはあるのでしょうか?
  • pkgディレクトリは外部からも利用可能なものを配置する認識ですが、プロジェクト依存の高いもの(pkgディレクトリ外への参照を行なっているなど)の場合、配置すべきではないのでしょうか?
  • 上記の理由からいわゆるビジネスロジック的な実装はinternalディレクトリに配置しましたが、使い方間違ったりしてるでしょうか?

@nagaa052 nagaa052 mentioned this pull request Jun 11, 2019
6 tasks
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.

1 participant