Избегайте совершать несколько вызовов Find.find ("./") в Ruby

Question

Я не уверен, что является лучшей стратегией для этого. У меня есть класс, где я могу искать файловую систему для определенного шаблона файлов. Я хочу выполнить Find.find ("./") только один раз. как бы я подойти к этому:

def files_pattern(pattern) 
    Find.find("./") do |f| 
     if f.include? pattern 
      @fs << f 
     end 
    end 
    end 

Answer 1

Запоминание (обычно вычислительно интенсивный) результат вызова метода, так что вам не нужно пересчитывать его в следующий раз, когда вызывается метод известен как memoization, поэтому вы, вероятно, захотите больше узнать об этом.

Один из способов достижения этой цели Ruby - использовать небольшой класс-оболочку, который сохраняет результат в переменной экземпляра. например

class Finder 
    def initialize(pattern) 
    @pattern = pattern 
    end 

    def matches 
    @matches ||= find_matches 
    end 

    private 

    def find_matches 
    fs = [] 
    Find.find("./") do |f| 
     if f.include? @pattern 
     fs << f 
     end 
    end 
    fs 
    end 
end 

И тогда вы можете сделать:

irb(main):089:0> f = Finder.new 'xml' 
=> #<Finder:0x2cfc568 @pattern="xml"> 
irb(main):090:0> f.matches 
find_matches 
=> ["./example.xml"] 
irb(main):091:0> f.matches # won't result in call to find_matches 
=> ["./example.xml"] 

---

Примечание: оператор ||= выполняет задание, только если переменная на левой стороне делает принимает значение ложь. то есть @matches ||= find_matches является сокращением для @matches = @matches || find_matches, где find_matches будет называться только в первый раз из-за оценки короткого замыкания. Есть много other questions, объясняя это на Stackoverflow.

---

Незначительное изменение: Вы можете изменить способ, чтобы вернуть список всех файлов, а затем использовать методы из Enumerable таких как grep и select выполнять несколько запросов с тем же списком файлов. Конечно, это имеет недостаток в сохранении всего списка файлов в памяти. Вот пример, хотя:

def find_all 
    fs = [] 
    Find.find("./") do |f| 
    fs << f 
    end 
    fs 
end 

И затем использовать его как:

files = find_all 
files.grep /\.xml/ 
files.select { |f| f.include? '.cpp' } 
# etc 

Answer 2

как о system "find/-name #{my_pattern}"

Answer 3

Если я правильно понимаю ваш вопрос вы хотите запустить Find.find, чтобы присвоить результат переменной экземпляра. Вы можете переместить то, что теперь является блоком для отдельного метода, и вызвать это, чтобы возвращать только файлы, соответствующие вашему шаблону.

Проблема только в том, что если каталог содержит много файлов, вы храните большой массив в памяти.