Go и плохой алгоритм простых чисел - PullRequest
0 голосов
/ 09 января 2011

Я написал этот алгоритм просеивания простых чисел, и он не работает должным образом. Я не могу найти ошибку в самом алгоритме. Может ли кто-нибудь помочь мне?

Это то, что он должен напечатать:

[2 3 5 7 11 13 17 19 23 29]

По сравнению с тем, что он на самом деле печатает:

[3 5 7 11 13 17 19 23 25 29]

.

package main

import "fmt"

func main() {
 var primes = sieve(makeNumbers(29))
 fmt.Printf("%d\n", primes);
}

func makeNumbers(n int) []int {
 var numbers = make([]int, n - 1)
 for i := 0; i < len(numbers); i++ {
  numbers[i] = i + 2
 }
 return numbers
}

func sieve(numbers []int) []int {
 var numCopy = numbers
 var max = numbers[len(numbers)-1]
 var sievedNumbers = make([]int, 0)
 for i := 0; numCopy[i]*numCopy[i] <= max; i++ {
  for j := i; j < len(numCopy); j++ {
   if numCopy[j] % numCopy[i] != 0 || j == i {
    sievedNumbers = append(sievedNumbers, numCopy[j])
   }
  }
  numCopy = sievedNumbers
  sievedNumbers = make([]int, 0) 
 }
 return numCopy
}

1 Ответ

3 голосов
/ 09 января 2011

Должно быть "для j: = 0", а не "для j: = i".

...