JList(与arraylist链接)添加元素导致高内存或消失

JList(linked with arraylist) adding element causes either high memory or dissapears

抱歉我的英语不好。我一直很难解决我在尝试使用 java swing 库作为其 GUI 的应用程序中遇到的问题。

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

我正在使用 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,然后添加一个,它将工作正常。如果我再添加一个,它将消失,这意味着,我猜,它的 capacity/size 个元素有问题).

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

P.S:为了完成这项工作,我尝试了很多方法。我几乎编写了所有包含 JList 的 Whosebug post(不是开玩笑)。我用多线程和 swingworker 尝试了很多东西,但没有任何改变。

我使用的一些东西是:

重新绘制我的滚动窗格 - 什么都不做。

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

替换整个 Jlistmodel - 每次都增加内存。

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

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

编辑: 我曾经从 Windows 任务管理器观察内存使用情况,但为了确保,我使用 jconsole...

你看起来是over-complicating东西有点。首先,我不会使用 JTextField 作为渲染器,而是使用 DefaaultListCellRenderer 并在需要时添加边框。我还建议不要扩展 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;
    }
}