質問

このアルゴリズムのRubyバージョンをネットからスクレイピングするのではなく、その説明に基づいて独自に作成したかったここ。ただし、2つのことがわかりません

def primeSieve(n)
  primes = Array.new

  for i in 0..n-2
   primes[i] = i+2
  end

  index = 0
  while Math.sqrt(primes.last).ceil > primes[index]
    (primes[index] ** 2).step(primes.length - 1, primes[index]) 
      {|x| x % primes[index] == 0 ? primes.delete(x) : ""}
    index += 1
  end

  primes
end
  1. なぜ配列の最後まで反復しないのですか?
  2. 上記のリンクの説明によると、配列の最後の要素の平方根が現在の素数よりも大きい場合、ループを中断する必要があります-私は前にこれを行います。

削除操作が配列の長さを変更することと関係があると確信しています。たとえば、n = 10を入力すると、私の関数は現在2,3,5,7,9,10を生成しますが、これは明らかに正しくありません。想定どおりに動作するようにこれを変更する方法についての提案はありますか?

役に立ちましたか?

解決

次のように動作するようです。浮動小数点演算を取り出し、平方根ではなく平方しました。また、削除ループを「select」に置き換えました。呼び出します。

while primes[index]**2 <= primes.last
      prime = primes[index]
      primes = primes.select { |x| x == prime || x%prime != 0 }
      index += 1
end

編集:どうやってこれをやろうとしているのかがわかったと思います。以下はうまくいくようで、元のアプローチに沿っているようです。

while Math.sqrt(primes.last).ceil >= primes[index]
    (primes[index] * 2).step(primes.last, primes[index]) do
      |x|
      primes.delete(x)
    end
    index += 1
end

他のヒント

www.scriptol.org には、より高速な実装があります:

def sieve_upto(top)
  sieve = []
  for i in 2 .. top
    sieve[i] = i
  end
  for i in 2 .. Math.sqrt(top)
    next unless sieve[i]
    (i*i).step(top, i) do |j|
      sieve[j] = nil
    end
  end
  sieve.compact
end

わずかに改善できると思います:

def better_sieve_upto(n)
  s = (0..n).to_a
  s[0] = s[1] = nil
  s.each do |p|
    next unless p
    break if p * p > n
    (p*p).step(n, p) { |m| s[m] = nil }
  end
  s.compact
end

...主に配列の初期化が速いためだと思いますが、それはわずかです。 (不要な nil sを排除するために、両方に #compact を追加しました)

これは、ビット配列。

#!/usr/bin/env ruby -w

require 'rubygems'
require 'bitarray'

def eratosthenes(n)

   a = BitArray.new(n+1)

   (4..n).step(2) { |i|
      a[i] = 1
   }

   (3..(Math.sqrt(n))).each { |i|
       if(a[i] == 0)
           ((i*i)..n).step(2*i) { |j|
               a[j] = 1
           }
       end
   }
   a
 end

def primes(n)
    primes = Array.new
     eratosthenes(n).each_with_index { |isPrime, idx|
        primes << idx if isPrime == 0
     }
     primes[2..-1]
end

これは、興味のある人向けのリファレンスです。コードはこのサイトからのものです。

このコードは、Sieve of Eratosthenesも使用しています。

n = 1000000
ns = (n**0.5).to_i + 1
is_prime = [false, false] + [true]*(n-1)
2.upto(ns) do |i|
  next if !is_prime[i]
  (i*i).step(n, i) do |j|
    is_prime[j] = false
  end
end

count = 0
list = (0..n).map do |i|
  count += 1 if is_prime[i]
  count
end

while gets
  puts list[

これは、興味のある人向けのリファレンスです。コードはこのサイトからのものです。

このコードは、Sieve of Eratosthenesも使用しています。

def eratosthenes(n)
  nums = [nil, nil, *2..n]
  (2..Math.sqrt(n)).each do |i|
    (i**2..n).step(i){|m| nums[m] = nil}  if nums[i]
  end
  nums.compact
end

p eratosthenes(100)

そして、別のです。

<*>.to_i] end

そして、別のです。

<*>

または

x = []
Prime.each(123) do |p|
  x << p
end

ここでインジェクトを使用する方法があるかもしれませんが、今日の開始の事は私の頭を痛めます。

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top