Ruby conditional array

Question

In ruby what's a nice way of choosing which array to traverse conditionally?

If condition is true -> use array 1 if not use array B.

I tried the following but it didn't work.

   <% if  @post.active ?  %>
        <% Post::A_OPTIONS.each do |option|  %>
    <%else %>
        <% Post::B_OPTIONS.each do |option| %>
    <%end%>
      <br><%= radio_button_tag 'option', option, @option == option %>
      <%= option.humanize %>
    <% end %>

fengd · Accepted Answer · 2014-01-28 09:12:47Z

4

You should put your business logic into your model. such as

class Post
  def options
    active ? Post::A_OPTIONS : Post::B_OPTIONS
  end
end

then in your view,

<% @post.options.each do |option| %>
  <%= radio_button_tag 'option', option, @option == option %>
  <%= option.humanize %>
<% end %>

In this case, your view is isolated from how the options are generated, and both options logic and view logic are simple and clean

edited Jan 28, 2014 at 9:12

answered Jan 26, 2014 at 15:54

fengd

7,5793 gold badges44 silver badges44 bronze badges

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

2 Comments

Spone Over a year ago

As you are in the Post class, you can simplify with: self.active ? A_OPTIONS : B_OPTIONS

Leo Correa Over a year ago

You shouldn't need the self either.

Spone · Accepted Answer · 2014-01-26 15:43:22Z

1

What about this?

<% (@post.active ? Post::A_OPTIONS : Post::B_OPTIONS).each do |option|  %>
  <br><%= radio_button_tag 'option', option, @option == option %>
  <%= option.humanize %>
<% end %>

answered Jan 26, 2014 at 15:43

Spone

1,3691 gold badge9 silver badges20 bronze badges

Comments

Patrick Oscity · Accepted Answer · 2014-01-26 15:55:16Z

0

How about you define a method on the Post model that simply returns the correct options? e.g.

class Post < ActiveRecord::Base
  A_OPTIONS = [...];
  B_OPTIONS = [...];

  # other stuff

  def options
    if active
      A_OPTIONS
    else
      B_OPTIONS
    end
  end
end

And then your view will be dead simple:

<% @post.options.each do |option|  %>
  <br><%= radio_button_tag 'option', option, @option == option %>
  <%= option.humanize %>
<% end %>

answered Jan 26, 2014 at 15:55

Patrick Oscity

54.9k19 gold badges149 silver badges173 bronze badges

Collectives™ on Stack Overflow

Ruby conditional array

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related