これは私にとって一般的で反復的なイディオムです。正規表現を使用して配列をフィルタリングし、サブ配列を返します。私のアプローチは、Rubyにあまり似ていません(Javaから来ました)。私はこのように見える多くのメソッドを持つことになります。
このコードを改善する慣用的なRuby方法は何ですか?
def get_all_gifs(items_)
output = Array.new
filter = /\.jpg$/
items_.each do |item|
next if item =~ filter
output << item
end
output
end
すべてのgifを検索する場合:
def get_all_gifs(files)
files.select{ |i| i[/\.gif$/] }
end
すべてのjpegを検索する場合:
def get_all_jpgs(files)
files.select{ |i| i[/\.jpe?g$/] }
end
それらを実行する:
files = %w[foo.gif bar.jpg foo.jpeg bar.gif]
get_all_gifs(files) # => ["foo.gif", "bar.gif"]
get_all_jpgs(files) # => ["bar.jpg", "foo.jpeg"]
ちょっと待って!さらにあります!
それらをすべてタイプごとにグループ化し、拡張子に基づいて抽出したい場合はどうしますか?:
def get_all_images_by_type(files)
files.group_by{ |f| File.extname(f) }
end
ファイルの種類は次のとおりです。
get_all_images_by_type(files).keys # => [".gif", ".jpg", ".jpeg"]
特定のタイプを取得する方法は次のとおりです。
get_all_images_by_type(files) # => {".gif"=>["foo.gif", "bar.gif"], ".jpg"=>["bar.jpg"], ".jpeg"=>["foo.jpeg"]}
get_all_images_by_type(files)['.gif'] # => ["foo.gif", "bar.gif"]
get_all_images_by_type(files).values_at('.jpg', '.jpeg') # => [["bar.jpg"], ["foo.jpeg"]]
Enumerable.grep をご覧ください。これは、列挙可能なあらゆるものを検索/フィルタリングする非常に強力な方法です。
$ cat foo.rb
images = %w[foo.gif foo.png foo.jpg bar.jpeg moo.JPG]
jpgs = images.select{|e| e =~ /\.jpe?g$/i}
puts jpgs.inspect
$ Ruby foo.rb
["foo.jpg", "bar.jpeg", "moo.JPG"]
正規表現の変更により、大文字と小文字を区別せずに「jpg」に加えて「jpeg」に一致させることができます。
images = %w[foo.gif foo.png foo.jpg bar.jpeg moo.JPG]
images.grep(/.jpg/i) # => ["foo.jpg", "moo.JPG"]
この特定の問題については、regexを使用しなくてもかまいません。つかいます - String#end_with?
images = %w[foo.gif foo.png foo.jpg bar.jpeg moo.JPG]
images.find_all{|e| e.end_with?(".jpg",".jpeg")}
# => ["foo.jpg", "bar.jpeg"]
ta, tb = files.partition{|f| f[/(.*\.jpe*g)/]}
ap ta
[
[0] "bar.jpg",
[1] "foo.jpeg"
]
ap tb
[
[0] "foo.gif",
[1] "bar.gif"
]
/ gucio