这里发生了什么？

Question

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

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

如果包含10位数字，则第一位数字应为7或8或9。

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

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

对于测试用例：

1
881906355596

此代码应产生Invalid，但显示为Valid。

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

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

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

您将看到为什么按如下方式安排它：

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'))