この部分のStringIndexOutOfBoundsExceptionを保持し続ける

Question

コードのこの部分についてStringIndexOutOfBoundsExceptionを取得し続けます。 理由は何でしょうか？接触は、この電話番号の検証

public static boolean phNumValidation(String contact){ 
    boolean valid = true; 
    String dash = contact.substring(3, 4); 
    if(contact.length() == 11){ 
     valid = true; 
     valid = contact.startsWith("01"); 
     valid = dash.matches("-"); 
    } 
    else{ 
     valid = false; 
    } 

    return valid; 
} 

Answer 1

移動contact.lengthの左側に< 11バリデーションまたは（while文の話）するための方法である

do{ 

     System.out.print("Contact Number (01X-XXXXXXX) :"); 
     contact = scan.next(); 
     if(!phNumValidation(contact)){ 
      System.out.println("Invalid Phone Number. Please try again."); 
      System.out.println(""); 
     } 
    }while(!phNumValidation(contact) || contact.length() < 11); 

Stringとして宣言されています。

それ以外の場合は、連絡先が11ch未満の場合でもphNumValidationに入ります。そしてその方法はそれのために準備されていません。

Answer 2

contactのlength <=4をに渡し、最初に長さを確認せずに部分文字列を使用する傾向があるため、例外が発生します。以下のコードは長さが最初にチェックされるので例外をスローしません。 whileループは、関数を再度呼び出す必要はありません。

boolean valid = false; 
    do{ 
     System.out.print("Contact Number (01X-XXXXXXX) :"); 
     contact = scan.next(); 
     if(!phNumValidation(contact)){ 
      System.out.println("Invalid Phone Number. Please try again."); 
      System.out.println(""); 
     }else{ 
      valid = true; 
     } 


    }while(!valid); 


public static boolean phNumValidation(String contact){ 
    boolean valid = true; 

    if(contact.length() == 11){ 
    String dash = contact.substring(3, 4); 
    valid = true; 
     valid = contact.startsWith("01"); 
     valid = dash.matches("-"); 
    } 
    else{ 
     valid = false; 
    } 

    return valid; 
} 

Answer 3

これは、私のvalidationメソッドがどのように見えるかなど、正規表現を使用するときです。

public static boolean phNumValidation(String contact){ 
    return contact.matches("01\\d-\\d{7}"); 
} 

この方法では、長さのフォーマットを一度に検証します。