Rails 条件分岐 ログインユーザーのみ自分の名前を確認でき、別のユーザーであれば名前を表示させない

Question

予約表を作っているのですが、ログインユーザーにだけ予約表でユーザーの名前を表示させたいです。
他のユーザーにはユーザーの名前でなく”予約済み”と表示させるような条件分岐を書きたいです。

自分なりに考え、試しに、

<% @time0900 = "予約済みです" if !(@time0900 == current_user)%>

とviewに書いてみたのでが、うまくいきませんでした。

あと、条件分岐はviewにもかけると思うのですが、こういう分岐はコントローラーに書く方が良いのでしょうか？

よろしくお願いします。

    <tr>
      <td>9:00~9:30</td>
      <% if @time0900.count == 4 %>
        <% @time0900.each do |f| %><td><%= f.user_name %></td><% end %>
        
      <% elsif @time0900.count == 3 %>
        <% @time0900.each do |f| %><td><%= f.user_name %></td><% end %><td>予約可能です</td>
        
      <% elsif @time0900.count == 2 %>
        <% @time0900.each do |f| %><td><%= f.user_name %></td><% end %><td>予約可能です</td><td>予約可能です</td>
        
      <% elsif @time0900.count == 1 %>
        <% @time0900.each do |f| %><td><%= f.user_name %></td><% end %><td>予約可能です</td><td>予約可能です</td><td>予約可能です</td>
        
      <% else %><td>予約可能です</td><td>予約可能です</td><td>予約可能です</td><td>予約可能です</td><% end %>
    </tr>

class TimetablesController < ApplicationController
  def timetable_index
    @timetables = Timetable.all
    @time0900 = Timetable.where(hour: "9", minute: "00")
    @time0930 = Timetable.where(hour: "9", minute: "30")
    @time1000 = Timetable.where(hour: "10", minute: "00")
    @time1030 = Timetable.where(hour: "10", minute: "30")
    @time1100 = Timetable.where(hour: "11", minute: "00")
    @time1130 = Timetable.where(hour: "11", minute: "30")
    @time1200 = Timetable.where(hour: "12", minute: "00")
    @time1230 = Timetable.where(hour: "12", minute: "30")
    @time1500 = Timetable.where(hour: "15", minute: "00")
    @time1530 = Timetable.where(hour: "15", minute: "30")
    @time1600 = Timetable.where(hour: "16", minute: "00")
    @time1630 = Timetable.where(hour: "16", minute: "30")
    @time1700 = Timetable.where(hour: "17", minute: "00")
    @time1730 = Timetable.where(hour: "17", minute: "30")
    @time1800 = Timetable.where(hour: "18", minute: "00")
    @time1830 = Timetable.where(hour: "18", minute: "30")
    @time1900 = Timetable.where(hour: "19", minute: "00")
    @time1930 = Timetable.where(hour: "19", minute: "30")
  end
  
  def new
    @timetable = Timetable.new 
  end 
  
  def create
    @timetable = Timetable.new(user_id: current_user.id, user_name: current_user.name, date: timetable_params[:date], hour: timetable_params[:hour], minute: timetable_params[:minute])
    if @timetable.save
      flash[:success] = "ご予約ありがとうございます"
      redirect_to root_path
    else
      flash[:success] = "申し訳ございません。予約を受け付けられませんでした"
      redirect_to root_path
    end
  end
  
  def show
    @time0900 = Timetable.find()
  end
  
  def destroy
  end 
  
  private
  def timetable_params
    params.require(:timetable).permit(:user_id, :user_name, :date, :hour, :minute)
  end
end

Answer 1

<% @time0900 = "予約済みです" if !(@time0900 == current_user)%>
@time0900はTimetableでcurrent_user は Userですからけして == は成立しません。
<% @time0900 = "予約済みです" if !(@time0900.user_id == current_user.id)%>
としてみてください。

＞あと、条件分岐はviewにもかけると思うのですが、こういう分岐はコントローラーに書く方が良いのでしょうか？
私の好みとしては、viewにはできるだけrubyを書かないようにしています。
<% %> だとか <%= %> だとかあって読みにくい。
ですので 私好みに書いて頂くとすると「controllerで書けるならそっちで書いて」です。

viewに書くとしても

<% if @time0900.count == 4 %>
        <% @time0900.each do |f| %><td><%= f.user_name %></td><% end %>

の連続はつらいです。
ifやcaseがあると、単なる場合分けなのか例外処理なのか緊張感が走りますし、
@time1930　までこれが１８組もあるかと思うと、、、、、

<td><%= f.user_name %></td>
<% ([e-mail]).times do %><td>予約可能です</td><% end %>
<% @time0900.count.times do %><tf></td><% end %>

とか。
更に　
@time['0900'] = Timetable.where(hour: "9", minute: "00")
として、viewの方を二重loopで処理したい。

Comments

[muraishi]

ありがとうございます！
やっぱりコントローラーに書いた方が良いですよね！でもviewに書く方が簡単そうなのでviewで書いたのですが、コントローラーに書くように頑張ります。view用に書いてくださったリファクタリング？すごいです！とてもスッキリしました。わたしも、「これはもっと手短に書けるんだろうけど...」と思っていたのですが、どう書いて良いのかわかりませんでした。

問題の
<% @time0900 = "予約済みです" if !(@time0900.user_id == current_user.id)%>
を試しに入れてみたですが、エラーが出てしまいます。
これの原因と打開策をずっと考えていたのですが、よくわかりませんでした。
やはり、コントローラー側に記述した方が良いと思うので、そっちの方向で考えます。
いつもありがとうございます！

NoMethodError in StaticPages#home

Showing /home/ec2-user/environment/portfolio/app/views/helpers/_time_table.html.erb where line #12 raised:

undefined method `user_id' for #<Timetable::ActiveRecord_Relation:0x00007f7a9449a1b8>

Extracted source (around line #12):

10
11
12
13
14
15

<tr>
<td>9:00~9:30</td>
<% @time0900 = ["予約済みです"] if !(@time0900.user_id == current_user.id)%>
<% @time0900.each do |f| %><td><%= f.user_name %></td><% end %>
<% ([e-mail]).times do %><td>予約可能です</td><% end %>
</tr>

Trace of template inclusion: app/views/static_pages/home.html.erb

Rails.root: /home/ec2-user/environment/portfolio
Application Trace | Framework Trace | Full Trace

app/views/helpers/_time_table.html.erb:12:in `_app_views_helpers__time_table_html_erb___1918112975281294955_70082194706980'
app/views/static_pages/home.html.erb:28:in `_app_views_static_pages_home_html_erb___1763037265118203_41238900'

Request

Parameters:

None

Toggle session dump
Toggle env dump
Response

Headers:

None

[winterboum]

<% @time0900 = "予約済みです" if !(@time0900.user_id == current_user.id)%>
ですが
<% @time0900 = "予約済みです" if !(@time0900.first.user_id == current_user.id)%>
ですね。@time0900が単数形なので騙されました。
変数名の単数形複数形は注意深く区別して命名してください

[winterboum]

あと、
viewに書くほうが楽なものもあります。
それはviewでも良いかと。
頑張らないとCONTROLLERに書けないものは適度に頑張るということで

[muraishi]

あーーー！！
複数だったからどのuser_idかわからなかったんですね！
いやーほんと名前って大事ですね。気をつけます！
ありがとうございます！

[muraishi]

そこの判断は経験とかなんでしょうね！
精進します！