単純選択法がうまくできません＞＜

Question

単純選択法（選択ソート）を書いてみたのですが意図した値が返ってきません。
どこがおかしいのでしょうか＞＜

配列{12, 13, 11, 14, 10}を小さい順に並べ替えたいと思っています。

//返ってくる値
10 13 14 11 12

①sortメソッドの呼び出し
②最小値の初期値を設定、最小値の更新
③最小値を探す処理
④値を入れ替える処理
⑤入れ替えた配列を表示

//記述
class sort1{
  public static void main(String[] args){
    int[] ary = {12, 13, 11, 14, 10};
    sort(ary); //①
    for(int a : ary){　//⑤
      System.out.print(a + " ");
    }
  }  

  public static void sort(int[] ary){
    for(int i = 0; i < ary.length - 1; i++){
      int min = i; //②

      for(int k = 1; k < ary.length; k++){
        if(ary[k] < ary[min]){
          min = k;　//③
        }
      }
　　　　//④
      int w = ary[i];
      ary[i] = ary[min];
      ary[min] = w;
    }  
  }
}

Comments

[K_3578]

返ってくるとか以前に全角スペースがあるからコンパイルエラー出るのでは

Answer 1

単純検索法／選択ソートとはいったん整列（ソート）したものは次の整列処理のループで対象としないようにするのですから、2つ目のkの変数を使うループで常に1から始めてしまっては正しく処理できません。したがって、2つ目のループでは1つ目のループの添え字+1を開始点とすれば良いです。

[質問への追記・修正の依頼]欄で指摘いただいている問題を修正した上で：

class sort1{
  public static void main(String[] args){
    int[] ary = {12, 13, 11, 14, 10};
    sort(ary); //①
    for(int a : ary){ //⑤
      System.out.print(a + " ");
    }
  }  

  public static void sort(int[] ary){
    for(int i = 0; i < ary.length - 1; i++){
      int min = i; //②

      // 常に1から始めてしまっては、せっかくソート済みであるものを再度使ってしまいます。
      //for(int k = 1; k < ary.length; k++){
      for(int k = i + 1; k < ary.length; k++){
        if(ary[k] < ary[min]){
          min = k; //③
        }
      }
	  //④
      int w = ary[i];
      ary[i] = ary[min];
      ary[min] = w;
    }  
  }
}

実行結果です。

$ javac sort1.java
$ java sort1
10 11 12 13 14 $

Comments

[dodox86]

まぁ、「コードを書いた→実行した→うまく（意図したように）動かない→分からん、聞こう。」だったとすれば、そうではなく、机上デバッグをし、机上で分からなければEclipseなりVisual Studio Codeなり、デバッガーを使ってデバッグしましょう。

[ooaai]

解決しました
机上でバックまではしたのですが見つけられませんでした＞＜
この機会にデバッグの使い方も勉強してみます　ありがとうございます！！