0

我正在尝试学习如何做 MVC 和 DAO,但我的实现现在失败了,我无法绕过它。

这是我的观点,或者它的相关部分:

package gui;

import javax.swing.JFrame;

public class LoginFrame{

    private JPanel contentPane;
    private final JLabel credentialsLabel = new JLabel("Credentials");
    private JTextField usernameField;
    private JPasswordField passwordField;
    private JButton btnLogin;

    /**
     * Create the frame.
     */
    public LoginFrame() {
        createAndShowGUI();

    }

    private void createAndShowGUI()
    {
            EventQueue.invokeLater(new Runnable() {
            public void run() {
            try {
                    JFrame frame = new JFrame();
                    frame.setTitle("Login");
                    frame.setContentPane(createContentPane());
                    frame.setBounds(100, 100, 480, 237);
                    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            } 
        });

    }


    //A function that sets up the basic structure of the GUI.
    private JPanel createContentPane() {

        contentPane = new JPanel();
        contentPane.setForeground(Color.LIGHT_GRAY);
        contentPane.setBackground(Color.DARK_GRAY);
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        GridBagLayout gbl_contentPane = new GridBagLayout();
        gbl_contentPane.columnWidths = new int[]{106, 0, 0, 153, 0};
        gbl_contentPane.rowHeights = new int[]{65, 27, 39, 36, 0, 0};
        gbl_contentPane.columnWeights = new double[]{0.0, 0.0, 0.0, 1.0, Double.MIN_VALUE};
        gbl_contentPane.rowWeights = new double[]{0.0, 0.0, 0.0, 0.0, 0.0, Double.MIN_VALUE};
        contentPane.setLayout(gbl_contentPane);



        //login button
        btnLogin = new JButton("login");
        GridBagConstraints gbc_btnLogin = new GridBagConstraints();
        gbc_btnLogin.anchor = GridBagConstraints.WEST;
        gbc_btnLogin.insets = new Insets(0, 0, 5, 5);
        gbc_btnLogin.gridx = 2;
        gbc_btnLogin.gridy = 3;
        contentPane.add(btnLogin, gbc_btnLogin);

        //register button 
        JButton btnRegister = new JButton("register");
        btnRegister.setFont(new Font("Helvetica", Font.PLAIN, 13));
        GridBagConstraints gbc_btnRegister = new GridBagConstraints();
        gbc_btnRegister.anchor = GridBagConstraints.WEST;
        gbc_btnRegister.insets = new Insets(0, 0, 5, 0);
        gbc_btnRegister.gridx = 3;
        gbc_btnRegister.gridy = 3;
        contentPane.add(btnRegister, gbc_btnRegister);
        return contentPane; 
    }

    //the Action Listener for the Login Button passed by controller
    public void buttonActionListeners(ActionListener al) {
        btnLogin.setActionCommand("login");
        btnLogin.addActionListener(al); 
    }


}

我的模型:

package functions;

import database.LoginDAO;

public class LoginModel {
    LoginDAO logindao; 
    public LoginModel(LoginDAO logindao){
        this.logindao = logindao;
    }

    public void attemptLogin(String username, char[] password) {
        System.out.println("Testing login attempt"); 
        logindao.attemptLogin(username, password); 
    }


}

我的控制器:

package controller;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import functions.LoginModel;
import gui.LoginFrame;

public class LoginControl implements ActionListener {
    LoginModel model; 
    LoginFrame view; 

    public LoginControl(LoginModel model, LoginFrame view){
        this.model = model; 
        this.view = view;

        //add action listener from control to view. 
        view.buttonActionListeners(this); 
    }

    //action performed by view
    public void actionPerformed(ActionEvent ae)
    {
        System.out.println("Testing Action Performed");
        String action = ae.getActionCommand(); 
        if(action.equals("login")){
            System.out.println("Testing Action Performed");
            model.attemptLogin(view.getUsername(),view.getPassword()); 
        }
    }
}

主要的:

import functions.LoginModel;
import java.io.IOException;
import java.sql.SQLException;
import controller.LoginControl;
import database.LoginDAO;
import gui.LoginFrame;

public class Main {

    public static void main(String[] args) throws IOException, SQLException {
        LoginFrame frame = new LoginFrame(); 
        LoginDAO loginDao = new LoginDAO(); 
        LoginModel model = new LoginModel(loginDao);
        LoginControl controller = new LoginControl(model, frame); 
    }

}

错误:

Exception in thread "main" java.lang.NullPointerException
    at gui.LoginFrame.buttonActionListeners(LoginFrame.java:160)
    at controller.LoginControl.<init>(LoginControl.java:16)
    at Main.main(Main.java:14)

错误引用的行:

btnLogin.setActionCommand("login"); //in View
view.buttonActionListeners(this);  //in Control
LoginControl controller = new LoginControl(model, frame); //in main.

所以从我试图理解的内容来看:view.buttonActionListeners(this); 在这里的某个地方我传递了一个空值,所以这是空值还是视图是空值?如果我们启动 LoginFrame 视图,视图如何为空;

感谢您帮助我解决此错误。

4

1 回答 1

1

好的,我知道了。当您调用时,您的 btnLogin 属性为空

view.buttonActionListeners(this);

发生这种情况是因为它是在 Runnable 上初始化的,这是异步的,并且可能尚未运行。从这部分代码中,我认为不需要异步调用它。所以我建议同步设置属性。所以应该是:

 private void createAndShowGUI()
{

                JFrame frame = new JFrame();
                frame.setTitle("Login");
                frame.setContentPane(createContentPane());
                frame.setBounds(100, 100, 480, 237);
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                frame.setVisible(true);
}
于 2013-10-10T00:25:19.520 回答