我有一个ArrayList，我想用来容纳RaceCar对象，这些对象在执行完后立即扩展Thread类。名为Race的类使用RaceCar对象完成执行后调用的回调方法来处理此ArrayList。回调方法addFinisher（RaceCar finisher）将RaceCar对象添加到ArrayList。这应该给出线程完成执行的顺序。

我知道ArrayList不同步，因此不是线程安全的。我尝试使用Collections.synchronizedCollection（c Collection）方法，方法是传入新的ArrayList并将返回的Collection分配给ArrayList。但是，这给了我一个编译器错误：

Race.java:41: incompatible types
found   : java.util.Collection
required: java.util.ArrayList
finishingOrder = Collections.synchronizedCollection(new ArrayList(numberOfRaceCars));

以下是相关代码：

public class Race implements RaceListener {
    private Thread[] racers;
    private ArrayList finishingOrder;

    //Make an ArrayList to hold RaceCar objects to determine winners
    finishingOrder = Collections.synchronizedCollection(new ArrayList(numberOfRaceCars));

    //Fill array with RaceCar objects
    for(int i=0; i<numberOfRaceCars; i++) {
    racers[i] = new RaceCar(laps, inputs[i]);

        //Add this as a RaceListener to each RaceCar
        ((RaceCar) racers[i]).addRaceListener(this);
    }

    //Implement the one method in the RaceListener interface
    public void addFinisher(RaceCar finisher) {
        finishingOrder.add(finisher);
    }

我需要知道的是，我使用的是正确的方法吗？如果不正确，应该使用什么使我的代码具有线程安全性？谢谢您的帮助！

使用Collections.synchronizedList()。

例如：

Collections.synchronizedList(new ArrayList<YourClassNameHere>())

更改

private ArrayList finishingOrder;

//Make an ArrayList to hold RaceCar objects to determine winners
finishingOrder = Collections.synchronizedCollection(new ArrayList(numberOfRaceCars)

至

private List finishingOrder;

//Make an ArrayList to hold RaceCar objects to determine winners
finishingOrder = Collections.synchronizedList(new ArrayList(numberOfRaceCars)

List是ArrayList的超类型，因此您需要指定它。

否则，您正在做的事情似乎还不错。另一个选择是您可以使用Vector，它是同步的，但这可能是我要做的。

CopyOnWriteArrayList

使用CopyOnWriteArrayList类。这是的线程安全版本ArrayList。

您可能使用了错误的方法。仅仅因为一个模拟汽车的线程在另一个汽车模拟线程之前完成并不意味着第一个线程应该赢得模拟比赛。

这在很大程度上取决于您的应用程序，但是最好有一个线程以较小的时间间隔计算所有汽车的状态，直到比赛结束。或者，如果您希望使用多个线程，则可以让每辆汽车记录完成比赛所花费的“模拟”时间，然后选择获胜者作为最短时间。

您也可以将synchronized关键字用于这样的addFinisher方法

    //Implement the one method in the RaceListener interface
    public synchronized void addFinisher(RaceCar finisher) {
        finishingOrder.add(finisher);
    }

因此，您可以通过这种方式使用ArrayList add方法线程安全。

每当您要使用ant线程安全版本的ant集合对象时，请使用java.util.concurrent。*包的帮助。它几乎具有所有并发版本的非同步收集对象。例如：对于ArrayList，您具有java.util.concurrent.CopyOnWriteArrayList

您可以执行Collections.synchronizedCollection（任何集合对象），但请记住此经典同步。技术很昂贵，并且会带来性能开销。java.util.concurrent。*包的成本较低，并且可以通过使用类似的机制来更好地管理性能

写入时复制，比较和交换，锁定，快照迭代器等。

因此，最好从java.util.concurrent。*包中获取一些东西

您也可以改为使用Vector，因为vector是线程安全的，而arraylist不是。尽管矢量很旧，但它们可以轻松解决您的目的。

但是您可以使Arraylist像下面的代码一样同步：

Collections.synchronizedList(new ArrayList(numberOfRaceCars())); 

您可以从ArrayList更改为Vector类型，在其中每种方法都将同步。

private Vector finishingOrder;
//Make a Vector to hold RaceCar objects to determine winners
finishingOrder = new Vector(numberOfRaceCars);