Hi
Ich habe ein Problem Mit meinem Primes Prog und zwar denkt es anscheinend boolean isPrime ist immer true ich habe bereits alle möglichen Ursachen Idiotensicher gemacht aber ich finde den Fehler nicht java Primes 10 20 Ausgabe:11 12 13 14 15 16 17 18 19.
Ich bitte euch das mal anzuschauen sollte ja nicht allzu schwer sein

Wollte noch dazu sagen dass ich das Prog wohl ziemlich verstümmelt habe Ausgabe untere_grenze anstatt toProof zB habe schon so ziemlich alles probiert ich kann keinen fehler erkennen. Ich hoffe es kann mir jemand helfen
Danke im voraus

Deine erste For-Schleife sieht etwas seltsam aus.

was soll die machen?

wie schon gesagt sieht alles ein bischen komisch aus sie soll einfach nur den zahlenintervall durchrattern die zweite ist fürs prüfen der einzelnen Zahlen zuständig Ursprünglich war in der ersten vor dem ersten Semikolon ein zaehler eingebaut

was hälste hiervon:

Code :

1 2 3 4 5 6 7 8 9 10 11 12 13 14

public boolean primtest(int n){             for (teiler=2;teiler*teiler <= n; teiler++)         {             if(n % teiler == 0){               // keine Primzahl              return true;            else {                 return false;             }         }         // keinen Teiler gefunden, also Primzahl         return true;     }

Code :

1 2 3 4 5 6 7 8

for(x=untere_grenze;x<=obere_grenze;x++){         if (primtest(x)){             ...     }         else {             ...     }     }

Weiß jetzt aber nich genau, ob's ...

Sieht gut aus die Optimierung teiler*teiler hat ich auch drin bzw wurzel toProof zunächst mal bin ich ein Neuling und mit OOP hab ich mich bis jetzt noch nicht groß beschäftigt also ich suche kein anderes Prog sondern den Fehler in meinem Prog ich kann ihn einfach nicht finden trozdem dank ich dir schonmal

ich glaube, es ligt an Variable Teiler, weil sie global gilt, nicht local. Ich guck aber noch

Danke für deine Antwort ich habe es mal ausprobiert also ich habe sie jezz ma in dem zweiten schleifenkopf definiert und geht leider immernoch nicht

e voila:

Code :

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34

class Primes {    public static void main (String[] args) {     int teiler = 2;     int untere_grenze = 1;//Integer.parseInt(args[0]);     int obere_grenze = 20;//Integer.parseInt(args[1]);     int toProof;     boolean isPrime = false;           for(toProof = untere_grenze; toProof < obere_grenze + 1; toProof++) {               for(teiler = 2; teiler < toProof; teiler++) {                     //if (toProof % teiler == 0) {                       // isPrime = false;                        //break;                        //} else {                            if(toProof % teiler != 0) {                                  isPrime = true;                            } else {                                   isPrime = false;                                   break;                            }                       //}                 }           if(isPrime == true)           System.out.println(toProof);           isPrime = false;       }      }  }

bissel Variablen umstellen und unnötiges entfernen und schon geht's. ist getestet.

Mhm also bei mir geht es nicht kannst du mir bitte mal das fertige Prog schicken also den Compilierfertigen code

Bei mir läuft's... ?(
Siehe anhang.

Danke Hammerhart ich versteh das nicht du hast die Abfrage umgedreht und ein wenig Variablen umgestellt aber sobald ich das in mein Prog übertrage geht es nicht was soll das das Prog raubt mir echt mein letzten nerv

hier mal mein Prog

Wenn ich mal kurz drauf gucke gibst du unten nur die untere Grenze aus .. obwohl du toProof ausgeben müsstest .. so wie ich das sehe ..

Übrigens heißt Proof im englischen "sicher" und nicht prüfen

Bsp: bulletproof

Zitat von teppi

Übrigens heißt Proof im englischen "sicher" und nicht prüfen

Bsp: bulletproof

Der Zollstock zeigt auch nur Zentimeter an ....