Czy to jest DRYowa przesada?

Zakład patologii kodu
1

Czy to ma sens czy mnie poniosło?

[code]<% [“name”, “surname”, “email”, “twitter_id”, “facebook”, “website”, “phone_number”, “city”, “postal_code”].each do |attr_event| %>
<% if @event.send(attr_event.to_sym) != 0 %>

	<p><%= f.label attr_event.to_sym, attr_event.capitalize %><%= " - required" if @event.send(attr_event.to_sym) == 2 %><br/>
	<%= f.text_field attr_event %></p>
	
<% end -%>

<% end -%>[/code]

2

Poniosło Cię :slight_smile:

3

Pewnie przez EuRuKo :slight_smile:

4

Użyłeś kilka razy “end” i “if”, “f”.

5

[quote=pski][code] <% if @event.send(attr_event.to_sym) != 0 %>

<p><%= f.label attr_event.to_sym, attr_event.capitalize %><%= " - required" if @event.send(attr_event.to_sym) == 2 %><br/>
<%= f.text_field attr_event %></p>

<% end -%>[/code]
[/quote]
Jakbyś wrzucił ten fragment do helpera lub lepiej, napisał własny FormBuilder, to nie miałbym niczego przeciwko. :slight_smile: W tablicy zamiast stringów wrzuć symbole - będziesz mógł pozbyć się tych wszystkich to_sym.