質問
このアルゴリズムの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
- なぜ配列の最後まで反復しないのですか?
- 上記のリンクの説明によると、配列の最後の要素の平方根が現在の素数よりも大きい場合、ループを中断する必要があります-私は前にこれを行います。
削除操作が配列の長さを変更することと関係があると確信しています。たとえば、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
ここでインジェクトを使用する方法があるかもしれませんが、今日の開始の事は私の頭を痛めます。
所属していません StackOverflow