Как это имеет смысл?

Question

сегодня я нашел довольно забавную проблему. В основном этот код не работает, если вы удалите System.out.println. Он никогда не выходит внутрь, если без него !!! (Thread запускается из основного класса)

import java.util.LinkedList; 
import java.util.Vector; 
import java.util.Queue; 



public class Matchmaking extends Thread{ 
    public static Vector onlinePlayers = new Vector(); 
    public static Queue<Player> queuedPlayers = new LinkedList<Player>(); 


    @Override 
    public void run() { 
     while(true){ 
      System.out.println(queuedPlayers.size()); 
      if(queuedPlayers.size() >= 2){ 
       new Matchmaking_GameFoundThreads(queuedPlayers.remove(),queuedPlayers.remove()); 
      } 
     } 
    } 
} 

Answer 1

LinkedList несинхронизирован.

Изменения в нем в одном потоке могут быть не видны в другом потоке. Попробуйте использовать:

public static List<Player> queuedPlayers = 
    Collections.synchronizedList(new LinkedList<Player>()); 

Answer 2

Я решил его, сделав очереди volatile. Я не уверен, что это хорошо, потому что я не знаком с использованием volatile, но он работал ...