多个|| 一个if语句中的语句

Question

我试图让我的程序检查播放器输入的名称，以查看它是否使用了任何无效字符。 在我看来，如何使其有效似乎很有意义，但在实际环境中似乎无效。 当我运行它并且使用无效字符时，它将继续进行到程序的下一部分。 关于如何解决它的任何想法？

这是我的相关代码块：

if (!(nameInput.getText().equals(""))) {
    boolean allow = true;
    String check = nameInput.getText();
    for (int i = 0;i>check.length();i++) {
        if (check.charAt(i) == '`' || check.charAt(i) == '~' ||
                check.charAt(i) == '!' || check.charAt(i) == '@' ||
                check.charAt(i) == '#' || check.charAt(i) == '$' ||
                check.charAt(i) == '%' || check.charAt(i) == '^' ||
                check.charAt(i) == '&' || check.charAt(i) == '*' ||
                check.charAt(i) == '(' || check.charAt(i) == ')' ||
                check.charAt(i) == '_' || check.charAt(i) == '+' ||
                check.charAt(i) == '=' || check.charAt(i) == '[' ||
                check.charAt(i) == ']' || check.charAt(i) == '{' ||
                check.charAt(i) == '}' || check.charAt(i) == ';' ||
                check.charAt(i) == '\''|| check.charAt(i) == ':' ||
                check.charAt(i) == '"' || check.charAt(i) == '<' ||
                check.charAt(i) == '>' || check.charAt(i) == '?' ||
                check.charAt(i) == ',' || check.charAt(i) == '.' ||
                check.charAt(i) == '/' || check.charAt(i) == '\\'||
                check.charAt(i) == '|' || check.charAt(i) == '1' ||
                check.charAt(i) == '2' || check.charAt(i) == '3' ||
                check.charAt(i) == '4' || check.charAt(i) == '5' ||
                check.charAt(i) == '6' || check.charAt(i) == '7' ||
                check.charAt(i) == '8' || check.charAt(i) == '9' ||
                check.charAt(i) == '0') {
            allow = false;
        }
    }
    if (allow == true) {
        player = new Player(check);
        window.refresh();
        mainMenu();
    } else {
        JOptionPane.showMessageDialog(window, "Invalid Name(Uses invalid Characters","Invalid!", JOptionPane.OK_OPTION);
    }
} else {
    JOptionPane.showMessageDialog(window, "Can Not Leave Name Blank!", "Missing Name!", JOptionPane.OK_OPTION);
    window.refresh();
    createPlayer();
}
}

Answer 1

for(int i = 0;i>check.length();i++)

您永远不会输入for循环，因为我不认为String的长度可以为负。

您是否打算使用<代替？

另外，还有一些显著更好的方法来检查不是字符检查的一个巨大的链条有效的名称。

一种方法是使用正则表达式。 查看Java的Pattern类和Google以学习正则表达式。 这样，您就可以减少循环和阻塞的次数（我认为；我只接受字母字符，而基于对这种怪异的一瞥，就拒绝了其他所有内容）：

allow = String.matches("[a-zA-Z]{1,}");

Answer 2

如果您真的想只允许包含不在该列表中的字符组成的字符串，则可以使用这样的正则表达式来实现。

allow = check.matches("[^]`~!@#$%^&*()_+=[{};':\"<>?,./\\\\|1234567890]*");

如果check完全由列表中未包含的字符组成，则将allow设置为true 。

请注意，我在[ -的内部放置了^ -这表示“不是”。 我也将[ ]移到了^ ，因此不会被解释为关闭列表。

Answer 3

更改

for(int i = 0;i>check.length();i++)

至

for(int i = 0;i<check.length();i++)

由于i为0，显然这比check.length()程序流真正从未进入for循环。

另外，不只是检查""而是添加空检查。

if(nameInput.getText()!=null && !(nameInput.getText().equals(""))){..}