< Сильный > Цель :
Используйте 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")

2
Dreamspace President 1 Май 2019 в 15:23

2 ответа

Лучший ответ

Я еще не знаю источник проблемы (следовательно, это ответ как «вики сообщества», а не как личный ответ), но проблема исчезнет, если вы установите количество видимых строк 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();
    }
}
1
2 revs 1 Май 2019 в 13:25

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

valueList.setPrototypeCellValue("                ");

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

1
DontKnowMuchBut Getting Better 1 Май 2019 в 13:36