Methode in JTextArea anzeigen lassen

[xNk]

Mein Problem sollte einfach zu lösen sein, aber ich komme selber nicht auf die Lösung und bin zu unfähig die Lösung hier im Forum zu finden.

Der Code:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class GUI extends JFrame
                    implements ActionListener
{
    JLabel lb1;
    JButton bt1;
    JTextArea ta1;
    JScrollPane sp1;
    
    public GUI()
    {
        super("  Primzahlen-Simulator 2012");
        setLayout(null);
        setLocation(300,200);
        setSize(420,650);
        lb1 = new JLabel("            Drück den Button, wenn du die Primzahlen haben willst!" );
        add(lb1);
        lb1.setBounds(10,20,400,20);

        ta1 = new JTextArea("",380,450);
        add(sp1 =new JScrollPane(ta1));
        sp1.setBounds(10,60,400,500);
        bt1 = new JButton("Ich will Primzahlen!");
        add(bt1);
        bt1.addActionListener(this); //Regristrieren
        bt1.setBounds(55,580,300,20);
        setVisible(true);
    }
    
    public void actionPerformed(ActionEvent e) {
        if (e.getSource()==bt1) {
            prim(20);
        }
    }


    public void prim(int n)
    { // alle Primzahlen von 1 bis n
        int nr=0;
        for (int zahl = 2;zahl<=n;zahl++) {
            boolean ist_prim = true;
            for (int j=2;j<=zahl/2;j++) {
                if (zahl % j == 0) { // Rest Div
                    ist_prim=false;
                }
            }
            if (ist_prim) {
                nr = nr + 1;
            }
        }
    }


}

Jetzt möchte ich die Methode "prim" (Zeile 35) im JFrame, genauer im JTextArea anzeigen lassen, also nciht extern in einer Console. Wie mache ich das? :bahnhof:

Danke!

 

[Mujahiddin]

public void prim(int n)
    { // alle Primzahlen von 1 bis n
        String primzahlen = "";
        int nr=0;
        for (int zahl = 2;zahl<=n;zahl++) {
            boolean ist_prim = true;
            for (int j=2;j<=zahl/2;j++) {
                if (zahl % j == 0) { // Rest Div
                    ist_prim=false;
                }
            }
            if (ist_prim) {
                nr = nr + 1;
                primzahlen += nr + ", ";
            }
        }
        lb1.setText(primzahlen);
    }

 

[xNk]

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
public class GUI extends JFrame
                    implements ActionListener
{
    JLabel lb1;
    JButton bt1;
    JTextArea ta1;
    JScrollPane sp1;
    
    public GUI()
    {
        super("  Primzahlen-Simulator 2012");
        setLayout(null);
        setLocation(800, 300); 
        setSize(420,650);
        lb1 = new JLabel("            Drück den Button, wenn du die Primzahlen haben willst!" );
        add(lb1);
        lb1.setBounds(10,20,400,20);
        ta1 = new JTextArea("",380,450);
        add(sp1 =new JScrollPane(ta1));
        sp1.setBounds(10,60,400,500);
        bt1 = new JButton("Ich will Primzahlen!");
        add(bt1);
        bt1.addActionListener(this); //Regristrieren
        bt1.setBounds(55,580,300,20);
        setVisible(true);
    }
    
    public void actionPerformed(ActionEvent e) {
        if (e.getSource()==bt1) {
            prim(20);
        }
    }
 
    public void prim(int n)
    { // alle Primzahlen von 1 bis n
        String primzahlen = "";
        int nr=0;
        for (int zahl = 2;zahl<=n;zahl++) {
            boolean ist_prim = true;
            for (int j=2;j<=zahl/2;j++) {
                if (zahl % j == 0) { // Rest Div
                    ist_prim=false;
                }
            }
            if (ist_prim) {
                nr = nr + 1;
                //primzahlen += nr;
            }
        }
            ta1.setText(primzahlen);
    }
 
}

Das ist jetzt der Code, allerdings wird mir in der JTextArea nicht angezeigt. Danke für die schnelle Hilfe.

 

[Mujahiddin]

Wenn du

primzahlen += nr + ", ";

auskommentierst, kein Wunder.

EDIT:

primzahlen+=nr+",";

muss eine Zeile drüber...

 

[hüteüberhüte]

[...]

GUI updates immer nur im EDT

@xNk: Entweder Observable erweitern oder Ergebnis zurückgeben lassen

 

[Mujahiddin]

GUI updates immer nur im EDT

@xNk: Entweder Observable erweitern oder Ergebnis zurückgeben lassen

Ist im EDT, wenn "actionPerformed" ausgeführt wird.

Übrigens, die Methode muss so sein:

public void prim(int n) { // alle Primzahlen von 1 bis n
		String primzahlen = "";
		int nr = 0;
		for( int zahl = 2; zahl <= n; zahl++ ) {
			boolean ist_prim = true;
			for( int j = 2; j <= zahl / 2; j++ ) {
				if( zahl % j == 0 ) { // Rest Div
					ist_prim = false;
				}
			}
			System.out.println( zahl + " == prim == " + ist_prim );
			if( ist_prim ) {
				primzahlen += zahl + ", ";
				nr++;
			}
		}
		ta1.setText( primzahlen );
	}

 

