Adding a generic arraylist to an arraylist

Question

so I have these four classes/interface:

  public class Box <S extends SomeClass> implements Comparable <Box<S>> {...}
  public interface SomeClass <T extends Comparable<T>> {...}
  public class ThisItem implements SomeClass {...}
  public class OtherItem implements SomeClass {...}

And I am trying to create a list of Box that holds a list of instances of ThisItem. I'm unsure as to why this is giving me an error.

  public ArrayList<ArrayList<Box>> variable = new ArrayList<ArrayList<Box>>();
  this.variable.add(new ArrayList<Box<ThisItem>>(5));

Make sure to include the exact error messages in questions. — user2246674
– user2246674, Commented May 10, 2013 at 0:17
Is it just me or do sometimes people just take generics too far? — Kevin Bowersox
– Kevin Bowersox, Commented May 10, 2013 at 0:22

zw324 · Accepted Answer · 2013-05-10 00:20:31Z

5

Box is a generic class, so when use it just as Box, it is a raw type, which is different from Box<ThisItem>, which has a type parameter specified. This is similar to ArrayList<Box>, where Box it the type parameter.

Change this:

public ArrayList<ArrayList<Box>> variable = new ArrayList<ArrayList<Box>>();

To:

public ArrayList<ArrayList<Box<ThisItem>>> variable = new ArrayList<ArrayList<Box<ThisItem>>>();

answered May 10, 2013 at 0:20

zw324

27.2k16 gold badges88 silver badges119 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

TheGandhi Over a year ago

This gets rid of the error, however I need to instantiate the ArrayList, then either add ThisItem or OtherItem ArrayList to it in a constructor for a new class so I can add items of their respective type to it.

Pshemo · Accepted Answer · 2013-05-10 00:50:57Z

1

How do you think, would it be safe to let variable store list like ArrayList<Box<ThisItem>>?

If Java would let that happen then while getting that list from variable it would be cased to ArrayList<Box>. Because of that returned list would let you add any kind of Box object to that list originally ArrayList<Box<ThisItem>> was suppose to store only Box<ThisItem> object.

To get rid of that problem you should declare your variable as

public ArrayList<ArrayList<Box<ThisItem>>> variable
 = new ArrayList<ArrayList<Box<ThisItem>>>();

edited May 10, 2013 at 0:50

answered May 10, 2013 at 0:32

Pshemo

125k25 gold badges194 silver badges280 bronze badges

2 Comments

TheGandhi Over a year ago

The problem is I dont know if the box needs to be of type ThisItem or OtherItem until I get into the constructor that requires arraylist of arraylists. So would there be a way to create an instance variable in the constructor w.o defining the variable outside it?

Pshemo Over a year ago

@TheGandhi Purpose of this error is to stop you from situation described in my answer, but if you are absolutely sure that you wont add incorrect object to incorrect lists, you can get rid of this error by casting your new list to raw type like variable.add((ArrayList)new ArrayList<Box<ThisItem>>(5));. But this way of coding is dangerous for type safety so don't use it unless you really have no other option left (in most cases people can redesign their projects to avoid this kind of situations).

Bohemian · Accepted Answer · 2013-05-10 02:46:05Z

1

ThisItem is a raw type of SomeClass; it's roughly the same as declaring it as implements SomeClass<Object>, so the compiler can't verify that it's appropriate to be used in that way.

Instead declare it as typed:

public class ThisItem implements SomeClass<SomeComparableClass> {...}

edited May 10, 2013 at 2:46

answered May 10, 2013 at 0:22

Bohemian♦

427k103 gold badges603 silver badges750 bronze badges

2 Comments

TheGandhi Over a year ago

I'm not quite sure what you mean by <SomethingComparable>. Could you explain that please?

Bohemian Over a year ago

SomeClass is defined as ` public interface SomeClass <T extends Comparable<T>> , which means to implement it is a typed` interface (not a raw type) you must provide a type for the generic parameter, and that type is bounded by the Comparable type, so SomethingComparable is meant to be that type. I'll change its name then - see edit

itsfarseen · Accepted Answer · 2013-05-10 00:29:53Z

0

What about this..

public ArrayList<ArrayList<Box>> variable = new ArrayList<ArrayList<Box>>();
this.variable.add(new ArrayList<Box<ThisItem>>(5));

to...

public ArrayList<ArrayList<Box>> variable = new ArrayList<ArrayList<Box>>();
ArrayList<Box<ThisItem>> tempInstance = new ArrayList<>();
tempInstance.add(new Box<ThisItem>()); //add new Boxes manually as you wish
this.variable.add(tempInstance);

answered May 10, 2013 at 0:29

itsfarseen

1,19810 silver badges26 bronze badges

1 Comment

TheGandhi Over a year ago

Got an error, "Cannot covert ArrayList<?> to ArrayList<Box<ThisItem>>"

rees · Accepted Answer · 2013-05-10 02:59:48Z

0

This should work:

public ArrayList<ArrayList<? extends Box>> variable = new ArrayList<ArrayList<? extends Box>>();

answered May 10, 2013 at 2:59

rees

1,5861 gold badge12 silver badges19 bronze badges

Collectives™ on Stack Overflow

Adding a generic arraylist to an arraylist

5 Answers 5

1 Comment

2 Comments

2 Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

2 Comments

2 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related