これをシングルパスで行いますか？

Question

while(i < bArray.length) 
    if(aArray.include?(bArray[i]) == false) then 
    return false 
    end 
    i+=1 
end 

私は上記を書いていますが、私はそれが醜いと肥大していることがわかります。これを1つのステートメントで行うには、きれいにカットされた方法が必要です。

Answer 1

これは同等でなければなりません：あなたはaArrayかどうかを確認しようとしているように見えます

bArray.all? { |item| aArray.include?(item) } 

Answer 2

bArray.all?(&aArray.method(:include?)) 

Answer 3

：

while i < bArray.length 
    return false unless aArray.include?(bArray[i]) 

    i += 1 
end 

は、ここで私は同等であるべきだと思うワンライナーですbArrayのすべての要素が含まれています。

aArray = %w[a b c d] 
bArray = %w[a b c z] 

(bArray - aArray).empty? # => false 

は私が近い元のコードを見て、それは、論理爆弾ティッキングを持っている：

def cmp_array(aArray, bArray) 
    i = 0 
    while (i < bArray.length) 
    if (aArray.include?(bArray[i]) == false) then 
     return false 
    end 
    i += 1 
    end 
end 

def cmp_array2(a, b) 
    (b - a).empty? 
end 

cmp_array(%w[ a b c ], %w[ a b c ]) # => nil 
cmp_array(%w[ a b c ], %w[ c b a ]) # => nil 
cmp_array(%w[ a b c ], %w[ a b  ]) # => nil 
cmp_array(%w[ a b c ], %w[ a b c d ]) # => false 
cmp_array(%w[ a b ], %w[ a b c ]) # => false 

cmp_array2(%w[ a b c ], %w[ a b c ]) # => true 
cmp_array2(%w[ a b c ], %w[ c b a ]) # => true 
cmp_array2(%w[ a b c ], %w[ a b  ]) # => true 
cmp_array2(%w[ a b c ], %w[ a b c d ]) # => false 
cmp_array2(%w[ a b ], %w[ a b c ]) # => false 

私はそうRubyは初期化されていない変数不満i = 0初期化子を追加する必要がありました。

cmp_arrayは、nilまたはfalseのいずれかを返します。nil?またはfalse?の追加テストでは何か便利なことが行われます。

if (cmp_array(%w[ a b c ], %w[ a b c ]).nil?) ... 

または：

if (cmp_array(%w[ a b c ], %w[ a b c d ]).false?) ... 

に比べ：一貫して真/偽を返し

if (cmp_array2(%w[ a b c ], %w[ a b c ])) ... 

。