Swing JList сильно замедляется, если выбрана 1-я запись (Windows 10)

Цель:
использовать JList для любых целей.

Проблема.
Если выбрана первая запись, а затем вы очистите модель и добавите новый контент, это будет чрезвычайно медленно, независимо от того, как часто вы это делаете Это. Только после того, как вы выберете другую запись — ту, которая не первая — поведение снова станет быстрым = нормальным.

Временное решение.
Каждый раз перед очисткой модели программно добавляйте две произвольные записи (две, потому что список может быть полностью пуст), а затем выбирайте запись с самым высоким допустимым индексом.

Вопрос.
Можете ли вы воспроизвести ошибку? Это известная ошибка? Если нет, может ли кто-нибудь опубликовать его, чтобы OpenJDK был исправлен?

Тестовая среда:
стандартная загрузка выпуска Oracle Java JDK8 202.
java version "1.8.0_202"
Java(TM) SE Runtime Environment (build 1.8.0_202-b08)
Java HotSpot(TM) 64-Bit Server VM (build 25.202-b08, mixed mode)
Windows 10 Домашняя, версия 1809 (сборка ОС 17763,437)
ЦП Intel(R) Core(TM) i7-6700K с тактовой частотой 4,00 ГГц 4,01 ГГц

Также воспроизводится с OpenJDK 11
openjdk version "11.0.2" 2019-01-15
OpenJDK Runtime Environment 18.9 (build 11.0.2+9)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.2+9, mixed mode)

SSCCE

Этот код минимален, поэтому он даже использует LookAndFeel по умолчанию. Изначально я столкнулся с проблемой при использовании L&F Windows.

package jlistslowdownbugdemo;

import javax.swing.*;
import java.awt.*;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Random;




final public class Main {


    final private static boolean ENABLE_FIX = false;


    public static void main(final String[] args) {


        SwingUtilities.invokeLater(() -> {


            final DefaultListModel<String> valueListModel = new DefaultListModel<>();
            final JList<String> valueList = new JList<>(valueListModel);


            final DefaultListModel<String> keyListModel = new DefaultListModel<>();
            for (String key : Main.DATA.keySet()) {
                keyListModel.addElement(key);
            }
            final JList<String> keyList = new JList<>(keyListModel);


            keyList.addListSelectionListener(e -> {

                if (e.getValueIsAdjusting()) { // To prevent SUPER-slowdown when the bug kicks. Not an important measure, just for comfortable demoing.


                    // Workaround for the bug: Add two entries (two, because list may be entirely empty),
                    // and select the last available entry (in the worst case, this is the 2nd).
                    // Just so that one is selected that IS NOT THE FIRST.
                    // This problem AND fix occurred in Oracle JDK 8 and Open JDK 11.
                    if (ENABLE_FIX) {
                        valueListModel.addElement(null);
                        valueListModel.addElement(null); // Make sure you add something else instead of null if your GUI/JList setup requires it.
                        valueList.setSelectedIndex(valueListModel.size() - 1);
                    }


                    valueListModel.clear();

                    final String key = keyList.getSelectedValue();
                    if (key != null) {
                        for (String value : Main.DATA.get(key)) {
                            valueListModel.addElement(value);
                        }
                    }

                }
            });


            final JPanel contentPane = new JPanel(new GridLayout(1, 0, 0, 0));
            contentPane.add(new JScrollPane(keyList));
            contentPane.add(new JScrollPane(valueList));


            final JFrame window = new JFrame();
            window.setContentPane(contentPane);
            window.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
            window.setMinimumSize(new Dimension(1200, 1000));
            window.setLocationRelativeTo(null);
            window.setVisible(true);


            final String msg = "Click&drag in the left list. This rapidly changes the content of the right list.\n" +
                    "\n" +
                    "Now select any entry EXCEPT the first entry of the right list. Click&drag again in the left list,\n" +
                    "it still works just as rapidly. Now click THE FIRST entry of the right list.\n" +
                    "\n" +
                    "If you NOW click&drag again in the left list, you will experience BRUTAL slowdown.\n" +
                    "\n" +
                    "Once Swing has calmed down, click any entry EXCEPT the first entry of the right list.\n" +
                    "Click&drag again in the left list - the problem is gone.\n" +
                    "\n" +
                    "The built-in workaround (off by default) simulates this.";

            JOptionPane.showMessageDialog(window,
                                          msg,
                                          "How to reproduce the Swing bug:",
                                          JOptionPane.INFORMATION_MESSAGE);


        });

    }


