Range オブジェクトを #each や #map で回すときは無限ループに注意する

なんか特定の条件でサーバが重くなるなぁ、と思ったら無限ループでした。

注意すべきコード

以下のようなループは無限ループとなる可能性があるので注意

(0..max).each do |i|
  # 何か処理
end

なぜ?

Range クラスの仕様で終端が省略されている or nil の場合の Range オブジェクトは「終端なしのRangeオブジェクト」となる。

An “endless range” represents a semi-infinite range. Literal notation for an endless range is:

(1..)
# or similarly
(1...)

Which is equivalent to

(1..nil)  # or similarly (1...nil)
Range.new(1, nil) # or Range.new(1, nil, true)

よって、前述のコードでは max が nil となる場合、無限ループとなる。

じゃあどうしたら?

以下のように #upto() のループに置き換える

0.upto(max) do |i|
  # 何か処理
end

max に nil が入る余地を無くすというのも手ですが、そもそも無限ループしない書き方にしたい。

upto に nil を渡すと例外が発生するので、少なくとも無限ループすることはあり得ない。