同じ数字を入力したらbreakしたい

Question

java初心者です。以下にコードを貼ります。

import java.util.Scanner;

class Matiuke{

  public static void main(String[] args){
    Scanner stdIn = new Scanner(System.in);

    System.out.println("患者様の数");
    int n = stdIn.nextInt();
    int[] u = new int[n];
    int[] key = new int[n];
    int p = 0;

    for(int i = 0; i < n; i++){


      //  System.out.println("患者様の番号(1~n)");
      u[i] = stdIn.nextInt();
      key[i] = u[i];//uの配列の中を探索する時に他の配列に入れたら後で便利になると思った。

      System.out.print("\n");
      System.out.print("\n");
      System.out.print("\n");
      System.out.print("\n");
      System.out.print("\n");
      for(p = 0; p <= i; p++){
        if(u[i] == key[p]){
          break;
        }
      }

      for(int q = 0; q <= i; q++){

        System.out.print(u[q] + " ");

      }


    }
    /*for(int i = 0; i < n; i++){
    System.out.print(u[i] + " ");
  }*/

}



}

今の段階ではとても見づらい実行結果となってしまっています…。
後で改良するのですが、今は同じ数字が入力されたら、入力した数字を格納してある配列の中を1から探って、同じ数字があった場合にbreakするように書いたつもりなのですが、同じ数字を入力してもそのまま動いてしまいます…、
どなたか解決策、自分の何がいけないのかを教えていただけると幸いです。よろしくお願いします。

Answer 1

例えばですが…
配列keyの中にu[i]と同じものがあるか?
を判断して返す関数を作ってみたらどうでしょう。
たぶん読みやすくなると思います。

あ、原因はswordoneさんが回答されてますので
そちらを参照ください。

Comments

[m.ts10806]

質問者さんが仰っている実行結果だけにフォーカスすると毎回5行も空で改行があるのも見やすいとは言えないですね・・

[rink]

なるほど、ありがとうございます！
見にくいですよね…後で改良します！

Answer 2

      for(p = 0; p <= i; p++){
        if(u[i] == key[p]){
          break;
        }
      }

この部分のbreakは、あくまでこのfor文から抜けるだけで、その外にある入力のループから抜けられません。
外側のループで抜けるためのフラグを用意するか、ラベル付きループで抜けるかする必要があります。

Comments

[rink]

そうでした…
ありがとうございます。。