- CheckStyle.

This commit is contained in:
Chris
2012-11-07 13:51:08 +00:00
parent eb3ad526bd
commit 1c7bfb4d49

View File

@@ -79,9 +79,10 @@ public class CardDamageHistory {
* a boolean. * a boolean.
*/ */
public final void setCreatureAttackedLastTurnOf(final Player p, boolean value) { public final void setCreatureAttackedLastTurnOf(final Player p, boolean value) {
if ( value && !creatureAttackedLastTurnOf.contains(p) ) if (value && !creatureAttackedLastTurnOf.contains(p)) {
creatureAttackedLastTurnOf.add(p); creatureAttackedLastTurnOf.add(p);
while ( !value && creatureAttackedLastTurnOf.remove(p)) {} // remove should return false once no player is found in collection }
while (!value && creatureAttackedLastTurnOf.remove(p)) { } // remove should return false once no player is found in collection
} }
/** /**
* <p> * <p>
@@ -197,8 +198,9 @@ public class CardDamageHistory {
* @param player * @param player
*/ */
public void registerCombatDamage(Player player) { public void registerCombatDamage(Player player) {
if ( !damagedThisTurnInCombat.contains(player) ) if (!damagedThisTurnInCombat.contains(player)) {
damagedThisTurnInCombat.add(player); damagedThisTurnInCombat.add(player);
}
} }
/** /**
* TODO: Write javadoc for this method. * TODO: Write javadoc for this method.
@@ -212,8 +214,9 @@ public class CardDamageHistory {
* @param player * @param player
*/ */
public void registerDamage(Player player) { public void registerDamage(Player player) {
if ( !damagedThisTurn.contains(player) ) if (!damagedThisTurn.contains(player)) {
damagedThisTurn.add(player); damagedThisTurn.add(player);
}
} }
} }