ฉันมี ArrayList ที่ฉันต้องการใช้เพื่อเก็บวัตถุ RaceCar ที่ขยายคลาสเธรดทันทีที่ดำเนินการเสร็จสิ้น คลาสที่เรียกว่า Race จะจัดการ ArrayList นี้โดยใช้วิธีการเรียกกลับที่อ็อบเจ็กต์ RaceCar เรียกใช้เมื่อดำเนินการเสร็จสิ้น วิธีการโทรกลับ 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 เพิ่มเมธอดเธรดได้อย่างปลอดภัยด้วยวิธีนี้

เมื่อใดก็ตามที่คุณต้องการใช้อ็อบเจ็กต์การรวบรวมมดเวอร์ชันปลอดภัยของเธรดให้ใช้ความช่วยเหลือของแพ็คเกจjava.util.concurrent. * มีเกือบทั้งหมดของวัตถุคอลเลกชันที่ไม่ซิงโครไนซ์พร้อมกัน เช่นสำหรับ ArrayList คุณมี java.util.concurrent.CopyOnWriteArrayList

คุณสามารถทำ Collections.synchronizedCollection (วัตถุคอลเลกชันใดก็ได้) แต่จำการซิงค์แบบคลาสสิกนี้ไว้ เทคนิคมีราคาแพงและมาพร้อมกับค่าใช้จ่ายในการแสดง แพ็คเกจ java.util.concurrent. * มีราคาไม่แพงและจัดการประสิทธิภาพได้ดีขึ้นโดยใช้กลไกเช่น

คัดลอกเมื่อเขียนเปรียบเทียบและสลับล็อคตัวทำซ้ำสแนปชอต ฯลฯ

ดังนั้นต้องการบางอย่างจากแพ็คเกจ java.util.concurrent. *

คุณยังสามารถใช้เป็น 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);