ошибка переполнения стека во время цикла

Question

Я пытаюсь перебрать всех игроков и

1> Выход линии
2> Создать объект

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

public void refuteSuggestion(Weapon w, Character c, int currentPlayerNumber) { 
    for (Player p : game.getPlayers()) { 
     if(currentPlayerNumber != 6) { 
      if (p.getNum() == currentPlayerNumber + 1) { 
       Set<Card> cardSet = new HashSet<>(); 
       cardSet.addAll(p.getPlayerCards()); 
       Card card = (Card) getRandomObject(cardSet); 
       System.out.println(card.getName(0)); 
      } else { 
       refuteSuggestion(w, c, currentPlayerNumber + 1); 
      } 
     } 
     else{ 
      refuteSuggestion(w, c, 0); 
     } 

    } 
    System.out.println("No one refuted..."); 
} 

Answer 1

Не уверен, что вы пытаетесь сделать здесь, но вы вызываете метод из того же метода, который является бесконечным циклом. Следовательно, ошибка переполнения стека.

В ваших других условиях вы изменяете значение currentPlayerNumber, так что просто делайте это вместо того, чтобы вызывать одну и ту же функцию после изменения текущего слоя.

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

public void refuteSuggestion(Weapon w, Character c, int currentPlayerNumber) { 
    for (Player p : game.getPlayers()) { 
     if(currentPlayerNumber != 6) { 
      if (p.getNum() == currentPlayerNumber + 1) { 

        nextFunction(p); 
      } else { 
       currentPlayerNumber += 1; 
      } 
     } 
     else{ 
      currentPlayerNumber = 0; 
     } 

    } 
    System.out.println("No one refuted..."); 
} 

public void nextFunction(Player p){ 
    Set<Card> cardSet = new HashSet<>(); 
    cardSet.addAll(p.getPlayerCards()); 
    Card card = (Card) getRandomObject(cardSet); 
    System.out.println(card.getName(0)); 
    }