Java画图“结巴”-列表并发

Question

我的一个应用程序通过读取数组列表将对象绘制到屏幕上：

简单代码摘要：

@Override
public synchronized void paintComponent(Graphics g) {
  for(Object gO:paintList) {
    g.drawImage( gO.getObjImage(), gO.getXLoc(), gO.getYLoc(), outer.getTransparent(), null);
  }
}

问题是每次用户点击鼠标时我都会添加更多的对象，所以如果用户点击的速度足够快，我会导致程序绘画结结巴巴，因为它在写入时无法读取（arrayList是同步的）。开发人员处理这种并发问题的常用做法是什么？

编辑：下面是调用重新绘制的代码：

byte ticks = 0;
    while(true) {
        currentTime = System.nanoTime();
        if(ticks == 25) {
            drawPanel.repaint();
            ticks = 0;
        } else if (ticks%5 == 0) {//if ticks is a multiple of 5 (5,10,15,...)
            drawPanel.operations();
            ticks++;
        } else if(ticks < 25) {
            ticks++;
        }
        try {
            /*
            Refer to: 'http://stackoverflow.com/questions/1036754/difference-between-wait-and-sleep'
            on differences between Thread.sleep() and wait()
            */
            wait(1);//old timings: (long)(refreshRate*1000)
        } catch (InterruptedException ex) {
            Logger.getLogger(DeathWish.class.getName()).log(Level.SEVERE, null, ex);
        }
        //Debugging
        //System.out.println(ticks);
        currentTime = System.nanoTime();

*其中operations()计算“paintable”对象属性的更改，移除满足特定条件的对象，并将新对象添加到“paint”列表中。在我看来，加法和写法应该分开？如果没有足够的信息，我可以发布operations（）方法，但我尽量不发布大量的代码，这样更容易解释。

Answer 1

我想你在同步方面有点不对劲：

• ArrayList是不同步实现的列表
• 同步方法意味着每次只有一个线程可以访问该方法，但函数中的变量根本不同步

您想要的是临时同步您的列表。

你可以这样做：

@Override
public void paintComponent(Graphics g) {
  synchronized(paintList) {
    for(Object gO:paintList) {
      g.drawImage( gO.getObjImage(), gO.getXLoc(), gO.getYLoc(), outer.getTransparent(), null);
    }
  }
}

在添加对象的代码中，您的列表也做了一些相同的操作。

从此处编辑：

如果您想要删除add线程和paint线程之间的所有并发问题，下面是您可以做的：

在添加图像的方法中：

public synchronized void addImage(...) {
  Something newImage = .....
  List<Something> newPaintList = new ArrayList<>(paintList.size() + 1);
  newPaintList.addAll(paintList);
  newPaintList.add(newImage);
  paintList = newPaintList;
}

并且在paint方法中，去掉同步部分。

@Override
public void paintComponent(Graphics g) {
  for(Object gO:paintList) {
    g.drawImage( gO.getObjImage(), gO.getXLoc(), gO.getYLoc(), outer.getTransparent(), null);
  }
}

这样，读取和写入之间就不会有任何并发，因为对paintList执行的唯一操作是读取。

应该同步Addmage以避免两个不同的线程同时添加图像，这可能会忽略一个Addmage。