Очень медленное повторное заселение JList - PullRequest
3 голосов
/ 26 августа 2010

У меня есть компонент JList, который должен быть очищен и заполнен.Следующий код (на основе моего исходного кода) показывает простое окно с JList и JButton:

import java.awt.BorderLayout;
import javax.swing.*;

public class JListTest extends javax.swing.JFrame{
    JList jList;
    JButton button;
    DefaultListModel model;

    public JListTest() {
        jList = new JList();
        model = new DefaultListModel();
        jList.setModel( model );
        button = new JButton();

        getContentPane().add(jList, java.awt.BorderLayout.CENTER);

        button.setText("add 10000 items");
        button.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                model.clear();
                for( int i=0; i<10000; ++i ) {
                    model.addElement( "aaaa");
                }
            }
        });
        getContentPane().add(button, BorderLayout.PAGE_START);        
        pack();        
    }

    public static void main(String args[]) {
        JListTest jlt =new JListTest();
        jlt.setSize(300, 300);
        jlt.setVisible( true );
    }
}

Если я нажимаю кнопку, вставка (10000 элементов) происходит очень быстро.Если я нажимаю его снова и снова, он все еще очень быстрый.

Если я выберу третий элемент и нажму кнопку, результат будет таким же, вставка будет очень быстрой.

Если явыберите первый элемент и нажмите кнопку, программа работает очень медленно (на самом деле я должен остановить ее).

Почему выбор первого элемента замедляет выполнение?

У меня естьпроверил это, используя JDK 1.5 и 1.6.

Ответы [ 3 ]

2 голосов
/ 26 августа 2010

Я бы предложил написать свою собственную модель, которая позволяет добавлять несколько значений сразу.Я предполагаю, что это не дополнение к модели, а вызванные этим вещи с графическим интерфейсом, которые убивают производительность.

1 голос
/ 26 августа 2010

Я не уверен, почему выбор элемента вызывает проблемы с производительностью.Но каждый раз, когда вы добавляете элемент, происходит событие, которое заставляет список перекрашивать себя.Поэтому, возможно, тот факт, что элемент выбран, требует дополнительной перекраски.

В любом случае, лучший способ сделать это - создать новую модель, а затем просто добавить ее в список:

    button.addActionListener(new java.awt.event.ActionListener() {
        public void actionPerformed(java.awt.event.ActionEvent evt) {
            DefaultListModel dlm = new DefaultListModel();
            for( int i=0; i<10000; ++i ) {
                dlm.addElement( "aaaa");
            }
            jList.setModel(dlm);
        }
    });

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

1 голос
/ 26 августа 2010

Вы не должны добавлять много элементов в модель в цикле событий, подобном этому. Гораздо лучше было бы, чтобы ваш слушатель действий порождал поток, чтобы добавить элементы, и чтобы этот поток вызывал SwingUtilities.invokeLater () для запуска события change в списке.

Обратите внимание, что согласно приведенному ниже комментарию вам необходимо создать AbstractListModel (или его подкласс), сделать его моделью и вызвать fireContentsChanged для него в invokeLater.

...