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

Question

わからないこと

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

Answer 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は

<%= 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 %>

とすることで

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

が可能です。

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

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

まぁ、つまり

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

こうですね。

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

Comments

[natecosan]

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

[asm]

> 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]

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