RuboCop错误：函数的分配分支条件大小太高。我该如何减少它？

Question

1.我刚刚开始在Ruby上工作，并开始使用Rubocop来遵循Ruby的一般指南，但是我遇到了分配分支条件错误，有人可以帮助我吗？

@months = [{1 =＆gt; 'jan'}，{2 =＆gt; 'feb'}，{3 =＆gt; 'Mar'}，{4 =＆gt; 'apr'}，{5 =＆gt; 'May'}，{6 =＆gt; 'jun'}，{7 =＆gt; 'jul'}，{8 =＆gt; 'Aug'}，{9 =＆gt; 'sep'}，{10 =＆gt; 'oct'}，{11 =＆gt; 'nov'}，{12 =＆gt; 'dec'}]]

   def calc_high_low_temp_of_month(date, address)

      high_temp_of_month = []
      min_temp_of_month = []
      max_humid_of_month = []

      @data_exists = false
      year, month = date.split('/')
      month = month.to_i
      month_hash = @months.detect { |x| x[month] }
      month = month_hash[month]
      begin
        arr = Dir.children(address)
      rescue Errno::ENOENT
        puts 'file not found!!!'
        return ''
      end
      arr.each do |elem|
         next unless elem.include?(year.to_s) && elem.include?(month.to_s)

        file_data = []
        File.foreach("#{address}/#{elem}") do |line|
          unless line.strip.empty?
            file_data.push(line)
            @data_exists = true
          end
        end

        splited_data = []
        splited_data = file_data[0].split ','
        index_of_high_temp = splited_data.find_index('Max TemperatureC')
        index_of_min_temp = splited_data.find_index('Min TemperatureC')
        index_of_humidity = splited_data.find_index('Max Humidity')

        (1...file_data.count).each do |index|
          splited_data = file_data[index].split ','
          if index_of_high_temp <= (splited_data.count - 1) && !splited_data[index_of_high_temp].empty?
            high_temp_of_month.push(splited_data[index_of_high_temp].to_i)
          end
          if index_of_min_temp <= (splited_data.count - 1) && !splited_data[index_of_min_temp].empty?
            min_temp_of_month.push(splited_data[index_of_min_temp].to_i)
          end
          next unless index_of_humidity <= (splited_data.count - 1)

          max_humid_of_month.push(splited_data[index_of_humidity].to_i) unless splited_data[index_of_humidity].empty?
        end
      end
      if @data_exists
        min_max_temp_of_month_hash = {}
        min_max_temp_of_month_hash['highTempOfMonth'] = high_temp_of_month
        min_max_temp_of_month_hash['minTempOfMonth'] = min_temp_of_month
        min_max_temp_of_month_hash['maxHumidOfMonth'] = max_humid_of_month
        min_max_temp_of_month_hash
      else
        ''
      end
    end

Answer 1

分配，分支和条件度量（ABC）计数分配，分支和条件，作为名称暗示。

如果您的作业，分支机构和条件太多，那么修复方法是减少至少一个

• 任务，
• 分支或
• 条件之一。

例如：

这里：

splited_data = []
splited_data = file_data[0].split ','

您分配splited_data，然后您立即覆盖该分配，而无需在介于两者之间做任何其他事情。这意味着第一个任务是完全没有用的，它根本没有任何作用，而您可以写作

splited_data = file_data[0].split ','

。

另一个例子：

min_max_temp_of_month_hash = {}
min_max_temp_of_month_hash['highTempOfMonth'] = high_temp_of_month
min_max_temp_of_month_hash['minTempOfMonth'] = min_temp_of_month
min_max_temp_of_month_hash['maxHumidOfMonth'] = max_humid_of_month

这4个作业可以很容易地写成没有作业：

{
  'highTempOfMonth' => high_temp_of_month,
  'minTempOfMonth' => min_temp_of_month,
  'maxHumidOfMonth' => max_humid_of_month
}

很难提供好的建议，因为您的代码非常复杂且令人费解。最好的方法是将代码分解为多个单独的方法。