Loop через ThreadGroup - пожалуйста, помогите мне отладить

Question

Я стараюсь найти проблему в этом Java-коде, но я не могу ее найти - вы можете мне помочь?

Надеюсь, код, который я предоставляю, достаточно, но при необходимости отправлю больше.

Далее извиняюсь, я не сделал минимального примера.

game.getGroupPlayers().list(); 
MoverThread[] playerThread = game.getPlayers(); 
System.out.println(playerThread.length); 
for (int i = 0; i < playerThread.length; i++) { 
    try { 
     System.out.println(i + " -> " +playerThread[i].toString()); 
     returnString += playerThread[i].toString() + "\n"; 
    } catch(NullPointerException e) { 
     System.out.println("Problem at i = " + i); 
     e.printStackTrace(); 
    } 
    game.getGroupPlayers().list(); 
} 

иногда дает мне следующий вывод:

java.lang.ThreadGroup [имя = Игроки, maxpri = 10]
игрока-0: 113
игрока-1: 277
Игрок-2: 0
0 -> Игрок-0: 113
1 -> Игрок-1: 277
Проблема при я = 2
java.lang.NullPointerException
на Referee.goalFound (Referee.java:70)
на DebugTestReferee.goalFound (DebugTestReferee.java:42)
в Player.checkGoal (Player.java:61)
на Player.run (Player.java:94)
в java.lang.Thread.run (Thread.java:636)
java.lang.ThreadGroup [имя = Игроки, maxpri = 10]
Player- 0: 113
Игрок-1: 277
Игрок-2: 0

[править] вот источник getPlayers()

/* 
* post returns the games players as an array 
*/ 
public MoverThread[] getPlayers() { 
synchronized(movers) { 
    MoverThread[] playerList = new MoverThread[players.activeCount()]; 
    players.enumerate(playerList); 
    return playerList; 
} 
} 

[править] вот как генерируется игроки

private ThreadGroup movers; 
private ThreadGroup players; 
private ThreadGroup ghosts; 

private Observer observer; 

/* 
* constructor 
*/ 
public Game(Maze maze, Referee referee) { 
this.maze = maze; 
this.referee = referee; 
threadList = new ArrayList<MoverThread>(); 
movers = new ThreadGroup("Movers"); 
players = new ThreadGroup(movers, "Players"); 
ghosts = new ThreadGroup(movers, "Ghosts"); 
observer = null; 
} 

[править]

Вот как я называю метод, который порождает проблему:

/* 
* post checks if the players thread was interrupted - if not if hostfield pretends to be a goal the game gets stopped and referee is called to perform "goal-found-actions" 
*/ 
private void checkGoal() { 
if (!getThread().isInterrupted()) { 
    synchronized(getGame().getMovers()) { 
     if (!getThread().isInterrupted()) { 
      if (getHostField().isGoal()) { 
       Field goal = getHostField(); 
       getGame().getReferee().goalFound(this, goal); 
       getGame().setGameOver(); 
      } 
     } 
    } 
} 
} 

и вот весь goalFound()

/* 
* post action to be performed if a player finds a goal 
* print some information 
*/ 
public void goalFound(Player player, Field at) { 
//FIXME get the Bug!!! 
String returnString = "Game over - player " + player.getName() + " found a goal on (" + at.getPos()[0] + ", " + at.getPos()[1] + ")!\n"; 
game.getGroupPlayers().list(); 
MoverThread[] playerThread = game.getPlayers(); 
System.out.println(playerThread.length); 
for (int i = 0; i < playerThread.length; i++) { 
    try { 
     System.out.println(i + " -> " +playerThread[i].toString()); 
     returnString += playerThread[i].toString() + "\n"; 
    } catch(NullPointerException e) { 
     System.out.println("Problem at i = " + i); 
     e.printStackTrace(); 
    } 
} 
game.getGroupPlayers().list(); 
returnString += game.mazeString(); 
System.out.println(returnString); 
} 

Answer 1

Я нашел решение - или, может быть, больше обходной путь ...

рядом с использованием ThreadGroups Я храню темы в ArrayList Aswell (возможно, вектор был бы еще лучше, но я в порядке с ArrayList) ,

Я не знаю почему, но когда я пытаюсь вызвать все темы в ThreadGroup, часто бывает, что некоторые потоки не учитываются. Однако с ArrayList он работает отлично.

Было бы интересно, почему ThreadGroups не работают должным образом и для чего они нужны в этом случае.

Answer 2

game.getPlayers(); возвращается MoverThread[] длиной 3, но третий один null.

Answer 3

Невозможно перечислить Thread s ThreadGroup. Это хорошо известный ужасный дизайн.

Между звонками и ThreadGroup.enumerate(Thread[]) потоки, возможно, начались или умерли. Лучшее, что вы можете сделать, это добавить фактор вымывания activeCount при распределении массива. Если возвращаемое значение соответствует длине массива, возможно, вы пропустили некоторые из них и должны повториться с большим размером массива (возможно, фактором больше, а не просто добавлением константы). В случае успеха вам необходимо соответствующим образом обрезать массив (или рассматривать его как таковой).