• Post Reply Bookmark Topic Watch Topic
  • New Topic

Generics and method compareTo giving me a warning  RSS feed

 
Neil Cartmell
Ranch Hand
Posts: 150
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Hello there i'm very new to learning about Generics. There is an exercise in Java How to Program by Paul and Harvey Deitel where i'm supposed to rewrite a method used in an earlier example but as a generic method. I've done it and it works fine but i keep getting this warning...

Note: GenericSelectionSort.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.


So i recompile with Xlint and it say the following...


GenericSelectionSort.java:22: warning: [unchecked] unchecked call to
compareTo<T> as member of raw type java.lang.Comparable

Here is the code



So the problem is where i'm trying to use the compareTo method but i don't understand why. I mean the method makes sure that the array implements the Comparable interface. Can anyone point out what i'm doing wrong please?
 
Rob Spoor
Sheriff
Posts: 21133
87
Chrome Eclipse IDE Java Windows
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Comparable is generic as well. So your <T extends Comparable> should be <T extends Comparable<T>> or even better <T extends Comparable<? super T>>. This second form allows you to use classes like java.sql.Date, since java.sql.Date indirectly implements Comparable<java.util.Date>, not Comparable<java.sql.Date>.
 
Neil Cartmell
Ranch Hand
Posts: 150
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Rob Prime wrote:Comparable is generic as well. So your <T extends Comparable> should be <T extends Comparable<T>> or even better <T extends Comparable<? super T>>. This second form allows you to use classes like java.sql.Date, since java.sql.Date indirectly implements Comparable<java.util.Date>, not Comparable<java.sql.Date>.


Thank you! i can't believe i didn't spot that as i'm looking right at examples using compareTo! Oh well problem solved, thanks for your help.
 
Campbell Ritchie
Marshal
Posts: 56527
172
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
It's not something you know until you see it written down.
 
With a little knowledge, a cast iron skillet is non-stick and lasts a lifetime.
  • Post Reply Bookmark Topic Watch Topic
  • New Topic
Boost this thread!