🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

Q&A

解決済

1回答

707閲覧

勤務中のユーザーの一覧表示をしたい

ikt_erk122

総合スコア40

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

0グッド

1クリップ

投稿2019/09/10 06:23

【やりたいこと】
勤務中ユーザーの一覧表示したい
【困っていること】
一応表示はできているが重複して表示されている
【質問】
何が原因で重複しているのかわからないので原因と解決方法を教えていただけないでしょうか?
⬇️githubも載せておきます
github

![イメージ説明
users_controller.rb

def on_duty @now_users = [] @now_users_employee_number = [] User.all.each do |user| if user.attendances.any?{|day| ( day.worked_on == Date.today && !day.started_at.blank? && day.finished_at.blank? ) } @now_users.push(user.name) @now_users_employee_number.push(user.employee_number) end end end

views/users/on_duty.htnl.erb

<% provide(:title, '出勤中社員一覧') %> <h1>出勤中社員一覧</h1> <div> <table class="table-bordered table-striped table-condensed"> <thead> <tr> <th>社員番号</th> <th>名前</th> </tr> </thead> <tbody> <% @now_users_employee_number.each do |employee_number| %> <% @now_users.each do |name| %> <tr> <td> <%= employee_number %> </td> <td> <%= name %> </td> </tr> <% end %> <% end %> </tbody> </table> </div>

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

原因としては、on_dutyで生成した
@now_users
@now_users_employee_number
これらの配列にある値の数だけループをループで回しているのが原因かと。
※上記の例だと、2×2通り表示している

on_dutyで連想配列をつくってでループで回せば期待している結果が得られると思います。
下記ではとりあえず、viewの部分だけ変更

<% ary = [@now_users_employee_number, @now_users].transpose %> <tbody> <% ary.each do |key, value| %> <tr> <td> <%= key %> </td> <td> <%= value %> </td> </tr> <% end %> </tbody>

投稿2019/09/10 06:51

編集2019/09/10 06:53
退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ikt_erk122

2019/09/10 07:25

回答ありがとうございます。 重複せずに表示することができました。 丸投げのような質問をしてしまい申し訳ありませんでした。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問