优化触发I18n.l日期格式的条件

Question

型号TravelReport有两个属性：departure_date和arrival_date

助手arrival_and_departure_months(travel_report)产生以下字符串：

'February - July 2014'
'December 2013 - January 2014'
'March 2014'
''

取决于起始月和结束月是否在同一年。更糟糕的是，两者都可以为零，这会导致一个空字符串。

这是目前的方法：

def arrival_and_departure_months(travel_report)
  if travel_report.arrival_date && travel_report.departure_date
    output = I18n.l travel_report.arrival_date, :format => '%B'
    if travel_report.arrival_date.year != travel_report.departure_date.year
      output += I18n.l travel_report.arrival_date, :format => ' %Y'
    else
      if travel_report.arrival_date.month == travel_report.departure_date.month
        return I18n.l(travel_report.departure_date, :format => '%B %Y')
      end
    end
    output + I18n.l(travel_report.departure_date, :format => ' - %B %Y')
  else
    ''
  end
end

我们如何简化或优化代码？CCD_ 5和CCD_。

Answer 1

您可以通过使用正则表达式替换字符串中不需要的部分来消除一些if条件并获得更干净的代码：

def arrival_and_departure_months(travel_report)
  if travel_report.arrival_date && travel_report.departure_date
    output = I18n.l(travel_report.arrival_date, :format => '%B %Y') + 
             I18n.l(travel_report.departure_date, :format => ' - %B %Y')
    output = output.gsub(/([^ ]+ d{4}) - 1/, '1')
    output.gsub(/([^ ]+) (d{4}) - ([^ ]+) 2/, '1 - 3 2')
  else
    ''
  end
end

您可以在这里查看并使用正则表达式：

• http://rubular.com/r/mY3SiN7N14
• http://rubular.com/r/1p5P5BUCVs