Railsのview画面のif分岐が反映されない

Question

Railsのviewにてif文を記述し、DBに値が入っている、いない、で表示を変更させたいのですが思うように変わってくれず、助力を頂きたく質問をいたします。

実行したいこと

index画面にてuserをeach do で一覧表示している中で、userテーブル以外のDBに入っている値を取得し、その値が入っているかいないかでif文で表示を分けようとしています。

確認したこと

画面をリロードした際に、ターミナルを確認するとuserテーブルとは別のテーブルのDBに入っている値は取得できているように思います。（見方があっていれば…）

コード

index.html.erb抜粋

  <div class="employee__box">
  <% @users.each do |user| %>
  <ul class="employee__box__list">
  <%= render partial: "employee", locals: { user: user} %>
  </ul>
  <% end %>
  </div>

<%= render partial: "employee", locals: { user: user} %></ul>のrender部分

<% if @sch.present? %>
      <li class="employee__box__list__name", id="leave", data-user-id="<%=user.id%>"><%= user.name %></li>
      <% else %>
      <li class="employee__box__list__name_select", id="attend", data-user-id="<%=user.id%>"><%= user.name %></li>
<% end %>

すべて
<li class="employee__box__list__name", id="leave", data-user-id="<%=user.id%>"><%= user.name %></li>の方で表示されてしまいます。

users_controller.rbのindexアクション

  def index
    @users = User.all.order("id")
    @users.each do |user|
      @schedule = Schedule.find(user.schedules.ids.last)
      @sch = @schedule.out
    end
  end

コントローラーの@users = User.all.order("id")でuserの一覧を表示しています。
usersとschedulesはmodel側でhas_manyとbelongs_toでアソシエーションを組んでいます。

@schedule = Schedule.find(user.schedules.ids.last)により@scheduleでschedulesテーブルでそれぞれのuser_idを持つ、最後のレコードを取得しています。
@sch = @schedule.outはoutカラムを指定し、view側で<% if @sch.present? %>とすることで、schedulesテーブルのoutカラムの中に値が入っているか、いないか、で分岐させようとしています。

index画面をリロードした際に、ターミナルでの表示の詳細を見ると

Started GET "/" for ::1 at 2019-12-29 22:35:41 +0900
Processing by UsersController#index as HTML
  User Load (0.3ms)  SELECT `users`.* FROM `users` ORDER BY id
  ↳ app/controllers/users_controller.rb:5
   (0.3ms)  SELECT `schedules`.`id` FROM `schedules` WHERE `schedules`.`user_id` = 1
  ↳ app/controllers/users_controller.rb:6
  Schedule Load (0.5ms)  SELECT  `schedules`.* FROM `schedules` WHERE `schedules`.`id` = 14 LIMIT 1
  ↳ app/controllers/users_controller.rb:6
   (0.5ms)  SELECT `schedules`.`id` FROM `schedules` WHERE `schedules`.`user_id` = 2
  ↳ app/controllers/users_controller.rb:6
  Schedule Load (0.2ms)  SELECT  `schedules`.* FROM `schedules` WHERE `schedules`.`id` = 5 LIMIT 1
  ↳ app/controllers/users_controller.rb:6
   (0.2ms)  SELECT `schedules`.`id` FROM `schedules` WHERE `schedules`.`user_id` = 3
  ↳ app/controllers/users_controller.rb:6
  Schedule Load (0.2ms)  SELECT  `schedules`.* FROM `schedules` WHERE `schedules`.`id` = 15 LIMIT 1
  ↳ app/controllers/users_controller.rb:6
   (0.2ms)  SELECT `schedules`.`id` FROM `schedules` WHERE `schedules`.`user_id` = 4
  ↳ app/controllers/users_controller.rb:6
  Schedule Load (0.2ms)  SELECT  `schedules`.* FROM `schedules` WHERE `schedules`.`id` = 18 LIMIT 1
  ↳ app/controllers/users_controller.rb:6
   (0.2ms)  SELECT `schedules`.`id` FROM `schedules` WHERE `schedules`.`user_id` = 5
  ↳ app/controllers/users_controller.rb:6
  Schedule Load (0.2ms)  SELECT  `schedules`.* FROM `schedules` WHERE `schedules`.`id` = 16 LIMIT 1
  ↳ app/controllers/users_controller.rb:6
   (0.2ms)  SELECT `schedules`.`id` FROM `schedules` WHERE `schedules`.`user_id` = 6
  ↳ app/controllers/users_controller.rb:6
  Schedule Load (0.2ms)  SELECT  `schedules`.* FROM `schedules` WHERE `schedules`.`id` = 17 LIMIT 1
  ↳ app/controllers/users_controller.rb:6
   (0.3ms)  SELECT `schedules`.`id` FROM `schedules` WHERE `schedules`.`user_id` = 7
  ↳ app/controllers/users_controller.rb:6
  Schedule Load (0.2ms)  SELECT  `schedules`.* FROM `schedules` WHERE `schedules`.`id` = 12 LIMIT 1
  ↳ app/controllers/users_controller.rb:6
  Rendering users/index.html.erb within layouts/application
  Rendered users/_header.html.erb (0.2ms)
  Rendered users/_employee.html.erb (0.5ms)
  Rendered users/_employee.html.erb (0.1ms)
  Rendered users/_employee.html.erb (0.0ms)
  Rendered users/_employee.html.erb (0.0ms)
  Rendered users/_employee.html.erb (0.0ms)
  Rendered users/_employee.html.erb (0.1ms)
  Rendered users/_employee.html.erb (0.1ms)
  Rendered users/index.html.erb within layouts/application (12.3ms)
