java - JList(与数组列表链接)添加元素导致内存不足或消失

Question

抱歉我的英语不好。我在尝试使用 java swing 库作为 GUI 的应用程序中遇到了一个问题，这让我遇到了很大的困难。

我将解释一个更简单的问题，以便您清楚地理解(如果我发布应用程序的源代码将更难理解，因为我为每个组件使用不同的类等...)

我正在使用 JList 来显示人员数组列表。看一下下面的代码:

import java.awt.BorderLayout;
import java.awt.Component;
import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;

import javax.swing.AbstractListModel;
import javax.swing.DefaultListModel;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JScrollPane;
import javax.swing.JTextField;
import javax.swing.ListCellRenderer;

public class Test {

    private JFrame frame;
    private ArrayList<Person> persons = new ArrayList<>();

    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    Test window = new Test();
                    window.frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }

    public Test() {
        initialize();
    }

    private void initialize() {
        frame = new JFrame();
        frame.setBounds(100, 100, 450, 300);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.getContentPane().setLayout(new BorderLayout());
        for (int i = 0; i < 111111; i++) {
            Person p = new Person(i);
            persons.add(p);
        }
        PersonList pl = new PersonList(persons);
        JScrollPane sp = new JScrollPane(pl);
        frame.getContentPane().add(sp, BorderLayout.CENTER);
        JButton addPerson = new JButton("add person");
        addPerson.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                Person p = new Person(166);
                /*
                 * This will make the list dissapear.
                 */
                // persons.add(p);
                // pl.repaint();
                // pl.revalidate();
                /*
                 * This will cause high memory usage on each button click.
                 */
                jlist.Test.PersonList.PersonModel m = (jlist.Test.PersonList.PersonModel) pl.getModel();
                m.add(p);
            }
        });
        frame.getContentPane().add(addPerson, BorderLayout.PAGE_END);

    }

    @SuppressWarnings("serial")
    private class PersonList extends JList<Person> {
        private ArrayList<Person> listPersons;

        public PersonList(ArrayList<Person> persons) {
            this.listPersons = persons;
            setModel(new PersonModel());
            setCellRenderer(new PersonRenderer());
        }

        private class PersonRenderer extends JTextField implements ListCellRenderer<Person> {

            @Override
            public Component getListCellRendererComponent(JList<? extends Person> list, Person value, int index,
                    boolean isSelected, boolean cellHasFocus) {
                setText("height: " + value.getHeight());
                return this;
            }

        }

        private class PersonModel extends AbstractListModel<Person> {
            public void add(Person p) {
                listPersons.add(p);
                fireContentsChanged(this, listPersons.size(), listPersons.size() - 1);
            }

            @Override
            public Person getElementAt(int arg0) {
                return listPersons.get(arg0);
            }

            @Override
            public int getSize() {
                return listPersons.size();
            }

        }
    }

    private class Person {
        private int height;

        public Person(int height) {
            this.setHeight(height);
        }

        @SuppressWarnings("unused")
        public int getHeight() {
            return height;
        }

        public void setHeight(int height) {
            this.height = height;
        }
    }
}

如果我将新的 Person 添加到我的主数据数组列表中，然后重新绘制并重新验证，列表将消失(在我的主应用程序中，如果我先删除一个 Person，然后添加一个，它将正常工作。如果我再添加一个后，它就会消失，这意味着，我猜，它的容量/元素大小出了问题)。

如果我使用 fireContentsChanged 方法，JList 将获取新的 Person 并将被重新绘制。但是，如果我的 Jlist 中有很多项目，这会导致每次单击按钮时使用的内存都会增加(大量)。

P.S:为了完成这项工作，我尝试了很多方法。我几乎写过所有包含 JList 的 StackOverflow 帖子(不是开玩笑)。我已经尝试了很多多线程和 swingworker 的方法，但没有任何改变。

我使用的一些东西是:

重新绘制我的滚动 Pane - 什么也不做。

删除并替换我的 ScrollPane - 每次都会增加内存。

更换整个 Jlistmodel - 每次都会占用更多内存。

更换整个面板(在我的主应用程序中)- 每次都会增加内存。

对模型使用 setSize - 每次都会增加内存。

编辑:我过去常常从 Windows 任务管理器中观察内存使用情况，但为了确保这一点，我使用 jconsole...

Answer 1

你看起来有点把事情过于复杂化了。其一，我不会使用 JTextField 作为渲染器，而是使用 DefaultListCellRenderer 并根据需要添加边框。我还建议不要扩展 JList，而是更简单地使用 JList。如果需要，您可以扩展模型，或者更简单地使用 DefaultListCellModel<Person> .

例如:

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.event.ActionEvent;

import javax.swing.*;

@SuppressWarnings("serial")
public class Test2 extends JPanel {
    private static final int MAX_PERSONS = 111111;
    private DefaultListModel<Person> personModel = new DefaultListModel<>();
    private JList<Person> personJList = new JList<>(personModel);

    public Test2() {
        personJList.setPrototypeCellValue(new Person(11111111));
        personJList.setCellRenderer(new PersonRenderer());
        personJList.setVisibleRowCount(12);
        JScrollPane scrollPane = new JScrollPane(personJList);
        scrollPane.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);

        for (int i = 0; i < MAX_PERSONS; i++) {
            personModel.addElement(new Person(i));
        }

        setLayout(new BorderLayout());
        add(scrollPane);
        add(new JButton(new AddPersonAction()), BorderLayout.PAGE_END);
    }

    private class AddPersonAction extends AbstractAction {
        public AddPersonAction() {
            super("Add Person");
        }

        @Override
        public void actionPerformed(ActionEvent arg0) {
            Person p = new Person(166);
            personModel.addElement(p);
        }
    }

    private class PersonRenderer extends DefaultListCellRenderer {
        @Override
        public Component getListCellRendererComponent(JList<?> list, Object value, int index,
                boolean isSelected, boolean cellHasFocus) {

            if (value != null) {
                value = ((Person) value).getHeight();
            } else {
                value = "";
            }
            JComponent renderer = (JComponent) super.getListCellRendererComponent(list, value,
                    index, isSelected, cellHasFocus);

            // if you want a border around your renderer:
            renderer.setBorder(BorderFactory.createLineBorder(Color.BLACK));
            return renderer;
        }
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> createAndShowGui());
    }

    private static void createAndShowGui() {
        Test2 mainPanel = new Test2();
        JFrame frame = new JFrame("Test2");
        frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        frame.add(mainPanel);
        frame.pack();
        frame.setLocationRelativeTo(null);
        frame.setVisible(true);
    }
}

public class Person {
    private int height;

    public Person(int height) {
        this.setHeight(height);
    }

    @SuppressWarnings("unused")
    public int getHeight() {
        return height;
    }

    public void setHeight(int height) {
        this.height = height;
    }
}