Comparable Missing One traitを実装しています

Question

私はComparableを実装する非常に基本的なコードを、年、アーティスト、タイトルに基づいて比較しています。

しかし、私のコードは、タイトル、年、アーティストだけで絵を比較していません。

public class Main { 

    /** 
    * @param args the command line arguments 
    */ 
    public static void main(String[] args) { 
     // TODO code application logic here 


     Painting p1 = new Painting(Year.NINETYEIGHT, artist.ART1, title.TIT1); 
     Painting p2 = new Painting(Year.NINETYEIGHT, artist.ART1, title.TIT2); 

     System.out.println("p1: " + p1.toString()); 
     System.out.println("p2: " + p2.toString()); 

     if(p1.compareTo(p2)> 0){ 
      System.out.println(p1.toString() + " beats " + p2.toString()); 

     } else if(p1.compareTo(p2) < 0){ 
      System.out.println(p2.toString() + " beats " + p1.toString()); 
     } else{ 
      System.out.println("Same Everything"); 
     } 

    } 

} 

public enum Year { 
    NINETYSEVEN, NINETYEIGHT, NINETYNINE, TWOTHOUSAND 

} 

public enum artist { 
    ART1, ART2, ART3, 

} 
public enum title { 
    TIT1, TIT2,TIT3, 

} 

public class Painting implements Comparable { 

    private title title; 
    private Year year; 
    private artist artist; 

    public Painting(Year y, artist a, title t) { 
     title = t; 
     year = y; 
     artist = a; 

    } 

    @Override 
    public int compareTo(Object o) { 
     //compare values 
     Painting other = (Painting) o; 
     int yearCompare = this.year.compareTo(other.year); 
     int artistCompare = this.artist.compareTo(other.artist); 
     if (yearCompare == 0) { 
      //same year, compare artist 
      return this.artist.compareTo(other.artist); 

     } else if (artistCompare == 0) { 
      return this.title.compareTo(other.title); 

     } else { 

      return yearCompare; 

     } 
    } 

    @Override 
    public String toString() { 
     return title.name() + " by " + artist.name() + " produced " + year.name(); 
    } 

} 

Answer 1

if-elseロジックにはいくつかの点で欠陥があります。あなたはジェネリック、サイドノートで

int yearCompare = this.year.compareTo(other.year); 
if (yearCompare != 0) { 
    return yearCompare; 
} 

int artistCompare = this.artist.compareTo(other.artist); 
if (artistCompare != 0) { 
    return artistCompare; 
} 

return this.title.compareTo(other.title); 

なければならないと鋳造を避ける：それはより次のようになります

public class Painting implements Comparable<Painting> { 
    @Override 
    public int compareTo(Painting other) { 
     // no casting necessary 
    } 
} 

Answer 2

ダン。数秒で遅くなります。ハハ。私はshmoselと同じ解決策を思いついた。

public int compareTo(Painting other) { 

    int yearCompare = year.compareTo(other.year); 
    if (yearCompare != 0) 
     return yearCompare; 

    int artistCompare = artist.compareTo(other.artist); 
    if (artistCompare != 0) 
     return artistCompare; 

    return title.compareTo(other.title); 
} 

1つの違い。私はあなたのクラスのヘッダーを変更することを検討します。具体的には、私が変更になります。

public class Painting implements Comparable 

へ：

public class Painting implements Comparable<Painting> 

、代わりに "生" オブジェクトタイプを使用するこの方法は、あなたが絵画しているクラスののcompareTo（）メソッドのシグネチャはなります：

public int compareTo(Painting o) 

つまり、引数がペイントのインスタンスであるかどうかを確認する必要はありません。