Is a for-loop the right way of removing an object from an ArrayList?

Asked Jun 30 '15 at 22:23

Active Jun 30 '15 at 22:23

Viewed 40 times

So, I'm trying to remove an object from an ArrayList. When I do it and print the ArrayList again, the object is still there.

     else if (cmd==4){
        System.out.println("Enter the product name: ");
        String nameToRemove = input.next();
        for (Product x : inventoryList){
           if (nameToRemove == x.getName()){
              inventoryList.remove(x);
           }
        }
     }

Why won't the remove() method do anything when I implement it?

asked Jun 30 '15 at 22:23

hellomello

1

You are comparing strings with == instead of equals() – pedromss Jun 30 '15 at 22:24
2

Apart from String comparison, use iterator to perform remove operation to avoid concurrentmodifcationexception. – kosa Jun 30 '15 at 22:26
1

see `ArrayList.removeIf()` method – ZhongYu Jun 30 '15 at 22:30
@bayou.io Ooooh that's shiny! Happy to see that Java is getting more Haskellian. – ApproachingDarknessFish Jun 30 '15 at 22:45
@TuttiFruttiJacuzzi - no...... Java will never be any where near haskel in type system:) ever! – ZhongYu Jun 30 '15 at 22:56

Is a for-loop the right way of removing an object from an ArrayList?

0 Answers0