检查字符串是否只包含数字，然后仅在字符串不包含数字以外的字符时进行设置

Question

/**
 * Set the person's mobile phone number
 */
public void setMobile(String mobile) {
    for (int i = 0; i < mobile.length(); i++) {
if (!Character.isDigit(mobile.charAt(i))) {}
}
this.mobile = mobile;
}

所以我基本上需要确保字符串只包含数字，如果它包含非数字的方法，就什么也不做。我的问题是，如果一个数字串中有一个随机字符，即“0343A45645”，它仍然会设置方法。感谢您的帮助，谢谢！

Answer 1

添加一个布尔字母

然后在你的for循环中

无论何时找到一封信，请使用else语句将letterFound设置为true

然后立即停止循环i=mobile。length（）在您的else语句中

Answer 2

你有两个问题：

第一：如果if条件为false，则没有退出循环

第二：为什么要使用循环来实现这样的tryParse

boolean tryParseInt(String value) {  
     try {  
         Integer.parseInt(value);  
         return true;  
      } catch (NumberFormatException e) {  
         return false;  
      }  
}

if(tryParseInt(mobile)){
  this.mobile = mobile;
}

Answer 3

您可以使用String.matches（String regex）：

boolean onlyDigits = mobile.matches("[0-9]+");

有了for循环，当找到一个非数字时，你可以直接中断。

boolean onlyDigits = true;
for (int i = 0; i < mobile.length(); i++) {
    if (!Character.isDigit(mobile.charAt(i))) {
        onlyDigits = false;
        break;
   }
}

你也可以直接返回，而不是打破循环。听上去你不想再发生什么事了。这样就不需要从onlyDigits变量开始。

请注意，如果mobile。length（）==0那么就上述for循环而言，数字仍然是true。因此，假设onlyDigits应为false如果mobile为空字符串，则可以将其初始化为：

boolean onlyDigits = !mobile.isEmpty()

检查后，您可以分配它，如果onlyDigits是true。

if (onlyDigits)
    this.mobile = mobile;