java小程序链接分享,问题与链接列表（Java小程序）

In this code, I am trying to initialize a linked list of ovals that are sent to a class to be drawn.

public void actionPerformed(ActionEvent e)

{

oval p;

Graphics g = this.getGraphics();

int x, y, height, width, fill;

ListIterator li;

LinkedList list = new LinkedList();

li = list.listIterator();

x = Integer.parseInt(xfield.getText());

y = Integer.parseInt(yfield.getText());

height = Integer.parseInt(heightf.getText());

width = Integer.parseInt(widthf.getText());

list.add(new oval(x,y,height,width));

repaint();

while (li.hasNext())

{

p = (oval)li.next();

p.draw(g);

}

When I run this, I get this exception:

Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException

at java.util.LinkedList$ListItr.checkForComodification(Unknown Source)

at java.util.LinkedList$ListItr.next(Unknown Source)

at ovalapplet.actionPerformed(ovalapplet.java:97)

How can I fix this? (Line 97 is p = (oval)li.next(); if that helps.)

解决方案

You get the list's iterator before you change the state of the list, and then you use the iterator to iterate through, but now the iterator is no longer appropriate. Solution: change that order. Get the iterator only after adding items to the list, or use a ListIterator and add items with the ListIterator.

Other problems: you're drawing incorrectly as you should not get a Graphics object by calling getGraphics() on a component as this returns a short-lived object. To see what I mean, once you get your GUI working, minimize and then restore it after doing this drawing and watch your drawings disappear. Instead use a LinkedList field, change its state in your actionPerformed method, call repaint() and have your component's paintComponent(Graphics g) method (or the paint(Graphics g) method if this is an AWT program) do the drawing for you.