ambidextrous/gemp-lotr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixing the infinite loop when sending state with unattached attached cards.

Browse Source
This commit is contained in:
marcins78@gmail.com
2012-07-31 08:28:59 +00:00
parent 6f4c5c91be
commit 69e5aa92a1
1 changed files with 13 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
gemp-lotr/gemp-lotr-logic/src/main/java/com/gempukku/lotro/game/state/GameState.java
View File

@@ -175,29 +175,24 @@ public class GameState {
for (Map.Entry<String, Integer> stringIntegerEntry : _playerPosition.entrySet())
listener.setPlayerPosition(stringIntegerEntry.getKey(), stringIntegerEntry.getValue());
Set<PhysicalCard> cardsLeftToSent = new HashSet<PhysicalCard>(_inPlay);
Set<PhysicalCard> sentCardsFromPlay = new HashSet<PhysicalCard>();
Set<PhysicalCard> attachedCardsToSend = new HashSet<PhysicalCard>();
// First non-attached cards
for (PhysicalCardImpl physicalCard : _inPlay) {
if (physicalCard.getAttachedTo() == null) {
listener.cardCreated(physicalCard);
sentCardsFromPlay.add(physicalCard);
} else
attachedCardsToSend.add(physicalCard);
}
int passes = 4;
// Now the attached ones
while (!attachedCardsToSend.isEmpty()) {
Iterator<PhysicalCard> iter = attachedCardsToSend.iterator();
while (iter.hasNext()) {
PhysicalCard attachedCard = iter.next();
if (sentCardsFromPlay.contains(attachedCard.getAttachedTo())) {
listener.cardCreated(attachedCard);
sentCardsFromPlay.add(attachedCard);
iter.remove();
while (cardsLeftToSent.size() > 0 && passes > 0) {
Set<PhysicalCard> newCardsLeft = new HashSet<PhysicalCard>();
for (PhysicalCard physicalCard : cardsLeftToSent) {
PhysicalCard attachedTo = physicalCard.getAttachedTo();
if (attachedTo == null || sentCardsFromPlay.contains(attachedTo)) {
listener.cardCreated(physicalCard);
sentCardsFromPlay.add(physicalCard);
} else {
newCardsLeft.add(physicalCard);
}
}
passes--;
cardsLeftToSent = newCardsLeft;
}
// Finally the stacked ones