Java

Question

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

我有一组验证码图片：

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

我还有一组验证码字符串：

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

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

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);)，但出现错误 "local variable defined in an enclosing scope"。我无法将原始 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

你似乎有一个上帝-class，一个承担太多责任和不必要的复杂性的 class，为此，考虑重构该代码，着眼于简化并将责任分配给更小的人粒度 classes。基于 M-V-C 或模型-视图-控制器的程序结构将是一个好的开始。

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

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

Answer 2

您可以使用另一种方法生成随机数，然后在单击按钮时调用该方法。这是获得所需效果的最简单方法。

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

Java - Click JButton to change value of int to a random number between specific range

random

indexing

captcha

scope