ArrayList size error

[wepwep]

Hi, im using arraylist to store a collection of objects. I have a menu that do a lot of things, also loading/saving. Saving works well, and loading too, but im having a hard time trying to copy the loaded collection in the actual one, because i get IndexOutOfBoundsException in the collection, when calling collections.copy.

public void cargarRacionals() {
    	ObtenirRacional or = new ObtenirRacional();
    	System.out.println("XX:" + (or.retornarRacionals().size()));
    	//racionals.ensureCapacity((or.retornarRacionals().size()));
    	ArrayList<Racional> racionals = new ArrayList<Racional>(or.retornarRacionals().size()+1);
        Collections.copy(racionals, or.retornarRacionals());
    }

Here is the code. I translate and coment each thing:
racionals : the arraylist of racional objects.
ObtenirRacional :the class that loads and deserializes.
retornar racionals: return the array

Ok. The error is in the last line, and i dont get why. I have tried with the two ways:
//racionals.ensureCapacity((or.retornarRacionals().s ize()));
ArrayList<Racional> racionals = new ArrayList<Racional>(or.retornarRacionals().size()+ 1);

I create a new array with the needed size, or i set the new size. Note that size is ok, its says its 8 and it goes well.

pd: sorry for my bad english

[ozzyman]

try this instead:
racionals.AddRange(or.retornarRacionals());

[wepwep]

Array list dont have that method.

[ozzyman]

my bad, it should be
racionals.addAll

[UJJAL DHAR]

when you are tring to copy,A stack overflow is being occured.That means,the destinaton array-size is smaller than the [copy-from]array size.Now what to do,you know very well..


Regards,
Ujjal

[ozzyman]

when you are tring to copy,A stack overflow is being occured.That means,the destinaton array-size is smaller than the [copy-from]array size.Now what to do,you know very well..

he's done this though:
or.retornarRacionals().size()+1

to ensure its larger. so i'm not sure where the problem is coming from but thats why i was suggesting to use .addAll(Collections c) method rather than copy, because you can add infinitely can't you?

[wepwep]

Yes, now i use a clear + add all and works fine. But i dont know why the resize option or the constructor one dont work...

thanks anyway

[Tolls]

What was the full error you got?
Stack trace and all?

Might give us an idea of what it thought the problem was.

[wepwep]

stack trace?

i dont remember anything strange in the errror, only all the calls like main-objecta,object-method,etc.

[Tolls]

That's the stack trace.
And the full error text will tell you what index it is trying to use as well.