java - 像这样创建 Java Frame 是一个好习惯吗？

Question

我问这个是因为当我开始学习 GUI 时，我不太喜欢其他人使用的方法，所以我自己制作了这个方法。下面的代码是创建框架的类。

  class Frame{
        private JFrame frm;
        private JLabel desc;
        private JTextField username;
        private JPasswordField password;

        Frame(){
            //Creating and setting the frame
            frm = new JFrame();
            frm.setLayout(new FlowLayout());
            frm.setDefaultCloseOperation(frm.EXIT_ON_CLOSE);
            frm.setResizable(true);
            frm.setSize(300, 300);
            frm.setLocationRelativeTo(null);
            frm.setTitle("Default title");

            //Initializing variables
            desc = new JLabel("This is the description");
            username = new JTextField("Username");
            password = new JPasswordField("Password");
            EventHandler handler = new EventHandler();          

            //Adding components to the frame
            frm.add(desc);
            frm.add(username);
            frm.add(password);

            //Handling the components
            username.addActionListener(handler);
            password.addActionListener(handler);

            //Showing the frame
            frm.setVisible(true);
        }

        class EventHandler implements ActionListener{   
            public void actionPerformed(ActionEvent event){
                if(event.getSource() == username){
                    JOptionPane.showMessageDialog(null, "You typed \"" + username.getText() + "\" inside the username box.");
                } else if(event.getSource() == password){
                    JOptionPane.showMessageDialog(null, "You typed \"" + password.getText() + "\" inside the password box.");
                }
            }
        }
    }

在主类中简单地放一些类似的东西

Frame frm = new Frame();

创建它。

Answer 1

看起来不错。

1. Java 类名应该以大写字母开头。 Frame 是一个 Java 类，因此您应该将您的类命名为 MyFrame(或 Frame 以外的任何名称)。

2. 你需要把你的类(class)放在Event Dispatch thread上(美东时间)。您可以使用如下代码执行此操作:

.

SwingUtilities.invokeLater(new Runnable() {
    public void run() {
        new MyFrame();
    }
});