[Hilfe] einfache Aufgabe

[aspire]

Hey ich bin Java Anfänger und bräuchte die Lösung für Aufgabe 55 der folgenden PDF.
Es geht mir darum zu verstehen wie ich das Programm aufbaue.
Könnte mir jemand eine möglichst einfach Lösung zeigen.

PDF aufgabe 55: http://calug.de/~mmeeh/uni/Informatik.pdf

Mfg aspire

 

[Gonzo17]

Also die meiste Arbeit, nämlich zu ermitteln, ob es eine Primzahl ist, wurde ja schon gemacht, die Methode istPrimzahl(int x) steht ja da. Jetzt musst du nur noch eine for-Schleife von 3 bis zur Obergrenze machen und für jede Zahl diese Methode aufrufen. Liefert die Methode "true" zurück, dann soll dir das Programm die Zeile System.out.println(i); ausführen (wobei i hier die aktuelle Zahl ist, die getestet wurde).

 

[aspire]

Ja soweit ist mir das klar, aber ich muss ja das eingabefeld mit dem berchnebutton und dem ausgabefeld verknüpfen. da liegt schonmal mein erstes Problem.

 

[Gonzo17]

Achso, daran hab ich garnicht gedacht. ^^
Ja ähm, wie gestaltet ihr denn normalerweise eure Oberflächen? Haste die Oberfläche schon erstellt? Wenn ja, dann braucht dein Button einen Listener und wenn der betätigt wird, sollte das ausgeführt werden, was ich eben beschrieben hab.

 

[aspire]

bisher bin ich soweit gekommen:

import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.text.DecimalFormat;

import javax.swing.JButton;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;

public class Aufgabe55 extends MeinTypischesFenster {
	public static void main(String[] args) {
		new Aufgabe55();
	}

	public Aufgabe55() {
		this.add(new A55BorderLayout());
		this.pack();
		this.setResizable(false);
		this.setVisible(true);
	}
}
class A55BorderLayout extends JPanel {
	public A55BorderLayout() {
		this.setLayout(new BorderLayout());
		A55Rechenzeile rechenzeile = new A55Rechenzeile();
		this.add(rechenzeile, BorderLayout.CENTER);
		JTextArea ausgabe = new JTextArea(8, 0);
		JScrollPane scroll = new JScrollPane(ausgabe);
		this.add(scroll, BorderLayout.SOUTH);
		A55Kopfzeile kopfzeile = new A55Kopfzeile(rechenzeile, ausgabe);
		this.add(kopfzeile, BorderLayout.NORTH);
	}
}

class A55Kopfzeile extends JPanel {
	public A55Kopfzeile(A55Rechenzeile rechenzeilepara, JTextArea ausgabepara) {
		this.add(new JLabel("Primzahlen Programm "));
		A55BerechnenButton berechnenbutton = new A55BerechnenButton(
				rechenzeilepara, ausgabepara);
		this.add(berechnenbutton);

	}
}

 

[Landei]

Wie kann man bloß einen Panel A55BorderLayout nennen? Ich würde den Panel gar nicht ableiten, sondern sowas machen:

class Aufgabe55 extends MeinTypischesFenster {
...
private JTextArea ausgabe = new JTextArea(8, 0);
//alle anderen Komponenten, auf die du später zugreifen willst

public Aufgabe55() {
  JPanel myPanel = new JPanel(new BorderLayout());
  getContentPane().add(myPanel);
  JPanel northPanel = new JPanel();
  ... usw. ...
  ... adde alles auf die Panel...
  pack();
  setResizable(false);
  setVisible(true);
}
...
}

Generell sollte man Komposition ("ich benutze einen JPanel und packe was drauf") der Vererbung ("ich bin ein spezieller JPanel") vorziehen, wenn es keinen guten Grund für letztere gibt (und den sehe ich hier nicht).


Übrigens solltest du Code im Forum in [java] Tags packen.

 

[aspire]

