yubachiri / ideas_stock Goto Github PK

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/app/models/memo.rb#L19

<% if [email protected]? && [email protected]? %>

否定条件を組み合わせると、人はコードを理解しにくくなる。
条件が１つだけならまだしも、組み合わせるとさらにわかりにくくなる。

できる限り、否定を使わない方がスッキリ。

<% if @memos.present? && @memos.count > 0 %>

<% if @memos.try(:count) > 0 %>

でもいい

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/app/models/memo.rb#L18

current_user.books.find_by(id: {book_id})

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/config/routes.rb#L3

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/app/controllers/memos_controller.rb#L12

    user = current_user
    @memo = user.books.first.memos.build(memo_params)

userを宣言したけど利用していないので

@memo = current_user.books.first.memos.build(memo_params)

でいいけど、敢えて代入するなら
first_book = current_user.books.first
@memo = first_book.memos.build(memo_params)

やりたい事単位で区切って、変数に代入した方がわかりやすいかな。

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/config/routes.rb#L8

https://copy-app.herokuapp.com/users/3?memo_id=7

http://moro.hatenablog.com/entry/2015/08/20/081131

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/app/models/memo.rb#L8

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/app/controllers/memos_controller.rb#L37

この手のチェック処理はモデルに書くと良い。

コントローラーは表示する画面をコントロールに専念させ、
業務ロジックはモデルに書く。

SkinnyController, FatModelがRailsの作法として基本形。
用語を検索してみよう。
そうすると色々諸説が入り乱れて混乱すると思うけど、まず基本を抑えよう。

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/app/controllers/memos_controller.rb#L16

flash.nowを使っている所が:+1
redirect_toの所との使い分けは？

https://github.com/kakimoso/ideas_stock/blob/160619906edd096726a3cb2fcddb1ff3bdb29989/app/models/memo.rb#L8

テストケースレビューでの指摘。
ここはenumにしてバリデーションは削除できる

gemfileに記載するgemは本当に利用しているgemのみに書くようにしよう。
全てのgemの動きを理解する必要な無いけど、”良く分からないけどコピペ”はNG。
大まかで良いのでgemの要不要は把握しておこう。
ここは大事なところなので意識してみて。