重複する値が配列に入力されないようにするにはどうすればよいですか？

Question

for (int i = 0; i < 6; i++) { 
     lineOne[i] = Integer.parseInt(JOptionPane.showInputDialog(null,""); 

     if (lineOne[i] > 47 || lineOne[i] < 1)//Number also have to be inside these parameters 
      JOptionPane.showMessageDialog(null, "Please try again!!!"); 
      lineOne[i] = Integer.parseInt(JOptionPane.showInputDialog(null, "")); 
     } 
    } 

Answer 1

if文に問題があります。入力された値が1〜47の範囲外にある場合にコードが書き込まれる方法で、値を再度要求しました。しかし、 "間違った"値が与えられた場合、配列に保持してforループの先頭に戻ります。

これを試してください。

for (int i = 0; i < 6; i++) { 
     lineOne[i] = Integer.parseInt(JOptionPane.showInputDialog(null,""); 

     while (lineOne[i] > 47 || lineOne[i] < 1) { //Number also have to be inside these parameters 
      JOptionPane.showMessageDialog(null, "Please try again!!!"); 
      lineOne[i] = Integer.parseInt(JOptionPane.showInputDialog(null, "")); 
     } 
    } 

Answer 2

boolean b = true; 
    int[] lineOne = new int[6]; 
    for (int i = 0; i < 6; i++) { 
     b = true; 
     int k = Integer.parseInt(JOptionPane.showInputDialog(null, "")); 
     for (int j : lineOne) { 
      if (k == j) { 
       System.out.println("error"); 
       b = false; 
       break; 
      } 
     } 
     if (b && k < 47 && k > 1) { 
      lineOne[i] = k; 
     } else { 
      JOptionPane.showInputDialog(null, "Duplicate value or value out of bounds."); 
      i--; 
     } 
    } 

これは私が考えることができる最高の方法です。ブール値は、値がすでに存在するかどうかを決定します。

Answer 3

配列を使用する代わりに、Setを使用します。

例：

Set<Integer> numbers=new HashSet<>(); 

次にセットに番号を追加します。重複は許可されません。