Completed 200 OK in 89ms (Views: 48.4ms | ActiveRecord: 9.4ms)

と表示され、
(0.3ms) SELECT schedules.id FROM schedules WHERE schedules.user_id = 1
で、@users = User.all.order("id")の一覧が表示され、Schedule Load (0.5ms) SELECT schedules.* FROM schedules WHERE schedules.id = 14 LIMIT 1
で、@schedule = Schedule.find(user.schedules.ids.last)の値が取得できているのが分かるのですが、上記viewのように設定してもifで分岐される表示になりません。

試したこと

<% if @sch.present? %>
      <li class="employee__box__list__name", id="leave", data-user-id="<%=user.id%>"><%= user.name %></li>
      <% else %>
      <li class="employee__box__list__name_select", id="attend", data-user-id="<%=user.id%>"><%= user.name %></li>
<% end %>

の<% if @sch.present? %>を<% if @sch.blank? %>にすると、全部 <li class="employee__box__list__name_select", id="attend", data-user-id="<%=user.id%>"><%= user.name %></li>側の表示になります。
present?、blank?の他に、<% if @sch.nil? %>では.blank?の時と同じくなりました。

controllerの@sch = @schedule.outの直下にbinding.pryを入れてリロードした場合、@schの値は取得できていて、@schの中には日時が入っています。そのため、あるか、ないかでtrue、falseの判定をさせて、falseの場合にelse側の表示をさせようとしておりますが、上記htmlのif文で分岐できておりません。
その理由が分からず、困っております。

エラーなどは出ず、.present?だと全部else前の表示になり、.blank?だと全部else後の表示になってしまう状態です。
何か、お気づきの点がございましたらご回答いただけましたら幸いです。
不明な点がありましたらコメントにてお伝えいただけましたら、ご提示いたします。

Comments

[KazuSaka]

気になるところがあっので…
コントローラのIndexアクションの所ですが、
@usersには全ユーザーが入ってますが、@schにはeach文の最後のユーザーの日付データしか入ってないのでは？(上書きされるから)
だから、ビューでは、ユーザー毎に処理しても、@schは毎回同じ値が使われるのでは？
@schは配列にすべきかと思います。

[Dizzyboy]

ご回答頂き、ありがとうございます。

@users.each do |user|
@schedule = Schedule.find(user.schedules.ids.last)
end
のeachはuserは@schedule = Schedule.find(user.schedules.ids.last)をして、それぞれのuser_idを持っている、最後のschedule.idを取得するためにeachをかけております。（userが無いとuser.schedules.ids.lastでfindメソッドが使え無いのです…）

質問文内のterminalの表示をみる限りでは、user_idが1の時、schedule.idの14が取れてて、これがscheduleテーブル内でのuser_id1がついている、最後のレコードです。
今userは7名いて、それぞれに最後のレコードの値が付与されていますが、これが上書きされてしまっているということでしょうか？

@schを配列にするという方法が、わかりません…調べてみたのですがmapメソッドというのを使えばよいのでしょうか？
せっかくご回答をいただけたのに、理解が追いつかず申し訳ございません…。

[KazuSaka]

回答欄に私の考えを記載しました。

Answer 1

###ZIPメソッドについて

一つだけ気になったのですが、zipメソッドを使うとeachの文字がなくてもきちんと表示されるのはなぜなのでしょうか？試しに <% @users.zip(@sch).each do |user, sch| %>と記述しても、同じ挙動が得られたので、zipはeachも包括していると考えてよいのでしょうか？（この場合、eachを置くと重服する？？）

