Zugriffe in einer Klasse / Funktion

[breakplease]

Hallo liebe Java Community

Für mein Studium bin ich mich in Java am einarbeiten und versuche dafür eine Sportplatzverwaltung zu programmieren. Bei einem GUI habe ich nun ein Problem:
Die Klasse guiPlatzErfassen soll in zukunft weitervererbt werden, desshalb möchte ich eine eigene Methode/Funktion haben, welche das Formular aufbereitet. Die habe ich initialisiereGUI() benennt. Im Konstruktor rufe ich dann diese Funktion auf. Das klappt auch soweit, aber leider kann ich dann nicht auf die Elemente zugreifen, also besipielsweise bei dem JButton die Farbe ändern.
Hat jemand eine Idee wo es hier fehlen könnte?

Vielen Dank!!
Jürg

public class guiPlatzErfassen extends JFrame {

   private JPanel contentPane;
   private JTextField textField;
   private JComboBox comboBoxSportart;
   private JComboBox comboBoxPlatznummer;
   private JButton btnSpeichern;
   private JButton btnAbbrechen;


   public void initialisiereGUI() {
     setTitle("Sportplatz erfassen");
     setVisible(true);
     setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
     setBounds(100, 100, 354, 268);
     contentPane = new JPanel();
     contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
     setContentPane(contentPane);
     contentPane.setLayout(null);
     JLabel lblSportart = new JLabel("Sportart");
     lblSportart.setBounds(45, 40, 92, 14);
     contentPane.add(lblSportart);

     JButton btnSpeichern = new JButton("Speichern");
     btnSpeichern.setBounds(45, 171, 103, 23);
     btnSpeichern.setBackground(new Color(255, 255, 204));
     contentPane.add(btnSpeichern);   
   }

   public guiPlatzErfassen() {
     initialisiereGUI();
     btnSpeichern.setBackground(new Color(255, 255, 204));
}

 

[Joose]

Hat jemand eine Idee wo es hier fehlen könnte?

Mit setVisible(true) weist du Java an das Fenster und all seine bisher hinzugefügten Komponenten zu zeichen.
Alle Änderungen welche du danach an Komponenten vornimmst (Farbwechsel von Button) sind damit nicht sichtbar, du muss erst neu zeichnen lassen -> revalidate

Anmerkungen zu deinem Code:
Klassennamen werden in UpperCamelCase geschrieben und Beschreiben ein Objekt, Methoden beschreiben Tätigkeiten. Also ist "....Erfassen" eher der Name einer Methode.
Man erbt nicht von JFrame wenn man dessen Funktionalität nicht erweitern will (Layout festlegen usw. ist keine Funktionalität )

 

[breakplease]

Hallo Joose
Danke für Deine Hinweise! Die Klasssenbezeichnungen muss ich in dem Fall nochmals überdenken Das extends JFrame ist da reingerutscht nachdem ich das GUI mit dem Window Builder erstellt hatte. Funktioniert nun aber auch ohne, wenn ein neues JFrame instanziert wird. Danke Dir!

Ich habe nun das setVisible=True in den Konstruktorgenommen, also nachdem die Funktion initialisiereGUI() aufgerufen wurde. Aber ich kann dort die Objekte aus der Funktion nicht ansprechen. Weder das btnSpeichern noch das frmFrame. Muss dafür in der initialisereGUI() etwas anders deklariert oder zurückgegeben werden?

Sorry für die Anfängerfragen.

Die Klasse sieht eigentlich jetzt so aus:

public class GuiPlatzErfassen {

   private JPanel contentPane;
   private JTextField textField;
   private JComboBox comboBoxSportart;
   private JComboBox comboBoxPlatznummer;
   private JButton btnSpeichern;
   private JButton btnAbbrechen;
   private JFrame frmFrame;

   
   java.sql.Connection connection=null;

   public void fuelleComboSportart(){
     connection=MysqlConnector.dbConnector();
     
     try {

       Statement statement = connection.createStatement();
       String query = "SELECT * FROM tblSportart";
       ResultSet rs = statement.executeQuery(query);
       
       while(rs.next()){
         comboBoxSportart.addItem(rs.getString("saBezeichnung"));
       }
       connection.close();

     } catch (Exception e) {
       e.printStackTrace();
     }
   }
   
   public void fuelleComboPlatznummer(){
     try {
       
       int i = 1;
       while(i<11){
         
         comboBoxPlatznummer.addItem(i);
         i = i+1;
       }

     } catch (Exception e) {
       e.printStackTrace();
     }
   }
   
