StackOverflowError по рекурсивному методу

Question

Я получаю StackOverflowError при использовании рекурсивного метода. Я строю поверх API Spigot.

Это мой метод:

private Location spawnPlayer(Player p) { 
    int r = new Random().nextInt(ServerManager.getInstance().getServer(p).getSpawns().size()); 
    final Spawn s = ServerManager.getInstance().getServer(p).getSpawns().get(r); 

    if (s.hasPlayer()) { 
     spawnPlayer(p); 
    } else { 
     s.setPlayer(p); 

     Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(Arsenal.p, new Runnable() { 
      public void run() { 
       spawnClear--; 

       if (spawnClear == 0) { 
        s.setPlayer(null); 
       } 
      } 
     }, 0L, 20L); 
    } 
    return s.getLocation(); 
} 

Я получаю ошибку при spawnPlayer(p), которая называет себя, если порождение уже есть игрок.

Answer 1

Похоже s.hasPlayer() всегда верно, что делает это так:

if(s.hasPlayer()){ 
    spawnPlayer(p); 
} 

запускается на выполнение внутри spawnPlayer(Player) метода, который для них вызывает метод бежать бесконечно, в результате чего StackOverflowError.

Чтобы исправить это, вы могли бы подождать до вызова spawnPlayer(p):

if(s.hasPlayer()){ 
    long timeToWait = 20L;//set the time to wait to 20 ticks (1 second) 
    Bukkit.getServer().getScheduler().runTaskLater(Arsenal.p, new Runnable(){ 
     public void run(){ 
      spawnPlayer(p); 
     } 
    },timeToWait); 
} 

Итак, вот что ваш код может выглядеть следующим образом:

private Location spawnPlayer(Player p) { 
    int r = new Random().nextInt(ServerManager.getInstance().getServer(p).getSpawns().size()); 
    final Spawn s = ServerManager.getInstance().getServer(p).getSpawns().get(r); 

    if(s.hasPlayer()){ 
     long timeToWait = 20L;//set the time to wait to 20 ticks (1 second) 
     Bukkit.getServer().getScheduler().runTaskLater(Arsenal.p, new Runnable(){ 
      public void run(){ 
       spawnPlayer(p); 
      } 
     },timeToWait); 
    } 
    else{ 
     s.setPlayer(p); 

     Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(Arsenal.p, new Runnable() { 
      public void run() { 
       spawnClear--; 

       if (spawnClear == 0) { 
        s.setPlayer(null); 
       } 
      } 
     }, 0L, 20L); 
    } 
    return s.getLocation(); 
} 

Другое затруднительное бы сделать это так s.hasPlayer() не всегда например, используя ArrayList, чтобы убедиться, что игрок еще не был порожден:

List<String> spawned = new ArrayList<String>(); 

private Location spawnPlayer(Player p){ 
    int r = new Random().nextInt(ServerManager.getInstance().getServer(p).getSpawns().size()); 
    final Spawn s = ServerManager.getInstance().getServer(p).getSpawns().get(r); 

    if(s.hasPlayer() && !spawned.contains(p.getName()){ 
     spawned.add(p.getName()); 
     spawnPlayer(p); 
    } 
    else{ 
    //the rest of your code... 
    } 
} 

Answer 2

Я не уверен, но логически я думаю, что вы хотите:

if (!s.hasPlayer()) { 
    spawnPlayer(p); 

Примечание к!.

В целом, обратите внимание на рекурсивные функции в minecraft :) После того, как я написал один, который превратил бы дерево в журналы, но в итоге он перевернул листья - взрыв был довольно впечатляющим - уничтожил весь форрест и убил моего клиента из-за всей активности.