【问题标题】:Breaking for loop from within ActionListener从 ActionListener 中中断 for 循环
【发布时间】:2017-11-22 17:33:37
【问题描述】:

我正在做一个学校项目,我正在开发一种游戏。在这个游戏中,我需要用户登录,我遇到了一些困难。

这是我的代码的相关部分:

import java.util.*;
import java.io.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class User{
    int rank;
    String name;
    String pass;

User(){
    Scanner s = new Scanner(System.in);

    System.out.println("  Login\n1.New user\n2.Old user");
    int in = s.nextInt();

    for(;;){
        if(in == 1){
            //create new user
        }else if(in == 2){
            JFrame loginFrame = new JFrame();
            loginFrame.setVisible(true);
            loginFrame.setLayout(null);
            loginFrame.setSize(120+14,180+35);

            JLabel enterName = new JLabel("Enter Username:");
            enterName.setBounds(10,10,100,20);

            JTextField nameField = new JTextField();
            nameField.setBounds(120,10,130,20);

            JLabel enterPass = new JLabel("Enter Password:");
            enterPass.setBounds(10,40,100,20);

            JPasswordField passField = new JPasswordField();
            passField.setBounds(120,40,130,20);

            JButton hitEnter = new JButton("Login");
            hitEnter.setBounds(10,70,250,20);

            loginFrame.add(enterName);
            loginFrame.add(nameField);
            loginFrame.add(enterPass);
            loginFrame.add(passField);
            loginFrame.add(hitEnter);
            loginFrame.setSize(270+14,100+36);

            hitEnter.addActionListener(new ActionListener(){
                    public void actionPerformed(ActionEvent enter){
                        name = nameField.getText();
                        pass = new String(passField.getText());

                        boolean validUser = checkUser(filename,name,pass);

                        if(validUser){
                            loginFrame.setVisible(false);
                            //some how break the for loop from here
                        }
                    }
                });

        }else{
            System.out.println("Invalid input.");
        }
    }
}

如您所见,我需要从 actionlistener 内部以某种方式退出 for 循环,但我不确定是否有任何方法可以做到这一点。

请帮帮我!!! 提前致谢!!!

【问题讨论】:

  • 为什么不使用一个while循环来检查登录状态并在成功后退出而不是无限循环?
  • 您不应该混合使用事件驱动的 GUI 代码和扫描仪控制台代码。留在其中一个或另一个。

标签: java swing jbutton actionlistener break


【解决方案1】:

问题:

  • 首先,您在同一个程序中混合了 Swing 和控制台代码,而且方式很糟糕。理解控制台(使用基于 System.in 的 Scanner 对象)是“线性”代码,程序员完全指导程序流程方向的代码,我们经常使用“阻塞”代码来获取用户输入,完全阻塞程序的代码在输入输入之前一直流动,这适用于您的 while 循环。另一方面,Swing GUI(和大多数 GUI)代码主要是非线性事件驱动代码,用户可以更好地控制何时调用什么代码的代码,主要避免阻塞代码流的代码(除了模态对话框)。
  • 如果您绝对必须将两者结合起来,那么您应该使用 JFrame,因为显示 JFrame 不会暂停 while 循环,它会无限循环, 而不是一个好办法。相反,您需要使用带有 JDialog 或 JOptionPane 的 Swing 版本的阻塞代码,一个 modal 对话框。这些将暂停 while 循环,允许更清晰、更线性的输入。
  • 无论您使用哪一个,用户界面代码都不应该在 User 类中。此类应包含单个用户的状态(字段)和行为(方法),但同样不应包含 UI 代码。那属于别处;否则,您最终会得到难以调试和增强的意大利面条代码的可怕混合。因此,为您的用户提供包括姓名和等级在内的字段,为它提供主程序在希望用户执行某项操作时可以调用的用户方法,但将您的 UI 代码放在主方法中。
  • 将您的 Swing GUI 用于创建 JPanel,然后您可以将 JPanel 放置在任何需要的地方,从而使代码更加灵活。例如,如果您这样做,那么您可以将 JPanel 放入 JOptionPane 中,这意味着它将显示为 modal 对话框,完全按照您的意愿阻止 while 循环。
  • 小问题:避免使用字符串字段作为密码。这会让你的程序很容易被攻破,这不是一个好习惯。

如果你绝对必须混合 Swing 和控制台,我会这样做:

首先是 User.java 类。同样,只关注用户状态和行为:

public class User {
    private int rank;
    private String name;
    private char[] pass; // ***** Don't store password as a String
    // ?? other fields if needed

    public User(String name, char[] pass) {
        this.name = name;
        this.pass = pass;
    }

    public void setRank(int rank) {
        this.rank = rank;
    }

    public int getRank() {
        return rank;
    }

    public String getName() {
        return name;
    }

    // again, if this were a real-world program, you wouldn't make password accessible
    public char[] getPass() {
        return pass;
    }

    // other User methods would go here

    @Override
    public String toString() {
        return "User [rank=" + rank + ", name=" + name + "]";
    }

    // you'll want to override equals(Object o) and hashCode() here
}

然后您可以创建一个用于获取用户登录信息的 JPanel。当想要显示这种类型的面板时,我喜欢使用 GridBagLayout。例如:

// inports here....

@SuppressWarnings("serial")
public class GetUserInfo extends JPanel {
    private static final Insets INSETS = new Insets(4, 4, 4, 4);
    private JTextField nameField = new JTextField(10);
    private JPasswordField passField = new JPasswordField(10);

    public GetUserInfo() {
        // gridbaglayout works well for your needs
        setLayout(new GridBagLayout());
        GridBagConstraints gbc = new GridBagConstraints();
        gbc.gridx = 0;
        gbc.gridy = 0;
        gbc.insets = INSETS;
        gbc.anchor = GridBagConstraints.WEST;
        gbc.fill = GridBagConstraints.HORIZONTAL;
        add(new JLabel("Name:"), gbc);
        gbc.gridy = 1;
        add(new JLabel("Password:"), gbc);

        gbc.gridx = 1;
        gbc.gridy = 0;
        gbc.anchor = GridBagConstraints.EAST;
        add(nameField, gbc);
        gbc.gridy = 1;
        add(passField, gbc);
    }

    // allow classes to query this JPanel for the user name 
    public String getName() {
        return nameField.getText();
    }

    // and password data
    public char[] getPass() {
        return passField.getPassword();
    }

}

然后在控制台程序中结合以上内容,在 JOptionPane 中显示这个 JPanel,因为这会创建一个 modal 对话框,该对话框会阻止程序流程直到它被处理。你可以使用这样的代码:

    // user interface code can go here
    Scanner s = new Scanner(System.in);

    System.out.println("  Login\n1.New user\n2.Old user: ");
    int in = s.nextInt();
    s.nextLine();
    User user = null;  // hold our user object
    boolean inputNotOK = true; // keep looping until this is false
    GetUserInfo getUserInfo = new GetUserInfo();  // our JPanel for getting user sign in information
    if (in == 1) {
        // code to get a new user
    } else if (in == 2) {
        // code to sign in existing user
        while (inputNotOK) {
            String title = "Get User Name and Password";
            int optionType = JOptionPane.OK_CANCEL_OPTION;
            int msgType = JOptionPane.PLAIN_MESSAGE;
            int value = JOptionPane.showConfirmDialog(null, getUserInfo, title, optionType, msgType);
            if (value == JOptionPane.OK_OPTION) {
                // if the user presses "OK" on the dialog
                String name = getUserInfo.getName();
                char[] pass = getUserInfo.getPass();

                // validUser is a method that you have that checks if the user sign in is appropriate
                if (validUser(name, pass)) {
                    user = new User(name, pass);
                    System.out.println("new user: " + user);
                    inputNotOK = false;
                } else {
                    // show an error JOptionPane here to warn the user
                    // that their sign-on information was incorrect
                }
            }
        }
    }
    s.close();

// method that should check to see if user name and password are acceptable
private static boolean validUser(String name, char[] pass) {
    // TODO code to test if username and password are OK
    // TODO: change this to an actual test
    return true;
}   

使用这样的代码,如果您以后决定要删除 所有 控制台(扫描仪)代码,您可以,因为您现在已经有了一个可以在桌面 Swing 中使用的 JPanel图形用户界面。

【讨论】:

  • 非常感谢您的回答。我只是一个学生,所以我不知道你在这里谈论的所有东西,但再次感谢。一定会采纳您的所有反馈意见。
【解决方案2】:

你来试试看

import java.util.*;
import java.io.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class User{
    int rank;
    String name;
    String pass;
    //Added boolean up here
    boolean validUser;

    User(){
        Scanner s = new Scanner(System.in);

        System.out.println("  Login\n1.New user\n2.Old user");
        int in = s.nextInt();

        //Make sure it is set to false
        validUser = false;
        while(!validUser){
            if(in == 1){
                //create new user
            }else if(in == 2){
                JFrame loginFrame = new JFrame();
                loginFrame.setVisible(true);
                loginFrame.setLayout(null);
                loginFrame.setSize(120+14,180+35);

                JLabel enterName = new JLabel("Enter Username:");
                enterName.setBounds(10,10,100,20);

                JTextField nameField = new JTextField();
                nameField.setBounds(120,10,130,20);

                JLabel enterPass = new JLabel("Enter Password:");
                enterPass.setBounds(10,40,100,20);

                JPasswordField passField = new JPasswordField();
                passField.setBounds(120,40,130,20);

                JButton hitEnter = new JButton("Login");
                hitEnter.setBounds(10,70,250,20);

                loginFrame.add(enterName);
                loginFrame.add(nameField);
                loginFrame.add(enterPass);
                loginFrame.add(passField);
                loginFrame.add(hitEnter);
                loginFrame.setSize(270+14,100+36);

                hitEnter.addActionListener(new ActionListener(){
                    public void actionPerformed(ActionEvent enter){
                        name = nameField.getText();
                        pass = new String(passField.getText());

                        validUser = checkUser(filename,name,pass);

                        if(validUser){
                            loginFrame.setVisible(false);
                            //some how break the for loop from here
                        }
                    }
                });

            }else{
                System.out.println("Invalid input.");
            }
        }
    }

【讨论】:

    猜你喜欢
    • 2015-04-09
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2020-07-17
    • 1970-01-01
    相关资源
    最近更新 更多