LinkedList and iterators

[wdh321]

I have two problems, problem one is an error messages problem two is a how to question.

First off i have this error message when trying to run my program: EDIT

Exception in thread "main" java.util.ConcurrentModificationException

at java.util.LinkedList$ListItr.checkForComodificatio nLinkedList.java:761)
at java.util.LinkedList$ListItr.next(LinkedList.java: 696)
at Registry.deleteAssignment(Registry.java:25)
at RegistryTester.main(RegistryTester.java:17)
Java Result: 1

The message appears here when i try to delete from the linkedlist, the linked list contains two assignments which have already been added and printed to check they are there.

        System.out.println("Delete assignment with title Java 2");
        System.out.println("***********************************\n");
        
        newAssignment.deleteAssignment("Java");
        System.out.println(newAssignment.toString());
        
    }

This is the delete method im using, i think the problem is something to do with the iterator from what i have manged to see from searching, as to how to fix it i haven't seen a clear answer, hopefully you can help

public class Registry 
{
    LinkedList<Assignment> assignmentList;
    ListIterator<Assignment> iterator; 
    
    public Registry()
    {
        assignmentList = new LinkedList<Assignment>();
        iterator = assignmentList.listIterator(0);
    }
    
    public void deleteAssignment(String delete)
    {
        
        while(iterator.hasNext())
        {
            if(iterator.next().getATitle().equals(delete))
                iterator.remove();
        }
            
    }

Problem two i will elaborate on when this one is sorted, unless it would be easier to start a new topic.

[Tolls]

You've missed off most of the exception, chiefly the actual exception name.

[wdh321]

Sorry net beans had split the lines up and i missed them when copying. Its in full below.

From what ive seen, im removing an item from the list that the iterator is running through and that is causing the problem. Not sure what to do to fix this though.

Exception in thread "main" java.util.ConcurrentModificationException

at java.util.LinkedList$ListItr.checkForComodificatio nLinkedList.java:761)
at java.util.LinkedList$ListItr.next(LinkedList.java: 696)
at Registry.deleteAssignment(Registry.java:25)
at RegistryTester.main(RegistryTester.java:17)
Java Result: 1

[Tolls]

Normally an iterator is created where it is used, and not held onto as an attribute.
In your case you got hold of the iterator in your constructor and, in between getting it and reading it (ie using next()), you added something to the List the Iterator came from.
That's not allowed.

[wdh321]

Thank you, ive created the iterator in the delete method and that now works how i need it.