fireTableDataChanged() funktioniert nicht was tun?

[Jaensk]

package GUI;

import RawMaterial.Customer;
import RawMaterial.CustomerCollection;
import java.awt.BorderLayout;
import java.awt.Font;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.JTextField;
import javax.swing.table.DefaultTableModel;

/**
 *
 * 
 */
public class AddCustomerFrame extends JFrame{
    
    private JPanel southPanel = new JPanel();
    private JButton addCustomerBtn = new JButton("Kunde hinzufügen");
    private JPanel kundeneingabe = new JPanel();
    private JTextField kundenNr = new JTextField(5);
    private JTextField firma = new JTextField(20);
    private JTextField strasse = new JTextField(20);
    private JTextField zip = new JTextField(5);
    private JTextField ort = new JTextField(20);
    private JTextField state = new JTextField(15);
    private JTextField telefon = new JTextField(10);
    private JTextField anspName = new JTextField(20);
    private JTextField anspVorname = new JTextField(20);
    private JTextField anspDurchwahl = new JTextField(10);
    private String[] anredeArray = {"-","Frau", "Herr"};
    private JComboBox anrede = new JComboBox(anredeArray);
    private DefaultTableModel model = 
      new DefaultTableModel(CustomerCollection.customerGetRowData(CustomerCollection.customerCollection), CustomerCollection.customerColumnNames);
    private JTable kundenTabelle = new JTable(model);
    
    
    public AddCustomerFrame(){
        setDefaultCloseOperation(HIDE_ON_CLOSE);
        setLayout(new BorderLayout());
        kundeneingabe.setLayout(new GridBagLayout());
        GridBagConstraints gbc = new GridBagConstraints();
        gbc.gridx = 0;
        gbc.gridy = 0;
        gbc.anchor = gbc.WEST;
        kundeneingabe.add(new JLabel("Kunden Nr."));
        gbc.gridx++;
        kundeneingabe.add(kundenNr, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Firma"), gbc);
        gbc.gridx++;
        kundeneingabe.add(firma, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Strasse"), gbc);
        gbc.gridx++;
        kundeneingabe.add(strasse, gbc);
        gbc.gridx = 0;
        gbc.gridy = 3;
        kundeneingabe.add(new JLabel("ZIP"), gbc);
        gbc.gridx++;
        kundeneingabe.add(zip , gbc);
        gbc.gridx = 0;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Ort"), gbc);
        gbc.gridx++;
        kundeneingabe.add(ort, gbc);
        gbc.gridx = 0;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("State"), gbc);
        gbc.gridx++;
        kundeneingabe.add(state, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Telefon"), gbc);
        gbc.gridx++;
        kundeneingabe.add(telefon, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel(" "), gbc);
        gbc.gridy++;
        gbc.gridx++;
        JLabel anspLabel = new JLabel("Ansprechpartner");
        anspLabel.setFont(new Font("Arial", Font.BOLD, 15));
        kundeneingabe.add(anspLabel, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Anrede"), gbc);
        gbc.gridx++;
        kundeneingabe.add(anrede, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Name"), gbc);
        gbc.gridx++;
        kundeneingabe.add(anspName, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Vorname"), gbc);
        gbc.gridx++;
        kundeneingabe.add(anspVorname, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Durchwahl "), gbc);
        gbc.gridx++;
        kundeneingabe.add(anspDurchwahl, gbc);
        
        
        
        add(kundeneingabe, BorderLayout.NORTH);
        add(new JScrollPane(kundenTabelle), BorderLayout.CENTER);
        southPanel.add(addCustomerBtn);
        add(southPanel, BorderLayout.SOUTH);
        
        pack();
        setVisible(true);
        
        addCustomerBtn.addActionListener(new ActionListener(){

            public void actionPerformed(ActionEvent e) {
                CustomerCollection.customerCollection.add(new Customer(firma.getText(), strasse.getText(), zip.getText(),ort.getText(), state.getText(), anspVorname.getText(), anspName.getText(), anspDurchwahl.getText(), anrede.getSelectedIndex()));
               model.fireTableDataChanged();
            }
            
        });

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

Die Methode fireTableDataChanged() funktionier nicht was habe ich falsch gemacht bzw. was könnte ich machen das es funktioniert. (Zeile 132)

Danke

Gruß

 

[SlaterB]

was CustomerCollection so alles macht kann ein externer nicht nachvollziehen,
ansonsten siehts gut aus,

folgende Umstellung auf Vector funktioniert bei mir:

class AddCustomerFrame
    extends JFrame
{

    private JPanel southPanel = new JPanel();
    private JButton addCustomerBtn = new JButton("Kunde hinzufügen");
    private JPanel kundeneingabe = new JPanel();
    private JTextField kundenNr = new JTextField(5);
    private JTextField firma = new JTextField(20);
    private JTextField strasse = new JTextField(20);
    private JTextField zip = new JTextField(5);
    private JTextField ort = new JTextField(20);
    private JTextField state = new JTextField(15);
    private JTextField telefon = new JTextField(10);
    private JTextField anspName = new JTextField(20);
    private JTextField anspVorname = new JTextField(20);
    private JTextField anspDurchwahl = new JTextField(10);
    private String[] anredeArray =
        {"-", "Frau", "Herr"};
    private JComboBox anrede = new JComboBox(anredeArray);
    private Vector data = new Vector();
    private DefaultTableModel model = null;
    private JTable kundenTabelle ;


    public AddCustomerFrame()
    {
        Vector row = new Vector();
        row.add("A1");
        row.add("B1");
        data.add(row);
        
        Vector col = new Vector();
        col.add("A");
        col.add("B");
        model = new DefaultTableModel(data,col);
        kundenTabelle = new JTable(model);

        setDefaultCloseOperation(HIDE_ON_CLOSE);
        setLayout(new BorderLayout());
        kundeneingabe.setLayout(new GridBagLayout());
        GridBagConstraints gbc = new GridBagConstraints();
        gbc.gridx = 0;
        gbc.gridy = 0;
        gbc.anchor = gbc.WEST;
        kundeneingabe.add(new JLabel("Kunden Nr."));
        gbc.gridx++;
        kundeneingabe.add(kundenNr, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Firma"), gbc);
        gbc.gridx++;
        kundeneingabe.add(firma, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Strasse"), gbc);
        gbc.gridx++;
        kundeneingabe.add(strasse, gbc);
        gbc.gridx = 0;
        gbc.gridy = 3;
        kundeneingabe.add(new JLabel("ZIP"), gbc);
        gbc.gridx++;
        kundeneingabe.add(zip, gbc);
        gbc.gridx = 0;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Ort"), gbc);
        gbc.gridx++;
        kundeneingabe.add(ort, gbc);
        gbc.gridx = 0;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("State"), gbc);
        gbc.gridx++;
        kundeneingabe.add(state, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Telefon"), gbc);
        gbc.gridx++;
        kundeneingabe.add(telefon, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel(" "), gbc);
        gbc.gridy++;
        gbc.gridx++;
        JLabel anspLabel = new JLabel("Ansprechpartner");
        anspLabel.setFont(new Font("Arial", Font.BOLD, 15));
        kundeneingabe.add(anspLabel, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Anrede"), gbc);
        gbc.gridx++;
        kundeneingabe.add(anrede, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Name"), gbc);
        gbc.gridx++;
        kundeneingabe.add(anspName, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Vorname"), gbc);
        gbc.gridx++;
        kundeneingabe.add(anspVorname, gbc);
        gbc.gridx--;
        gbc.gridy++;
        kundeneingabe.add(new JLabel("Durchwahl "), gbc);
        gbc.gridx++;
        kundeneingabe.add(anspDurchwahl, gbc);


        add(kundeneingabe, BorderLayout.NORTH);
        add(new JScrollPane(kundenTabelle), BorderLayout.CENTER);
        southPanel.add(addCustomerBtn);
        add(southPanel, BorderLayout.SOUTH);

        pack();
        setVisible(true);

        addCustomerBtn.addActionListener(new ActionListener()
            {

                public void actionPerformed(ActionEvent e)
                {
                    Vector row = new Vector();
                    row.add("A2");
                    row.add("B2");
                    data.add(row);
                    
                    model.fireTableDataChanged();
                }

            });

    }

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

Tipp: wenn man solche Testdaten verwendet wie ich in meinem Beispiel, dann kann man den ganzen Quark mit den TextFeldern eigentlich weglassen,
stören nur

 

[Jaensk]

hmm hab dein programm ausprobiert da funktioniert alles. ich habe statt einem vector ein object[][] array verwendet aber sollte doch eigentlich auch funktionieren. kann es eventuell daran liegen, dass der vector erst im konstruktor initialisiert wird?

 

[SlaterB]

wenn du vorher null übergeben hast, dann ist das natürlich schlecht

ein Object[][] wird glaube ich bei der Übergabe in einen Vector kopiert, sonst könnte man ja keine neuen Zeilen einfügen