[xNk]

Oh, sry. Hab den flaschen Code kopiert. Es wird mir in der JTextArea aber trotzdem nur

1, 2, 3, 4, 5, 6, 7, 8

angezeigt. Verstehe ich nicht..

 

[Mujahiddin]

Oh, sry. Hab den flaschen Code kopiert. Es wird mir in der JTextArea aber trotzdem nur

1, 2, 3, 4, 5, 6, 7, 8

angezeigt. Verstehe ich nicht..

weil die Zeile

primzahlen += nr

war, muss

primzahlen += zahl

sein, mein Fehler.

 

[xNk]

Klappt nicht so, wie es eig soll, aber danke!

 

[Mujahiddin]

Deine Anforderung war lediglich, dass die Primzahlen aufs TextArea kommen.
Wenn du was anderes willst, musst du dich genauer ausdrücken und spezifischer werden.

 

[hüteüberhüte]

Mujahiddin & xNk: Vielleicht etwas zu viel des Guten, aber ich würde das immer so machen:

import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;
import javax.swing.WindowConstants;

/**
 * @author hüte
 */
public class PrimTest {

    private JFrame frame = new JFrame("PrimTest");
    private JScrollPane scroll = new JScrollPane();
    private JTextArea area = new JTextArea();
    private JMenuBar bar = new JMenuBar();
    private JMenuItem item = new JMenuItem("berechne...");

    public PrimTest() {
        scroll.setViewportView(area);
        scroll.setPreferredSize(new Dimension(200, 400));
        frame.add(scroll);
        bar.add(item);
        frame.setJMenuBar(bar);

        item.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                berechne(Integer.parseInt(JOptionPane.showInputDialog("n:")));
            }
        });

        frame.pack();
        frame.setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
        frame.setVisible(true);
    }

    private void berechne(int n) {
        final StringBuilder builder = new StringBuilder();
        a:
        for (int i = 2; i <= n; i++) {
            int max = (int) Math.sqrt(i); // round?
            for (int j = 2; j <= max; j++) {
                if (i % j == 0) {
                    continue a;
                }
            }
            builder.append(i);
            builder.append('\n');
        }
        SwingUtilities.invokeLater(new Runnable() {

            @Override
            public void run() {
                area.append(builder.toString());
            }
        });
    }

    public static void main(String[] args) {
        new PrimTest();
    }
}

 

[hüteüberhüte]

Ach so, Primzahlen beginnen bei 2. Ist es denn jetzt überflüssig, invokeLater aufzurufen? (Brauche erst mal 'ne' Kaffee^^)

 

[Spacerat]

Ihr habt ohnehin 'ne merkwürdige Art, Primzahlen zu berechnen:

	public static boolean isPrime(long value)
	{
		value = abs(value);
		if(value == 2) {
			return true;
		}
		if(value < 2 || value % 2 == 0) {
			return false;
		}
		long max = (long) sqrt(value);
		for(long n = 3; n <= max; n += 2) {
			if(value % n == 0) return false;
		}
		return true;
	}

1. Primzahlen sind positiv.
2. Primzahlen beginnen bei 2, 2 ist die einzig gerade Primzahl.
3. Als Zenit (maximal zu testende Zahl) genügt die Quadratwurzel der vermeintlichen Primzahl.

Wenn "invokeLater" schon bei 1 mit den Primzahlen anfängt würde man hier wohl ein "invokeMoreLater" benötigen... ohoh, was hat das eigentlich mit der Berechnung zu tun? :lol:

So ganz vollständig ist meine Berechnung allerdings auch nicht, es genügt nämlich, wenn man versucht, die Probanten ausschliesslich durch Primzahlen zu teilen. Allerdings müssten dazu alle irgendwann gefundenen Primzahlen gespeichert werden.

 

[hüteüberhüte]

Wie gesagt, ich habe die Methode größtenteils übernommen. Es gibt doch sicher auch noch effizientere Strategien, als alle Zahlen bis sqrt(value) zu testen, oder?

Ob 1 jetzt die kleinste Primzahl ist oder 2, ist doch wurscht ^^

 

[hüteüberhüte]

[OT]Dann halt so:

    private void berechne(int n) {
        final StringBuilder builder = new StringBuilder();
        a:
        for (int i = 2; i <= n; i++) {
            int max = (int) Math.sqrt(i); // round?
            for (int j = 2; j <= max; j++) {
                if (i % j == 0) {
                    continue a;
                }
            }
            builder.append(i);
            builder.append('\n');
        }
        SwingUtilities.invokeLater(new Runnable() {

            @Override
            public void run() {
                area.append(builder.toString());
            }
        });
    }

Muss denn Math.sqrt(i) gerundet werden? Ich kenne mich damit absolut nicht aus.

[EDIT]Habe auch alles in OT gesetzt.[/EDIT][/OT]

 

[Spacerat]

