Ruby: inested if statements

Question

Я писал код, и это оказалось слишком уродливым по своему вкусу. В любом случае, я могу реорганизовать его так, чтобы я не использовал вложенные операторы if?

def hours_occupied(date) 
    #assuming date is a valid date object  
    availability = get_work_hours(date) 
    focus = "work" 

    if availability.nil 
    availability = get_family_hours(date) 
    focus = "family" 

    if availability.nil 
     availability = get_friend_hours(date) 
     focus = "friends" 
    end 
    end 
end 

Я знаю, что я буду в состоянии сделать что-то вроде этого дост

availability = get_work_hours(date) || get_family_hours(date) || get_friend_hours(date) 

но как установить переменную фокусировки соответственно?

Answer 1

Еще один способ, это просто переназначить значения, если есть необходимость:

def hours_occupied(date) 
    availability, focus = get_work_hours(date), "work" 
    availability, focus = get_family_hours(date), "family" unless availability 
    availability, focus = get_friend_hours(date), "friend" unless availability 
end 

или с использованием итератора:

def hours_occupied(date) 
    availability = focus = nil 
    %w(work family friend).each {|type| availability, focus = self.send(:"get_#{type}_hours", date), type unless availability} 
end 

Answer 2

Я хотел бы сделать что-то вроде следующего, как это ясно показывает, что каждый случай взаимоисключающими:

def hours_occupied(date) 
    if availability = get_work_hours(date) 
    focus = "work" 
    elsif availability = get_family_hours(date) 
    focus = "family" 
    elsif availability = get_friend_hours(date) 
    focus = "friends" 
    end 
end 

Answer 3

Я бы написать:

def hours_occupied(date) 
    focus = if (availability = get_work_hours(date)) 
    "work" 
    elsif (availability = get_family_hours(date)) 
    "family" 
    elsif (availability = get_friend_hours(date)) 
    "friends" 
    end 
    # I guess there is more code here that uses availability and focus. 
end 

Однако, я не уверен, имеющие разные методы для разных типов - хорошая идея, это делает код сложнее писать. Другой подход с использованием Enumerable#map_detect:

focus, availability = [:work, :family, :friends].map_detect do |type| 
    availability = get_hours(date, type) 
    availability ? [type, availability] : nil 
end 

Answer 4

случай, когда также вариант:

focus = case availability 
when get_work_hours(date) 
    "work" 
when get_family_hours(date) 
    "family" 
when get_friend_hours(date) 
    "friends" 
end