External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-19 18:20:13 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

use for instead of while loop

Browse source
This commit is contained in:
androosss 2025-04-21 22:29:59 +02:00
parent 0ab9abd524
commit 00e51189e3
1 changed files with 4 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

11
Mage.Sets/src/mage/cards/g/GrenzosRebuttal.java
View file

@ -71,7 +71,7 @@ class GrenzosRebuttalEffect extends OneShotEffect {
public boolean apply(Game game, Ability source) {
UUID controllerId = source.getControllerId();
Player controller = game.getPlayer(controllerId);
if (controller == null || !controller.canRespond()) {
if (controller == null) {
return false;
}
List<Permanent> chosenPermanents = new ArrayList<>();
@ -79,13 +79,9 @@ class GrenzosRebuttalEffect extends OneShotEffect {
PlayerList playerList = game.getState().getPlayersInRange(controllerId, game, true);
Player currentPlayer = game.getPlayer(playerList.get());
Player nextPlayer = null;
boolean firstIteration = true;
while (!playerList.get().equals(controllerId) || nextPlayer == null) {
nextPlayer = game.getPlayer(playerList.getNext());
if (nextPlayer == null) {
return false;
}
for (Player nextPlayer = game.getPlayer(playerList.getNext()); !currentPlayer.getId().equals(controllerId) || firstIteration; nextPlayer = game.getPlayer(playerList.getNext())) {
if (currentPlayer != null) {
for (CardType cardType : cardTypes) {
@ -108,6 +104,7 @@ class GrenzosRebuttalEffect extends OneShotEffect {
}
}
}
firstIteration = false;
currentPlayer = nextPlayer;
}