文字列の長さをチェックし、Javaでdo-whileを使用する

Question

文字列を入力し、長さが9かどうかをチェックするプログラムを作成したいと思います。そうであれば、それは大丈夫であるというメッセージを出します。そうでない場合は、そうでないメッセージを表示し、ユーザーに再度文字列を入力するように促します。それを実行している間、私はいつもそれが間違っています。どこが間違っていますか？

import java.util.*; 
public class Sec { 
    public static void main(String[] args) { 
     Scanner Secnum = new Scanner(System.in); 
     System.out.println("Give Sec: "); 
     String Sec = Secnum.nextLine(); 
     do{ 
      if (Sec.length()!= 9); 
      System.out.println("Wrong lenght Sec,enter again"); 
      Secnum.nextLine(); 
     }while (Sec.length() == 9); 
     System.out.println("Sec lenght okay"); 
    } 
} 

Answer 1

あなたは同じ長さをだからあなたのループを条件としてこれを持っている必要があり9.

されていないため、新しい値を要求します。さらにあなたのifが間違っています。 Javaで中括弧が必要な場合は、次のステートメントのみに影響します。文はセミコロンだけでもかまいません。だからあなたのifステートメントはまったく役に立たない。

コードをさらにコンパクトにするために、ifを取り出してdo-whileループを入れ替えることができます。条件が真であり、条件に関係なく初めに一度でない場合にのみ実行されます。より良い動作するはず

コード：サイドノートで

import java.util.*; 

public class Sec { 

    public static void main(String[] args) { 
    Scanner secnum = new Scanner(System.in); 
    System.out.println("Give Sec: "); 
    String sec = secnum.nextLine(); 

    while (sec.length() != 9) { 
     System.out.println("Wrong lenght Sec,enter again"); 
     secnum.nextLine(); 
    } 

    System.out.println("Sec lenght okay"); 
    } 
} 

：使用小文字の変数名。それはより良いコードスタイルです:)

Answer 2

if文を終了します。

if (Sec.length()!= 9);

ので、次の印刷は常にかかわらず、それが本当であることかどうかの実行されます。

エラーのロジックをifブロックに入れます。

if (Sec.length()!= 9) { 
    // Print failure message 
    // Other logic 
}