如何从其他类或方法中“提取”变量？

Question

在使用 Python 很长一段时间后，我最近回到了 Java，我正在努力再次适应它。我想要的第一个项目是制作一个可以（首先）登录的小型应用程序。我正在使用 mySQL 来保存用户名和密码的数据库。

到目前为止，我已经使用 Java 的 swing GUI 制作了一个弹出框，询问登录信息。输入的用户名和密码将根据 SQL 表中的用户名和密码进行测试。

问题是我正在使用 while 循环来测试 SQL 表的输入。因此，输入的信息会根据 SQL 表创建的 Hashmap 中的每个键和值进行检查。

我不确定如何从 while 循环和 SQL 的 try 语句中“提取”我的变量 g（Hashmap），以便在它被来自 mySQL 的数据填充后使用它。

我正在使用 Eclipse (Java Neon)。

我不知道如何“提取”哈希图，当我尝试让 while 循环或 try 语句返回哈希图时，Eclipse 告诉我 void 方法无法返回值（很明显）。但是，我无法将返回类型从 void 更改为 HashMap)String, String>，因为“返回类型与 ActionListener.actionPerformed(ActionEvent e) 不兼容”和“实现了 java.awt.event.ActionListener.actionPerformed”。

这是我的代码：

import javax.swing.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.util.HashMap;
import java.util.Map;

public class Hello0 extends JFrame {
    private static final long serialVersionUID = 1487932324102279819L;

    public static void main(String[] args) {
        JFrame frame = new JFrame("Frame Demo");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setSize(350, 200);

        JPanel panel = new JPanel();
        frame.add(panel);
        placeComponents(panel);

        frame.setVisible(true);
    }


    private static void placeComponents(JPanel panel) {

        panel.setLayout(null);
        JLabel userLabel = new JLabel("Username");
        userLabel.setBounds(10, 20, 80, 25);
        panel.add(userLabel);

        JTextField userText = new JTextField(20);
        userText.setBounds(100, 20, 165, 25);
        panel.add(userText);

        JLabel passwordLabel = new JLabel("Password");
        passwordLabel.setBounds(10, 50, 80, 25);
        panel.add(passwordLabel);

        JPasswordField passwordText = new JPasswordField(20);
        passwordText.setBounds(100, 50, 165, 25);
        panel.add(passwordText);

        JButton loginB = new JButton("Login");
        loginB.setBounds(10, 80, 80, 25);
        panel.add(loginB);

        loginB.addActionListener(new ActionListener() {

            public HashMap<String, String> actionPerformed(ActionEvent e) {
                String username0 = "root";
                String password = "javaSQLmy98";
                try {

                    String url = "jdbc:mysql://localhost:3306/javabase?useSSL=false";
                    Connection connection = DriverManager.getConnection(url, username0, password);
                    PreparedStatement stmt0 = connection.prepareStatement("SELECT * from userids");

                    String pass0 = null;
                    String user1 = userText.getText().toString();
                    char[] pass1 = passwordText.getPassword();
                    pass0 = String.valueOf(pass1);
                    System.out.println(user1);
                    System.out.println(pass1);

                    ResultSet rs = stmt0.executeQuery();
                    rs = stmt0.executeQuery("SELECT * from userids ");
                    while (rs.next()) {

                        String user = rs.getString("username");
                        String pass = rs.getString("paswrd");

                        Map<String, String> g = new HashMap<>();
                        g.put(user, pass);
                        return g;
                        // This was the alternative: if(g.keySet().contains("Jacob") && g.values().contains("root")) {


                        /*This code below was originally outside the while loop but I could not
                        figure out how to make it work without it being inside, and accessible 
                        to the hashmap g. Now it is being checked each time the while loop 
                        is ran, with a new pair of usernames and passwords on each loop. */

                        if (user1.equals(user) && pass0.equals(pass)) {
                            System.out.println("Good!");
                        }

                        /*The problem here is that the checker IS inside the loop, so it 
                         tests the input against each of the entries in the SQL table. */
                        else {
                            System.err.println("The username or password is incorrect.");
                        }
                    }

                    connection.close();
                } catch (Exception e1) {
                    System.err.println(e1.getMessage());
                }

            }
        });

    }

}

干杯！

Answer 1

要在while 之外访问g，您需要在while 之外声明它，就像Java 范围内的每个{} 变量一样。因此，

Map<String, String> g = new HashMap<>();
while (condition) {
    // do something
    g.put(key, value);
}
// do something with g

但是在这种情况下，你可以直接在 SQL 中进行检查，这样应该会更有效。

---

旁注：

不要存储密码。 Don't store passwords.Hash+Salt passwords (or better, use a library).

Answer 2

如果您在查询中添加 WHERE 子句，它将仅返回符合条件的数据。换句话说，下面的查询（显然带有正确的值）如果存在则返回指定的userid条目，否则返回一个空的结果集。

SELECT * from userids
WHERE userName = 'user1'
AND passwrd = 'pass1'

然后你可以去掉整个while循环和HashMap，代码可以简化如下：

if (rs.next()) {
    System.out.println("Good!");
} else {
    System.err.println("The username or password is incorrect.");
}

另外，如前所述，如果这不仅仅是一个编程练习，密码应该被散列而不是以纯文本形式存储。

Answer 3

试试这个：

Map<String, String> g = new HashMap<String, String>();
while(rs.next()) {
    String user = rs.getString("username");
    String pass = rs.getString("paswrd");
    g.put(user, pass);

    .....
}

注意：在 HasmMap 中以用户名作为键并不是一个好的编程方式，因为可以有多个同名用户。