Probleme beim Aufruf von Listener in anderer Klasse

[MaikHSW]

Hey zusammen.
Ich bin momentan dabei mich an einem Projekt zu üben.
Dort wollte ich eigentlich GUI und Logik sauber in verschiedene Klassen trennen für die bessere Übersichtlichkeit.
Nun habe ich in meiner GUI einen Button definiert:

		Button startbutton = new Button();
		startbutton.setLabel("Start");
		startbutton.setBounds(36, 120, 132, 26);
		add(startbutton);

Soweit geht das auch.
Nur den Eventlistener wollte ich nun in die Logikklase auslagern da die GUI wirklich nur GUI sein soll.
Deklaration des Eventlisteners:

	class startbutton implements ActionListener
	{
		public void actionPerformed(ActionEvent evt)
		{Frame Fehler= new Frame ("Fehlermeldung");
			
	}

Das geht auch, solange er in einer Klasse mit dem Button bleibt.
Wechselt er die Klasse geht nichts mehr....

Ich rufe den Listener wie folgt auf:

startbutton.addActionListener(new startbutton());

So kann er ihn ja erstmal nur innerhalb der Klasse kennen.
Aber auch wenn ich ihn in der anderen Klasse als Public deklariere wird er nicht gesehen,
selbst eine Angabe in der Form "Klasse.Methode" bringt mich nicht weiter.

In der FH hat man sowas immer in eine Klasse gepackt, das geht bei kleinen und kurzen Projekten auch sicher.
Nur je größer das Projekt umso mehr Chaos erzeugt dieses Vorgehen ja.
Wahrscheinlich ist die Lösung auch banaler denn je,
aber kann sie mir einer von euch nennen?
Danke für eure Mühe.



Gruß

 

[Spin]

Wie meinst du dass den mit dem wechseln der Klasse?

Günstiger wäre es , wenn dein Button anders als dein Listener heißt.
Zudem würde ich dir JOptionPane empfehlen für Fehler meldungen.

Passiert denn kein Aufruf des Fensters?
Also passiert nichts beim Klicken?

 public void actionPerformed(ActionEvent event) {
        System.out.println(event.getActionCommand());
    }

Versuch doch sonst einfach mal ne Konsolenausgabe.
Sonst scheint dein Code doch richtig.
Also anhand was du hier postest kann ich nichts erkennen...:autsch:

gruß

 

[MaikHSW]

Hey Spin.
JOptionPane bevorzuge ich für Fehlermeldungen auch,
der Schnippsel ist nur kopiert um was zu haben.
Also mein Problem:
Der Button ist in der Klasse startgui definiert und deklariert.
Nun soll der Listener aber in die Klasse startlogic,
da ich diesen bereits als Teil der Logik sehe und somit trennen möchte.
Dies klappt jedoch nicht.
Kannst du dir nun vorstellen was ich meine?
Solange alles in der Klasse startgui abläuft geht es auch,
aber genau das will ich ja umgehen insofern möglich.
War vielleicht etwas unglücklich beschrieben.


Gruß

 

[javimka]

Wenn die innere Klasse StartButton (Klassen gross schreiben) in einer anderen Klasse ist, als wo sie gebraucht wird, musst du sie folgendermassen instanzieren:
Wenn sie static ist, mit [c]new AndereKlasse.StartButton()[/c] erstellen.
Wenn sie nicht static ist, brauchst du zuerst ein Objekt der anderen Klasse:
[c]AndereKlasse ak = new AndereKlasse; StartButton sb = ak.new StartButton()[/c]
oder direkt, aber schlecht:
[c] StartButton sb = new AndereKlasse().new StartButton()[/c].

Aber eigentlich sollte das nicht nötig sein. Wenn du Logik und GUI sauber trennen willst, kann es doch nicht sein, dass die eine Klasse eine Klasse verwendet, die in der anderen Klasse liegt
Vielleicht hast du schon vom MVC Prinzip gehört, das geht noch weiter und trennt Logik in Model und Controller auf. Dazu gibt es bereits einige Beiträge im Forum und der FAQ.
Ich würde den Listener in die GUI verfrachten.

 

[Spin]

/**
 *
 * @author Spin
 */
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class StartGui extends JFrame {

    public StartGui(String title) {
        super(title);
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.setSize(500, 500);


        this.setLayout(null);
        Button startbutton = new Button();
        startbutton.setLabel("Start");
        startbutton.setBounds(36, 120, 132, 26);
        startbutton.addActionListener(new StartLogic());
        add(startbutton);
        
        this.setVisible(true);

    }
    
    public static void main (String[]args) {
        
        new StartGui("Prgrammi");
//        new test();

    }
}

class StartLogic implements ActionListener {

    public void actionPerformed(ActionEvent evt) {
        JOptionPane.showInputDialog("Fehler");

    }
}

Haut doch alles hin , gruß :9

 

[MaikHSW]

Hey zusammen.
@ Javimka
Danke erstmal für deine Hinweise.
Mit Design kenne ich mich noch gar nicht aus da ich erstmal in Java selbst herein finden und mich näher damit beschäftigen muß.
Design bzw. Patterns sind ein weiterer Punkt auf meine großen Liste.
Listener in die GUI ist nicht so der Hit,
weil einige Listener auch später komplexere Rechnungen und Abläufe in sich darstellen sollen.
Sonst müßte ich die wieder auslagern und das wäre für mich aktuell noch nicht so logisch.
Ich schaue mir aber die FAQ mal an, dümmer sterben kann ich ja erstmal nicht.
@ Spin
Dir natürlich auch erstmal ein großes Danke für deine Mühe.
Genau so geht es auch.
So habe ich es ja bisher realisiert,
das passt mir aber nicht.
Ich will die Klassen komplett trennen,
das heißt in Eclipse als zwei eigenständige Dokumente sehen (doof erklärt aber ich kann es aktuell nicht besser)
Verstehst du was ich meine?
Sonst kommt man wieder zu so Monsterklassen die zwar im ersten Moment schnell und leicht gemacht sind,
im zweiten aber durch eine nahezu unmögliche Wartbarkeit gerächt werden.
Genau dies möchte ich tunlichst vermeiden da das aktuelle Projekt schon längerfristig laufen und erweiterbar sein soll.



Gruß

 

[Spin]

import javax.swing.*;
import java.awt.event.*;
public class StartLogic implements ActionListener {

    public void actionPerformed(ActionEvent evt) {
        JOptionPane.showInputDialog("Fehler");

    }
}

Denn nimm die innere Klasse raus und pack sie ( wie oben) einfach in eine neue, schreib publick davor fertig.

Ich verstehe dein Problem nicht..
Mein Listener kennt nichts von der Gui. Nur die Gui kennt den Listener, gruß

 

[MaikHSW]

Hey Spin.
Genau dies habe ich auch getan.
Dann habe ich aber keinen direkten Aufruf machen können.
Ich werde es nun testen wie Javimka es vorgeschlagen hat.
Wird zwar nicht die sauberste Lösung, aber sei es drum.
Mit der Zeit und wachsender Erfahrung kann ich immer noch ein Refactoring machen.



Gruß

 

[Spin]

Äh , aber bei mir funktioniert das. .

änder doch mal : dein aufruf.
Also schreib mal JOption pane rein.
So wie ich es habe, denn sollte es auch gehen.:autsch:

 

[MaikHSW]

Hey Spin.
AKtuell bin ich nicht zuhause und habe somit auch kein Zugriff auf mein Eclipse (ich nehme mir bald nen Stick mit Eclipse und Workspace mit....)
Sobald ich zurück bin und einen Moment Zeit finde baue ich das mal so wie du es vorgegeben hast.
Dann schaue ich mal was passiert,
vielleicht liegt der Fehler ja auch bei mir und ich habe irgendwas anders gemacht.
Ich werde berichten,
danke für deine Mühe.


Gruß