CODEWITHSUNDEEP
javacastingwarnings
Sophie Sperner
Sophie Sperner

Reputation: 4546

Unchecked cast warning in Java

Eclipse says: Type safety: Unchecked cast from Object to ObjectArrayList<Car> when I do:

final ObjectArrayList<Car> icars = (ObjectArrayList<Car>) cars[i];

where cars is defined as:

final Object[] cars = new Object[1000];
for (int i = 0; i < 1000; i++) {
    cars[i] = new ObjectArrayList<Car>();
}

Eclipse suggests to add @SuppressWarnings("unchecked") to icars object. But I've read somewhere that annotations are deprecated in Java, so should I leave it as it is?

Upvotes: 2

Views: 1101

Answers (3)

Xavi L&#243;pez
Xavi L&#243;pez

Reputation: 27880

The warning is just, well, warning you that the objects in the cars array that are being casted aren't guaranteed to be an ObjectArrayList<Car>.

It turns out Java doesn't allow array declaration with generic types unless they're unbounded (see Java 1.6: Creating an array of List, or this bug report 6229728 : Allow special cases of generic array creation). But if you could declare the array like this, you wouldn't be getting the warning:

final ObjectArrayList<Car>[] cars=new ObjectArrayList<Car>[1000]; // not allowed

If you really want to avoid an unchecked cast you should use a strongly typed collection instead of an array (for instance, a List<ObjectArrayList<Car>>).

The @SuppressWarnings("unchecked") annotation will just tell the compiler not to show the warning. I wouldn't advice using it unless the warning really annoys you. Reality is you'll be doing an unchecked cast (not a problem if you're certain about the type of the elements in the array).

As a side note, annotations aren't in any way deprecated in Java. Actually, it seems they'll become more powerful with Java 8 (it seems they'll support JSR 308: Annotations on Java Types). Maybe you read that @Deprecated is an annotation instead.

Upvotes: 6

Joan
Joan

Reputation: 426

You defined cars as: Object[] cars so when you try to put a cars element in icars you are casting from Object to ObjectArrayList<Car>.

The compiler can't know if all the elements in cars array are ObjectArrayList<Car>, so it shows the warning

To avoid this you can change the cars array definition to:

final ObjectArrayList<Car>[] cars = new ObjectArrayList<Car>[1000];

Upvotes: 0

GameDroids
GameDroids

Reputation: 5662

If you can't change the Object[] cars to be an ObjectArrayList<Car>[] array, then yes, leave it as it is. The compiler only gives you a warning. When you know that you can cast, then you don't have to listen to what the compiler says.

If you find the Warning annoying, then yes you can add the @SuppressWarnings("unchecked").

Upvotes: 0

Related Questions