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

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

1 голос
/ 01 мая 2019

Цель:
Используйте 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 Home, версия 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")

Ответы [ 3 ]

1 голос

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

valueList.setPrototypeCellValue("                ");

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

1 голос
/ 01 мая 2019

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

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);
1 голос

Я пока не знаю источник проблемы (следовательно, это ответ в виде "вики сообщества", а не в качестве личного ответа), но проблема исчезнет, ​​если вы установите количество видимых строк 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();
    }
}
...