[OT]Ich schreib' das mal OT. Sicher gibt es effizientere Methoden, den Test nach Fermat z.B. Aber je effizienter sie werden, desto unsicherer werden sie auch. Dieses "Sieb des Eratosthenes" hier ist auf jeden Fall das sicherste, dafür auch das langsamste.[/OT]

Zur 2. Frage: Nein, runden muss man nicht, aber was wollte man bei Primzahlen denn mit dem Nachkommaanteil?
Nehmen wir z.B. mal die Zahl 149. Die Wurzel daraus ist 12,2... sonste was. Alles was grösser als diese Zahl ist, würde den anderen Faktor wieder kleiner machen.
149 / 12 = 12, 41... (also 12)
149 / 13 = 11, 46... (also 11, kam kurz vor 12)

 

[xNk]

Sieht gut aus. Aber dank meinen 5h Informatik verstehe ich davon noch nicht viel.
Also meine Art die Zahlen zu berechnen ist halt für Einsteiger..
Aber grundsätzlich für mich: Wie kriege ich es hin die Methode

[c] public void prim(int n)
{ // alle Primzahlen von 1 bis n
int nr=0;
for (int zahl = 2;zahl<=n;zahl++) {
boolean ist_prim = true;
for (int j=2;j<=zahl/2;j++) {
if (zahl % j == 0) { // Rest Div
ist_prim=false;
}
}
if (ist_prim) {
nr = nr + 1;
}[c/]

genau so in die JTextArea einzubinden. Also so, dass da hinterher

1. 2
2. 3
3. 5 ...

steht und nicht

2, 3, 5

?


--

Ich habs jetzt:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
public class GUI extends JFrame
                    implements ActionListener
{
    JLabel lb1;
    JButton bt1;
    JTextArea ta1;
    JScrollPane sp1;
    
    public GUI()
    {
        super("  Primzahlen-Simulator 2012");
        setLayout(null);
        setLocation(800, 300); 
        setSize(420,650);
        lb1 = new JLabel("            Drück den Button, wenn du die Primzahlen haben willst!" );
        add(lb1);
        lb1.setBounds(10,20,400,20);
        ta1 = new JTextArea("",380,450);
        add(sp1 =new JScrollPane(ta1));
        sp1.setBounds(10,60,400,500);
        bt1 = new JButton("Ich will Primzahlen!");
        add(bt1);
        bt1.addActionListener(this); //Regristrieren
        bt1.setBounds(55,580,300,20);
        setVisible(true);
        
    }
    
    public void actionPerformed(ActionEvent e) {
        if (e.getSource()==bt1) {
            prim(150);
        }
    }
 
    public void prim(int n)
    { // alle Primzahlen von 1 bis n
        String primzahlen = "";
        int nr=0;
        for (int zahl = 2;zahl<=n;zahl++) {
            boolean ist_prim = true;
            for (int j=2;j<=zahl/2;j++) {
                if (zahl % j == 0) { // Rest Div
                    ist_prim=false;
                }
            }
            if (ist_prim) {
                nr = nr + 1;
                //primzahlen += zahl + ", "; 
                        ta1.append(nr+". Primzahl = "+zahl+"\n"); //1);
            }
        }
        
    }
 
}

Danke für die Hilfe!

 

[Spacerat]

Hab' mich verlesen - "habe jetzt" statt "habs jetzt" - und beinahe noch geantwortet. Machst du noch klick auf "Erledigt" oder war das doch noch 'ne Frage?

 

[hüteüberhüte]

EIGENinitiative?

import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;
import javax.swing.WindowConstants;

/**
 * @author hüte
 */
public class PrimTest {

    private JFrame frame = new JFrame("PrimTest");
    private JScrollPane scroll = new JScrollPane();
    private JTextArea area = new JTextArea();
    private JMenuBar bar = new JMenuBar();
    private JMenuItem item = new JMenuItem("berechne...");

    public PrimTest() {
        scroll.setViewportView(area);
        scroll.setPreferredSize(new Dimension(200, 400));
        frame.add(scroll);
        bar.add(item);
        frame.setJMenuBar(bar);

        item.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                try {
                    berechne(Integer.parseInt(JOptionPane.showInputDialog("n:")));
                } catch (NumberFormatException numberFormatException) {
                    JOptionPane.showMessageDialog(frame, "Keine Zahl eingegeben.");
                }
            }
        });

        frame.pack();
        frame.setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
        frame.setVisible(true);
    }

    private void berechne(int n) {
        final StringBuilder builder = new StringBuilder();
        int builder_idx = 1;
        a:
        for (int i = 2; i <= n; i++) {
            int max = (int) Math.sqrt(i);
            for (int j = 2; j <= max; j++) {
                if (i % j == 0) {
                    // i is not prime
                    continue a;
                }
            }
            // i is prime
            builder.append(builder_idx++);
            builder.append(". ");
            builder.append(i);
            builder.append('\n');
        }
        SwingUtilities.invokeLater(new Runnable() {

            @Override
            public void run() {
                area.append(builder.toString());
            }
        });
    }

    public static void main(String[] args) {
        new PrimTest();
    }
}

Siehe ab Zeile 61. Bitte drücke auf Danke. Ich habe so ein schönes Grafik erstellt :toll: