実行が終了したらすぐにThreadクラスを拡張するRaceCarオブジェクトを保持するために使用するArrayListがあります。Raceと呼ばれるクラスは、RaceCarオブジェクトが実行の終了時に呼び出すコールバックメソッドを使用して、このArrayListを処理します。コールバックメソッドaddFinisher（RaceCar finisher）は、RaceCarオブジェクトをArrayListに追加します。これは、スレッドが実行を終了する順序を与えることになっています。

ArrayListは同期されていないため、スレッドセーフではありません。新しいArrayListを渡し、返されたコレクションをArrayListに割り当てることで、Collections.synchronizedCollection（c Collection）メソッドを使用してみました。ただし、これによりコンパイラエラーが発生します。

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のスーパータイプなので、指定する必要があります。

そうでなければ、あなたがやっていることは問題ないようです。他のオプションは、同期されているベクターを使用できることですが、これはおそらく私がすることです。

CopyOnWriteArrayList

CopyOnWriteArrayListクラスを使用します。これはのスレッドセーフバージョンですArrayList。

あなたは可能性が間違ったアプローチを使用すること。車をシミュレートする1​​つのスレッドが別の車シミュレーションスレッドの前に終了するからといって、最初のスレッドがシミュレートされたレースに勝つ必要があるわけではありません。

これはアプリケーションに大きく依存しますが、レースが完了するまで、短い時間間隔ですべての車の状態を計算する1つのスレッドを使用することをお勧めします。または、複数のスレッドを使用したい場合は、各車にレースの完了にかかった「シミュレーションされた」時間を記録させ、最短の勝者として勝者を選択することができます。

このようなメソッドに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はスレッドセーフであり、arraylistはそうではないため、代わりにVectorとして使用することもできます。ベクトルは古いですが、目的を簡単に解決できます。

ただし、次のコードのように、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);