JTable повторяющиеся значения в строке

У меня есть JTable, заполненный пользовательским DataModel (вставленный ниже), и когда я вызываю метод populate(), он заполняет таблицу дублирующимися данными - каждая строка заполняется с тем же значением и снова. Однако при ближайшем рассмотрении (просто используя println() в поле данных) модель данных не виновата - она содержит правильные данные в ожидаемом формате. Что дает?

import java.util.ArrayList;    
import javax.swing.table.AbstractTableModel;

@SuppressWarnings("serial") // we don't expect this app to ever use serialized classes.  EVER.
public class CollectionDataModel extends AbstractTableModel {
    private ArrayList<ArrayList<String>> data;

    public CollectionDataModel() {
        data = new ArrayList<ArrayList<String>>();
    }

    @Override
    public int getColumnCount() {
        if(data.isEmpty()) return 0;
        return data.get(0).size();
    }

    @Override
    public int getRowCount() {
        return data.size();
    }

    @Override
    public Object getValueAt(int rowIndex, int columnIndex) {
        if(rowIndex > getRowCount()) return null;
        if(columnIndex > getColumnCount()) return null;
        return data.get(rowIndex).get(columnIndex);
    }

    public void populate(Collection c) {
        data.clear();
        for(Item i : c.getItems()) {
            ArrayList<String> row = new ArrayList<String>();
            for(Property p : i.getProperties().values()) {
                row.add(p.toString());
            }
            data.add(row);
        }
        fireTableDataChanged();
    }

}

Ответ 1

Вот полный пример, который может оказаться полезным. Поскольку образец Map немодифицируется, я отсылаю вас к @mKorbel примеру о том, как переопределить isCellEditable() и setValueAt().

import java.awt.EventQueue;
import java.awt.GridLayout;
import java.util.Map;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.table.AbstractTableModel;

/** @see https://stackoverflow.com/questions/9132987 */
public class EnvTableTest extends JPanel {

    public EnvTableTest() {
        this.setLayout(new GridLayout());
        this.add(new JScrollPane(new JTable(new EnvDataModel())));
    }

    private static class EnvDataModel extends AbstractTableModel {

        private Map<String, String> data = System.getenv();
        private String[] keys;

        public EnvDataModel() {
            keys = data.keySet().toArray(new String[data.size()]);
        }

        @Override
        public String getColumnName(int col) {
            if (col == 0) {
                return "Key";
            } else {
                return "Value";
            }
        }

        @Override
        public int getColumnCount() {
            return 2;
        }

        @Override
        public int getRowCount() {
            return data.size();
        }

        @Override
        public Object getValueAt(int row, int col) {
            if (col == 0) {
                return keys[row];
            } else {
                return data.get(keys[row]);
            }
        }
    }

    private void display() {
        JFrame f = new JFrame("EnvTableTest");
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.add(this);
        f.pack();
        f.setLocationRelativeTo(null);
        f.setVisible(true);
    }

    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {

            @Override
            public void run() {
                new EnvTableTest().display();
            }
        });
    }
}

Ответ 2

Вы можете попытаться сделать изменения более атомарными.

public void populate(Collection c) {
    ArrayList<ArrayList<String>> data2 = new  ArrayList<ArrayList<String>>();
    for(Item i : c.getItems()) {
        ArrayList<String> row = new ArrayList<String>();
        for(Property p : i.getProperties().values()) {
            row.add(p.toString());
        }
        data2.add(row);
    }
    data = data2;
    fireTableDataChanged();
}

Я предполагаю, что populate вызывается снова перед завершением предыдущего вызова populate. И, вероятно, c изменяется во время его итерации.

Ответ 3

1) ваш TableModel не завершен, я пропускаю там много или требуемые методы для JTable's life_cycle, начиная с TableHeader и т.д.

2), так как есть много AbstactTableModels на основе HashMap, я бы предложил вернуть тип массивов, реализованный в API напрямую

Vector<Vector<Object or String>> data; 

String[][] or Object[][]

вместо

ArrayList<ArrayList<String>> data;

простые объяснения состоят в том, что XxxList возвращает столбец, а Vector или String [] возвращает строку

3) Я бы предложил использовать DefaultTableModel напрямую, тогда вам никогда не придется решать дубликаты или пропущенные столбцы/строки