如何在类的其他方法中使用在构造函数中初始化的对象

Question

在这里编程新手。 我正在使用Swing，JDBC（Oracle）创建一个简单的登录应用程序。 它在大多数情况下都可以正常工作。 该应用程序具有3类-逻辑，UI和DBConnection。

请在此处参考整个代码： 完整代码

1. LoginLogic.java

    public class LoginLogic {

    LoginUI lu;

    public LoginLogic() {
        lu = new LoginUI();

    }

    public LoginLogic(ResultSet rs) {
        process(rs);
    }

    private void process(ResultSet rs) {
        try {
            if (rs.next()) {
                lu.loginSuccess();

            } else {
                lu.loginFailed();
            }
        } catch (SQLException e) {
            // TODO: handle exception
        }
    }

    public static void main(String[] args) {

        new LoginLogic();

    }


 }

最初，我直接显示了一个JOptionPane来代替loginSuccess（）和loginFailed（）方法，它按预期工作。 但是我想将所有UI功能都委托给UI类，因此我在UI类中创建了这两个方法

void loginSuccess() {
        JOptionPane.showMessageDialog(null, "Login Successful!");
}

void loginFailed() {
    JOptionPane.showMessageDialog(null, "Login Failed!");
} 

但是不会使用我在构造函数中创建的UI对象调用这些方法。 没有错误，但是也没有JOptionPane。

如何在process（）方法中使用UI对象引用lu来调用UI类的方法？

Answer 1

您的主要方法调用了无参数构造函数。 此无参数构造函数实现为

lu = new LoginUI();

因此，它创建了LoginUI对象，但仅此而已。 特别是，它永远不会调用process()方法，该方法显示JOptionPane。

您从未调用过的另一个构造函数会调用process()方法。 但是它不会初始化LoginUI对象。

我也看不到如何调用process()方法，因为在代码中没有任何东西创建该方法需要作为参数的ResultSet。

Answer 2

正如JB Nizet解释的那样，您的代码中需要进行一些更改。 尝试以下代码（对我有用）。

public class LoginLogic {

LoginUI lu;

public LoginLogic() {
    lu = new LoginUI();

}

public LoginLogic(ResultSet rs) {
    lu = new LoginUI();
    process(rs);
}

private void process(ResultSet rs) {
    try {
        if (rs.next()) {
            lu.loginSuccess();

        } else {
            lu.loginFailed();
        }
    } catch (SQLException e) {
        // TODO: handle exception
    }
}

public static void main(String[] args) {

    /*/ Here you shld get your result rs first 

            con is your Connection object
            Statement stmt = con.createStatement(
            ResultSet.TYPE_SCROLL_INSENSITIVE,
            ResultSet.CONCUR_UPDATABLE);

            ResultSet rs = stmt.executeQuery("SELECT yourColumn FROM yourTable");
    //*/

    new LoginLogic(rs);
}
}

已编辑

根据我们的评论，问题在于您正在以下位置创建一个新对象：

 private void getConn(String uname, String pwd) {
    ...
        // 4. Process the result set
        new LoginLogic(rs);

    ...

}

这意味着您没有在if（）语句中调用正确的对象。 您的新对象是使用参数化构造函数构造的，该构造函数不会初始化lu或对其进行更新。

编辑2

我认为您必须对设计进行一些更改。 我认为以下内容对您有用，但您必须处理例外情况。

public class LoginLogic {

LoginUI lu;

public LoginLogic() {
    lu = new LoginUI();

}


public void process() {
    try {
        if (lu.getRs().next()) {
            lu.loginSuccess();

        } else {
            lu.loginFailed();
        }
    } catch (SQLException e) {
        // TODO: handle exception
    }
}

public static void main(String[] args) {

    LoginLogic loginLogic = new LoginLogic();
    loginLogic.process();
 }
 }

loginUI将更改为：

public class LoginUI {

      ...
       ResultSet rs;

       public ResultSet getRs() {
          return rs;
       }
       ...
      btnLogin.addActionListener(new ActionListener() {

        @Override
        public void actionPerformed(ActionEvent e) {

            try {
                String uname = tfUname.getText().trim();
                String pwd = tfPwd.getText().trim();

                LoginDbConn loginDbConn = new LoginDbConn(uname, pwd);// startDB after user presses     
                rs = loginDbConn.getConn();

            } catch (Exception ex) {
                // TODO: handle exception
            }

        }
    });