Wie gesagt, ich bin Anfänger, aber danke für den Tip.
Wär echt nett wenn einer das vielleicht kurz machen könnte,
ich steh total auf dem Schlauch wie ich das Eoingabefeld mit dem BerechneKnopf (und der istPrimzahl Methode)
mit der Ausgabe verbinde :/

 

[Landei]

Das Eingabefeld machst du zu einen Member-Variable...

class Aufgabe55 extends MeinTypischesFenster {
...
  private JTextField eingabe = new JTextField(); 
...
}

... und fügst sie im Konstruktor irgendeinem Panel hinzu.

Dann hast du ja irgendwo eine Aktion oder einen ActionListener an deinem Knopf hängen. In actionPerformed() liest du Feld dann einfach aus, so in der Art:

int nummer = Integer.parseInt(eingabe.getText()); //Fehlerbehandlung wäre natürlich noch cool

 

[Gonzo17]

Also das mit dem Button und dem Listener sollte etwa so aussehen:

Button testButton = new Button(shell, 0);
		testButton.addListener(SWT.Selection, new Listener() {
			public void handleEvent(Event event) {
				// hier passiert dann das, was der Klick auf den Button bewirken soll
			}
		});

 

[Landei]

Äh, Gonzo? Hier war bisher alles Swing, wie kommst du plötzlich auf SWT?

 

[aspire]

Ja, ich bin auch nur mit Swing vertraut

 

[Gonzo17]

Gute Frage. Hab geschlafen.

Dann eben sowas:

JButton testButton= new JButton("Klick mich!");
		testButton.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent e) {
// hier passierts dann				
                    }
		});

 

[aspire]

ok ich werds mal probieren ???:L

 

[aspire]

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.text.DecimalFormat;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.JTextField;

public class Aufgabe55 extends JFrame {

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

	Aufgabe55() {
		setSize(300, 400);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setVisible(true);
		add(new Aufgabe55HauptPanel());
	}
}

class Aufgabe55HauptPanel extends JPanel {
	Aufgabe55HauptPanel() {
		JTextField textfeld = new JTextField(25);
		JTextArea area = new JTextArea();
		area.setColumns(25);
		area.setText("Primzahlen: ");
		area.setRows(20);
		this.add(textfeld);
		this.add(new BerechneKnopf(area, textfeld));
		this.add(area);
	}
}

class BerechneKnopf extends JButton implements ActionListener {
	private JTextField EingabeFeld;

	private JTextArea AusgabeFeld;

	public BerechneKnopf(JTextArea Ausgabe, JTextField Eingabe) {

		super("Berechne");
		this.addActionListener(this);
		EingabeFeld = Eingabe;
		AusgabeFeld = Ausgabe;
		

	}

	public void actionPerformed(ActionEvent e) {
		//AusgabeFeld.append((EingabeFeld.getText()));
		int a = Integer.parseInt(EingabeFeld.getText());
		
		for(int i=a;i>=3;i--) System.out.println("" +i+ " " + istPrimzahl(i));
	}

		
static boolean istPrimzahl(int x) {
		int i=1;
		if(x<3) return true;
		while(++i<=x/2) if(x%i==0) return false;
		return true;
		}
}

Es geht jetzt nur noch darum den Wert in der Textarea auszugeben falls es eine Primzahl ist. Den Rest habe ich nun. Hilfe ?!

 

[Ezra]

textarea.setText(text);

 

[Landei]

Oh Mann, musst du alles über Vererbung lösen? Die abgeleiteten Panels sind nicht schön, aber class BerechneKnopf extends JButton implements ActionListener ist die absolute Härte!

(ungetestet)

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.text.DecimalFormat;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.JTextField;
 
public class Aufgabe55 extends JFrame {

    private JTextField textfeld = new JTextField(25);
    private JTextArea area = new JTextArea();
 
    public static void main(String[] args) {
        new Aufgabe55();
    }
 
    Aufgabe55() {
        setSize(300, 400);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setVisible(true);
        add(getPanel());
    }