他のサイトなどを確認すると、zipだけのところもあれば、eachもあわせて使用しているところもあり、どちらがより良い記述なのだろうか、と気になりました。
https://freesworder.net/ruby-each-multiple/
こちらのサイトも参照してzipについて、調べていた中でふと気になった所存です。

本来、zipメソッドは複数の配列を多次元配列にして、返すメソッドです。
ですので、["apple","banana","orange"].zip([100,120,80])
の結果は[ ["apple",100], ["banana",120], ["orange",80] ]（2次元配列）です。

eachを付けた場合、この2次元配列から順に要素を取り出して処理できます。

eachを省略した場合でも、同じ挙動になるのは、zipメソッドにブロックを渡すこともできるからです。これは、rubyのリファレンスに書いてあります。https://ref.xaio.jp/ruby/classes/array/zip
※ブロックとは、do|変数|...endのことです。

この場合、eachを置くと重服する？？）

重複はしていないと思います。

eachを付けた場合
ブロックを受け取っているのは、eachメソッドなので、zipメソッドは2次元配列を返しているだけです。（zipメソッド自体はeachみたいな振る舞いはしない）

eachを省略した場合
zipメソッドにブロックを渡しているので、eachみたいな振る舞いをする。

どちらがより良い記述なのだろうか、と気になりました。

リファレンスに書いてありますが、ブロックを渡した時は、zipメソッドの返り値はnilになります。

ブロックを渡さない場合

#retには[["apple",100],["banana",120],["orange",80]]が入ります。
ret = ["apple","banana","orange"].zip([100,120,80])

ブロックを渡す場合

#retはnilになります。
ret = ["apple","banana","orange"].zip([100,120,80]) do |fruit,price|
       #処理
end

ですので、eachを付けるか省略するかの違いは
zipメソッドの返り値（2次元配列）を使って、eachメソッドの繰り返し処理をする。
もしくは、返り値はnilだけど、eachみたいなループをさせるか。
の違いだと思います。
ビューではただ、ループさせたいだけなので、どちらの記述でも問題ないと思います。

以上

Comments

[Dizzyboy]

KazuSaka様

この度は、zipメソッドについての追加質問にも丁寧にお答えいただき誠にありがとうございます。
リファレンスのアドレスまでつけて頂き、より詳しく理解を深めることができました。
自分で書いたコードがどのように動いているのか、きちんと把握できて大変助かりました。

今回教えて頂いた情報を元に、引き続き勉強を続けていきます。
知らなかったメソッドなどの情報も得られ非常に勉強になりました。

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

Answer 2

質問の続きですが、

のeachはuserは@schedule = Schedule.find(user.schedules.ids.last)をして、それぞれのuser_idを持っている、最後のschedule.idを取得するためにeachをかけております。（userが無いとuser.schedules.ids.lastでfindメソッドが使え無いのです…）

ここはeach文で各ユーザーに紐づいたschedulesテーブルのレコードを取り出す処理なので問題ないと思います。ログからも確認できていると思います。

質問文内のterminalの表示をみる限りでは、user_idが1の時、schedule.idの14が取れてて、これがscheduleテーブル内でのuser_id1がついている、最後のレコードです。

今userは7名いて、それぞれに最後のレコードの値が付与されていますが、これが上書きされてしまっているということでしょうか？

@schには7人目のoutカラム情報しか入っていないと思います。
outカラム情報はユーザーによってユニークですよね？

以下のコードについて
each文：（１）（２）を7人分繰り返す。
（１）各ユーザーのscheduleテーブルの最終レコードを取り出す。
（２）（１）で取り出したレコードのoutカラムを@schに入れる。

eachループが終わった時にインスタンス変数として保持されているデータは下記だと思います。
@users => [id: 1, name: "hoge"]...[id:7, name: "xxx"]※7人分のユーザー情報
@sch => @usersの7人目のoutカラムのみ. ※7人分のoutカラム情報は保持できていない。
@schedule => @usersの7人目のscheduleテーブルの最終レコード.ビューで参照しないため、関係ない.

  def index
    @users = User.all.order("id")
    @users.each do |user|
      @schedule = Schedule.find(user.schedules.ids.last) #(1)
      @sch = @schedule.out #(2)
    end
  end

@schを配列にするという方法が、わかりません…調べてみたのですがmapメソッドというのを使えばよいのでしょうか？

7人分のoutカラム情報をビューから参照するには、@schを配列にして、7人分保持すればいいかと思いました。（each文の中で配列に代入）
以下のように配列に値をプッシュします。
** @sch.push(@schedule.out) **　※@sch = @schedule.outの書き換え

以下が私の考えるコードになります。
一度、試していただけると嬉しいです。

###コントローラの記述
@schを配列にして7人分のoutカラム情報を入れる

  def index
    @users = User.all.order("id")
    @sch=[] #配列初期化
    @users.each do |user|
      @schedule = Schedule.find(user.schedules.ids.last)
      @sch.push(@schedule.out) #配列に格納
    end
  end

##HTMLの記述
zipメソッドで@users配列と@sch配列から同時に値を取り出せます。
取り出したschをパーシャルでつかう。

  <div class="employee__box">
    <% @users.zip(@sch) do |user, sch| %>
      <ul class="employee__box__list">
        <%= render partial: "employee", locals: {user: user, sch: sch} %>
      </ul>
    <% end %>
  </div>

##パーシャルの記述
@schから取り出した各ユーザーのout情報で分岐する。

<% if sch.present? %>
      <li class="employee__box__list__name", id="leave", data-user-id="<%=user.id%>"><%= user.name %></li>
<% else %>
      <li class="employee__box__list__name_select", id="attend", data-user-id="<%=user.id%>"><%= user.name %></li>
<% end %>

以上

Comments

[Dizzyboy]

ご回答ありがとうございます。

お返事が遅くなり、大変申し訳ございません。
追加の質問にも細かく答えて頂き、重ね重ねありがとうございます。
まだ、試すことができていないのですが、先にご返答を頂いたお礼をさせていただきたくコメントいたしました。

これからお伝え頂いた内容を、試してみたいと思います。
pushは知っていたのですがzipメソッドは初耳なので、調べつつ内容をきちんと理解しながら作業したく思います。
追って、結果をお伝えしたいと考えておりますのでお時間をいただけましたら幸いです。

[Dizzyboy]

KazuSaka様、ご回答頂いた内容を元に記述いたしましたところ、無事希望する挙動を得ることができました。大変感謝いたします。

ご指摘頂いた、
eachループが終わった時にインスタンス変数として保持されているデータは下記だと思います。
@users => [id: 1, name: "hoge"]...[id:7, name: "xxx"]※7人分のユーザー情報
@sch => @usersの7人目のoutカラムのみ. ※7人分のoutカラム情報は保持できていない。
@schedule => @usersの7人目のscheduleテーブルの最終レコード.ビューで参照しないため、関係ない.

の部分で、画面表示に適用される部分とされない部分が明確になりました。
自分の記述では @sch = @schedule.outに入る値が、eachで排出した最後の部分しか適用されないというのに気づけていなかった、ビューの表示に影響する部分と、されない部分が明確に判断できていなかったのが、自身の勉強不足と痛感しました。
terminalで表示されているからviewにも伝わっている、と誤った判断をしておりました。

zipメソッドについても調べて内容を把握いたしました。
2つの配列をループさせて値を取り出す、というメソッドがあるのを知りませんでした…。
今回調べた中で、https://qiita.com/Jwataru/items/923a11f9e97a84c7982f#zip%E3%83%A1%E3%82%BD%E3%83%83%E3%83%89
の記事を読んだ時に「引き出しがないと「〇〇したいから〇〇メソッドを使えばいいか、そのメソッドどうやって使うんだっけ」という段階にすらいかない」という記載があり、正に今回の自分のケースだなと思いました。

pushについてもrubyでやった内容だったのにも関わらず、記憶から抜け落ちていて恥ずかしい限りです…。
zipのことを調べることで、色々メソッドについても勉強になり、知らなかったメソッドを知ることもできました。これに関しましても大変ありがたく感謝いたします。

一つだけ気になったのですが、zipメソッドを使うとeachの文字がなくてもきちんと表示されるのはなぜなのでしょうか？試しに <% @users.zip(@sch).each do |user, sch| %>と記述しても、同じ挙動が得られたので、zipはeachも包括していると考えてよいのでしょうか？（この場合、eachを置くと重服する？？）
他のサイトなどを確認すると、zipだけのところもあれば、eachもあわせて使用しているところもあり、どちらがより良い記述なのだろうか、と気になりました。
https://freesworder.net/ruby-each-multiple/
こちらのサイトも参照してzipについて、調べていた中でふと気になった所存です。

[KazuSaka]

色々と洞察されていて、素晴らしいです。
また、解決されたみたいで、良かったです。
zipメソッドについての解釈を回答欄にまとめました。