Kod

Zakład patologii kodu
1

Część, mógłby mi ktoś podpowiedzieć jak nie robić takiego kodu?

[code]


<% @groups.each do |group| %>
<% if group.bookmarks.exists? %>

<%= group.name %>




    <% @bookmarks.each do |bookmark| %>
    <% if group.id == bookmark.group_id %>
  • <%= link_to bookmark.name, bookmark.url, :target => ‘_blank’ %></li
    <% end %>
    <% end %>


<% end %>
<% end %> 
	</div>[/code]

Bookmark - zakładka, ma pod sobą wiele kategorii (group).

[code]MODEL

class Group < ActiveRecord::Base
belongs_to :user
has_many :bookmarks
end

class Bookmark < ActiveRecord::Base
belongs_to :user
belongs_to :group
end

KONTROLER

@groups = Group.where("user_id = ?", current_user.id)
@bookmarks = Bookmark.where("user_id = ?", current_user.id)[/code]
2
  1. w kontrolerze pobrać tylko te grupy, które mają bookmarki.
  2. nie ustawiać zmiennej @bookmarks, a zrobić coś takiego:
  <%  @group.bookmarks.each do |bookmark| %>
  ...
  1. Przesiąść się na HAMLa :wink:
3

Dzięki,

Z HAMLa oczywiście korzystam, to tylko dla testów zrobiłem. :slight_smile:

4 
@groups = current_user.groups.includes(:bookmarks)