mirror of
https://github.com/magefree/mage.git
synced 2026-01-22 19:29:59 -08:00
replace [list].size() == 0 and [string].length() == 0 to use .isEmpty() instead
This commit is contained in:
parent
3612332607
commit
3128bd2b4b
48 changed files with 90 additions and 109 deletions
|
|
@ -95,7 +95,7 @@ class CreaturesYouControlBecomesTargetTriggeredAbility extends TriggeredAbilityI
|
|||
if (object != null && object instanceof Spell) {
|
||||
Card c = (Spell) object;
|
||||
if (c.getCardType().contains(CardType.INSTANT) || c.getCardType().contains(CardType.SORCERY)) {
|
||||
if (getTargets().size() == 0) {
|
||||
if (getTargets().isEmpty()) {
|
||||
for (Effect effect : getEffects()) {
|
||||
effect.setTargetPointer(new FixedTarget(event.getTargetId()));
|
||||
}
|
||||
|
|
|
|||
|
|
@ -102,7 +102,7 @@ class NezumiGraverobberFlipEffect extends OneShotEffect {
|
|||
if (card != null) {
|
||||
Player player = game.getPlayer(card.getOwnerId());
|
||||
if (player != null) {
|
||||
if (player.getGraveyard().size() == 0) {
|
||||
if (player.getGraveyard().isEmpty()) {
|
||||
return new FlipSourceEffect(new NighteyesTheDesecratorToken()).apply(game, source);
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -112,7 +112,7 @@ class DesecratorHagEffect extends OneShotEffect {
|
|||
}
|
||||
}
|
||||
}
|
||||
if (cards.size() == 0) {
|
||||
if (cards.isEmpty()) {
|
||||
return true;
|
||||
}
|
||||
if (cards.size() > 1
|
||||
|
|
|
|||
|
|
@ -85,7 +85,7 @@ class OpponentHasNoCardsInHandCondition implements Condition {
|
|||
if (player != null) {
|
||||
for (UUID playerId : game.getOpponents(source.getControllerId())) {
|
||||
Player opponent = game.getPlayer(playerId);
|
||||
if (opponent != null && opponent.getHand().size() == 0) {
|
||||
if (opponent != null && opponent.getHand().isEmpty()) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -86,7 +86,7 @@ class DingusEggTriggeredAbility extends TriggeredAbilityImpl {
|
|||
if (zEvent.getFromZone() == Zone.BATTLEFIELD
|
||||
&& zEvent.getToZone() == Zone.GRAVEYARD
|
||||
&& zEvent.getTarget().getCardType().contains(CardType.LAND)) {
|
||||
if (getTargets().size() == 0) {
|
||||
if (getTargets().isEmpty()) {
|
||||
UUID targetId = zEvent.getTarget().getControllerId();
|
||||
for (Effect effect : this.getEffects()) {
|
||||
effect.setTargetPointer(new FixedTarget(targetId));
|
||||
|
|
|
|||
|
|
@ -84,7 +84,7 @@ class GorillaTitanCondition implements Condition {
|
|||
Player p = game.getPlayer(source.getControllerId());
|
||||
if (p != null)
|
||||
{
|
||||
return p.getGraveyard().size() == 0;
|
||||
return p.getGraveyard().isEmpty();
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -102,7 +102,7 @@ class CloneShellEffect extends OneShotEffect {
|
|||
cards.add(card);
|
||||
}
|
||||
|
||||
if (cards.size() == 0) {
|
||||
if (cards.isEmpty()) {
|
||||
return false;
|
||||
|
||||
}
|
||||
|
|
|
|||
|
|
@ -89,7 +89,7 @@ class AEtherFlashTriggeredAbility extends TriggeredAbilityImpl {
|
|||
UUID targetId = event.getTargetId();
|
||||
Permanent permanent = game.getPermanent(targetId);
|
||||
if (filter.match(permanent, getSourceId(), getControllerId(), game)) {
|
||||
if (getTargets().size() == 0) {
|
||||
if (getTargets().isEmpty()) {
|
||||
for (Effect effect : this.getEffects()) {
|
||||
effect.setTargetPointer(new FixedTarget(targetId));
|
||||
}
|
||||
|
|
|
|||
|
|
@ -134,7 +134,7 @@ class HollowbornBarghestTriggeredAbility extends TriggeredAbilityImpl {
|
|||
public boolean checkTrigger(GameEvent event, Game game) {
|
||||
if (game.getOpponents(controllerId).contains(event.getPlayerId())) {
|
||||
Player opponent = game.getPlayer(event.getPlayerId());
|
||||
if (opponent != null && opponent.getHand().size() == 0) {
|
||||
if (opponent != null && opponent.getHand().isEmpty()) {
|
||||
opponent.loseLife(2, game);
|
||||
return true;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -100,7 +100,7 @@ class ImmortalCoilAbility extends StateTriggeredAbility {
|
|||
@Override
|
||||
public boolean checkTrigger(GameEvent event, Game game) {
|
||||
Player player = game.getPlayer(this.getControllerId());
|
||||
return player != null && player.getGraveyard().size() == 0;
|
||||
return player != null && player.getGraveyard().isEmpty();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
|
|
|||
|
|
@ -94,7 +94,7 @@ class RivalryTriggeredAbility extends TriggeredAbilityImpl {
|
|||
|
||||
@Override
|
||||
public boolean checkTrigger(GameEvent event, Game game) {
|
||||
if (getTargets().size() == 0) {
|
||||
if (getTargets().isEmpty()) {
|
||||
for (Effect effect : this.getEffects()) {
|
||||
effect.setTargetPointer(new FixedTarget(event.getPlayerId()));
|
||||
}
|
||||
|
|
|
|||
|
|
@ -133,7 +133,7 @@ class QuestForTheNihilStoneTriggeredAbility2 extends TriggeredAbilityImpl {
|
|||
if (game.getOpponents(controllerId).contains(event.getPlayerId())) {
|
||||
Player opponent = game.getPlayer(event.getPlayerId());
|
||||
if (opponent != null
|
||||
&& opponent.getHand().size() == 0
|
||||
&& opponent.getHand().isEmpty()
|
||||
&& quest.getCounters(game).getCount(CounterType.QUEST) >= 2) {
|
||||
for (Effect effect : this.getEffects()) {
|
||||
effect.setTargetPointer(new FixedTarget(opponent.getId()));
|
||||
|
|
|
|||
|
|
@ -90,7 +90,7 @@ class GuulDrazSpecterCondition implements Condition {
|
|||
|
||||
Set<UUID> opponents = game.getOpponents(source.getControllerId());
|
||||
for (UUID opponentId : opponents) {
|
||||
result |= game.getPlayer(opponentId).getHand().size() == 0;
|
||||
result |= game.getPlayer(opponentId).getHand().isEmpty();
|
||||
}
|
||||
|
||||
return result;
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue