自己代入した後に並び替えを行いたい。

Question

railsでmessageモデルからwhereで情報を取得してから、自己代入をした後に.orderで並び替えをしたいのですがなぜか並び替えが動きません。

コードは下記です。

  def show
    @user = Room.find(params[:id])
    @message = Message.where(room_id: @user)
    @message += Message.where(room_id: @user.second_user_id).order(id: :DESC) #←これが動かない
  end

ブラウザで表示自体はされるのですが並び替えが動きません。
どのように解決すればよろしいでしょうか？

Answer 1

 def show
    @user = Room.find(params[:id])
    @message = Message.where(room_id: @user)
    @message += Message.where(room_id: @user.second_user_id).order(id: :DESC) #second_user_idに関するものはidの逆順に並び替えたよ
  end

となっているので

  def show
    @user = Room.find(params[:id])
    @message = Message.where(room_id: @user)#ActiveRecodeRelationだよーまだSQL実行してないよ
    @message += Message.where(room_id: @user.second_user_id)#ActiveRecodeRelationにプラスなんてねーから勝手に結果をarrayに格納して結合するよ
    @message.sort_by{|ms|ms.id}!.reverse!#arrayをIDの逆順で並び替えるよ
  end

とすればOK
rails5ならorで繋いで一つのActiveRecodeRelationにするのですが…。
とりあえずこれでいいでしょう。

Comments

[yoshimitsu41]

mokoさんのコードを試したのですが、結果は同じでした。
エラーは出ないのですが・・・。
何故でしょうか？

[moke]

実は、yoshimitsu41さんが
どうしたいかが、わからなく想像なんです。
あっ逆順でしたね。
reverseをつけ忘れてますね。
回答を修正しておきます。
これでも希望通りにならない場合、レコードの例を使って
yoshimitsu41さんがどうしたいか説明してください。

[yoshimitsu41]

修正したコードにしましたが希望通りの動きになっておりません。
説明不足でもうしわけありません。
ユーザー同士のメッセージ機能を実装してす。

roomsテーブルにはuser_idとsecond_user_idのカラムがあり
belongs_to :user
belongs_to :second_user, class_name: 'User'
has_many :messages

messagesテーブルにはtextとroom_idとuser_idのカラムがあり
belongs_to :room
belongs_to :user

usersテーブルにはemaiilとpasswordとnicknameのカラムがあり
has_many :messages
has_many :rooms
の状態です。

やりたいこととして
・ログインしているuserのidとroomsテーブルのuser_idが一致しているものを探す。
・ログインしているuserのidとroomsテーブルのsecond_user_idが一致しているものを探す。
・その2つを結合させる。
・ソートをかけて新しい順に上から並べてブラウザに表示させる。

どのようにすればいいでしょうか？

[moke]

その日本語だと、問題無い気がします。
具体的なレコードを書いて希望の並びかたと、現状の並びかたの例を示して下さい。
あと、いくらrailsが空気を読むからといって、あなたの意識を読み取り、貴方の意図と違う動きをしたらエラーを吐くなどということはありません。
貴方の書いたコードの通りに動こうとするだけです。

[yoshimitsu41]

具体的なレコードを書きます。
（レコードを書くという意味があまり良く分かっていなくてあってるかわかりませんが・・・。）

@message.sort_by{|ms|ms.id}.reverse
の下にbinding.pry
を記述して表示される結果が下記です。
***************************************************************************************
=> [#<Message:0x007f80a0942618
id: 16,
text: “ログインユーザーからのメッセージ”,
created_at: Tue, 25 Apr 2017 15:54:54 JST +09:00,
updated_at: Tue, 25 Apr 2017 15:54:54 JST +09:00,
room_id: 10,
user_id: 1>,
#<Message:0x007f80a09424d8
id: 22,
text: "ログインユーザーからのメッセージ",
created_at: Thu, 27 Apr 2017 14:21:46 JST +09:00,
updated_at: Thu, 27 Apr 2017 14:21:46 JST +09:00,
room_id: 10,
user_id: 1>,
#<Message:0x007f80a0942398
id: 23,
text: “セカンドユーザーからのメッセージ”,
created_at: Thu, 27 Apr 2017 14:21:49 JST +09:00,
updated_at: Thu, 27 Apr 2017 14:21:49 JST +09:00,
room_id: 10,
user_id: 3>]
***************************************************************************************
上記とは逆の順番で表示させたいということになります。

[moke]

おお
sort_byとreverse
に!が付いていません
sort_by!{|ms|ms.id}.reverse!
でオッケーです。
これで@messageに変更が適用されます。

[yoshimitsu41]

希望通り動きました！いつもありがとうございます。
ruby自体をもっと理解しないとだめですね。
ありがとうございました！

[moke]

すみません
!が付いていないと変更結果を表示して終わりです、新たな変数で補足するか、！をつけないといけません。