`reduce`にaccumulator引数を指定して` next`を使います

Question

警官があります：RuboCop::Cop::Lint::NextWithoutAccumulatorです。

この警官が何であるか説明できる人はいますか、どのようにコードを改善するのでしょうか？

読みやすく、効率が向上しますか？

github code。

# bad 
result = (1..4).reduce(0) do |acc, i| 
    next if i.odd? 
    acc + i 
end 

コンソールでこれをしようとした場合、あなたはnilオブジェクトのNoMethodError例外が発生します：

Answer 1

は、ドキュメントからのサンプルコードを考えてみましょう。これは、オブジェクトが指定されていない場合、nextはnilを返します。イテレータの場合はreturnとして扱うことができます。

reduce methodの場合、ブロックによって返される値が必要なため予期しない動作が発生することがあります。 iが奇数の場合、nextが評価され、結果としてnilがブロックされます。次のイテレータではaccはnilと等しく、整数を追加することはできません。この例では、最初の反復はi = 1、nextはブロックの結果としてacc〜nilに設定されています。

いくつかのケースでは、列挙可能な値を正しく得ることができますが、一般的にnextの値を指定する方が安全です。