    private JPanel getPanel() {
       JPanel panel = new JPanel();
        area.setColumns(25);
        area.setText("Primzahlen: ");
        area.setRows(20);
        panel.add(textfeld);
        panel.add(getButton());
        panel.add(area);
        return panel;
    }   

    private JButton getButton() {
       JButton button = new JButton("Berechnen");
       button.addActionListener(new ActionListener(){
          public void actionPerformed(ActionEvent e) {
             int a = Integer.parseInt(EingabeFeld.getText());
             area.setText("");
             for(int i=1;i<=a;i++) {
                 if(istPrimzahl(i)) area.append("" +i+ " ");
             }  
          }});
        return button;
    }

    private static boolean istPrimzahl(int x) {
        if(x<=1) return false;
        if(x<=3) return true;
        if(x % 2 == 0) return false;
        int limit = (int) Math.ceil(Math.sqrt(x));
        for(int i = 3; i <= limit; i += 2) {
           if(x % i==0) return false;
        } 
        return true;
    }
}

 

[Gonzo17]

Schau mal, in folgender Zeile ist deine Ausgabe (allerdings eben in der Konsole):

for (int i = a; i >= 3; i--)
			System.out.println("" + i + " " + istPrimzahl(i));

Wenn du stattdessen die Werte einfach der JTextArea hinzufügst, haste genau das, was du möchtest. Aber mal noch was anderes, wieso lässt du jede Zahl i eintragen und danach den Boolean-Wert, ob sie Primzahl ist oder nicht? Ich dachte da sollen nur die Primzahlen angezeigt werden? Und wieso zählst du von oben nach unten (wobei das ja auch nich so wichtig ist)?

Ich hab das jetzt mal ein bisschen geändert. Das einzige, was noch nicht passt, ist die Ausgabe in der JTextArea, da sind noch keine Zeilenumbrüche drin, weiss grad nicht, wie ich das hinbekomme. Ansonsten klappt es.

for (int i = 3; i <= a; i++)
			if (istPrimzahl(i))
				AusgabeFeld.append(Integer.toString(i));

 

[aspire]

Ja hammermäßig, das "Integer.toString" hat mir gefehlt.
Sonst hät ichs wahrscheinlich geschafft aber vielen dank Gonzo,
hast mir echt gut geholfen. Jetzt muss ich das mit dem Zeilenumbruch noch
hinbekommen dann wars das ! 1A


EDIT:

BZW. habs schon
"\n"+


Das wars danke !

 

[Gonzo17]

Achja, hät ich auch selbst drauf kommen können. ^^
Freut mich, dass ich dir helfen konnte.

Ach und was mir da noch einfällt. Das Integer.toString brauchst du nur, wenn du sonst nichts in der Klammer stehen hast. Sobald du aber nen String stehen hast, wird deine Integer-Variable automatisch als String angefügt.

AusgabeFeld.append("\n" + i);

Das ist somit kein Problem.

 

[aspire]

Ahh ok
man könnte theoretisch auch einfach ("" + i) schreiben dann hätte sich das Integer.toString auch erledigt

Eine Frage noch, ich schaffs nicht eine Scrollpane da drauf zu bekommen.
Wenn man zb. die Obergrenze auf 100 setzt ist das Fenster zu klein.

 

[Gonzo17]

Ahh ok
man könnte theoretisch auch einfach ("" + i) schreiben dann hätte sich das Integer.toString auch erledigt

Sieht find ich nicht schön aus. Dann lieber mit Integer.toString, da weiss man dann wenigstens auf den ersten Blick, was passiert.

Eine Frage noch, ich schaffs nicht eine Scrollpane da drauf zu bekommen.
Wenn man zb. die Obergrenze auf 100 setzt ist das Fenster zu klein.

this.add(new JScrollPane(area));

Damit sollte es klappen.

 

[aspire]

So dacht ich auch, tuts aber nicht ...

 

[Gonzo17]

