search

TsuMakoto / blog-system

0 stars 0 forks source link

Associationsを使おう #6

Open ippachi opened 5 years ago

ippachi commented 5 years ago 
@post = Post.find(params[:post_id])
@comments = Comment.where(post_id: @post.id)

みたいにしていくと処理が多くなったときにインスタンス変数が増えすぎてしまうので has_many や belongs_to で定義されるメソッドを調べて使ってみよう

例）

@post = Post.find(params[:post_id])
@post.comments # => Comment.where(post_id: @post.id)

うまく使えば Model.where(hoge_id: hoge.id) みたいなのがほぼ無くなるはずです

TsuMakoto commented 5 years ago

修正しましたー！ https://github.com/TsuMakoto/blog-system/commit/e0127cf16712c16e8d80ccbdb44985d66dfbd7cf

ippachi commented 5 years ago

上のリンク辿った先にコメントあります！

TsuMakoto commented 5 years ago

76cfd800b49c8799aaaa25fa969990d7bc556dfa 「current_user.」 を使って、処理変更してます！

TsuMakoto commented 5 years ago

追加修正 fbfe9c98dd8debb637cb286c38965c3909448da4

ippachi commented 5 years ago

merge(block_flg: 0) はその値に決まってるならDB側にdefault値を当てた方がいいね

TsuMakoto commented 5 years ago

困ってたけどたしかに ありがとう 6b382e73867a504699ea708a4821b4888d3c396d

ippachi commented 5 years ago 
  def change
    change_column :comments, :block_flg, :text, null: false, default: 0
  end

text 型だけど大丈夫？

TsuMakoto commented 5 years ago

20158cf1d6a0652505c710b477f810b5ff66f6b6 変更しました〜