0

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

简单的代码总结：

@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();

* where operations() 计算“可绘制”对象属性的变化，删除满足特定条件的对象并将新对象添加到绘制列表中。从逻辑上讲，添加和写入应该分开吗？如果没有足够的信息，我可以发布操作（）方法，但我尽量不发布大量代码，以便更容易解释。

4

1 回答 1

1

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

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

你想要的是让你的列表临时同步。

你可以做这样的事情：

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

并且在代码中添加对象你的列表做的有些相同。

从这里编辑：

如果您想消除添加线程和绘制线程之间的所有并发问题，您可以这样做：

在添加图像的方法中：

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 上完成的唯一操作是读取。

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

于 2014-10-03T17:06:35.053 回答