c++ - 埃拉托色尼筛法 C++

Question

目前我正在做一个项目，我想计算所有素数。当我编译（MINGW Windows Comp.）时，程序崩溃并返回一个随机错误号。这是我写的代码：

http://pastebin.com/4vVnAM2v

/*
    Sieb des Eratosthenes
*/

#include <iostream>
#include <math.h>
using namespace std;

main()
{
    //variablendeklaration
    unsigned int nmax=100;
    unsigned int i,j,erg;
    bool prim[nmax];

    //Initialisieren
    prim[0]=false;
    prim[1]=false;

    //array prim[i] erstellen
    for(i=2;i<nmax;i++)
    {
        prim[i]=true;
    }




    for(i=2;i<nmax;i++) //alle Primzahlen durchlaufen
    {
        if(prim[i] == true) //auf Prim prüfen
        {
            for(j=2;j<nmax;j++) //multiplizieren und wegstreichen
            {
                erg = j * i;
                prim[erg] = false;
            }
        }
    }

    for(i=2;i<nmax;i++)
    {
        cout << prim[i] << endl;
    }


}

Answer 1

解决此问题的另一种方法是避免循环中的额外步骤：

for(i=2; i < nmax; i++)
{
    if(prim[i] == true)
    {
        for (j = 2 * i; j < nmax; j += i) // state j at 2i, increment by i
        {
            prim[j] = false;
        }
    }
}

这具有以下效果：1）不循环遍历nmax嵌套的项目（将整体复杂性从 O(n^2) 降低到 O(n * (n/2 + n/6 + n/10 + ...) ，这实际上是 O (n log n) 和 2) 不需要额外的边界检查，因为您在循环条件下永远不会越界。

Answer 2

在此刻：

            erg = j * i;
            prim[erg] = false;

您最终将访问超出 的范围prim，因为两者i和j的值都可以达到nmax - 1，因此erg最大值将为(nmax - 1) * (nmax - 1)。您需要检查这种情况并打破 if erg >= nmax，例如

            erg = j * i;
            if (erg < nmax)          // if i * j still within bounds
                prim[erg] = false;   // set prim[i * j] to false
            else                     // else i * j now >= nmax
                break;               // break out of loop and try next i value