Event Handling Event Handling in eigene Klasse

[Gnaa]

Hallo

Ich bin recht neu im Programmieren und deshalb auch noch sehr unerfahren ,bitte nicht hauen wenn diverse Fehler im Code sind aber aufmerksam machen kann man mich dennoch. :rtfm:
Ich habe ein Programm geschrieben mit einem Table und Buttons und Textfeldern sowie eine JCalendarCombo Box.
Das ganze befand sich alles zumindest der grösste teil in einer Klasse "View" und funktionierte einwandfrei .dann entschloss ich mich alles zu splitten und in verschiedene Klassen auszulagern z.b NordPanel MiddlePanel SuedPanel anschließend wird im Controller alles zusammengefügt und im View dargestellt quasi.Das funktioniert in der Anzeige auch super ich bekomme nur leider das mit dem Actionlistener nicht hin. Die Buttons und Textfelder sowie die JCalendarCombo befinden sich im NordPanel und die Table im Middlepannel. Wie kann ich die Daten nun in das Table einfügen
hier mal der vollständige Code der beiden Panels.

NordPanel:

public class FWNordPanelAusgaben extends JPanel  {
    private static final long serialVersionUID = 1L;
    private JLabel labelout;
    private BevelBorder bevel;
    private  JButton jbtnedit;
    private  JButton jbtnadd;
    private  JButton jbtndelete;
    final JTextField jtfname;
    final JTextField jtfbetrag;
    final JCalendarCombo cal;
    private   JPanel calpanel;
    private FWMiddlePanelAusgaben mid;
    
    
    public FWNordPanelAusgaben() {
        super();
        this.setLayout(null);
        this.setSize(600, 250);
        this.setOpaque( true );    /////////////////sichtbarkeit auf true sonst funzt farbauswahl nicht///////////////
        this.setBorder(new EmptyBorder(0, 0, 0, 0));
        this.setBackground( new Color( 162,183,192 ) );
       
        {
            bevel = new BevelBorder(BevelBorder.RAISED);
            Border border =bevel;
            labelout = new JLabel("Daten Ausgaben");
            labelout.setFont(new Font("Arial Marrow", Font.BOLD, 18));
            labelout.setLayout(null);
            labelout.setBounds(40, 10, 150, 30);    //(x,y,Höhe,Breite))))))
            labelout.setBorder(border);
            labelout.setForeground(new Color(120,111,80));
            labelout.setHorizontalAlignment(SwingConstants.CENTER); 
        }
        {
            jbtnadd = new JButton("Hinzufügen");
            jbtnadd.setLayout(null);
            jbtnadd.setBounds(40, 50, 150, 30);
 //           mid = new FWMiddlePanelAusgaben();
//            jbtnadd.addActionListener(new ActionListener() {
//
//                @Override
//                public void actionPerformed(ActionEvent e) {
//                   if( e.getSource() == jbtnadd){
//                    //mid.model.addEinAusgaben(new        FWEinAusgaben(jtfname.getText(),cal.getDate(),Double.parseDouble(jtfbetrag.getText()),false));
//                       mid.model.addEinAusgaben(new FWEinAusgaben("huhu",new Date(),8.9,false));
//                }}
//            });
//            //jbtnadd.addActionListener((ActionListener) new FWMiddlePanelAusgaben());
//            
        }
        {
            jbtnedit = new JButton("edit");
            jbtnedit.setLayout(null);
            jbtnedit.setBounds(40, 100, 150, 30);
        }
        {
            jbtndelete = new JButton("edit");
            jbtndelete.setLayout(null);
            jbtndelete.setBounds(40, 150, 150, 30);
        }
        {
            jtfname = new JTextField();
            jtfname.setLayout(null);
            jtfname.setBounds(370, 50, 145, 25);
        }
        {
            jtfbetrag = new JTextField(); 
            jtfbetrag.setLayout(null);
            jtfbetrag.setBounds(370, 150, 145, 25);
        }
        {
        cal =new JCalendarCombo(JCalendarCombo.DISPLAY_DATE, false);
        cal.putClientProperty("JCalendar.headerStyleer);", "Modern_Arrow");
        cal.setDateFormat(new SimpleDateFormat("dd-MM-yyyy"));
        calpanel=new JPanel(new BorderLayout());
        calpanel.setBounds(370, 100, 145, 25);
        calpanel.add(cal,BorderLayout.NORTH);
        }
       
        this.add(labelout);
        this.add(jbtnadd);
        this.add(jbtnedit);
        this.add(jbtndelete);
        this.add(jtfbetrag);
        this.add(jtfname);
        this.add(calpanel);
        this.validate();
	this.repaint();
        
        
     }
   
      
}

