Java - 单击 JButton 将 int 值更改为特定范围内的随机数

Question

我正在开发一个涉及创建验证码的项目。

我有一系列验证码图像:

Image[] CAPTCHAimageArray = {2G4QH, 4FTD2, 7BJHL, etc.};

我还有一个验证码字符串数组:

String[] CAPTCHAstringArray = {"2G4QH", "4FTD2", "7BJHL", etc.};

我想生成一个索引 int，它实际上会随机选择验证码图像及其相应的字符串。看起来很简单 - 我创建了这段代码，认为我走在正确的道路上:

Random rn = new Random();
int i = rn.nextInt(46);

到目前为止一切顺利。我创建了一个显示验证码图像的 JLabel:

JLabel lblCAPTCHA = new JLabel("");
lblCAPTCHA.setBounds(162, 152, 300, 180);
panelCAPTCHA.add(lblCAPTCHA);
lblCAPTCHA.setIcon(new ImageIcon(CAPTCHAimageArray[i]));

最后，我添加了一个 JTextfield，用户可以在其中输入他们对验证码的解释。用户单击 JButton，我能够正确地将用户的输入与验证码字符串进行比较......但仅限一次!

问题是我无法更改此 int i 的值:

int i = rn.nextInt(46);

我尝试在提交按钮的 ActionListener 部分中将其值更改为新生成的数字 (i = rn.nextInt(46);)，但收到错误“在封闭范围中定义的局部变量”。我无法将原始 int i 更改为新的随机数。我能够以一种非常拙劣的方式解决这个问题，即通过将这段代码添加到 ActionListener 的末尾:

int i = rn.nextInt(46);
lblCAPTCHA.setIcon(new ImageIcon(CAPTCHAimageArray[i]));

这只能将图像更改为新的验证码图像，但索引仍然集中在原始验证码字符串上。基本上，该程序只能关注第一个验证码字符串。我无法更改原始 int i 的值。

解决这个问题的最佳方法是什么？

