train_refactoring.rb

train_path.each do |obj|
  result << obj.obj_id
end if train_path.any?

#
# 1. Если ты вызываешь метод .each, то как минимум переменная
# должна иметь множественное число, это ведь коллекция (массив),
# т.е. это train_paths
#
# 2. Если массив пуст, то метод .each сразу вернет пустой массив
# и if не нужно вызывать, чтобы узнать пустой ли массив
#
# train_paths = []
# train_paths.each do |train|
#   raise Exception
# end
# 
# => []
#
# 3. Чтобы собрать какой-то параметр у коллекции объектов, можно юзать inject.
# Тогда не нужно объявлять временную переменную:
# 
# [1,2,3].inject(0) { |n, sum| sum += n }
# => 6
#

train_paths.inject do |train, ids|
  ids += train.obj_id
  ids
end