MiddlePanel:

public class FWMiddlePanelAusgaben extends JPanel{
    public FWPTablemodel.FWTabelmodel model;
    public JTable table;
    private TableColumnModel tcm;
    private JTable lineTable;
    private JScrollPane scp;
    private FWNordPanelAusgaben npa;
    
    public FWMiddlePanelAusgaben() {
        super();
        this.setLayout(null);
        this.setSize(600, 250);
        this.setOpaque( true );    /////////////////sichtbarkeit auf true sonst funzt farbauswahl nicht///////////////
        this.setBorder(new EmptyBorder(0, 0, 0, 0));
        this.setBackground( new Color( 162,183,192 ) );
        {
            model = new FWTabelmodel();
            table = new JTable(model);
            table.setOpaque(true);
            table.repaint();
            table.revalidate();
        }
        {
        tcm = table.getColumnModel();
        tcm.getColumn(0).setPreferredWidth(200);    
        tcm.getColumn(1).setPreferredWidth(123);    
        tcm.getColumn(2).setPreferredWidth(80);
        tcm.getColumn(3).setPreferredWidth(150);
        table.getTableHeader().setResizingAllowed(false);
        table.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);
        }
        {
            lineTable = new FWLineNumberTable( table );
            lineTable.setBackground(Color.LIGHT_GRAY);
        }
        {
            scp = new JScrollPane(table);
            scp.setBackground(Color.LIGHT_GRAY);
            scp.setHorizontalScrollBarPolicy(HORIZONTAL_SCROLLBAR_NEVER);
            scp.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);;
            scp.getViewport().setBackground(Color.LIGHT_GRAY);
        }
       
            scp.setBounds(20, 0, 571, 487);
            this.add(scp);
            this.add(lineTable);
              
    }
    
}

mit // gekennzeichnete bereiche war so ne Sache die ich versucht hatte !
in der Klasse wo das Funktionierte lautet die Stelle

 {      
         jbtnadd.addActionListener( new ActionListener() {
         @Override public void actionPerformed( ActionEvent e ) {
          
             model.addEinAusgaben(new EinAusgaben(jtname.getText(),cal.getDate(),Double.parseDouble(jtbetrag.getText()),false));
      }         
   });   
}

evtl noch ne Frage kann man das komplette Event Handlich auch auslagern wenn ja wie?

vielen Dank schon mal im voraus

 

[Tom299]

Ich bin in Swing zwar nicht mehr so drin, aber ich denke mal, dein Problem ist, daß du in jedem Panel einen eigenen ActionListener benutzt. Schreib dir einen ActionListener, den du an die Panels weitergibst oder laß dein Parent-Panel/Frame, welches die SubPanels beinhaltet, den ActionListener implementieren.

 

[Harry Kane]