    final private static Random RAND = new Random(0);


    final private static Map<String, List<String>> DATA = createDataMap();


    private static Map<String, List<String>> createDataMap() {

        final Map<String, List<String>> ret = new HashMap<>();

        for (int i = 0; i < 30; i++) {


            final int listSize = 20 + RAND.nextInt(5000);


            final String key = generateRandomString('A');
            final List<String> value = new ArrayList<>(listSize);
            ret.put(key, value);


            for (int ii = 0; ii < listSize; ii++) {
                value.add(generateRandomString('a'));
            }

        }

        return ret;
    }


    private static String generateRandomString(final char baseChar) {

        final StringBuilder sb = new StringBuilder();

        final int len = 4 + RAND.nextInt(17);

        for (int i = 0; i < len; i++) {
            sb.append((char) (baseChar + RAND.nextInt(26)));
        }

        return sb.toString();
    }


}

(Название отредактировано: удалено, определенно ошибка Java)


person Dreamspace President    schedule 01.05.2019    source источник
comment
Спасибо. (Отредактировал заголовок.) ... Но, ммм, я меняю модель списка other. Никаких изменений в списке, в котором есть прослушиватель событий, не происходит.   -  person Dreamspace President    schedule 01.05.2019
comment
Что должен делать этот код?   -  person DontKnowMuchBut Getting Better    schedule 01.05.2019
comment
Очевидно, это должно продемонстрировать ошибку.   -  person Dreamspace President    schedule 01.05.2019
comment
Но 1+ за интересную задачу   -  person DontKnowMuchBut Getting Better    schedule 01.05.2019


Ответы (3)


Я еще не знаю источник проблемы (поэтому на это отвечает как «вики сообщества», а не как личный ответ), но проблема исчезнет, ​​если вы установите количество видимых строк JList и значение ячейки прототипа ( что является передовой практикой Swing).

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

Рабочий код:

import javax.swing.*;
import java.awt.*;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Random;

final public class Foo {

    public static void main(final String[] args) {
        SwingUtilities.invokeLater(() -> {
            final DefaultListModel<String> valueListModel = new DefaultListModel<>();
            final JList<String> valueList = new JList<>(valueListModel);
            final DefaultListModel<String> keyListModel = new DefaultListModel<>();
            for (String key : Foo.DATA.keySet()) {
                keyListModel.addElement(key);
            }
            final JList<String> keyList = new JList<>(keyListModel);
            keyList.addListSelectionListener(e -> {
                if (e.getValueIsAdjusting()) {
                    valueListModel.clear();
                    final String key = keyList.getSelectedValue();
                    if (key != null) {
                        for (String value : Foo.DATA.get(key)) {
                            valueListModel.addElement(value);
                        }
                    }
                }
            });

            keyList.setVisibleRowCount(25);
            valueList.setVisibleRowCount(25);
            String prototypeFormat = "%100s";
            keyList.setPrototypeCellValue(String.format(prototypeFormat, " "));
            valueList.setPrototypeCellValue(String.format(prototypeFormat, " "));
            final JPanel contentPane = new JPanel(new GridLayout(1, 0, 0, 0));
            contentPane.add(new JScrollPane(keyList));
            contentPane.add(new JScrollPane(valueList));
            final JFrame window = new JFrame();
            window.setContentPane(contentPane);
            window.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
            // window.setMinimumSize(new Dimension(1200, 1000));
            window.pack();
            window.setLocationRelativeTo(null);
            window.setVisible(true);
        });
    }

    final private static Random RAND = new Random(0);
    final private static Map<String, List<String>> DATA = createDataMap();

    private static Map<String, List<String>> createDataMap() {
        final Map<String, List<String>> ret = new HashMap<>();
        for (int i = 0; i < 30; i++) {
            final int listSize = 20 + RAND.nextInt(5000);
            final String key = generateRandomString('A');
            final List<String> value = new ArrayList<>(listSize);
            ret.put(key, value);
            for (int ii = 0; ii < listSize; ii++) {
                value.add(generateRandomString('a'));
            }
        }
        return ret;
    }

    private static String generateRandomString(final char baseChar) {
        final StringBuilder sb = new StringBuilder();
        final int len = 4 + RAND.nextInt(17);
        for (int i = 0; i < len; i++) {
            sb.append((char) (baseChar + RAND.nextInt(26)));
        }
        return sb.toString();
    }
}
person Community    schedule 01.05.2019

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

