Ответ 1
Ruby 1.9:
arr = ['x', 'o', 'x', '.', '.', 'o', 'x']
p arr.each_index.select{|i| arr[i] == 'x'} # =>[0, 2, 6]
Учитывая массив, как я могу найти все индексы элементов, которые соответствуют заданному условию?
Например, если у меня есть:
arr = ['x', 'o', 'x', '.', '.', 'o', 'x']
Чтобы найти все индексы, в которых элемент x
, я мог бы сделать:
arr.each_with_index.map { |a, i| a == 'x' ? i : nil }.compact # => [0, 2, 6]
или
(0..arr.size-1).select { |i| arr[i] == 'x' } # => [0, 2, 6]
Есть ли лучший способ достичь этого?
Ruby 1.9:
arr = ['x', 'o', 'x', '.', '.', 'o', 'x']
p arr.each_index.select{|i| arr[i] == 'x'} # =>[0, 2, 6]
Другой способ:
arr.size.times.select {|i| arr[i] == 'x'} # => [0, 2, 6]
EDIT:
Не уверен, что это даже необходимо, но вот они.
Ориентиры:
arr = 10000000.times.map{rand(1000)};
Benchmark.measure{arr.each_with_index.map { |a, i| a == 50 ? i : nil }.compact}
2.090000 0.120000 2.210000 ( 2.205431)
Benchmark.measure{(0..arr.size-1).select { |i| arr[i] == 50 }}
1.600000 0.000000 1.600000 ( 1.604543)
Benchmark.measure{arr.map.with_index {|a, i| a == 50 ? i : nil}.compact}
1.810000 0.020000 1.830000 ( 1.829151)
Benchmark.measure{arr.each_index.select{|i| arr[i] == 50}}
1.590000 0.000000 1.590000 ( 1.584074)
Benchmark.measure{arr.size.times.select {|i| arr[i] == 50}}
1.570000 0.000000 1.570000 ( 1.574474)
Небольшое улучшение по сравнению с вашей линией each_with_index.map
arr.map.with_index {|a, i| a == 'x' ? i : nil}.compact # => [0, 2, 6]
Эти методы немного длиннее, но вдвое быстрее
class Array
def find_each_index find
found, index, q = -1, -1, []
while found
found = self[index+1..-1].index(find)
if found
index = index + found + 1
q << index
end
end
q
end
end
arr = ['x', 'o', 'x', '.', '.', 'o', 'x']
p arr.find_each_index 'x'
# [0, 2, 6]
Здесь контрольный показатель AGS был взломан с помощью этого решения
arr = 10000000.times.map{rand(1000)};
puts Benchmark.measure{arr.each_with_index.map { |a, i| a == 50 ? i : nil }.compact}
puts Benchmark.measure{(0..arr.size-1).select { |i| arr[i] == 50 }}
puts Benchmark.measure{arr.map.with_index {|a, i| a == 50 ? i : nil}.compact}
puts Benchmark.measure{arr.each_index.select{|i| arr[i] == 50}}
puts Benchmark.measure{arr.size.times.select {|i| arr[i] == 50}}
puts Benchmark.measure{arr.find_each_index 50}
# 1.263000 0.031000 1.294000 ( 1.267073)
# 0.843000 0.000000 0.843000 ( 0.846048)
# 0.936000 0.015000 0.951000 ( 0.962055)
# 0.842000 0.000000 0.842000 ( 0.839048)
# 0.843000 0.000000 0.843000 ( 0.843048)
# 0.405000 0.000000 0.405000 ( 0.410024)
Не уверен, что вы считаете это улучшением или нет, но используя (map
+ compact
), поскольку фильтр чувствует себя очень неуклюжим для меня. Я бы использовал select
, так как для этого он нужен, а затем просто захватить часть результата, о котором я забочусь:
arr.each_with_index.select { |a,i| a == 'x' }.map &:last
Я определил Array#index_all
, который ведет себя как Array#index
, но возвращает все согласованные индексы. Этот метод может принимать аргумент и блокировать.
class Array
def index_all(obj = nil)
if obj || block_given?
proc = obj ? ->(i) { self[i] == obj } : ->(i) { yield self[i] }
self.each_index.select(&proc)
else
self.each
end
end
end
require 'test/unit'
class TestArray < Test::Unit::TestCase
def test_index_all
arr = ['x', 'o', 'x', '.', '.', 'o', 'x']
result = arr.index_all('x')
assert_equal [0, 2, 6], result
arr = [100, 200, 100, 300, 100, 400]
result = arr.index_all {|n| n <= 200 }
assert_equal [0, 1, 2, 4], result
end
end