@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | throw new PlayersCollectionException('The couple can not be the same player'); |
38 | 38 | } |
39 | 39 | |
40 | - if (!$this->isDuplicatePlayer($player) && !$this->isDuplicatePlayer($couple) ) { |
|
40 | + if (!$this->isDuplicatePlayer($player) && !$this->isDuplicatePlayer($couple)) { |
|
41 | 41 | $this->players[$player->id()] = $player; |
42 | 42 | $this->players[$couple->id()] = $couple; |
43 | 43 | |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | { |
103 | 103 | foreach ($players as $mainPlayer) { |
104 | 104 | foreach ($players as $player) { |
105 | - if ($mainPlayer->id() == $player->id()){ |
|
105 | + if ($mainPlayer->id() == $player->id()) { |
|
106 | 106 | continue; |
107 | 107 | } |
108 | 108 | $this->exclusivePlayers[$mainPlayer->id()][] = $player->id(); |
@@ -177,7 +177,9 @@ discard block |
||
177 | 177 | */ |
178 | 178 | private function shuffleAssoc(array $list) |
179 | 179 | { |
180 | - if (!is_array($list)) return $list; |
|
180 | + if (!is_array($list)) { |
|
181 | + return $list; |
|
182 | + } |
|
181 | 183 | |
182 | 184 | $keys = $this->forceShuffle(array_keys($list)); |
183 | 185 | $random = []; |
@@ -194,7 +196,9 @@ discard block |
||
194 | 196 | */ |
195 | 197 | private function forceShuffle(array $list) |
196 | 198 | { |
197 | - if (!is_array($list) || count($list) < 2) return $list; |
|
199 | + if (!is_array($list) || count($list) < 2) { |
|
200 | + return $list; |
|
201 | + } |
|
198 | 202 | |
199 | 203 | $shuffleList = $list; |
200 | 204 | while ($shuffleList == $list) { |
@@ -112,7 +112,7 @@ |
||
112 | 112 | |
113 | 113 | $retry = count($this->players) + $this->players->countExclusivePlayers(); |
114 | 114 | |
115 | - while (!$this->tryMatchSecretSantaPlayers() && $retry > 0 ) { |
|
115 | + while (!$this->tryMatchSecretSantaPlayers() && $retry > 0) { |
|
116 | 116 | $retry--; |
117 | 117 | } |
118 | 118 |