Неявные возвращаемые значения в Ruby

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

Я работаю над небольшой программой для grep-журналов Tomcat и сгенерировал CSV файлы с разделителями по каналам из соответствующих данных. Вот упрощенный пример, который я использую для генерации строк из записи журнала.

class LineMatcher
  class << self
    def match(line, regex)
      output = ""
      line.scan(regex).each do |matched|
        output << matched.join("|") << "\n"
      end
      return output
    end        
  end
end


puts LineMatcher.match("00:00:13,207 06/18 INFO  stateLogger - TerminationRequest[[email protected][accountNumber=0951714636005,srNumber=20]",
                       /^(\d{2}:\d{2}:\d{2},\d{3}).*?(\d{2}\/\d{2}).*?\[accountNumber=(\d*?),srNumber=(\d*?)\]/)

Когда я запускаю этот код, я возвращаю следующее: это то, что ожидается при явном возврате значения вывода.

00:00:13,207|06/18|0951714636005|20

Однако, если я изменяю LineMatcher на следующее и явно не возвращаю вывод:

    class LineMatcher
      class << self
        def match(line, regex)
          output = ""
          line.scan(regex).each do |matched|
            output << matched.join("|") << "\n"
          end
        end        
      end
    end

Тогда я получаю следующий результат:

00:00:13,207
06/18
0951714636005
20

Очевидно, что это не желаемый результат. Мне кажется, что я смогу избавиться от выходной переменной, но неясно, откуда возвращается возвращаемое значение. Кроме того, любые другие предложения/улучшения для удобочитаемости приветствуются.

Ответ 1

Любой оператор в ruby возвращает значение последнего оцениваемого выражения. Вам нужно знать реализацию и поведение самого используемого метода, чтобы точно знать, как будет действовать ваша программа.

#each возвращает собранную вами коллекцию. При этом следующий код вернет значение line.scan(regexp).

line.scan(regex).each do |matched|
  output << matched.join("|") << "\n"
end

Если вы хотите вернуть результат выполнения, вы можете использовать map, который работает как each, но возвращает измененную коллекцию.

class LineMatcher
  class << self
    def match(line, regex)
      line.scan(regex).map do |matched|
        matched.join("|")
      end.join("\n") # remember the final join
    end        
  end
end

Существует несколько полезных методов, которые вы можете использовать в зависимости от вашего конкретного случая. В этом случае вы можете использовать inject, если количество результатов, возвращаемых scan, велико (работа над массивами, тогда их объединение более эффективно, чем работа с одной строкой).

class LineMatcher
  class << self
    def match(line, regex)
      line.scan(regex).inject("") do |output, matched|
        output << matched.join("|") << "\n"
      end
    end        
  end
end

Ответ 2

В рубине возвращаемое значение метода - это значение, возвращаемое последним оператором. Вы также можете выбрать явный возврат.

В вашем примере первый фрагмент возвращает строку output. Однако второй фрагмент возвращает значение, возвращаемое методом each (который теперь является последним stmt), который оказывается массивом совпадений.

irb(main):014:0> "StackOverflow Meta".scan(/[aeiou]\w/).each do |match|
irb(main):015:1* s << match
irb(main):016:1> end
=> ["ac", "er", "ow", "et"]

Обновление: Однако это не объясняет ваш вывод в одной строке. Я думаю, что это ошибка форматирования, она должна печатать каждое из совпадений в другой строке, потому что, как puts печатает массив. Маленький код может объяснить это лучше меня..

irb(main):003:0> one_to_three = (1..3).to_a
=> [1, 2, 3]
irb(main):004:0> puts one_to_three
1
2
3
=> nil

Лично я нахожу ваш метод с явным возвратом более читаемым (в данном случае)