ルーティングで設定したURLの「:id」について

Question

Ruby on Rails 5　でアプリケーションを作成しております。

書類の名前が並んだ一覧表ページ(index)から、名前をクリックすると編集ページ(edit)に遷移するようにしました。編集ページのURLはルーティングで以下のように設定してあります。

edit_document GET /documents/:id/edit(.:format) documents#edit

ページの遷移はうまくいきます。
例えばIDが18の書類の名前をクリックすると、URLは /documents/18/edit となるのですが、編集ページで表示される書類の名前が全く違うIDのものになってしまいます。

確認のためコントローラを以下のように書いてみたところ、params[:id]は18でしたが、@document.idが16と食い違って出力されました。

documents_controller.rb

def edit
　　　　@document = Document.find_by(params[:id])
　　　　puts params[:id]　 #=> 18
　　　　puts @document.id   #=> 16
end

食い違うのはなぜなのでしょうか？

よろしくお願いいたします。

Answer 1

こんにちは

@document = Document.find_by(params[:id])

の find_by を、find にして、

@document = Document.find(params[:id])

と修正するとうまくいきませんか？

参考になれば幸いです。

Comments

[uraura]

回答ありがとうございます！
アドバイス通り、find_byをfindにしたらうまくいきました！ findとfind_byの違いについて調べたところ、＊ 該当するレコードがない場合の挙動の違い
＊ findはidしか検索しない
ということがわかりましが、以前はfind_byできちんと通っていたのに今回はfindでなくてはいけないのはどうしてなのでしょうか？引き続きご回答いただけると幸いです。

[jun68ykt]

rails c
で、rails コンソールを起動して、
Document.find_by(18)
とすると、
SELECT "documents".* FROM "documents" WHERE (18) LIMIT 1
というクエリが発行されているのを確認できると思います。
このクエリだと、たぶん id の一番小さいレコードが返ってくるものとは
思いますが、SQL の文法的には、どの１レコードが返ってくるかは
予測できないです。

こういう条件なら、find_by でもうまくいっているように見えてしまう、
ということで考えられることは以下です。

もし、documentsテーブルのレコードが１つしかなくて、
それの id が、仮にたとえば 10 だったとしましょう。
そのとき
Document.find_by(10) は
SELECT "documents".* FROM "documents" WHERE (10) LIMIT 1
となって、これが、たまたまその１つしかない id=10 のレコードを返しますので、
/documents/10/edit
というパスで入力編集画面のテストをしていたすると、
ちゃんと find_by が id=10 のレコードを探してきて表示させている
ように見えてしまいます。

ということで、find_by でうまくいっているように見えていた間は、
documentsに１つしかレコードがない状態で、
edit画面（または他の画面）を表示させていた、ということは
ないでしょうか？

[uraura]

大変丁寧な回答ありがとうございます。
Document.find_by(18)
ですと、
ArgumentError: Unsupported argument type: 17 (Integer)
となってしまいました。

＞たぶん id の一番小さいレコードが返ってくるものとは思いますが
この点はおっしゃられる通りで、レコードが３つ（id: 16, 17, 18)の時は find_by(params[:id])で一番小さいレコード id: 16 の名前が出力されていました。

＞find_by でうまくいっているように見えていた間は、documentsに１つしかレコードがない状態で
ちょっと記憶が曖昧なのですが確かにそうかもしれません・・・！

find_byを使うときはfind_by(id: params[:id])のように、idを検索することを明示的にすると問題ないようです。今後はfindかfind_byか、違いをもう一度おさらいしてどちらを使うか決めたいと思います。

ありがとうございました！