How to print an HTML table using Ruby?

Question

This is my code:

<table class="video_table">
  <% count = 0 %>
  <tr>
  <% @f_videos.each do |f_video| %>
     <td><%= f_video.name %></td>
     <td><%= f_video.date_added %></td>
     <td><%= f_video.views %></td>
     <% count +=1 %>
     <% if count == 4 %>
         </tr>
     <% end %>
  <% end %>
</table>

at each 4 videos placed I want the table to switch row. So I implemented a counter. But it is not working. Any ideas?

Well, it'll only equal 4 once, since you don't reset it. Generally this type of task is done with the % (modulo) operator so it doesn't have to be reset. — Dave Newton
– Dave Newton, Commented Dec 10, 2011 at 17:29
Please indent your code properly, and please don't use "thx". — Andrew Grimm
– Andrew Grimm, Commented Dec 11, 2011 at 22:24

maček · Accepted Answer · 2011-12-10 17:39:49Z

2

Your count will only be set to 4 once.

Instead of if count == 4 use if count % 4 == 0

This will repeat the </tr> for each multiple of 4

Alternatively, you could skip using the count variable and use each_with_index to get the same result

<table class="video_table">
<% @f_videos.each_with_index do |f_video, i| %>
  <tr>
    <td><%= f_video.name %></td>
    <td><%= f_video.date_added %></td>
    <td><%= f_video.views %></td>
    <% if (i+1) % 4 == 0 %>
      </tr>
    <% end %>
<% end %>
</table>

Even better! each_slice

<table class="video_table">
<% @f_videos.each_slice(4).to_a do |slice| %>
  <tr>
    <% slice.each do |f_video| %>
      <td><%= f_video.name %></td>
      <td><%= f_video.date_added %></td>
      <td><%= f_video.views %></td>
    <% end %>
  </tr>
<% end %>
</table>

edited Dec 10, 2011 at 17:39

answered Dec 10, 2011 at 17:27

maček

78k37 gold badges172 silver badges200 bronze badges

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

3 Comments

Test Test Over a year ago

thx! my code had also another error, <tr> should be above <% @f_videos.each do |f_video| %>

maček Over a year ago

@TestTest see the each_slice example; I think this the best way to do this.

maček Over a year ago

Also, welcome to StackOverflow! If this answer worked for you, please mark it as accepted :)

Marek Příhoda · Accepted Answer · 2011-12-10 18:48:20Z

1

Another solution:

<table class="video_table">
<% @f_videos.in_groups_of(4) do |group| %>
  <tr>
    <% group.each do |f_video| %>
      <td><%= f_video.name %></td>
      <td><%= f_video.date_added %></td>
      <td><%= f_video.views %></td>
    <% end %>
  </tr>
<% end %>
</table>

in_groups_of has the added value/advantage that, when needed, it allows for padding any remaining slots with eg ' ', which can be very useful. See the docs.

edited Dec 10, 2011 at 18:48

answered Dec 10, 2011 at 18:25

Marek Příhoda

11.2k3 gold badges43 silver badges53 bronze badges

Collectives™ on Stack Overflow

How to print an HTML table using Ruby?

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related