这里发生了什么事?

Question

有效印度手机号码的规则：

该数字应包含 10 位、11 位或 12 位数字。

如果它包含 10 位数字，则第一个数字应该是 7 或 8 或 9。

如果它包含 11 位数字，则第一个数字应为 0，第二个规则遵循。

如果它包含 12 位数字，则前两位数字应为 91，并遵循第二条规则。

对于测试用例：

1
881906355596

此代码应生成无效，但它显示有效。

import java.util.*;
import java.lang.*;
import java.io.*;
class GFG
{
public static void main (String[] args)
{
Scanner scan = new Scanner(System.in);
int t=scan.nextInt();
while((t--)!=0){
String s = scan.next();
int length = s.length();
if((length==10) &&((s.charAt(0)=='7')||(s.charAt(0)=='9')||(s.charAt(0)=='8')))
System.out.println("Valid");
else if((length==11) &&(s.charAt(0)=='0')&&(s.charAt(0)=='7')||(s.charAt(0)=='9')||(s.charAt(0)=='8'))
System.out.println("Valid");//code
else if((length==12) &&(s.charAt(0)=='9')&&(s.charAt(1)=='1'))
System.out.println("Valid");//code
else System.out.println("Invalid");
}
}
}

Answer 1

你的第二个和第三个条件是错误的。

第二个条件错误地返回881906355596输入的true。

如果你按如下方式排列，你就会明白为什么：

else if (
(length==11) &&           // false &&
(s.charAt(0)=='0') &&     // false &&
(s.charAt(0)=='7') ||     // false ||
(s.charAt(0)=='9') ||     // false || 
(s.charAt(0)=='8')        // true 
)                             // equals true

它应该是：

else if (length == 11 && s.charAt(0) == '0' && (s.charAt(1) == '7' || s.charAt(1) == '9' || s.charAt(1) == '8'))

第三个条件应该是：

else if (length == 12 && s.charAt(0) == '9' && s.charAt(1) == '1' && (s.charAt(2) == '7' || s.charAt(2) == '9' || s.charAt(2) == '8'))

Answer 2

你错过了整整一个()

else if((length==12) &&((s.charAt(0)=='9')&&(s.charAt(1)=='1')))