如果您想查看涉及验证码的整个代码部分，这里是:

    /* CAPTCHA MENU CONTENTS */

        //"Account Creation" Text
            JLabel lblAccountCreation_3 = new JLabel("Account Creation");
            lblAccountCreation_3.setBounds(263, 89, 109, 16);
            panelCAPTCHA.add(lblAccountCreation_3);

        //"Please input the above message:" Text
            JLabel lblInputMessage = new JLabel("Please input the above message:");
            lblInputMessage.setBounds(207, 383, 209, 16);
            panelCAPTCHA.add(lblInputMessage);

        //Initiate CAPSLOCK Filter
            DocumentFilter filter = new UppercaseDocumentFilter();

        //CAPTCHA TextField
            CAPTCHAtextField = new JTextField();
            CAPTCHAtextField.setBounds(258, 411, 130, 26);
            panelCAPTCHA.add(CAPTCHAtextField);
            ((AbstractDocument) CAPTCHAtextField.getDocument()).setDocumentFilter(filter);

        //CAPTCHA Image Array
            Image[] CAPTCHAimageArray = {c2G4QH, c4FTD2, c7BJHL, c7JDFV, c9PB43, c9TVB4, cADVE8, cAZQRV,
                    cBLTFT, cBYF4D, cD8URH, cDBVFX, cDQAXC, cECD6A, cERTYA, cGTJRD, cGY67E, cHDP7R,
                    cJU4RV, cK8CRW, cKJPHL, cKMFDM, cL9MBP, cLGU3W, cLKMDR, cLMRTD, cLMUFX, cLPDT2,
                    cLPTY2, cLXF49, cMKNLH, cMY62A, cPT7W2, cRDAVH, cRTLPQ, cRVBAZ, cT7TMW, cUL4B7,
                    cUW2CZ, cVBCHY, cVF4TU, cW36X9, cWX2DT, cYT782, cYWRQZ, cZKGF8};

        //CAPTCHA String Array
            String[] CAPTCHAstringArray = {"2G4QH", "4FTD2", "7BJHL", "7JDFV", "9PB43", "9TVB4", "ADVE8", "AZQRV",
                    "BLTFT", "BYF4D", "D8URH", "DBVFX", "DQAXC", "ECD6A", "ERTYA", "GTJRD", "GY67E", "HDP7R",
                    "JU4RV", "K8CRW", "KJPHL", "KMFDM", "L9MBP", "LGU3W", "LKMDR", "LMRTD", "LMUFX", "LPDT2",
                    "LPTY2", "LXF49", "MKNLH", "MY62A", "PT7W2", "RDAVH", "RTLPQ", "RVBAZ", "T7TMW", "UL4B7",
                    "UW2CZ", "VBCHY", "VF4TU", "W36X9", "WX2DT", "YT782", "YWRQZ", "ZKGF8"};

        //CAPTCHA Image Generation
            Random rn = new Random();
            int i = rn.nextInt(46);
            JLabel lblCAPTCHA = new JLabel("");
            lblCAPTCHA.setBounds(162, 152, 300, 180);
            panelCAPTCHA.add(lblCAPTCHA);
            lblCAPTCHA.setIcon(new ImageIcon(CAPTCHAimageArray[i]));

        //"Submit" Button
            JButton btnSubmit_4 = new JButton("Submit");
            btnSubmit_4.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    String CAPTCHAinput = CAPTCHAtextField.getText();
                    //Check if input matches CAPTCHA
                    for (int j = 0; j <= 46; ++j) {
                        if (i == j) {
                            String CAPTCHA = CAPTCHAstringArray[i];
                        //System.out is to see which captcha the program is focusing on
                            System.out.println("i = " + i);
                            System.out.println("CAPTCHA is " + CAPTCHA);
                            if (compareCAPTCHA (CAPTCHA, CAPTCHAinput)) {
                                panelEmail.setVisible(true);
                                panelCAPTCHA.setVisible(false);
                                CAPTCHAtextField.setText("");
                                int i = rn.nextInt(46);
                                lblCAPTCHA.setIcon(new ImageIcon(CAPTCHAimageArray[i]));
                            }
                            else {
                                JOptionPane.showMessageDialog(null, "Error - CAPTCHA input incorrect!");
                                CAPTCHAtextField.setText("");
                                int i = rn.nextInt(46);
                                lblCAPTCHA.setIcon(new ImageIcon(CAPTCHAimageArray[i]));
                            }
                        }
                    }
                }
            });
            btnSubmit_4.setBounds(260, 463, 117, 29);
            panelCAPTCHA.add(btnSubmit_4);

        //"New CAPTCHA" BUTTON
            JButton btnNewButton = new JButton("New CAPTCHA");
            btnNewButton.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    int i = rn.nextInt(46);
                    lblCAPTCHA.setIcon(new ImageIcon(CAPTCHAimageArray[i]));
                }
            });
            btnNewButton.setBounds(255, 504, 130, 29);
            panelCAPTCHA.add(btnNewButton);

Answer 1

您似乎有一个神级，一个具有太多责任和不必要的复杂性的类，为此，请考虑重构该代码，着眼于简化并将责任分配给较小粒度的类。基于 M-V-C 或模型- View - Controller 的程序结构将是一个好的开始。

至于您的主要问题，如何从内部类中访问 i ，使数组索引成为它所在类的实例字段，而不是方法或构造函数局部变量。

其他问题:我会创建一个类来将图像及其字符串作为一个逻辑单元保存在一起，我会考虑使用 ImageIcons 而不是图像，以便我可以轻松地将图标换入和换出 JLabels。我会避免使用 null 布局和 setBounds(...)。另请注意，ImageIcons 可以同时保存图像和字符串，该字符串包含其“描述”`，并包含 getter 和 setter。您可以只使用此类的项目，并使用说明字符串作为验证码字符串结果。