Java校验字符串仅含字母的代码问题排查（关系运算符相关）

存在的问题： #

• 变量名错误导致编译失败
  isValidString方法中循环使用了未定义的变量newStr，实际方法参数是str，应改为str.charAt(i)，否则代码无法通过编译。

• 不必要的StringBuilder转换
  校验字符串不需要修改操作，将输入的String转成StringBuilder完全多余，直接传递String类型参数更符合代码逻辑，也更高效。

• ASCII码判断可优化（可选）
  原代码用ASCII数值范围判断英文字母，虽然能实现功能，但代码可读性较差。可以用字符直接比较（c >= 'a' && c <= 'z'），或者借助Java内置方法Character.isLowerCase(c)/Character.isUpperCase(c)，甚至用正则表达式^[a-zA-Z]+$来实现，代码会更简洁直观。

修正后的代码（基础版本）： #

import java.util.Scanner;

public class ValidString {

    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);
        System.out.println("Enter the String: ");
        String str = scan.nextLine();
        System.out.println(isValidString(str));
        scan.close();
    }

    public static boolean isValidString(String str) {
        // 若需禁止空字符串，可添加此判断
        if (str.isEmpty()) {
            return false;
        }
        for (int i = 0; i < str.length(); i++) {
            char c = str.charAt(i);
            if (!((c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z'))) {
                return false;
            }
        }
        return true;
    }
}

更简洁的正则实现版本： #

如果追求代码简洁，校验方法可以简化为：

public static boolean isValidString(String str) {
    return str.matches("^[a-zA-Z]+$");
}

内容的提问来源于stack exchange，提问作者user16143615