JTable#repaint() не работает должным образом

Это приложение извлекает данные из текстового файла и вставляет их в JTable. Observer прикрепляется для проверки каждые 300 мс на наличие изменений в файле и перезагрузки данных. У меня есть setChanged() и notifyObservers() в моем Observer классе.

При добавлении данных в таблицу getRowCount() сообщает, что строка добавлена, уведомители работают. Практически все работает, кроме repaint(). Я пробовал revalidate() и fireTableDataChanged() безрезультатно. Буду признателен за помощь в этом. На протяжении всего процесса не сообщается об ошибках компиляции.

Настольная модель

package HardwareDbFile;

import java.io.BufferedReader;
import java.io.FileInputStream;
import java.io.InputStreamReader;
import java.util.StringTokenizer;
import java.util.Vector;
import javax.swing.table.AbstractTableModel;

public class HardwareFileTableModel extends AbstractTableModel{
    protected Vector data;
    protected Vector columnNames ;
    protected String datafile;

    public HardwareFileTableModel(String file){
        datafile = file;
        initVectors();  
    }

    public void initVectors() {
        String aLine ;
        data = new Vector();
        columnNames = new Vector();
        try {
            FileInputStream fin =  new FileInputStream(datafile);
            try (BufferedReader br = new BufferedReader(new InputStreamReader(fin))) {
                StringTokenizer st1 = new StringTokenizer(br.readLine(), "|");
                while(st1.hasMoreTokens()) {
                    columnNames.addElement(st1.nextToken());
                }
                // extract data
                while ((aLine = br.readLine()) != null) {
                    StringTokenizer st2 = new StringTokenizer(aLine, "|");
                    while(st2.hasMoreTokens()) {
                        data.addElement(st2.nextToken());
                    }
                }
            }
        }
        catch (Exception e) {
            e.printStackTrace();
        }
    }
    @Override
    public int getRowCount() {
        return data.size() / getColumnCount();
    }

    @Override
    public int getColumnCount(){
        return columnNames.size();
    }

    @Override
    public String getColumnName(int columnIndex) {
        String colName = "";

        if (columnIndex <= getColumnCount()) {
            colName = (String)columnNames.elementAt(columnIndex);
        }
        return colName;
    }

    @Override
    public Class getColumnClass(int columnIndex){
        return String.class;
    }

    @Override
    public boolean isCellEditable(int rowIndex, int columnIndex) {
        return false;
    }


    public Object getValueAt(int rowIndex, int columnIndex) {
        return (String)data.elementAt( (rowIndex * getColumnCount()) + columnIndex);
    }


    public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
    }
}

Класс наблюдателя

package HardwareDbFile;

import java.awt.event.*;
import java.io.*;
import java.util.*;
import javax.swing.Timer;

 public class HardwareFileObserver extends Observable implements ActionListener {
     Timer time = new Timer(300,this); // check every 300ms
     long lastModified;
     String file ;

     HardwareFileObserver (String string) {
         file = string;
         File f = new File(file);
         lastModified = f.lastModified(); // original timestamp
         time.start();
     }

     @Override
     public void actionPerformed(ActionEvent e) {
         File f = new File(file);
         long actualLastModified = f.lastModified();
         if (lastModified != actualLastModified) {
             // the file have changed
             lastModified = actualLastModified;
             setChanged();
             notifyObservers();
         }
     }
}

Основной класс

    public class HardwareInventoryUI extends javax.swing.JFrame implements Observer {

    private String datafile = "hardware.dat";
    private String dataFilePath = "C:\\Windows\\Temp\\hardware.dat";
    protected HardwareFileTableModel model;

    /**
     * Creates new form HardwareInventoryUI
     */
    public HardwareInventoryUI() {
        initComponents();

        model = new HardwareFileTableModel(dataFilePath);
        HardwareFileObserver  fileWD;

        // this Observable object is monitoring any file change
        fileWD = new HardwareFileObserver(dataFilePath);
        fileWD.addObserver(this);
    }

    @Override
    public void update(Observable o, Object arg) {
        // reload data because data file have changed
        model.initVectors();
        jTable.repaint();
    }

Добавить кнопку записи

    private void jAddRecordActionPerformed(java.awt.event.ActionEvent evt) {                                           

        String toolID = jToolID.getText();
        String toolName = jToolName.getText();
        String quantity = jQuantity.getText();
        String itemPrice = jItemPrice.getText();
        try {
            model = new HardwareFileTableModel(dataFilePath);
            FileWriter fstream = new FileWriter(dataFilePath, true);
            try (BufferedWriter newRecord = new BufferedWriter(fstream)) {
                newRecord.newLine();
                newRecord.write(toolID + "|"+ toolName + "|" + quantity + "|" + itemPrice );
            }
        } catch (IOException ex) {
            Logger.getLogger(HardwareInventoryUI.class.getName()).log(Level.SEVERE, null, ex);
        }

    }

person Fred    schedule 01.12.2012    source источник
comment
как уже упоминал @trashgod: ваша реализация TableModel неправильна - она ​​должна уведомлять своих слушателей об изменении, у вас нет никакой возможности прокрасться вокруг этого контракта. Учтите, что в модели с хорошим поведением вам никогда не нужно задумываться об обновлениях представления, они просто происходят автоматически волшебным образом :-) Или наоборот: если вручную перерисовывается представление (например, как таблица, дерево, список) кажется решением - реализация модели неверна.   -  person kleopatra    schedule 02.12.2012


Ответы (1)


  1. Ваш TableModel должен реализовывать Observer, а не ваше представление. В методе update() обновите data и fireTableDataChanged(). JTable слушает свой TableModel и автоматически обновляется в ответ.

  2. Также рассмотрите альтернативные реализации шаблона наблюдателя, упомянутого здесь.

  3. Как показано здесь, ваша реализация setValueAt() неверна. Вам нужно запустить событие, которое уведомит таблицу прослушивания об изменении:

    @Override
    public void setValueAt(Object value, int row, int column) {
        // update data
        fireTableCellUpdated(row, column);
    }
    
  4. Если содержимое вашего datafile может быть получено как стандартный вывод команды, вы можете удалить файл, используя подход, описанный здесь .

Приложение: Как комментирует @kleopatra: «Обратите внимание, что с моделью с хорошим поведением вам никогда не нужно думать об обновлениях представления, они просто происходят автоматически :-) Или наоборот: если вручную перерисовывается представление ( такие как таблица, дерево, список) кажутся решением, реализация модели неверна».

person trashgod    schedule 02.12.2012