ArrayList size error

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.

Code:

---

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

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

Array list dont have that method.

my bad, it should be
racionals.addAll

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

Quote:

---

Originally Posted by 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..

---

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?

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

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

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

stack trace?

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

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