GUI程序不工作

Question

我一直在一个应用程序上工作，在那里你设置了一个包含项目的类别列表，然后将其制成一个文本文件。但是我为它制作的GUI并不能完全工作。我认为错误位于称为Cat的JMenuItem(或"添加类别")或我显示类别(Update()方法)的方式中。它应该请求一个名称，创建一个以该名称命名的类别，并在JScrollPane中显示它，但什么也没有出现。下面是代码:

public class GUIBuilder {
    public JFrame frame;
    public JPanel LeftPanel;
    public JPanel RightPanel;
    public JScrollPane scroll;
    public JMenuBar bar;
    public JMenu File;
    public JMenu Add;
    Inventory inv;
    public void go() {
        frame = new JFrame();
        scroll = new JScrollPane();
        bar = new JMenuBar();
        File = new JMenu("File");
        Add = new JMenu("Add...");
        bar.add(File);
        bar.add(Add);
        JMenuItem Save = new JMenuItem(new AbstractAction("Save") {
            private static final long serialVersionUID = 1L;
            public void actionPerformed(ActionEvent arg0) {
            }
        });
        JMenuItem Load = new JMenuItem(new AbstractAction("Load") {
            private static final long serialVersionUID = 1L;
            public void actionPerformed(ActionEvent arg0) {
            }
        });
        JMenuItem Generate = new JMenuItem(new AbstractAction("Generate Text File") {
            private static final long serialVersionUID = 1L;
            public void actionPerformed(ActionEvent arg0) {
            }
        });
        File.add(Save);
        File.add(Load);
        File.add(Generate);
        JMenuItem Cat = new JMenuItem(new AbstractAction("Add Category") {
            private static final long serialVersionUID = 1L;
            JFrame Cat;
            public void actionPerformed(ActionEvent arg0) {
                Cat = new JFrame("Add Category");
                final JTextField name = new JTextField(15);
                JButton Submit = new JButton(new AbstractAction("Submit") {
                    private static final long serialVersionUID = 1L;
                    public void actionPerformed(ActionEvent arg0) {
                        String n = name .getText();
                        if (n != null) {
                            inv.addCategory(new Category(n));
                            pullThePlug();
                            GUIBuilder.this.Update();
                        }
                    }
                });
                Cat.setLayout(new BorderLayout());
                Cat.add(name, BorderLayout.CENTER);
                Cat.add(Submit, BorderLayout.SOUTH);
                Cat.setSize(250, 150);
                Cat.setVisible(true);
            }
            public void pullThePlug() {
                WindowEvent wev = new WindowEvent(Cat, WindowEvent.WINDOW_CLOSING);
                Toolkit.getDefaultToolkit().getSystemEventQueue().postEvent(wev);
            }
        });
        JMenuItem item = new JMenuItem(new AbstractAction("Add Item") {
            private static final long serialVersionUID = 1L;
            public void actionPerformed(ActionEvent arg0) {
            }
        });
        Add.add(Cat);
        Add.add(item);
        frame.setJMenuBar(bar);
        frame.setSize(500, 800);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);
        frame.add(scroll);
        inv = new Inventory();
    }
    public void Update() {
        for (int i = 0; i < inv.categories.size(); i++) {
            Category cat = inv.categories.get(i);
            JPanel p = new JPanel();
            JTextPane name = new JTextPane();
            name.setText(cat.getName());
            scroll.add(p);
        }
    }
}

Answer 1

您需要将JTextPane name添加到JPanel p。

(在我看来，你也缺少一个main方法，将构建一个GUIBuilder -但也许这是在另一个文件，你没有在你的问题中显示?)

---

同样，如果你创建一个空的JScrollPane，你需要通过scrollpane.getViewport().add(yourcomponent);添加组件。

您还需要一些布局管理。最简单的方法(可能也是您想要做的)是:

    public void Update() {
        JPanel p = new JPanel();
        for (int i = 0; i < inv.categories.size(); i++) {
            Category cat = inv.categories.get(i);
            JTextPane name = new JTextPane();
            p.add(name);        
            name.setText(cat.getName());                
        }
        scroll.getViewport().add(p);       
    }

Answer 2

最好在显示JFrame之前初始化inv。
此外，您应该交换语句

frame.setVisible(true);
frame.add(scroll);

frame.add(scroll);
frame.setVisible(true);

所以添加JScrollpane后不需要重新油漆。在组件已经显示的情况下添加组件是相当棘手的。
我真的建议你用frame.setVisible(true);结束你的方法，或者甚至把这个语句提取到一个自己的方法中，当go()结束时被调用。
…那么，最好将go()重命名为init()，但那是另一回事;)。在我看来，划分构造和显示框架是一种很好的做法。编辑:重读你的代码后，我看不到你的JScrollpane在哪里被填满。你在哪里添加了一些东西?
此外，您还应该阅读Java编码约定，因为属性不应该以大写字母开头。