Die Suche nach einer Primzahl

Ihnen gedruckte prime oder nicht prime, und fuhr Fort mit der Schleife, anstatt zu stoppen. Die %2 überprüfen ist nicht wirklich notwendig. Entsprechend modifiziert:

int a = Convert.ToInt32(number);

bool prime = true;
if (i == 1) prime = false;
for (int i = 2; prime && i < a; i++)
    if (a % i == 0) prime = false;
if (prime) Console.WriteLine("prime");
else Console.WriteLine("not prime");
Console.ReadLine();

Vermutlich dein code ausgeben, eine Vielzahl von Nachrichten, die ein bisschen Durcheinander und sinnlos? Es gibt 3 wichtige Fragen:

• Ihnen sorglosen brechen aus Ihrem for-Schleife wenn Sie sich entschieden haben, es kann nicht sein, prime

• Sind Sie der Annahme, dass es prime, wenn es vielleicht nicht sein, siehe die Kommentare im code unten.

• Sind Sie im Vergleich zu einem selbst, und das wird immer teilbar durch a, die <= in der for-Bedingung muss sein, <

Code:

int a = Convert.ToInt32(number);

if (a % 2 != 0)
{
    for (int i = 3 i < a; i += 2) //we can skip all the even numbers (minor optimization)
    {
        if (a % i == 0)
        {
            Console.WriteLine("not prime");
            goto escape; //we want to break out of this loop
        }
        //we know it isn't divisible by i or any primes smaller than i, but that doesn't mean it isn't divisible by something else bigger than i, so keep looping 
    }
    //checked ALL numbers, must be Prime
    Console.WriteLine("prime");
}
else
{
    Console.WriteLine("not prime");
}
escape:
Console.ReadLine();

Wie andere erwähnt haben, man konnte nur die Schleife, um die Quadratwurzel der eine, von pro-Bewertung der Quadratwurzel und ersetzen Sie diese Zeile:

for (int i = 3 i < a; i += 2)

mit dabei:

float sqrRoot = (Int)Math.Sqrt((float)a);
for (int i = 3 i <= sqrRoot; i += 2)

Ist es wichtig, pro-bewerten Sie Sie sonst Ihr Programm ausgeführt wird viel langsamer, anstatt schneller, als jede iteration umfasst einen square root operation.

Wenn Sie nicht wie springen Aussagen (ich Liebe goto-Anweisungen), schreiben Sie einen Kommentar und ich werde es zu ersetzen, wird ein breakout-boolean (oder siehe Dukeling mehr neue Antwort).

Doch eine andere optimierte Möglichkeit ist die Verwendung Sieb des Eratosthenes Algorithmus.

Vom Wikipedia

Finden Sie alle Primzahlen kleiner oder gleich einer gegebenen Ganzzahl n von Eratosthenes' Methode:

1. Erstellen Sie eine Liste von aufeinander folgenden ganzen zahlen von 2 bis n: (2, 3, 4, ..., n).

2. Zunächst sei p gleich 2, die erste Primzahl.

3. Ausgehend von p, zählen in Schritten von p und markieren Sie jede dieser zahlen größer als p selbst in der Liste. Diese werden ein Vielfaches von p: 2p, 3p, 4p, etc.; beachten Sie, dass einige von Ihnen möglicherweise bereits markiert.

4. Hier finden Sie die erste Zahl, die größer als p in der Liste, die nicht markiert ist. Wenn es keine derartige Zahl, zu stoppen. Andernfalls sei p nun gleich diese Nummer (das ist die nächste Primzahl), und wiederholen Sie das Verfahren ab Schritt 3.

When the algorithm terminates, all the numbers in the list that are not marked are prime.

C# - code

  int[] GetPrimes(int number) //input should be greater than 1
    {
        bool[] arr = new bool[number + 1];
        var listPrimes = new List<int>();

        for (int i = 2; i <= Math.Sqrt(number); i++)
        {
            if (!arr[i])
            {
                int squareI = i * i;
                for (int j = squareI; j <= number; j = j + i)
                {
                    arr[j] = true;
                }
            }

            for (int c = 1; c < number + 1; c++)
            {
                if (arr[c] == false)
                {
                    listPrimes.Add(c);
                }
            }

        }
        return listPrimes.ToArray();
    }