JMS 队列到 JTable 更新 [现在在更改后出现异常]

Question

我正在编写自己的 JMS 浏览器，并且对来自 JMS 服务器的消息的 JTable 更新感到震惊。 我尝试过AbstractTableModel TableModelListener在数据添加到LinkedList时使Jtable刷新。下面的逻辑有效，但它不实时更新，意味着我希望在从 QueueBrowser 添加到 LinkedList 时立即显示添加到 Jtable 的每一行。

我已根据以下建议更新了代码。

我做错了什么吗？谁能帮我吗？

QueueBrowser qb = session.createBrowser(q);
MsgTable mt = (MsgTable) queueTable.getModel();
mt.load(qb.getEnumeration(),mt);
qb.close();




 class MsgTable extends AbstractTableModel implements TableModelListener{


      final String[] columnNames = { "#", "Timestamp", "Type", "Mode",
        "Priority" };

      public void setRowSize(){

      }
      LinkedList queueList = new LinkedList();

      public int getRowCount() { if (queueList == null) { return 0; } else { return queueList.size();}}
      public int getColumnCount() { return columnNames.length;}
      public String getColumnName(int column) {return columnNames[column];}

      public Object getValueAt(int row, int column) {
       if(queueList == null){
        return null;
       }
                        Message m = (Message) queueList.get(row);
                        ...  
      }

      void load(Enumeration e,MsgTable mt) {
                         mt.addTableModelListener(this);
        while(e.hasMoreElements()){
         queueList.add(e.nextElement());

        }
                      fireTableDataChanged();   
      }

      Message getMessageAtRow(int row) {
       if (queueList == null)
        return null;
       return ((Message) queueList.get(row));
      }

                @Override
  public void tableChanged(TableModelEvent arg0) {
   // TODO Auto-generated method stub
   fireTableDataChanged();
  }

}

并得到这个异常。

Exception in thread "AWT-EventQueue-0" java.lang.StackOverflowError  at javax.swing.table.AbstractTableModel.fireTableRowsInserted(Unknown Source)

是不是错了？

Answer 1

需要添加几点供考虑：

• 您的 load(Enumeration e) 方法不需要抛出 JMSException，因为您只是迭代 Enumeration代码>.

• 您应该确保事件触发是在 EDT 内完成的。这可能很简单，只需将对 load 的调用包装在可运行的文件中并将其放入 SwingUtilities.invokeLater() 中即可：

  MsgTable mt = (MsgTable)queueTable.getModel();
  最终枚举 e = qb.getEnumeration();
  SwingUtilities.invokeLater(new Runnable() {
      公共无效运行（）{
          mt.load(e);
      }
  });
  

Answer 2

提高性能的一种方法是仅在 load() 方法末尾调用 fireTableDataChanged()，而不是在加载每一行之后调用。

这应该有帮助。

即：

void load(Enumeration e) throws JMSException {
                 while(e.hasMoreElements()){
                         queueList.add(e.nextElement());
                         fireTableDataChanged();
                 }                      
        }

到

void load(Enumeration e) throws JMSException {
                 boolean dataAdded = false;
                 while(e.hasMoreElements()){
                         queueList.add(e.nextElement());
                         dataAdded = true;
                     }
                 fireTableDataChanged();

        }

Answer 3

那么，正确的设计是创建一个 addRow(...) 方法，该方法接收一行数据并更新 TableModel 的内部存储。然后，此方法应调用 fileTableRowsInserted() 方法。您的 getValueAt() 方法也没有意义。根据您的模型，您有 5 列数据，但您从未检查列变量以返回正确的列对象。

查看 DefaultTableModel 的源代码，了解如何编码 insertRow() 和 getValueAt() 方法。

Answer 4

据我所知，当 TableModel 发生更改时，您的 JTable 应该会自动更新。查看关于使用表的 sun 教程特别是监听数据更改部分，这可能有帮助。也就是说，我有几点意见：

• 我并没有真正理解 getValueAt(int row, int col) 方法。难道你不应该得到第 row-th 消息和消息的第 col-th 属性吗？

• 我会向 TableModel 的 MsgTable 实现添加 addRow(...) 和 addRows(...) 来更新内部模型并触发适当的事件。

• 您不需要实现 TableModelListener （无论如何，我看不到任何对 addTableModelListener(...) 的调用）

（编辑：OP 已用新代码更新了他的问题，所以我'我在下面相应地更新了我的答案。）

您已经修改了 load(...) 签名和正文以添加对 addTableModelListener(...) 的调用，我认为两种修改都不正确。

关于 addTableModelListener(...)，文档说：

将侦听器添加到每次数据模型发生更改时收到通知的列表。

关于各种 fireFooXxx(...) 方法：

通知所有侦听器[发生了更改]

因此，使用以下 TableModelListener 实现：

@Override
public void tableChanged(TableModelEvent arg0) {
    // TODO Auto-generated method stub
    fireTableDataChanged();
} 

您最终将进行无限递归调用（侦听器收到更改通知并触发一个事件来通知他再次等），因此 java.lang.StackOverflowError< /a>.

实际上，我仍然认为您不需要 TableModelListener （并且您注册它的方式在我看来是不正确的，请参阅 监听数据更改）。因此，我将删除 implements TableModelListener 并实现 load(...) 方法，如下所示：

void load(Enumeration e) {
    while(e.hasMoreElements()) {
        queueList.add(e.nextElement());
    }
    fireTableDataChanged();   
}