Rails loop through array don't show duplicate strings

Question

I am looping through an array of id's like so:

<% @tip.horse_ids.each do |i| %>
  <% @horse = Horse.find(i) %>
  <%= @horse.race.meeting.racecourse.strip %><br />
  <%= @horse.race.time.to_s(:time) %> - <%= @horse.name.strip %><br />
<% end %>

Which outputs something like this:

Kempton (AW)
01:30 - Light Gunner
Kempton (AW)
01:30 - Mesophere
Kempton (AW)
01:30 - Ragtag Rascal
Kempton (AW)
01:30 - Many Waters
Kempton (AW)
02:40 - Judicial
Stratford
01:40 - Rabunda
Stratford

I only want "Kempton (AW)" to appear once in the loop (the first one) and same for the time so "1:30" should only appear once too I want to remove all the duplicated content. Any help would be much appreciated, thanks!

Eyeslandic · Accepted Answer · 2017-04-10 19:04:26Z

1

Use uniq

@tip.horse_ids.uniq.each do |i|

or better yet

<% Horse.where(id: @tip.horse_ids).each do |horse| <%
  <%= horse.race.meeting.racecourse.strip %><br />
<% end %>

and even better, you should have access to horses if everything is set up correctly.

<% @tip.horses.each do |horse| <%
  <%= horse.race.meeting.racecourse.strip %><br />
<% end %>

answered Apr 10, 2017 at 19:04

Eyeslandic

14.9k13 gold badges44 silver badges55 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

coderhs · Accepted Answer · 2017-04-10 19:08:11Z

0

Adding a uniq to your array would solve your problem

<% @tip.horse_ids.uniq.each do |i| %>
  <% @horse = Horse.find(i) %>
  <%= @horse.race.meeting.racecourse.strip %><br />
  <%= @horse.race.time.to_s(:time) %> - <%= @horse.name.strip %><br />
<% end %>

answered Apr 10, 2017 at 19:08

coderhs

4,9021 gold badge19 silver badges26 bronze badges

Comments

fidler2326 · Accepted Answer · 2017-04-12 15:59:57Z

0

If anyone has a similar problem to this the way I fixed it was by using group_by

<% @tip.horses.group_by(&:meeting).each do |meeting, horses| %>
  <%= meeting.racecourse %><br />
  <% horses.each do |horse| %>
    <%= horse.race.race_time %> - <%= horse.name %><br />
  <% end %>
<% end %>

answered Apr 12, 2017 at 15:59

fidler2326

911 silver badge4 bronze badges

Collectives™ on Stack Overflow

Rails loop through array don't show duplicate strings

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related