JOptionPane.showMessageDialog

[SiDeu]

Hallo an alle ich habe mal wieder ein Problem
Warum gibt System.out.println alles richtig aus und JOptionPane.showMessageDialog nicht ?

if (teiler == 2)
					System.out.println(zahl
							+ " ist eine Primzahl, da sie genau " + teiler
							+ " Teiler hat");
				JOptionPane.showMessageDialog(null, zahl
				+ " ist eine Primzahl, da sie genau " + teiler
				+ " Teiler hat");

 

[faetzminator]

Ohne Fehlermeldung oder unerwünschtes Verhalten zu nennen können wir nur in die Glaskugel schauen, und die ging bei mir gerade zu bruch...

Edit: wie wärs, wenn du noch eine Klammerung um den Block machst, falls das [c]if[/c] auch auf die JOptionPane zutreffen sollte. Man merke sich: Auch wenn in einem Block (if, Schleifen, ...) nur ein Statement folgt, immer {} verwenden.

 

[Trolllllll]

Mache mal einen Klammernblock um deine Anweisungsblock

ein if sollte immer so aussehen

if (bedingung){
   anweisung1;
   anweisung2;
   ...
}

[EDIT]Zu langsam[/EDIT]

 

[SiDeu]

Danke an euch
das war der Fehler

 

[SiDeu]

Aber jetzt noch etwas
Kann ich irgendwie die zahlen alle auf einmal anzeigen lassen und ich nicht für jede Primzahl Enter drücken muss?
Wenn ich alle Primzahlen zwischen 1 und 10 anzeigen lassen will geht es ja noch aber wenn ich alle Primzahlen zwischen 1 und 1000 anzeigen lassen will muss ich 1000 mal Enter drücken

 

[faetzminator]

Und schon wieder stellst du eine Frage, ohne jeglichen Code o.ä. zu posten...

 

[Trolllllll]

...Kann ich irgendwie die zahlen alle auf einmal anzeigen lassen und ich nicht für jede Primzahl Enter drücken muss?
...

Ja

 

[SiDeu]

Der selbe Code

if (teiler == 2) {
					System.out.println(zahl
							+ " ist eine Primzahl, da sie genau " + teiler
							+ " Teiler hat");
					JOptionPane.showMessageDialog(null, zahl
							+ " ist eine Primzahl, da sie genau " + teiler
							+ " Teiler hat");
					// else
					// System.out.println(zahl + " ist KEINE Primzahl");
				}

wenn ich es jetzt ausführe zeigt es bei System.out.println alles an, aber bei JOptionPane.showMessageDialog muss ich für jedes ergebnis die Enter Taste oder OK drücken

 

[faetzminator]

Danke, dieses Codesnippet kenn ich bereits. Und es sagt nichts darüber aus, was dein Programm macht. Da müsste mindestens irgendwo noch eine Schleife o.ä. sein.

 

[SiDeu]

Er gibt alle Primzahlen zwischen 2 Werten aus aber bei JOptionPane.showMessageDialog gibt er sie der Reihe nach hintereinander aus bei jedem betätigen der OK Taste die nächste Primzahl

for (zahl = erstezahl; zahl <= zweitezahl; zahl++) {
			if (zweitezahl == 1)
				System.out.println("");
			else {
				teiler = 0;
				for (zaehler = 1; zaehler <= zweitezahl; zaehler++)

				{
					zahl2 = zahl % zaehler;
					if (zahl2 == 0)
						teiler++;

				}
				if (teiler == 2) {
					System.out.println(zahl
							+ " ist eine Primzahl, da sie genau " + teiler
							+ " Teiler hat");
					JOptionPane.showMessageDialog(null, zahl
							+ " ist eine Primzahl, da sie genau " + teiler
							+ " Teiler hat");

 

[X5-599]

An der Stelle, wo du die JOptionPane aufmachst einfach die Primzahlen sammeln. Also z.B. in einer ArrayList. Dann nach ablauf der Schleife ausgeben z.B. indem du die ArrayList durchläufst und zu einem grossen String zusammen kettest.

 

[Trolllllll]

Ja

Wie schon geschrieben ja.

String primzahlen="";
for (zahl = erstezahl; zahl <= zweitezahl; zahl++) {
   if (zweitezahl == 1){
      System.out.println("");
   }else {
      teiler = 0;
      for (zaehler = 1; zaehler <= zweitezahl; zaehler++){
          zahl2 = zahl % zaehler;
            if (zahl2 == 0){
               teiler++;
            }
         }
         if (teiler == 2) {
            primzahlen = primzahl+zahl+";";
         }
      }
   }
System.out.println(primzahlen
                            + " sind Primzahl, da sie genau " + 2
                            + " Teiler hat");
JOptionPane.showMessageDialog(null, primzahlen
                            + " sind Primzahl, da sie genau " + 2
                            + " Teiler hat";

Ich habe auch mal deine ifs wie schonb beschrieben mit klammern geschrieben, sollte man immer machen !!!!

 

[faetzminator]

List<Integer> primZahlen = new ArrayList<Integer>();
// ...
if (teiler == 2) {
    primZahlen.add(zahl);
}
// ...
JOptionPane.showMessageDialog(... Arrays.toString(primZahlen.toArray()) ...);

?

 

[Trolllllll]

Ja so kann man es auch machen ;-)
Aber was spricht dagegen, es direkt in den String zu schreiben, ok, man könnte auch anstelle meines codes den StringBuilder nehmen, aber warum die Zahlen in eine Liste Speichern, hat das einen Vorteil, außer man will später damit weiterarbeiten, dann macht das natürlich Sinn ;-)

 

[faetzminator]

[...] außer man will später damit weiterarbeiten, dann macht das natürlich Sinn ;-)

Man will z.B. diese Liste mit den Primzahlen ausgeben :bae: ?

 

[Trolllllll]

Man will z.B. diese Liste mit den Primzahlen ausgeben :bae: ?

Das geht auch mit dem String :-D

 

[SiDeu]

Hallo faetzminator

kannst du mir das vieleicht mit JOptionPane.showMessageDialog(... Arrays.toString(primZahlen.toArray()) ...); genauer erklären?

ArrayList<Integer> primzahlen = new ArrayList<Integer>();

				if (teiler == 2) {

					primzahlen.add(zahl);

					
					
					System.out.println(primzahlen
							+ " ist eine Primzahl, da sie genau " + teiler
							+ " Teiler hat");
					JOptionPane.showMessageDialog(null, primzahlen
							+ " ist eine Primzahl, da sie genau " + teiler
							+ " Teiler hat");
					// else
					// System.out.println(zahl + " ist KEINE Primzahl");
				}
			}
		}

 

[Trolllllll]

Nimm meinen Code, den ich gepostet habe und ersetze mein primzahl String durch seinen geposteten Code und fertig, oder nimm meinen, wenn du den besser verstehst.

http://www.java-forum.org/java-basics-anfaenger-themen/143916-joptionpane-showmessagedialog.html#post959141

 

[SiDeu]

Super danke danke funktioniert nun!
Aber könntest du mir sagen wie du [Nimm meinen Code, den ich gepostet habe und ersetze mein primzahl String durch seinen geposteten Code und fertig] gemeint hast!

 

[faetzminator]

Halt irgendwas wie...

JOptionPane.showMessageDialog(null, "SiDeu sollte Java lernen und ich hab folgende Primzahlen gefunden: " + Arrays.toString(primZahlen.toArray()));

 

[SiDeu]

Danke funktioniert jetzt beides! :toll:
Ich bin dabei es zu lernen! :rtfm: