如何使这些“ if”语句更干燥？

Question

I'm trying to layout some partials as shown below:

if controller == home
   render partail 0
else
   render partial 1
end

render partial 2

render partial 3

if controller == home
   render partail 4
end

render partial 5

I've repeated the if controller == home part three times. Is there any way to make the code follow the "DRY" approach?

Answer 1

You could use an array:

partials = (controller == home) ? [0,2,3,4,5] : [1,2,3,5]

partials.each { |p| render partial p }

Answer 2

Here's how I might do it. It's not any more DRY, but it reads better, IMO:

flag = controller == home

render flag ? :partial0 : :partial1
render :partial2
render :partial3
render :partial4 if flag
render :partial5

Answer 3

You can use an array and since the partials are ordered, sort it, then render:

partials = [2,3,5];                     # 2,3,5 are going to be rendered regardless of controller

case controller
when home
   partials.push(0,4);                  # if it's home use 0 and 4
else
   partials.push(1);                    # if not home use 1
end

partials.sort.each { |p| render partial p }  # sort and render in order

Note: there is more code in this example, but it is more flexible if you expect to use other conditionals in addition to home . I often find the more robust I make my code up front, it saves me more time in the long run; though, I admit this is less readable as compared to FMc 's answer