    ....
 }

而LoginDbConn对此：

 public class LoginDbConn {

String uname;
String pwd;
ResultSet rs;
/*
 * public LoginDbConn() { // TODO Auto-generated constructor stub }
 */

public LoginDbConn(String uname, String pwd) {
    this.uname = uname;
    this.pwd = pwd;
}

public ResultSet getConn() {
    try {
        // 0. Register the JDBC drivers
        String driverClass = "oracle.jdbc.driver.OracleDriver";
        Class.forName(driverClass);
        // or DriverManager.registerDriver(new oracle.jdbc.OracleDriver());

        // 1. Get a connection to the Database
        String dbUrl = "jdbc:oracle:thin:@localhost:1521:xe";
        String dbuname = "scott";
        String dbpwd = "tiger";
        Connection conn = DriverManager.getConnection(dbUrl, dbuname, dbpwd);

        // 2. Create a statement
        // String sql = "SELECT * FROM users WHERE name = '"+uname+"' and
        // password = '"+pwd+"'";
        // Statement st = conn.createStatement();
        String sql = "select * from users where name = ? and password = ?";
        PreparedStatement pst = conn.prepareStatement(sql);

        pst.setString(1, uname);
        pst.setString(2, pwd);

        // 3. Execute SQL query
        rs = pst.executeQuery();

        // 5. Close Connection
        // conn.close();

        return rs;
    } catch (SQLException | ClassNotFoundException e) {

            // THIS EXCEPTION MUST ABSOLUTELY BE HANDLED
    }
    return rs;
}

}

希望这会有所帮助（如果是，请投票给答案）。

Answer 3

除了使用两个构造函数，您还可以通过一个构造函数实现此目的：

public LoginLogic(ResultSet rs) {
    lu = new LoginUI();
    process(rs);
}

您只是从您要呼叫的地方拨打了此电话。 您可以通过new LoginLogic(resultSet);来调用它new LoginLogic(resultSet);

Answer 4

感谢+ Guizmoo03指出了问题的根源。

如您在此代码中看到的，该类具有2个构造函数。 创建UI时，默认构造函数首先初始化从null开始的LoginUI对象引用。

public class LoginLogic {

    LoginUI lu;

    public LoginLogic() {
        lu = new LoginUI();

    }
     .....

但是，当Logic类的对象传递要处理的ResultSet到参数化构造函数时，将再次在DbConn类中创建该对象。 此实例取消引用UI对象变量lu 。 因此，正如我在最初的问题中指出的那样，它不再可以用来访问process（）方法中的UI类的方法。

...

LoginUI lu;

...

public LoginLogic(ResultSet rs) {
        process(rs);
    }

    private void process(ResultSet rs) {
        try {
            if (rs.next()) {
                lu.loginSuccess();

            } else {
                lu.loginFailed();
            }
        } catch (SQLException e) {
            // TODO: handle exception
        }
    }
...

DbConn类片段

// 4. Process the result set
   new LoginLogic(rs);

所以解决方案：可能的解决方法是像这样在构造函数外部实例化UI对象变量lu

...
LoginUI lu = new LoginUI();

public LoginLogic() {

}

public LoginLogic(ResultSet rs) {
    process(rs);
}

private void process(ResultSet rs) {
...

尽管这解决了解引用问题，但在DbConn创建逻辑类的新对象时，导致GUI重新初始化。

public class LoginUI {

        // Container declarations
        ...

        // Component declarations
       ...

        public LoginUI() {
            createFrame();
        }

        private void createFrame() {
        ...

实际上没有太大副作用的另一种解决方法是使用静态方法 。 但是在OO代码中使用太多静态元素不是一个好习惯。

因此，我切换到具有Eager初始化的Singleton模式 ，从而有效地解决了该问题。

登录逻辑

public class LoginLogic {

    static final LoginUI lu = LoginUI.getInstance(); // immutable singleton
                                                        // object reference

    public LoginLogic() {
        // TODO Auto-generated constructor stub
    }

    public LoginLogic(ResultSet rs) { // parameterized constructor
        process(rs);
    }

    void process(ResultSet rs) {...

登录UI

public class LoginUI {

    // Container declarations

    // Component declarations

    private static final LoginUI lu = new LoginUI(); // immutable singleton
                                                        // object

    private LoginUI() { // prevent external instantiation
        initUI();
    }

    static LoginUI getInstance() { // getter for the singleton object
        return lu;
    }...