tags:

views:

1015

answers:

5

Offending bit of code

Vector moves = new Vector();

moves.add(new Integer(x));

Error:

ConnectFour.java:82: warning: [unchecked] unchecked call to add(E) as a member of the raw type java.util.Vector moves.add(new Integer(x));

Not really sure how much info is needed for an error like this....

+5  A: 

The problem is that the code above is not using generics.

The following will work:

Vector<Integer> moves = new Vector<Integer>();

move.add(new Integer(x));

The type name inside the <> (in the case of Vector, the type parameter E for the element to hold) tells the compiler what type of object it should expect.

If one tries to add an object that is of the specified type, such as in this case, trying to add an String to and Vector<Integer>, an compile-time error will occur, indicating that a type of object that is not of the expected type is being added.

That said, one should try not to use the Vector class. For more purposes, a class implementing List such as ArrayList from the Java Collections Framework would be sufficient, and better performing.

Edit

Although not directly related to the question about generics, Adam Paynter brought up a good point in the comments about the use of auto-boxing.

Since Java 5, primitives and their wrapper classes, e.g. int and Integer will be automatically converted between each other as necessary.

Therefore, it is possible to add an value specified as an int or an int literal into a class expecting an Integer:

Vector<Integer> v = new Vector<Integer>();
v.add(5);    // Not necessary to use an Integer value.
coobird
+1. Also, it appears that they're compiling for Java 5 or higher. Because of this, they could also take advantage of auto-boxing: `move.add(x);`
Adam Paynter
+3  A: 

That's not an error, it's just a compiler warning. Vector is usually parametized, so to get rid of the warning, just use generics:

Vector<Integer> moves = new Vector<Integer>();
moves.add(new Integer(x));
Andreas_D
+1  A: 
  1. initialize your vector like this

    Vector<Integer> moves = new Vector<Integer>();
    
  2. Preferably use java.util.ArrayList - it's a replacement of Vector

Bozho
A: 

If you have no choice but to use the non-generic data structure, you can put @SuppressWarnings("unchecked") at the start of the method to silence the warning.

This only be done if you have no choice but to use the non-generic vector. This usually happens when you're working with older libraries or certain parts of the Java runtime libraries.

James Schek
A: 

Not directly related to the code, but it is recommended to use (from version >= 5):

Integer.valueOf(x);

instead of

new Integer(x);

Because, some integer values {-128,...,127) are cached and it will always return the same object. This is very useful especially regarding to autoboxing.

fikovnik