【问题标题】:Why never change the notifier in receiving a change event为什么在接收更改事件时从不更改通知程序
【发布时间】:2011-11-11 19:42:58
【问题描述】:

我在Java TableModelListener and Live Feed Listener? 上发布了一个答案,但我收到了来自 – kleopatra 的评论

nonono - you never change the notifier in receiving a change event. 
As to probable effects, think: nasty loops. As to code sanity, think: 
indecent intimacy. It's the task of the model itself to internally 
update related values if necessary.

可以向我解释什么是在接收更改事件时更改通知程序,可能发生什么,她的真正意思是什么,因为当我尝试了所有我知道我只从非常快速的循环中收到 RepaintManager 异常的事情时,

我再也没有遇到过异常,

  • 我将它与 50 x 1000 矩阵相乘,

  • 使用prepareRenderer(更改正/负值的颜色)

  • 刷新率为 175 毫秒

代码演示了如何更改通知器和另外两个(可能是正确的)方法

import java.awt.*;
import java.awt.event.ActionEvent;
import java.util.Random;
import java.util.concurrent.*;
import javax.swing.*;
import javax.swing.event.*;
import javax.swing.table.DefaultTableModel;

public class ChangeNotifiersOnEvent extends JFrame implements Runnable {

    private static final long serialVersionUID = 1L;
    private boolean runProcess = true;
    private Random random = new Random();
    private javax.swing.Timer timerRun;
    private Executor executor = Executors.newCachedThreadPool();
    private String[] columnNames = {"Source", "Hit", "Last", "Ur_Diff"};
    private JTable table;
    private Object[][] data = {{"Swing Timer", 2.99, 5, 1.01},
        {"Swing Worker", 7.10, 5, 1.010}, {"TableModelListener", 25.05, 5, 1.01}};
    private DefaultTableModel model = new DefaultTableModel(data, columnNames);

    public ChangeNotifiersOnEvent() {
        table = new JTable(model) {

            private static final long serialVersionUID = 1L;

            @Override
            public Class getColumnClass(int column) {
                return getValueAt(0, column).getClass();
            }
        };
        model.addTableModelListener(new TableModelListener() {

            @Override
            public void tableChanged(TableModelEvent tme) {
                if (tme.getType() == TableModelEvent.UPDATE) {
                    if (tme.getColumn() == 1 && tme.getLastRow() == 2) {
                        double dbl = ((Double) table.getModel().getValueAt(2, 1))
                                - ((Integer) table.getModel().getValueAt(2, 2));
                        table.getModel().setValueAt(dbl, 2, 3);
                    } else if (tme.getColumn() == 1 && tme.getLastRow() == 0) {
                        prepareUpdateTableCell();
                    } else if (tme.getColumn() == 1 && tme.getLastRow() == 1) {
                        executor.execute(new MyTask(MyTask.UPDATE_TABLE_COLUMN));
                    }
                }
            }
        });
        table.setRowHeight(30);
        table.setFont(new Font("Serif", Font.BOLD, 20));
        table.getColumnModel().getColumn(0).setPreferredWidth(180);
        table.setPreferredScrollableViewportSize(table.getPreferredSize());
        JScrollPane scrollPane = new JScrollPane(table);
        add(scrollPane, BorderLayout.CENTER);
        new Thread(this).start();
    }

    private void prepareUpdateTableCell() {
        timerRun = new javax.swing.Timer(10, UpdateTableCell());
        timerRun.setRepeats(false);
        timerRun.start();
    }

    private Action UpdateTableCell() {
        return new AbstractAction("Update Table Cell") {

            private static final long serialVersionUID = 1L;

            @Override
            public void actionPerformed(ActionEvent e) {
                double dbl = ((Double) table.getModel().getValueAt(0, 1))
                        - ((Integer) table.getModel().getValueAt(0, 2));
                table.getModel().setValueAt(dbl, 0, 3);
            }
        };
    }

    @Override
    public void run() {
        while (runProcess) {
            try {
                Thread.sleep(250);
            } catch (Exception e) {
                e.printStackTrace();
            }
            changeTableValues();
        }
    }

    private void changeTableValues() {
        Runnable doRun = new Runnable() {

            @Override
            public void run() {
                table.getModel().setValueAt(random.nextInt(128) + random.nextDouble(), 0, 1);
                table.getModel().setValueAt(random.nextInt(256) + random.nextDouble(), 1, 1);
                table.getModel().setValueAt(random.nextInt(512) + random.nextDouble(), 2, 1);

                table.getModel().setValueAt(random.nextInt(128), 0, 2);
                table.getModel().setValueAt(random.nextInt(128), 1, 2);
                table.getModel().setValueAt(random.nextInt(128), 2, 2);
            }
        };
        SwingUtilities.invokeLater(doRun);
    }

    private class MyTask extends SwingWorker<Void, Integer> {

        private static final String UPDATE_TABLE_COLUMN = "update";
        private String namePr;
        private double dbl;

        MyTask(String str) {
            this.namePr = str;
        }

        @Override
        protected Void doInBackground() throws Exception {
            dbl = ((Double) table.getModel().getValueAt(1, 1))
                    - ((Integer) table.getModel().getValueAt(1, 2));
            return null;
        }

        @Override
        protected void done() {
            SwingUtilities.invokeLater(new Runnable() {

                @Override
                public void run() {
                    table.getModel().setValueAt(dbl, 1, 3);
                }
            });
        }
    }

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

            @Override
            public void run() {
                ChangeNotifiersOnEvent frame = new ChangeNotifiersOnEvent();
                frame.setDefaultCloseOperation(EXIT_ON_CLOSE);
                frame.setLocation(150, 150);
                frame.pack();
                frame.setVisible(true);
            }
        });
    }
}

【问题讨论】:

  • 为什么你的 MyTask.done() 实现使用 invokeLater ?如果你需要在 EDT 上做某事 done() 是你的机会。
  • @Ryan 代码用于测试目的,使用 invokeLater ---> a) 通知 EDT,b) 将所需代码移动到 EDT 末尾(多次提到 JTextComponents 及其 XxxListeners、Focus 等...),确保 API 中的 done、process、publish、setProcess 实现相当保证所有事件都在 EDT 上完成,对不起,我讨厌 SwingWokrer
  • 只执行 invokeLater 的 SwingWorker.done() 方法闻起来像反模式。我理解“b/c 测试”。我仍然不相信 a) 或 b) 在这个例子中完成任何事情。如果您试图理解具有不必要的线程怪癖的东西并没有帮助。

标签: java swing events jtable


【解决方案1】:

我认为她的意思是,如果您不真正考虑您的代码,那么您可以引入无限循环。

大多数人在创建表格时可能会将第 1、2 列设为可编辑,而将第 3 列设为不可编辑,因为第 3 列只是两列之间的区别。

因此,当他们编写 TableModelListener 时,他们会检查 UPDATE 事件,但忘记检查哪一列已更新,因为他们认为该表不允许他们更新第 3 列。

他们忘记了当 TableModelListener 更新第 3 列时,将生成另一个 UPDATE 事件,从而导致无限循环。当然,正确的编码,如您的示例,将防止循环。

一般来说,它应该不会导致异常。

第二点是关于业务规则的。业务规则应该在一个地方定义,在这种情况下是模型。数据本身和数据的更新应该在一个地方完成。

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 2022-10-04
    • 1970-01-01
    • 2018-01-17
    • 1970-01-01
    • 1970-01-01
    • 2010-10-20
    • 2011-05-26
    相关资源
    最近更新 更多