我正在做一个涉及创建验证码的项目。
我有一系列验证码图像:
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);
你似乎有一个上帝类,一个责任太大和不必要的复杂性的类,为此,请考虑重构该代码,着眼于简化责任并将其分配给较小的粒度类。基于M-V-C或模型-视图-控制器的程序结构将是一个良好的开端。
至于你的主要问题,如何从内部类中访问i
,使数组索引成为它所在的类的实例字段,而不是方法或构造函数局部变量。
其他问题:我将创建一个类来将图像及其字符串作为一个逻辑单元保存在一起,我会考虑使用 ImageIcon 而不是图像,以便我可以轻松地将图标交换到 JLabel 中。我会避免空布局和setBounds(...)
.另请注意,ImageIcon 可以同时保存图像和字符串,字符串用于其"描述",包括 getter 和 setter。您可以只使用此类的项目,并使用验证码字符串结果的描述字符串。
您可以在其他方法中生成随机数,并在单击按钮时调用该方法。这是获得所需效果的最简单方法。