   public void initialisiereGUI() {
     JFrame frmFrame = new JFrame();
     frmFrame.setTitle("Sportplatz erfassen");
     
     frmFrame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
     frmFrame.setBounds(100, 100, 354, 268);
     contentPane = new JPanel();
     contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
     frmFrame.setContentPane(contentPane);
     contentPane.setLayout(null);
     JLabel lblSportart = new JLabel("Sportart");
     lblSportart.setBounds(45, 40, 92, 14);
     contentPane.add(lblSportart);
     
     JLabel lblPlatznr = new JLabel("Platznr");
     lblPlatznr.setBounds(45, 85, 46, 14);
     contentPane.add(lblPlatznr);
     
     JLabel lblPreis = new JLabel("Preis / h");
     lblPreis.setBounds(45, 128, 68, 14);
     contentPane.add(lblPreis);
     
     textField = new JTextField();
     textField.setBounds(158, 125, 46, 20);
     contentPane.add(textField);
     textField.setColumns(10);
     
     JButton btnSpeichern = new JButton("Speichern");
     btnSpeichern.setBounds(45, 171, 103, 23);
     btnSpeichern.setBackground(new Color(255, 255, 204));
     contentPane.add(btnSpeichern);
     
     JButton btnAbbrechen = new JButton("Abbrechen");
     btnAbbrechen.setBounds(158, 171, 108, 23);
     btnAbbrechen.setBackground(new Color(255, 255, 204));
     contentPane.add(btnAbbrechen);
     
     comboBoxSportart = new JComboBox();
     comboBoxSportart.setBounds(158, 37, 127, 20);
     
     
     comboBoxPlatznummer = new JComboBox();
     comboBoxPlatznummer.setBounds(158, 82, 46, 20);
     
     contentPane.add(comboBoxPlatznummer);
     contentPane.add(comboBoxSportart);
     
     fuelleComboSportart();
     fuelleComboPlatznummer();

     
   }

   public GuiPlatzErfassen() {

     initialisiereGUI();
     System.out.println("ich sollte doch");              <-------- GEHT
     frmFrame.setTitle("Testtitel");       <-------- GEHT NICHT
     btnSpeichern.setBackground(new Color(255, 255, 204));      <-------- GEHT NICHT 
     frmFrame.setVisible(true);         <-------- GEHT NICHT

   }
   
}

 

[Joose]

Weil du in der Methode initialisiereGUI lokale Variablen erstellst welche initialisiert werden und dem JFrame hinzugefüt werden.

public class GuiPlatzErfassen {

   private JPanel contentPane;
   private JTextField textField;
   private JComboBox comboBoxSportart;
   private JComboBox comboBoxPlatznummer;
   private JButton btnSpeichern;
   private JButton btnAbbrechen;
   private JFrame frmFrame;

.....
  
   public void initialisiereGUI() {
     JFrame frmFrame = new JFrame();
     contentPane = new JPanel();
     contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
     frmFrame.setContentPane(contentPane);
     contentPane.setLayout(null);
     JLabel lblSportart = new JLabel("Sportart");    
     JLabel lblPlatznr = new JLabel("Platznr");
     JLabel lblPreis = new JLabel("Preis / h");
     textField = new JTextField();
     textField.setBounds(158, 125, 46, 20);
     contentPane.add(textField);
     textField.setColumns(10);
     JButton btnSpeichern = new JButton("Speichern");
     JButton btnAbbrechen = new JButton("Abbrechen");
     comboBoxSportart = new JComboBox();
     comboBoxPlatznummer = new JComboBox();
   }
}

Lass den Typ vor dem Variablennamen weg dann wird es funktionieren
Bei contentPane oder textField oder deinen ComboBoxen hast du es doch auch so gemacht und da funktioniert es

 

[breakplease]

Du bist mein Held!!
Ich habe mir die Infos für den Code aus unterschiedlichen Quellen (Buch und Netz) geholt, deswegen ist es nicht ganz so einheitlich

Vielen Dank!

 

[breakplease]

Guten Sonntag allerseits

Leider hat sich bei mir noch ein weiteres Problem eingeschlichen:
Ich stelle mit der Methode uebersichtPlaetze() eine Tabelle mit Werten aus einer DB dar. Mit einem Klick auf die entsprechende Zeile und anschliessend einen JButton "btnPlatzLoeschen" rufe ich eine Methode einer anderen Klasse auf, welche den entsprechenden EIntrag aus der DB löschen sollte. Das funktioniert im Grunde, nur nicht ganz zufriedenstellen
Wenn ich in der Tabelle mehrere EInträge nacheinander anwählen, dann werden anschliessend beim Klick auf den JButton alle jemals angewählten gelöscht, anstelle des nur aktiv gewählten. Und wenn die Methode uebersichtPlaetze() mehrfach aufgerufen wird (wenn ich noch ein zweiten mal auf btnPlaetze drücke) dann wird bei einem Klick auf den JButton btnPlatzEditieren das Gui gleich mehrfach geladen. Alles läuft dann mehrfach durch.


Hier wird die Platzübersicht aufgerufen:

    // Platzübersicht aufrufen
     btnPlaetze.addActionListener(new ActionListener() {
       public void actionPerformed(ActionEvent arg0) {
         uebersichtPlaetze();
         System.out.println("Knopf plaetze gedrueckt");
       }
       
     });

Hier wird die Tabelle und der Button für das löschen sichtbar gemacht. Wenn nun die Methode uebersichtPlaetze() ein zweites mal aufgerufen wird, läuft der ActionListener "public void actionPerformed(ActionEvent arg0)" mehrfach durch.
Ebenfalls wenn in der Tabelle mehrere Zeilen angeklickt werden, wird nicht nur ein Eintrag gelöscht, sondern alle die angeklickt wurden.

