Problem mit Objekten Vererbung deklarieren initialisieren usw

[do1emu]

Hallo Leute,

damals in der Schule haben wir gelernt Objektorientiert zu Programmieren, ob ich das richtig verstanden habe!? Keine Ahnung

Also folgendes:

Ich habe ein Main Frame mit Borderlayout

für die 5 Fixpunkte (North, East usw) habe ich jeweils ein Panel welches in einer eigenen Klasse z.B. "PanelNorth extends JPanel" erstellt wird und mit

add(new PanelNorth(),Borderlayout.NORTH) direkt initialisiert und hinzugefügt wird. Im SOUTH mache ich das ganze mit einer JProgressbar...

Wenn ich nun einen Button klicke, dann wird in einer extra Klasse (Thread) eine Aktion berechnet die etwas länger dauern kann. Fortschritte sollen in der Progressbar dargestellt werden. Wie kann ich nun auf diese Progressbar zugreifen? Im Thread kann ich leider kein neues Objekt erstellen der Progressbar, das würde ja nix bringen und im Main Frame kann ich auch auf nix zugreifen, da sie ja nur initialisiert wird aber nicht deklariert also habe keine Objektvariable...

Gibt es da eine Lösung oder muss es auf jeden Fall deklariert werden?? Hoffe ich habe mit "initialisiert" und "deklariert" die richjtigen Ausdrücke gewählt


Danke schonmal

 

[Gastredner]

Dein Button bzw. dessen umgebendes Panel muss eine Referenz auf deine ProgressBar besitzen, also in etwa sowas:

// Imports fehlen

public class SomePanel extends JPanel {
	
	private JProgressBar progressBar;
	
	// GUI-Elemente
	private JButton btn;

	public SomePanel(JProgressBar progressBar) {
		this.progressBar = progressBar;
		btn = new JButton("Klick mich!");
		btn.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent event) {
				SwingUtilities.invokeLater(new Runnable() {
					public void run() {
						int i = 0;
						while (i <= 100) {
							SomePanel.this.progressBar.setValue(i);
							Thread.sleep(250);	// 250ms schlafen - try/catch müsste fehlen
							i++;
						}
					}
				});
			}
		});
		this.add(btn);
	}
}

Ist jetzt aus dem Kopf getippt, kann daher Fehler beinhalten.

 

[Ein Keks]

Und nimm einen SwingWorker statt dem Thread, swing ist nicht Thread-Safe.

 

[do1emu]

erstmal denke für die schnelle Antwort...

In deiner Zeile 5 steckt mein Problem. Das umgebene Panel des Buttons beinhaltet nicht die Progressbar die wird im MainFrame genau wie das panel des buttons so initialisiert:

...
add(new NorthPanel(), Borderlayout.NORTH); //Panel mit dem Button
add(new SouthProgress(), Borderlayout.SOUTH); //Progressbar
...

Wenn ich nun im Buttonpanel einen Actionlistener aufrufe, dann habe ich eben diese Referenz nicht. Sollte ich im ButtonPanel ein neues Objekt der Progressbar-Klasse aufrufen, dann ist das ja ein eigenes Objekt und nicht das, was im Main Frame eingefügt wurde, oder ist da nen Denkfehler??

Wenn nicht, dann müsste es ja mit Setter und Getter gehen!?!

@Ein Keks Danke für den Tipp, ist das der grund warum mein Prog so oft abschmiert wenn ich die Größe verändere??

 

[Murray]

Du musst im MainFrame eine Membervariable für das Panel haben, dieses Member initialisieren (also einen Wert zuweisen) und dann genau dieses Objekt auch dem Frame hinzufügen.

Etwa so:

public class MainFrame {
  NorthPanel thePanel;
  SouthProgess theProgress;

/* ... */

thePanel = new NorthPanel();
add( thePanel, Borderlayout.NORTH); //Panel mit dem Button
theProgress = new SouthProgress();
add(theProgress, Borderlayout.SOUTH); //Progressbar
}

 

[Tomate_Salat]

Man könnte auch eigene Listener für nehmen.

 

[do1emu]

Habs nun so gelöst wie Murray es vorgeschlagen hat, Frage mich allerdings warum das Objekt immer static sein muss wenn ich aus einer anderen Klasse drauf zugreifen will... Naja egal so funzt es danke

 

[Murray]

Frage mich allerdings warum das Objekt immer static sein muss wenn ich aus einer anderen Klasse drauf zugreifen will...

Wer sagt denn das? Auch auf Instanzmethoden kann man aus anderen Klassen zugreifen - man muss aber natürlich die konkrete Instanz kennen.

 

[do1emu]

und wie kann ich das rausfinden?? Wenn ich im Thread auf die Progressbar zugreifen will, dann mache ich das so:

SouthPanel.jpb.setValue(i);

Dann meckert mein Eclipse ich soll das Objekt public und static machen anderes habe ich nicht kennengelernt

 

[Tomate_Salat]

SouthPanel.jpb.setValue(i);

Dann meckert mein Eclipse ich soll das Objekt public und static machen anderes habe ich nicht kennengelernt

Weil du hier auf keine Referenz sondern auf die Klasse selbst drauf zugreifst.

SouthPanel southPanel = new SouthPanel();

das [c]southPanel[/c] kannst du dann mittels Methoden an andere Klassen weitergeben. Somit besitzen diese die Referenz und können auf diese Einwirken.

 

[do1emu]

ist meine Lösung denn "schmutzig" oder ist es halt einfach anders?

 

[Tomate_Salat]

Da das ganze auf deinem Hauptfenster ist und ich annehme das es nur ein einziges mal offen sein wird, hmm wäre denke ich noch annehmbar (aber sauber würde ichs trotzdem nicht bezeichnen). Aber ich würde dafür keine static nutzen, lieber eine Referenz übergeben. Wobei man das auch mit vorsicht genießen sollte, denn auf diese Weise schaffst du abhängigkeiten die später zieml. lästig werden! Am besten du schaust dir vllt mal das MVC an und machst übungen dazu um die Code-Bereich sauber zu trennen.

Ich verwende Static z.B. nur für factory-instanzen, Singletons, Ini-dateien und Konstanten (ja ich glaub das wars)

 

[do1emu]

Ja ich erinnere mich glaube ich dunkel wenn man gegenseite Referenzen erstellt kommts zu Fehlern daher habe ich ja alles soweit ausgelagert, damit sowas nicht passiert... Naja es geht nur um die Progressbar, die muss dann mal static abkönnen

Danke auf jeden Fall an alle für die Hilfe und Ratschläge!!!!