Du erstellst Unterklassen von JPanel, ohne dessen Funktionalität zu erweitern. Sowas sollte man sich gar nicht erst angewöhnen. Es ist nicht Sinn des Erweiterns, die erweiterte Klasse nur zu benutzen. Zu dem Thema gibt es zahlreiche Beiträge.
Die Aufteilung von Code (von Gui Code in deinem Fall) auf mehrere Klassen hat zwar zunächste den scheinbaren Vorteil, dass der Code pro Klasse kürzer wird (zu große Klassen sollte man ja vermeiden), aber bringt den Nachteil mit sich, daß die Verbindung zwischen Komponenten, die zusammenarbeiten und damit voneinander wissen müssen, erschwert wird, wenn die Komponenten in verschiedenen Klassen liegen.
Als dirty workaround kannst du folgendes machen: In der "Hauptklasse" (um sie mal so zu nennen), in der du deine GUI erzeugst und die Instanzen von deinen ganzen Panels anlegst, übergibst du der Instanz von FWNordPanelAusgaben eine vorher erzeuget Instanz von FWMiddlePanelAusgaben. Dazu musst du den Konstruktor von FWNordPanelAusgaben ändern. Dann kannst du in der actionPerformed von FWNordPanelAusgaben (bzw. in der inneren, anonymen Klasse) auf FWMiddlePanelAusgaben.model zugreifen.
Nachteil: du erzeugst auf diese Weise Abhängigkeiten zwischen ziemlich speziellen Klassen --> Wiederverwendbarkeit gleich null.
Für ein eigenes Projekt stand ich vor kurzem vor einer ähnlichen Aufgabe: Bei einer Benutzeraktion sollen Objekte erzeugt und in einem anderen Objekt gespeichert werden.
Ich habe es so gelöst:

public interface Receiver<T> {
    void receive(T item);
}
public interface Creator<T> {
    T create();
}
public class ItemBroker<T> extends AbstractAction{
    private Creator<T> creator;
    private Receiver<T> receiver;
    
    public Receiver<T> getReceiver(){
        return this.receiver;
    }
    
    public Creator<T> getCreator(){
        return this.creator;
    }

    public void setReceiver(Receiver<T> r){
        this.receiver = r;
    }
    
    public void setCreator(Creator<T> c){
        this.creator = c;
    }

    public void actionPerformed(ActionEvent ae){
        if(creator == null || receiver == null) return;
        receiver.receive(creator.create());
    }
}

Instanzen von ItemBroker können z. B. zur Erzeugung von JMenus oder JButtons verwendet werden.
Angewandt auf deinen Fall würde das zu folgender Struktur führen:
FWNordPanelAusgaben implementiert Creator<EinAusgaben>. In der create-Methode würde FWNordPanelAusgaben den Inhalt der entsprechenden JTextFields oder JCalendarCombo auslesen, daraus ein EinAusgaben Objekt machen und dieses zurückgeben.
FWMiddlePanelAusgaben implementiert Receiver<EinAusgaben>. In seiner receive(EinAusgaben ea) Methode würde FWMiddlePanelAusgaben das übergebene EinAusgaben Objekt in sein TableModel einfügen.
Vorteil: FWNordPanelAusgaben muss nicht wissen, wie FWMiddlePanelAusgaben das EinAusgaben Objekt genau speichert, und muss auch von der Existenz eines TableModels in FWMiddlePanelAusgaben nix wissen, d. h. das TableModel von FWMiddlePanelAusgaben muss nicht public sein. Es ist allein Sache von FWMiddlePanelAusgaben wie das Objekt gespeichert wird. Und FWMiddlePanelAusgaben muss nicht wissen, wie genau FWNordPanelAusgaben eine Instanz von EinAusgaben erzeugt, und auf welche JComponents dabei genau zugegriffen werden muss. Das Erzeugen des EinAusgaben Objektes ist allein Sache des FWNordPanelAusgaben.

Davon abgesehen: null Layout is evil. Benutze lieber einen gescheiten Layoutmanager.

 

[Gnaa]

Ich Danke erstmal allen Antwortenden!!!

Zu Harry Kane´s Beitrag ein paar Fragen.
Wenn ich das im oberen Text richtig verstanden habe rätst du mir besser die Variante zu benutzen die ich vorher hatte ? Alles im View außer mein Tablemodel sowie Renderer. Was mach ich wenn ich nehmen wir mal an ich habe 20 Buttons 5 Tabel´s incl model und jede menge anderes Zeugs dann wird die View Klasse ja mega groß! Gibt es da eine gescheite Möglichkeit das sinnvoll zu teilen?

