ruby： each文を全て回してからunlessで一致していないかを判定する処理を実現したい。

Question

###前提
roomsの中に入っているclient_id下記の配列とします。
room: [1,3,2,6,4]
params[:id]: 2

  def show_room_check
    rooms = Room.where(customer_id: current_user.id)
    rooms.each do |room|
      unless room.client_id == params[:id].to_i
        return redirect_to users_path
      end
    end
  end

実現したいこと

showページに入る前にbefore_actionでそのページに入ってもいいユーザーかどうかをチェックする処理になります。
やりたいこととしては、もしroomのclient_idがparams[:id]と等しくなっかたらindex_pathに返すようにしたいです。

困っていること

現状の実装では、一つ目の要素がもし違うとindex_pathに変えられてしまうので、最後までeachを回して一致しなかった場合のみindex_pathに返すようにしたい。

Answer 1

redirect_to users_path unless Room.find_by(client_id: params[:id], customer_id: current_user.id)

Comments

[nanase21]

ご教示いただきありがとうござます。
まさか一行で、私が意図する処理ができるとは思いませんでした。
今回は、他の方をベストアンサーとしましたが本当にありがとうございました。

Answer 2

全部違ってて遷移であれば.none?という非常に便利な機能があるんですが、最後まで取り敢えず回したい場合は、フラグを使うのはどうでしょうか

Flag = 0
rooms.each do |room|
  unless room.client_id == params[:id].to_i
    Flag = 1
  end
end
if Flag==1 then
    return redirect_to users_path  
end

Comments

[nanase21]

ご教示いただきありがとうございます。
none?メソッドなんてあるんですね。
勉強になりました。
ちなみに、.none?を使った場合、redirirect_to はどこに書けばいいのでしょうか？

[nanase21]

rooms.none? {|w| w.client_id == params[:id].to_i }
と書いたのですが、trueの場合にredirect_toをどこに書けば良いのかわからないです。

[bamboo-nova]

ifとかでrooms.none?{ |room| room.client_id == params[:id].to_i}とかやった後ですかね。
以下が具体例になります。
http://railsdoc.com/references/none

[nanase21]

ご教示いただきありがとうございます。
上記のURLを参考に出来ました。

[bamboo-nova]

よかったです。多分、今回の実装だけならasmさんのが最適ですね...僕も高評価させて頂きました笑

[nanase21]

確かに今回の実装に限っては、asmさんの回答が1番の最適解なような気がします笑
find_byは一件しかデータを取得できないので、普段あまり使うことがなかったのですがこういう時に使えるんだなと勉強になりました。

Answer 3

  def show_room_check
    id = []
    rooms = Room.where(customer_id: current_user.id)
    rooms.each do |room|
      if room.client_id == params[:id].to_i
        id = room.client_id
      end
    end
    redirect_to users_path unless id.present?
  end

コード自体は汚いですが、一応このような形で意図する処理になりました。
もし問題点や気になる点があればご教示いただけると幸いです。