配列に格納した数値について

Question

以下の様なコードを書いて、配列内に格納した数値を並べ替えようと思ったのですが、結果を println()で表示しても入れ替わっていません。
まだ慣れていない言語の為、どうかご教授お願いします。

public class Sort {
	public static void main(String[] args)
	{
                // 練習の為 String から int に変換している
		String strings[] = {"25", "11", "8", "5", "50"};

		int data[] = new int[strings.length];

		for(int i = 0; i < data.length; i++)
		{
			data[i] = Integer.valueOf(strings[i]).intValue();
		}

		for(int i = 0; i < data.length - 1; i++)
		{
			for(int j = 0; j < data.length; j++)
			{
				if(data[i] > data[j])
				{
					int tmp = data[i];
					data[i] = data[j];
					data[j] = tmp;
				}
			}
		}
		
		// ここで確認
		for(int i = 0; i < data.length; i++)
		{
			System.out.println(data[i]);
		}
	}
}

Answer 1

順番に追ってみましょう．
最初：{25, 11, 8, 5, 50}
i=0,j=1のときに25>11なので入れ替え→{11, 25, 8, 5, 50}
i=0,j=2で(11>8)入れ替え→{8, 25, 11, 5, 50}
i=0,j=3で(8>5)入れ替え→{5, 25, 11, 8, 50}
i=1,j=0で(25>5)入れ替え→{25, 5, 11, 8, 50}
i=2,j=1で(11>5)入れ替え→{25, 11, 5, 8, 50}
i=3,j=2で(8>5)入れ替え→{25, 11, 8, 5, 50}
すでに確定した0番目の要素が2ループ目以降も入れ替え対象となるため，戻ってしまいます．

Comments

[rabit_box]

一度紙か何かに書き起こして確認してみるべきだったんですね。
ありがとうございました。

[swordone]

入れ替えを実行するif文で「何と何を入れ替えているか」を出力してみると見つけられたと思います．

Answer 2

変数jの初期値に問題があります。

Comments

[rabit_box]

確認して、改めてコンパイルを通したら正常に動きました。
ありがとうございます。