zu dem letzten Satz ! Ich höre immer nur null Layout ist Evil warum?

Danke schon mal im voraus für die Antworten.

 

[Harry Kane]

Wenn ich das im oberen Text richtig verstanden habe rätst du mir besser die Variante zu benutzen die ich vorher hatte ? Alles im View außer mein Tablemodel sowie Renderer. Was mach ich wenn ich nehmen wir mal an ich habe 20 Buttons 5 Tabel´s incl model und jede menge anderes Zeugs dann wird die View Klasse ja mega groß! Gibt es da eine gescheite Möglichkeit das sinnvoll zu teilen?

Wenn es deine Applikation erfordert, 20 Buttons und Tables zu haben, und dadurch die View "megagross" wird, dass soll das wohl so sein. Wie oben schon angedeutet, halte ich es für Augenwischerei, wenn dem Vorwurf entgehen möchte, eine zu große Klasse produziert zu haben, und deshalb Komponenten die zusammengehören, über verschiedene Klassen verteilt.
Auch wenn nur eine View-Klasse hast, sollte es möglich sein, durch eine saubere Programmierung ihre Lesbarkeit und ihren Umfang in einem brauchbaren Rahmen zu halten. Dabei können folgenden Punkt hilfreich sein:
- Verwendung von Arrays um verwandte Komponenten abzulegen (also JButton[20] statt jbutton1, jbutton 2, ...)
- Verwendung von Methoden, die bestimmte Foramtierungsaufgaben erledigen, z. B. statt mehrfach

 JLabel jl = new JLabel();
jl.setBorder(...);
jl.setBackground(...);
jl.setFont(...);

eine Hilfsmethode:

 JLabel jl = new JLabel();
formatLabel(jl);
private void formatLabel(JLabel jl){
    jl.setBorder(...);
    jl.setBackground(...);
    jl.setFont(...);
}

- überflüssige/sinnlose Methodenaufrufe entfernen. Du setzt mehrfach bei JButtons und JTextFields das Layout auf null --> sinnlos.
- Anwendungslogik auslagern in intelligente, flexibel anwendbare Actions (wie oben beschrieben), intelligente Datenmodelle wie z. B. eine TableModel (du hast eine Klasse FWPTablemodel.FWTabelmodel, aber ich habe kA, wie "schlau" die tatsächlich ist, und wieviel du von der Programmlogik dahin auslagern könntest. Nebenbemerkung: Das Interface Receiver könnte auch von einem selbstgeklöppelten TableModel implementiert werden).
- Statt eines Hauptscreens die Anwendung in mehrere Dialoge/Tabs/etc. aufteilen, aber so dass keine Komponenten die zusammenarbeiten müssen, getrennt werden. Die notwendige Kommunikation zwischen den Dialogen/Tabs sollte am besten über möglichst flexible interfaces erfolgen. Um eine interface basierte Kommunikation zwischen JPanels zu erreichen, wäre es auch statthaft, JPanel zu erweitern.
- Layoutmanager verwenden. Bei einem geeigneten Layoutmanager kannst du es dir sparen, Größe und Position der Komponenten einzeln anzugeben. Die in Java standardmäßig vorhandenen Layputmanager halte cich aber für eher mau. Ich mache seit Jahren das meiste mit Miglayout und habe bisher auch komplexe, in der Größe veränderliche Layouts stets so hinbekommen wie ich es wollte.

zu dem letzten Satz ! Ich höre immer nur null Layout ist Evil warum?

Uff. Dazu gibt es so viel im Netz... Kurzfassung: Warum sollte ich die Größe und Position von JComponents in einem komplexen Layout von hand berechnen, wenn ein LayoutManager das für mich erledigt? Und feste Größen und Positionen crashen spätestens dann, wenn du deine Applikationsfenster in der Größe veränderbar machen und bei verschiedenen Bildschirmauflösungen benutzen möchtest.