valueList.setPrototypeCellValue("                ");

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

person DontKnowMuchBut Getting Better    schedule 01.05.2019
comment
Действительно, эта одна строка (даже с одним пробелом) решает проблему. Хорошо знать. Я все еще думаю, что это ошибка из-за странных обстоятельств и эффектов. Кстати, в JavaDocs сказано: вычисляет свойства fixedCellWidth и fixedCellHeight, запрашивая компонент рендеринга ячеек для заданного значения (и индекса 0) из рендерера ячеек - Таким образом, есть что-то особенное в отношении индекса 0 в отношении этого метода!< /я> - person Dreamspace President; 01.05.2019
comment
... Нет, все поведение (включая мой обходной путь и ваше исправление) остается прежним без JScrollPanes. - person Dreamspace President; 01.05.2019
comment
@DreamspacePresident тогда, чтобы получить полное решение, кому-то, вероятно, потребуется изучить исходный код. Этим кем-то буду не я, так как я в отпуске и собираюсь в поход. Удачи, будем смотреть. - person DontKnowMuchBut Getting Better; 01.05.2019
comment
Да, да, давайте потеряем весь день для этого. Проблема действительно в BasicListUI и ListDataListener, который регистрируется в ListModel. Спасибо за эту строчку. Удаление слушателей данных, затем добавление элемента, затем добавление слушателей обратно (возможно, до добавления последнего элемента) тоже работает. - person George Z.; 22.06.2021

Каждый раз, когда вы выполняете это:

valueListModel.addElement(value);

Вы делаете правильную перерисовку JList. Выполнение этого десятки раз является причиной вашего замедления.

Вам нужно изменить всю модель сразу. Замените это:

final String key = keyList.getSelectedValue();
if (key != null) {
    for (String value : Main.DATA.get(key)) {
        valueListModel.addElement(value);
    }
}

с этим:

final String key = keyList.getSelectedValue();
if (key != null) {
    valueListModel.addAll(Main.DATA.get(key));
}

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

Обновление:

В Java 8 вы хотите добиться того же эффекта «установить всю модель сразу», просто создав новый ListModel. Замените это:

valueListModel.clear();

final String key = keyList.getSelectedValue();
if (key != null) {
    for (String value : Main.DATA.get(key)) {
        valueListModel.addElement(value);
    }
}

с этим:

DefaultListModel<String> newListModel = new DefaultListModel<>();

final String key = keyList.getSelectedValue();
if (key != null) {
    for (String value : JListSlowDown.DATA.get(key)) {
        newListModel.addElement(value);
    }
}

valueList.setModel(newListModel);
person VGR    schedule 01.05.2019
comment
Две вещи неверны в вашем ответе, но он поднимает хорошие вопросы и допускает третье решение, так что все в порядке. 1) Сделав мой JList анонимным классом и переопределив методы paint и paintComponent с вызовами super и println, я мог определить, что методы вызываются только один раз для каждого щелчка в левом списке, а не для каждой записи. Кроме того, почему проблема иногда возникает, а иногда нет? 2) DefaultListModel не имеет метода addAll. Однако вы вдохновили каждый раз заменять всю модель списка. ЭТО ТОЖЕ ИЗБЕГАЕТ ПРОБЛЕМЫ! - person Dreamspace President; 01.05.2019
comment
Конечно, похоже, что есть метод DefaultListModel.addAll. А система рисования Swing сложна и сильно оптимизирована. Вы также проверяли, как часто вызывался модуль рендеринга ячеек? - person VGR; 01.05.2019
comment
Нет, я этого не проверял. Я бы не знал, какой из методов отслеживать, и мне пришлось бы отделять несколько вызовов для одного и того же объекта от нескольких вызовов из-за нескольких записей в списке. И я только что увидел, что метод addAll действительно существует — начиная с Java 11. Я в основном работаю с 8. - person Dreamspace President; 01.05.2019
comment
Обновлено для Java 8. Вы можете создать новую ListModel и вызвать с ней метод setModel JList, достигнув того же эффекта, а именно, одного события данных списка вместо многих. - person VGR; 01.05.2019
comment
Что я и написал в своем первом комментарии. Но спасибо. И читатели оценят правку. - person Dreamspace President; 01.05.2019