Also ich hab deinen Code mal kopiert und das probiert, bei mir klappts. Hier mal der ganze Code, vielleicht hast du was geändert, was bei mir nicht so ist.

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;

public class Aufgabe55 extends JFrame {

	private final JTextField textfeld = new JTextField(25);
	private final JTextArea area = new JTextArea();

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

	Aufgabe55() {
		setSize(300, 400);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setVisible(true);
		add(getPanel());
	}

	private JPanel getPanel() {
		JPanel panel = new JPanel();
		area.setColumns(25);
		area.setText("Primzahlen: ");
		area.setRows(20);
		panel.add(textfeld);
		panel.add(getButton());
		panel.add(new JScrollPane(area));
		return panel;
	}

	private JButton getButton() {
		JButton button = new JButton("Berechnen");
		button.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				int a = Integer.parseInt(textfeld.getText());
				area.setText("Primzahlen: ");
				for (int i = 3; i <= a; i++)
					if (istPrimzahl(i))
						area.append("\n" + i);

			}
		});
		return button;
	}

	private static boolean istPrimzahl(int x) {
		if (x <= 1)
			return false;
		if (x <= 3)
			return true;
		if (x % 2 == 0)
			return false;
		int limit = (int) Math.ceil(Math.sqrt(x));
		for (int i = 3; i <= limit; i += 2) {
			if (x % i == 0)
				return false;
		}
		return true;
	}
}

 

[aspire]

hm seltsam, mit deinem kopierten Code gehts bei mir auch, aber ich seh keine nennenswerten unterschiede zu meinem

 

[aspire]

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;

public class Aufgabe55 extends JFrame {

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

	Aufgabe55() {
		setSize(300, 400);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setResizable (false);
		add(new Aufgabe55HauptPanel());
		setVisible(true);
	}
}

class Aufgabe55HauptPanel extends JPanel {
	Aufgabe55HauptPanel() {
		JTextField textfeld = new JTextField(25);
		JTextArea area = new JTextArea();
		JLabel label = new JLabel("Geben sie die Obergrenze an");
		
		
		 area.setColumns(25);
		 area.setRows(20);
	     area.setText("Primzahlen: ");
	        


	    this.add(new JScrollPane(area));

		this.add(label);
		this.add(textfeld);
		this.add(new BerechneKnopf(area, textfeld));
		this.add(area);
	}
}

class BerechneKnopf extends JButton implements ActionListener {
	private JTextField EingabeFeld;
	private JTextArea AusgabeFeld;

	public BerechneKnopf(JTextArea Ausgabe, JTextField Eingabe) {

		super("Berechne");
		this.addActionListener(this);
		EingabeFeld = Eingabe;
		AusgabeFeld = Ausgabe;

	}

	public void actionPerformed(ActionEvent e) {

		int a = Integer.parseInt(EingabeFeld.getText());
		for (int i = 3; i <= a; i++)
			if (istPrimzahl(i))
				AusgabeFeld.append("\n" + i);
		AusgabeFeld.append("\n \n *** Zeue Primzahl-Obergrenze setzen ***");
	}

	static boolean istPrimzahl(int x) {
		int i = 1;
		if (x < 3)
			return true;
		while (++i <= x / 2)
			if (x % i == 0)
				return false;
		return true;
	}
}

vielleicht siehst du einen

 

[Gonzo17]

this.add(new JScrollPane(area));
this.add(label);
this.add(textfeld);
this.add(new BerechneKnopf(area, textfeld));
this.add(area);

Das haste da drin stehen. Da tuste zweimal die JTextArea hinzufügen. Kann sein, dass es deshalb ist, probiers mal ohne die letzte Zeile (bzw tu die erste mal anstelle der letzten, so wie bei mir).

Edit: Ich hab den Quellcode oben mal aktualisiert und auf den von Landei aufgebaut, der sieht einfach sauberer aus. Klappt damit auch wunderbar.

 

[aspire]

Ok wunderbar jetzt funktioniert alles
Danke, das Forum rockt echt !