Как предотвратить вывод дубликата записи в цикле?

Question

[Max]

Здравствуйте, у меня есть такой цикл:

<% @session.number_of_session_places.times do |session| %>
<% n = n+1 %>
  <div class="place_block" style=""><%= n %></div>
  <% @session.places.each do |p| %>
    <% if p.place_number == n %>
        <div class="place_block" style="background-color: black"><%= n %></div>
    <% end %>
  <% end %>
<% end %>

По понятным причинам он выводит сначала блок
<div class="place_block" style=""><%= n %></div>
А потом

<div class="place_block" style="background-color: black"><%= n %></div>

С одинаковым номером.
Остальные записи выводятся без дубликатов. Подскажите как решить эту проблему, не могу логически продумать решение.

Comments

[Fahrenhe17]

что-то я вообще не могу уловить логику циклов. Что они отображают и что выводят? Что имеется в виду под session.place?

[Fahrenhe17]

А вообще, следуя логике Вашего кода - нужно на второй строке объявить переменную как 0(или какое число там нужно), потом 3 строка, потом инкремент переменной

[Max]

Fahrenhe17: Есть сеанс в кинотеатре (@session), и есть количество мест в зале (number_of_session_places). Первый цикл выводит например 100 мест в зале. Далее есть таблица в БД (places), которая связана с таблицей сеансов (то есть в таблицу (places) добавляются только забронированные места, например 21 с указанием id сеанса). Выходит, что второй цикл проходит по всех забронированных местах для данного сеанса и выводит блок с черным фоном, если это место забронировано. Логика проста, если место забронировано - выводиться блок с стандартным фоном, иначе - блок с черным фоном.

[Fahrenhe17]

Max: Какая же странная структура) Сейчас подумаю.

[Fahrenhe17]

Max: а number_of_session_places это тоже строка в бд или метод модели?

[Max]

Fahrenhe17: строка из БД

[Max]

Fahrenhe17: Как придумал)

Answer 1

[Михаил Белошицкий]

Основная проблема с данным кодом - неправильно подобраны структуры данных. Поэтому у вас тут два цикла, хотя по логике вещей нужен всего-лишь один. Отталкиваясь от того, что уже есть, я бы сделал как-нибудь вот так:

<% reserved = Hash[ @session.places.map { |p| [p.place_number, p] } ] %>
<% @session.number_of_session_places.times do |session| %>
  <% n = n+1 %>
  <div style="<%= reserved.include? n ? 'background: black;' : '' %>">
    <%= n %>
  </div>
<% end %>

Но по-хорошему, вам нужно модели/контроллеры дорабатывать, в шаблонах кода

<% reserved = Hash[ @session.places.map { |p| [p.place_number, p] } ] %>

быть не должно.