X-Git-Url: https://git.hedgecode.org/?p=snooker-score-api.git;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Forg%2Fhedgecode%2Fsnooker%2Fjson%2FJsonPlayers.java;h=2ba59d3a6b2eac844b5a86f73e88503cc49d3e77;hp=26c68e2a7844014875d5396c94e2e59fbbd37a54;hb=edb206228987f506c5dc7a20585537b71f7a2e9d;hpb=20a8093246c9bb7def0c07a03bceff6f31412f35 diff --git a/src/main/java/org/hedgecode/snooker/json/JsonPlayers.java b/src/main/java/org/hedgecode/snooker/json/JsonPlayers.java index 26c68e2..2ba59d3 100644 --- a/src/main/java/org/hedgecode/snooker/json/JsonPlayers.java +++ b/src/main/java/org/hedgecode/snooker/json/JsonPlayers.java @@ -30,6 +30,8 @@ import org.hedgecode.snooker.compare.PlayerComparators; */ public class JsonPlayers extends JsonCollectionEntity implements Players { + private static final JsonPlayers EMPTY = new JsonPlayers(new Player[0]); + protected JsonPlayers(Player[] entities) { super(entities); } @@ -44,7 +46,7 @@ public class JsonPlayers extends JsonCollectionEntity implements Players for (Player player : this) if (player.type() == type) players.add(player); - return players.isEmpty() ? null : new JsonPlayers(players); + return players.isEmpty() ? EMPTY : new JsonPlayers(players); } @Override @@ -53,7 +55,7 @@ public class JsonPlayers extends JsonCollectionEntity implements Players for (Player player : this) if (player.shortName().contains(name)) players.add(player); - return players.isEmpty() ? null : new JsonPlayers(players); + return players.isEmpty() ? EMPTY : new JsonPlayers(players); } @Override @@ -62,7 +64,7 @@ public class JsonPlayers extends JsonCollectionEntity implements Players for (Player player : this) if (player.nationality().equals(nationality)) players.add(player); - return players.isEmpty() ? null : new JsonPlayers(players); + return players.isEmpty() ? EMPTY : new JsonPlayers(players); } @Override @@ -71,7 +73,7 @@ public class JsonPlayers extends JsonCollectionEntity implements Players for (Player player : this) if (player.nationality().equals(sex)) players.add(player); - return players.isEmpty() ? null : new JsonPlayers(players); + return players.isEmpty() ? EMPTY : new JsonPlayers(players); } @Override