ruby on rails チュートリアル 14章　フォロー機能のdestroyアクションの記述でわからないところがあります。

わからないこと

Ruby on Rails チュートリアル１４章「14.2.4 [Follow] ボタン (基本編)」
のdestroyアクション実装の部分でわからないところがあるので質問させてください。

user = Relationship.find(params[:id).followed
ということは、
「Relatihonshipテーブルにおいて、idカラムがparams[:id]のフォロー中ユーザー」をuserとしているということですよね。
この場合、unfollowボタンのビュー記述部分に記載している「followed_id:@user.id」が上記のparams[:id]に当てはまるという理解で合っていますでしょうか？
そして、合っていたとしたら、find(params[:follwed_id]).followedと書かないのはなぜでしょうか？

雑多な説明で恐縮ですが、よろしくおねがいします・・・！

該当のソースコード

リスト 14.33: Relationshipsコントローラ
app/controllers/relationships_controller.rb

class RelationshipsController < ApplicationController
 before_action :logged_in_user

 def create
   user = User.find(params[:followed_id])
   current_user.follow(user)
   redirect_to user
 end

 def destroy
   user = Relationship.find(params[:id]).followed　#ここの部分
   current_user.unfollow(user)
   redirect_to user
 end
end

unfollowボタンのビュー記述部分

<%= form_for(current_user.active_relationships.find_by(followed_id: @user.id),
      html:{ method: :delete}) do |f| %>
    <%= f.submit "Unfollow", class: "btn" %>
<% end %>

user.rb

 def unfollow(other_user)
    active_relationships.find_by(followed_id: other_user.id).destroy
  end

回答1件

ベストアンサー

user = Relationship.find(params[:id).followed

ということは、
「Relatihonshipテーブルにおいて、idカラムがparams[:id]のフォロー中ユーザー」をuserとしているということですよね。

「Relatsionshipテーブルにおいて、idカラムがparams[:id]の行のフォロー中ユーザー」
という意味ならばそうです。

この場合、unfollowボタンのビュー記述部分に記載している「followed_id:@user.id」が上記のparams[:id]に当てはまるという理解で合っていますでしょうか？

いいえ、別です。
図 14.7にidカラムがないのが誤解の原因なのかな？と思いますが
Relatihonshipテーブルにもidカラムがあり、Userテーブルとは無関係にユニークな番号が振られているわけです。

図 14.7をこう読み替えるといいかもしれません。

follower_id	id	followed
1	1	2
1	2	7
3	3	1
7	4	2
1	5	10
2	6	1
1	7	8
9	8	1

ただし、userは

erb
<%= form_for(current_user.active_relationships.find_by(followed_id: @user.id),
      html:{ method: :delete}) do |f| %>
    <%= f.hidden_field :followed_id, @user.id %>
    <%= f.submit "Unfollow", class: "btn" %>
<% end %>

とすることで

ruby
def destroy
  user = User.find(params[:followed_id])
  # 省略
end

が可能です。

こうしなかった理由はCRUDとしての統一感を重視したのかなと思います。

Relationshipコントローラーのdestroyメソッドなんだから
Relationshipのidを渡して削除しようという考えかなと思います。

まぁ、つまり

ruby
class RelationshipsController
  def destroy
    Relationship.find(params[:id]).destroy
  end
end

こうですね。

しかしこれでは、他人のフォロワーを勝手に解除できてしまいます。
そこで遠回りして、other_userを取得しcurrent_userを介して解除しています。

投稿2019/04/16 23:51

asm

総合スコア15127

natecosan

2019/04/17 12:05

回答ありがとうございます。アプリケーション側でのコード実行順序がよく分かっておらず、混乱していたみたいです。。。つまり、form_forタグのcurrent_user.active_relationships.find_by(followed_id: @user.id)の部分で先にfollowed_id: @user.idであるユーザーを探しておいて、そのレコードにあるidカラムの値をparams[:id]として受け取る→Relationshipの該当レコードを削除ということですか？

asm

2019/04/17 13:16 編集

> form_forタグのcurrent_user.active_relationships.find_by(followed_id: @user.id)の部分で先にfollowed_id: @user.idであるユーザーを探しておいて current_user.active_relationships.find_by(followed_id: @user.id)は follower_id: current_user.idかつfollowed_id: @user.idなRelationshipを探しておりそのidをparams[:id]としてコントローラに渡しています。そして、コントローラ側はそのparams[:id]からRelationshipを探して(最終的に)削除しています

natecosan

2019/04/17 23:58

なるほど！わかりました。ありがとうございます！！

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
87.20%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する