Skip to content

[レビュー] SaKu#1

Open
SaKu2110 wants to merge 1 commit into
mainfrom
review
Open

[レビュー] SaKu#1
SaKu2110 wants to merge 1 commit into
mainfrom
review

Conversation

@SaKu2110
Copy link
Copy Markdown
Collaborator

@SaKu2110 SaKu2110 commented Jun 6, 2022

レビュー観点

  1. : goの言語仕様に従っているか
    package名は小文字かつ1単語で抑えられているかどうか
    errorは,呼び出し元が処理されているまたは処理できるようerrorを返却しているかどうか
  2. : MVCの責務を理解しているかどうか
    controller層で,ユーザのリクエストに適したモデルが呼び出されているかどうか
    controller層は,モデルの値に対してユーザにレスポンスが返却されているかどうか
    model層に,データの加工や処理などメインロジックが書かれているかどうか

@SaKu2110 SaKu2110 added the enhancement New feature or request label Jun 6, 2022
@SaKu2110 SaKu2110 self-assigned this Jun 6, 2022
@SaKu2110
Copy link
Copy Markdown
Collaborator Author

SaKu2110 commented Jun 6, 2022

Step. 1

・MVCモデルを意識したフォルダ構成に変更する
・goの言語仕様に準拠したプログラムを理解する

参考

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant