Javaのオブジェクト配列の最小値と最大値の切り替え

Question

Javaのオブジェクト配列の最大値と最小値を交換するように指示されています。私は自分のコードを書いて、合理的だと思われますが、何らかの理由で値が切り替わっていませんか？私は自分のコードに何が間違っているのかはっきりしていない、誰かが正しい方向に私を向けることができますか？ありがとうございました！！

これは私のコードです：

public class Measurables 
{ 
    /** 
     Swaps the values with the smallest and largest measure. 
     @param objects an array of objects of a class that implements the 
     Measurable interface. 
    */ 
    public static void swapMinAndMax(Measurable[] objects) 
    { 

     Measurable largest = objects[0]; 
     Measurable smallest = largest; 
     for (int i = 1 ; i < objects.length ; i++) 
     { 
     Measurable current = objects[i]; 
     if (largest.getMeasure() < current.getMeasure()) 
     { 
      largest = current; 
     } 
     if (smallest.getMeasure() < current.getMeasure()) 
     { 
      smallest = current; 
     } 
     Measurable temp = largest; 
     largest = smallest; 
     smallest = temp; 

     } 
     } 
} 

これは私がテスターの実行時に取得されるものです：

Testers 

Running Tester.java 

fail 
[Uruguay, Thailand, Belgium] 
Expected: [Uruguay, Belgium, Thailand] 
Running Tester2.java 

fail 
[BankAccount[balance=1000.0], BankAccount[balance=3000.0], BankAccount[balance=2000.0]] 
Expected: [BankAccount[balance=3000.0], BankAccount[balance=1000.0], BankAccount[balance=2000.0]] 
Running Tester3.java 

pass 
[Uruguay] 
Expected: [Uruguay] 

Answer 1

をあなたが実行できるようにあなたの要素のインデックスを格納する必要がありますスワップの後に、最小と最大のインデックスを決定します。また、あなたは最小のために比較を交換したようです。何かのように、

public static void swapMinAndMax(Measurable[] objects) { 
    int largest = 0, smallest = 0; 
    for (int i = 1; i < objects.length; i++) { 
     Measurable current = objects[i]; 
     if (objects[largest].getMeasure() < current.getMeasure()) { 
      largest = i; 
     } 
     if (objects[smallest].getMeasure() > current.getMeasure()) { 
      smallest = i; 
     } 
    } 
    // Now swap, we know the indexes. 
    Measurable temp = objects[largest]; 
    objects[largest] = objects[smallest]; 
    objects[smallest] = temp; 
}