なぜ私のfindMaxメソッドは、最大要素の代わりに配列の最初の要素の結果を返しますか？

Question

私は、静的なfindMaxメソッドを使用して最大周囲長の矩形を見つけようとしています。どうにかして、配列内で最大長の配列ではなく、生成されたRectangleが得られます。私は何が間違っているのかわからないのは、私が打ち勝ったcompareToメソッドがうまく動作しているようだからです。ここ

は、Rectangleクラスのコードです：

public class Rectangle implements Comparable<Rectangle> { 
    private double length; 
    private double width; 
     private double perimeter; 


    public Rectangle(double l, double w){ 
     this.length = l; 
     this.width = w; 
    } 

    public double getLength() { 
     return length; 
    } 

    public double getWidth(){ 
     return width; 
    } 

     public void setLength(double l){ 
      length= l; 
     } 
     public void setWidth(double w){ 
      width = w; 
     } 

    public double getPerimeter(){ 
       perimeter = 2*(length+width); 

       return perimeter; 
    } 




     @Override 
    public int compareTo(Rectangle other){ 
       return Double.compare(this.perimeter, other.perimeter); 
     } 

     @Override 
     public String toString(){ 
      return "Rectangle: "+ width +" by "+ length ; 
     } 




} 

そして、これはfindMaxを有している主なある：私はあなたではない

public class Problem1{ 

    public static <Anytype extends Comparable<Anytype>> Anytype findMax(Anytype[] arr){ 
       int maxIndex = 0; 
     for (int i = 1; i < arr.length; i++) 
       if (arr[i].compareTo(arr[maxIndex]) > 0) 
          maxIndex = i; 
      return arr[maxIndex]; 
     } 


       public static void main(String[] args){ 

     Rectangle[] arr = new Rectangle[5]; 

     for(int i = 0; i < 5; i++) 
     { 
     Rectangle r = new Rectangle(10, 20); 
     r.setWidth((Math.random()*10)+10); 
     r.setLength((Math.random()*10)+10); 

     arr[i] = r; 
     } 

       // Rectangle max = findMax(<Rectangle>[] arr); 

       Rectangle max = findMax(arr); 
       double maxP =max.getPerimeter(); 
       System.out.println("The rectangle that has the max perimeter is "+findMax(arr)+" maxP is "+maxP); 
       for(Rectangle rec: arr){ 
        System.out.println(rec.getPerimeter()); 
        System.out.println(rec); 


       } 

    } 


} 

Answer 1

を物事をテストしていたのcuzそれはちょっと厄介です境界のデフォルト値が0になるようにgetPerimeter（）を呼び出すと、本体のgetPerimeterメソッドを呼び出すcompareToを編集することをお勧めします：

public int compareTo(Rectangle other){ 
    return Double.compare(this.getPerimeter(), other.getPerimeter()); 
} 

Answer 2

@Override 
public int compareTo(Rectangle other){ 
    System.out.println(this.perimeter + " :: " + other.perimeter);//all 0.0 :: 0.0 
    return Double.compare(this.perimeter, other.perimeter); 
} 

変更リターン：

return Double.compare(this.getPerimeter(), other.getPerimeter());