  public void uebersichtPlaetze(){

     System.out.println("Start Methode uebersichtPlaetze()");

     anzeigeUntermenu("Plaetze");
     try {   
       
       Plaetze platz = new Plaetze();
       ResultSet platzListe = platz.lesePlatz();
       table.setModel(DbUtils.resultSetToTableModel(platzListe));
       table.getColumn("saBezeichnung").setHeaderValue("Sportart");
       table.getColumn("plNummer").setHeaderValue("Platz Nummer");
       table.getColumn("plPreis").setHeaderValue("Preis / h");
       table.getColumnModel().getColumn(0).setMaxWidth(0);
       table.getColumnModel().getColumn(0).setMinWidth(0);
       table.getColumnModel().getColumn(0).setPreferredWidth(0);
       table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
   
       table.addMouseListener(new MouseAdapter() {
         public void mouseClicked(java.awt.event.MouseEvent e) {
           System.out.println("Zeile geklickt mit ID" + table.getValueAt(table.getSelectedRow(), 0));
             
          btnPlatzLoeschen.setVisible(true);
          btnPlatzEditieren.setVisible(true);
           
          // Lesen der PlatzID aus der Tabelle für Übergabewert in die Methoden
          int plID = (int) table.getValueAt(table.getSelectedRow(), 0);
          // Platz löschen
          btnPlatzLoeschen.addActionListener(new ActionListener() {
               public void actionPerformed(ActionEvent arg0) {
                 System.out.println("Knopf löschen gedrückt");
                 Plaetze platz = new Plaetze();
                   platz.loeschePlatz(plID);
                   System.out.println(plID);

             }
             });
             // Bestehender Platz editieren
             btnPlatzEditieren.addActionListener(new ActionListener() {
                 public void actionPerformed(ActionEvent e) {
                   GuiPlatz guiPlatz = new GuiPlatz();
                   Plaetze platz = new Plaetze();
               //      guiPlatz.editierePlatz(plID);
                 }
               });

          }
       });
     }
     catch (Exception exc) {
       exc.printStackTrace();
     }
     return;
   }

Hat wer evtl. eine Idee was da die Ursache sein könnte?

Vielen Dank

 

[breakplease]

Also vielleicht hier noch kurz vereinfacht.
So sieht das Programm aus:


Wenn ich oben rechts auf Plätze klicke und dann eine Zeile auswähle passt es. Wenn ich aber oben rechts mehrfach auf Plätze drücke und anschliessend eine einzelne Zeile wähle läuft der ActionListener so viel mal durch, wie ich Plätze geklickt habe:


Muss diese Methode irgendwie wie entladen werden?

 

[Joose]

Weil du sobald du einen Eintrag in der Tabelle auswählst den Button einen ActionListener hinzufügst. Wählst du eine andere Zeile aus entfernst du den ActionListener aber nicht
Sobald du auf den Button klickst reagieren natürlich alle ActionListener.

Anstatt bei jeder Änderung der Auswahl einen ActionListener hinzuzufügen füge ihn doch nur einmal hinzu

 

[breakplease]

Achso achso, jetzt wird mir einiges klar! Danke Dir!

Ich hab den ActionListener nun mal direkt in den Konstruktur platziert. Wenn nun eine Zeile geklickt wird sollte er mir die ID (aus Spalte 0) in die Variabel rowID schreiben. Wenn ich aber im ActionListener drin dem rowID einen neuen Wert zuweisen will geht das nicht. Die rowID lesen und ausgeben funktioniert hingegen problemlos.

public GuiUebersicht() {
     
     int rowID = 7;
     initialisiereUebersicht();
     connection=MysqlConnector.dbConnector();
     frmFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     frmFrame.setBounds(100, 100, 1024, 631);
     frmFrame.setVisible(true);
    
     //ActionListener Tabelle
     table.addMouseListener(new MouseAdapter() {

     public void mouseClicked(java.awt.event.MouseEvent e) {
         System.out.println("Zeile geklickt mit ID" + table.getValueAt(table.getSelectedRow(), 0));
         System.out.println(rowID);  <- Funktioniert. Gibt 7 aus
         rowID = 8; <- Das funktioniert nicht. Ich kann diese Variabel nicht neu beschreiben.
           
        btnPlatzLoeschen.setVisible(true);
        btnPlatzEditieren.setVisible(true);
         
        // Lesen der Zeilen ID aus der Tabelle für Übergabewert in die Methoden
        //rowID = (int) table.getValueAt(table.getSelectedRow(), 0);
        }
     });

Hast Du evtl. noch eine Idee wo es hier scheitert?
Sorry für die blöden Fragen.

 

[breakplease]

Hat sich erledigt. Habe nun zuoberst in der Klasse eine Static Variabel gemacht, nun passts:

public class GuiUebersicht {

   static String Navigation;
   static int TabellenID;

Good night