Сравнение двух переменных

Question

Сравнение двух переменных

Am I comparing the pipe symbols |x| & |f| correctly below ?

| F | соответствует названию каталога | Х | соответствует имени файла

  Dir.foreach(@sortedFilesDir) do
  |f|
    @fileArray.each() {
      |x| 
      if(x.match(/^#{f}/))
        puts "match - "+x+","+f
      end
    }
  end

1

добавлено 28 Октябрь 2011 в 01:01 автор blue-sky отредактировано 28 Октябрь 2011 в 01:44

Просмотры: 3

источник

ro fr es pt de zh hi bn ar uk

2 ответы

Я внес несколько изменений:

I made the variable names more informative
I replaced the regular expression with String#start_with?
I made the code more functional-programming than imperative programming, by replacing two eachs with a map and a find.
I used "#{expression}" rather than "match - "+x+","+f

Однако я не проверял, что это работает.

directory_names = Dir.foreach(@sorted_files_dir)
matching_directory_names = @file_array.map do |filename|
  directory_names.find do |directory_name|
    filename.start_with?("/" + filename + "/")
  end
end
@file_array.zip(matching_directory_names) do |filename, matching_directory_name|
  next if matching_directory_name.nil?
  puts "match - #{filename},#{matching_directory_name}"
end

Надеюсь, что directory_names не будет вычисляться каждый раз, когда цикл запускается! Если это так, то вам может понадобиться directory_names = Dir.foreach (@sorted_files_dir) .to_a в начале скрипта.

1

добавлено 28 Октябрь 2011 в 01:42 автор Andrew Grimm

источник

автор Gazler · Answer 1 · 2011-10-28T01:05Z

В общем случае вы используете нотацию do | var | , если ваш блок больше 1 строки. Если это 1 строка, используйте синтаксис {| var |} .

Строчная интерполяция также предпочтительна для конкатенации, поскольку она использует меньше методов.

Другое соглашение состоит в том, что если ваш метод не принимает аргументы, не используйте скобки в конце. (Как вы делали в каждом из своих методов)

Also, in the ruby world, we usually underscore_our_variable_names instead of camelCasing them. The exception being Constants (including class names) e.g. MyClass or ThisIsOneOfMyConstants

 Dir.foreach(@sorted_files) do |f|
    @file_array.each do |x| 
      if(x.match(/^#{f}/))
        puts "match - #{x},#{f}"
      end
    end
  end

Вы используете их правильно, если ваша цель - перечислить файлы в указанной директории, соответствующие записи в вашем файле @file_array.