IndexOutOfBoundsException

[aldo1987]

Well i have e f**k off project in for thursday and i have a little runtime error that goes as such :-

notifyPassengerArrival(passengers.get(0));

on this line i want to return the first element in the array (a Passenger object) and pass it to the notifyPassengerArrival method, but i keep getting an IndexOutOfBoundsException.

I have confirmed that the array is not empty by using a println, so there is definetly an object at 0 in the array.

I just dont get it!

[sukatoa]

Can you show as your get(int index) method?

[Eranga]

Hi aldo1987,

Welcome to our community. :)

Yes, put some code here we can check/test. You say that, it is confirmed using println(), so what is the output? May be after that array content can be changed.

[Zosden]

Please don't you curse words, they just make you sound incompetent. If you post some code then we can help you.

[aldo1987]

public void act()
{
//ADDED ENTIRE ACT FUNCTIONALITY
if(destinations.isEmpty() == true && pickups.isEmpty() == true)
{
incrementIdleCount();
}

else
{
chooseTargetLocation();
Location target = getTargetLocation();
if(target != null) {
// Find where to move to next.
Location next = getLocation().nextLocation(target);
setLocation(next);
if(next.equals(target))
{
if(target == pickups.get(0))
{
notifyPickupArrival();
}
else if(target == destinations.get(0))
{
System.out.println(passengers.get(0));
notifyPassengerArrival(passengers.get(0));
offloadPassenger();
}


}
}

}


}

this is the method my problem appears in, The result of my println is "Passenger travelling from location 2,28 to location 18,21", in accordance with the toString method of the passenger class.

[Zosden]

Congratulation's for maintaining the ability to write Unmaintainable code. isEmpty = true is extremely redundant. Thats like saying is 1 = 1. I mean come on. Second thing I have absolutely know idea what this code does. Your passing in magic numbers into other methods that I can't see. Also please put your code in the code blocks next time it makes it more readable.

[Eranga]

yes aldo, your question is not clear. And some coding styles are pointless, as Zosden says,

if(destinations.isEmpty() == true && pickups.isEmpty() == true)

no real mean of this. It's legal to use, but not good to use. isEmpty() always return a boolean value. So why did you comparing it again.