Solved index out of bounds help

Code:Java

 
List<String> lore = item.getItemMeta().getLore();
            for (int index = 0; index <= lore.size(); index++) {
                System.out.println(index + ", " + lore.get(index));
                if (lore.get(index).contains("Merchant")
                        || lore.get(index).contains("Price")) {
                    lore.remove(index);
                }
            }
 

The exception is on line 4, here is the error

Caused by: java.lang.IndexOutOfBoundsException: Index: 2, Size: 2

 

@1Camer0471

That for loop is running for all elements in the List object lore, plus one. Since indexes start at 0, the last index value will be list.size() -1, which is "out of bounds" and why your code is throwing an exception.

Tl;Dr: Change index <= lore.size() to index < lore.size().

 

@1Camer0471 You're going outside of the the List size bounds.

 

what do you mean? there are 3 lines of lore: 0, 1, 2 it's reading lines 0 and 1, but then stopping at line 2?

 

@1Camer0471 Check your for-loop condition.

 

@1Camer0471 Whenever an item in your lore matches the criteria you specified in your if statement, it will get removed from the list. I assume one of the items in the lore list does match with the criteria, and so it is removed from the list, leaving you with two indexes (0 and 1). That is why the number being given in the exception is 2, because the size is 2 and the last possible index is the size minus 1, so its 1.

 

*cough* less than or equals *cough*

 

Why u here using for(i) loop? U don't need it. Just use Foreach.

Code:

for(String line : lore) {
    if (line.contains("Merchant") || line.contains("Price")) {
        lore.remove(line);
    }
}

 

@sionzee Are you trying to throw errors on purpose?!?!

 

@teej107 What? What about u talking? That what I said is correct. No errors, no throwing...

 

Try out your "working" code yourself. Have fun with your ConcurrentModificationException

 

That piece of code can throw a ConcurrentModificationException.

 

You can clone list and then replace him. (Alternate solution)
Ok, I don't know why getLore is unmodifitable list (they can @Override add, remove method to minecraft usage)
But ok. Probably all have own reason. It's a Bukkit core....

And I can't know if item is ItemStack and getLore is from ItemMeta#getLore...
He can use it as own classes.

BTW:
replace
for (int index = 0; index <= lore.size(); index++) {
to
for (int index = 0; index < lore.size(); index++) {
Because it start from index zero.

 

lmao a mini fire war went down while I was eating supper
@teej107 @sionzee @FisheyLP

I fixed it, thanks for your help! <3

 

@1Camer0471
Mark as solved if solved.