[ruby-dev:36128] Re: Refactoring of enumerating prime numbers

けいじゅ＠いしつかです.

In [ruby-dev :36067 ] the message: "[ruby-dev:36067] Re: Refactoring
of enumerating prime numbers ", on Sep/01 00:07(JST) "Yugui (Yuki
Sonoda)" writes:

>Yuguiです。

どもです.

>ありがとうございます。コミットしようかと思いましたがテストを書いていたら
>2点問題に気づきました。結果としてちょっと変更が必要になりそうですので、
>恐れ入りますが再度確認をお願いします。

はい.

>先にお送りしたprime.rbでは2点問題がありました。
>1. Prime.eachが返すenumerator(実際にはgenerator)が、uboundを記憶しない。
>   Prime.each(100){|p| }は停止するのに、Prime.each(100).each{|p| }は停止
>しないという奇妙な結果になります。

これは, まずいですね.

>2. Prime#nextがない。
>  互換性にこだわって見せた割にうっかりPrime#nextの実装を書き忘れていました。
>  また、先のバージョンを元に単純にPrime#nextを実装するとPrime#nextによっ
>て進んだ内部ポインタとPrime#eachが同期しません。Prime#eachは先立つ
>Prime#nextに関わらず2から列挙を開始してしまいます。
>  一方、Prime::eachやPrime::instance.eachのほうは先立つPrimeのインスタン
>スメソッド呼び出しには影響されずに2から列挙することを期待されると思い
>ます。

うーん.  言われていることは分かりますが, eachとかコードが非常にわかり
ずらいですねぇ(^^;; 

ちなみに確認ですが, 新版の方は, Prime.succ は当然必要ないわけですね?
古い版では, Prime#eachはunboound, generatorの指定とかいらないですよね?

なら, こんなのはいかがです? 
これなら, 昔のコンパチビリティが必要なくなればすぐ削除できます.

よければ, 

  OldCompatibility

を Primeクラスの最後にもっていっていただ, 豊福さんのを反映してコミット
していただければと思います.


--- prime.rb	2008-09-03 11:01:29.000000000 +0900
+++ prime2.rb	2008-09-03 20:41:10.000000000 +0900
@@ -41,9 +41,6 @@
 class Prime
   include Enumerable
 
-  def initialize # :nodoc:
-    @generator = nil
-  end
   @the_instance = Prime.new
 
   # obsolete.
@@ -51,13 +48,22 @@
   # Use +Prime::instance+ or class methods of +Prime+.
   def initialize
     @generator = EratosthenesGenerator.new
+    extend OldCompatibility
     warn "Prime::new is obsolete. use Prime::instance or class methods of Prime."
   end
 
-  def succ
-    @generator.succ
+  module OldCompatibility
+    def succ
+      @generator.succ
+    end
+    alias next succ
+
+    def each(&block)
+      loop do
+	yield succ
+      end
+    end
   end
-  alias next succ
 
   class<<self
     extend Forwardable
@@ -71,16 +77,11 @@
   end
 
   # iterates the given block over all prime numbers.
-  def each(ubound = nil, generator = nil, &block)
-    generator ||= (@generator || EratosthenesGenerator.new)
-    orig_ubound = generator.upper_bound
+  def each(ubound = nil, generator = EratosthenesGenerator.new, &block)
     generator.upper_bound = ubound
     generator.each(&block)
-  ensure
-    generator.upper_bound = orig_ubound
   end
 
-
   # returns true if +value+ is prime, false for a composite.
   #
   # +value+:: an arbitrary integer.

__
---------------------------------------------------->> 石塚 圭樹 <<---
---------------------------------->> e-mail: [e-mail] <<---