tags:

views:

108

answers:

2

Take the following method, which just returns a map of fields by name:

public static < T > HashMap< String, Field > getFields( Class< T > klass ) {
    HashMap< String, Field > fields = new HashMap< String, Field >();
    for ( Field f : klass.getFields() ) {
        fields.put( f.getName(), f );
    }
    return fields;
}

The method behaves identically if you remove the generic typing in the method signature, except that you get a warning for using a raw type. I've run into other similar things, especially around reflection, where you don't necessarily have the input type. It seems like reflection is just naturally going to have problems with generics, given that reflection is built to let you work with objects when you don't know (or care about) the type.

Aside from just pasting an "@SuppressWarning" on everything, does anyone have any good ideas about more elegant way of handling reflection without being constantly scolded by generics?

+4  A: 

Effective Java, Chapter 5 (Generics):

  • Don't use raw types in new code
  • Favor generic methods

So - don't remove the type parameter.

Bozho
+1 for succinct answer.
fastcodejava
The issue is more that you're required to specify a type parameter that doesn't serve a purpose.
Steve B.
+4  A: 

How about this (you don't need the template parameter T, so you can skip it):

public static HashMap< String, Field > getFields2( Class<?> klass ) {
    HashMap< String, Field > fields = new HashMap< String, Field >();
    for ( Field f : klass.getFields() ) {
        fields.put( f.getName(), f );
    }
    return fields;
}
tangens