Heroyt / tournament-generator

src/classes/class_group.php

Braces +146 added lines, -70 removed lines

@@ -33,23 +33,36 @@ 
 		foreach ($settings as $key => $value) {
 			switch ($key) {
 				case 'name':
-					if (gettype($value) === 'string') $this->name = $value;
-					else throw new \Exception('Expected string as group name '.gettype($value).' given');
+					if (gettype($value) === 'string') {
+						$this->name = $value;
+					} else {
+						throw new \Exception('Expected string as group name '.gettype($value).' given');
+					}
 					break;
 				case 'type':
-					if (in_array($value, groupTypes)) $this->type = $value;
-					else throw new \Exception('Unknown group type: '.$value);
+					if (in_array($value, groupTypes)) {
+						$this->type = $value;
+					} else {
+						throw new \Exception('Unknown group type: '.$value);
+					}
 					break;
 				case 'ordering':
-					if (in_array($value, orderingTypes)) $this->ordering = $value;
-					else throw new \Exception('Unknown group ordering: '.$value);
+					if (in_array($value, orderingTypes)) {
+						$this->ordering = $value;
+					} else {
+						throw new \Exception('Unknown group ordering: '.$value);
+					}
 					break;
 				case 'inGame':
 					if (gettype($value) === 'integer') {
-						if ($value === 2 || $value === 3 || $value === 4) $this->inGame = $value;
-						else throw new \Exception('Expected 2,3 or 4 as inGame '.$value.' given');
+						if ($value === 2 || $value === 3 || $value === 4) {
+							$this->inGame = $value;
+						} else {
+							throw new \Exception('Expected 2,3 or 4 as inGame '.$value.' given');
+						}
+					} else {
+						throw new \Exception('Expected integer as inGame '.gettype($value).' given');
 					}
-					else throw new \Exception('Expected integer as inGame '.gettype($value).' given');
 					break;
 				case 'maxSize':
 					if (gettype($value) === 'integer') {
@@ -98,10 +111,11 @@ 
 					'second' => 0,
 					'third'  => 0
 				];
-			}
-			elseif (gettype($team) === 'array') {
+			} elseif (gettype($team) === 'array') {
 				foreach ($team as $team2) {
-					if ($team2 instanceof Team) $this->teams[] = $team2;
+					if ($team2 instanceof Team) {
+						$this->teams[] = $team2;
+					}
 					$team2->groupResults[$this->id] = [
 						'group' => $this,
 						'points' => 0,
@@ -113,16 +127,20 @@ 
 						'third'  => 0
 					];
 				}
+			} else {
+				throw new \Exception('Trying to add team which is not an instance of Team class');
 			}
-			else throw new \Exception('Trying to add team which is not an instance of Team class');
 		}
 		return $this;
 	}
 	public function getTeams($filters = []) {
 		$teams = $this->teams;
 
-		if (gettype($filters) !== 'array' && $filters instanceof TeamFilter) $filters = [$filters];
-		elseif (gettype($filters) !== 'array') $filters = [];
+		if (gettype($filters) !== 'array' && $filters instanceof TeamFilter) {
+			$filters = [$filters];
+		} elseif (gettype($filters) !== 'array') {
+			$filters = [];
+		}
 
 		// APPLY FILTERS
 		foreach ($filters as $key => $filter) {
@@ -130,27 +148,31 @@ 
 				switch (strtolower($key)) {
 					case 'and':
 						foreach ($teams as $tkey => $team) {
-							if (!$this->filterAnd($team, $filter)) unset($teams[$tkey]); // IF FILTER IS NOT VALIDATED REMOVE TEAM FROM RETURN ARRAY
+							if (!$this->filterAnd($team, $filter)) {
+								unset($teams[$tkey]);
+							}
+							// IF FILTER IS NOT VALIDATED REMOVE TEAM FROM RETURN ARRAY
 						}
 						break;
 					case 'or':
 						foreach ($teams as $tkey => $team) {
-							if (!$this->filterOr($team, $filter)) unset($teams[$tkey]); // IF FILTER IS NOT VALIDATED REMOVE TEAM FROM RETURN ARRAY
+							if (!$this->filterOr($team, $filter)) {
+								unset($teams[$tkey]);
+							}
+							// IF FILTER IS NOT VALIDATED REMOVE TEAM FROM RETURN ARRAY
 						}
 						break;
 					default:
 						throw new \Exception('Unknown opperand type "'.$key.'". Expected "and" or "or".');
 						break;
 				}
-			}
-			elseif ($filter instanceof TeamFilter) {
+			} elseif ($filter instanceof TeamFilter) {
 				foreach ($teams as $tkey => $team) {
 					if (!$filter->validate($team, $this->id, 'sum', $this)) {
 						unset($teams[$tkey]); // IF FILTER IS NOT VALIDATED REMOVE TEAM FROM RETURN ARRAY
 					}
 				}
-			}
-			else {
+			} else {
 				throw new \Exception('Filer ['.$key.'] is not an instance of TeamFilter class');
 			}
 		}
@@ -161,20 +183,24 @@ 
 			if (gettype($value) === 'array') {
 				switch (strtolower($key)) {
 					case 'and':
-						if ($this->filterAnd($team, $value)) return false;
+						if ($this->filterAnd($team, $value)) {
+							return false;
+						}
 						break;
 					case 'or':
-						if ($this->filterOr($team, $value)) return false;
+						if ($this->filterOr($team, $value)) {
+							return false;
+						}
 						break;
 					default:
 						throw new \Exception('Unknown opperand type "'.$key.'". Expected "and" or "or".');
 						break;
 				}
-			}
-			elseif ($value instanceof TeamFilter) {
-				if (!$value->validate($team, $this->id, 'sum', $this)) return false;
-			}
-			else {
+			} elseif ($value instanceof TeamFilter) {
+				if (!$value->validate($team, $this->id, 'sum', $this)) {
+					return false;
+				}
+			} else {
 				throw new \Exception('Filer ['.$key.'] is not an instance of TeamFilter class');
 			}
 		}
@@ -185,20 +211,24 @@ 
 			if (gettype($value) === 'array') {
 				switch (strtolower($key)) {
 					case 'and':
-						if ($this->filterAnd($team, $value)) return true;
+						if ($this->filterAnd($team, $value)) {
+							return true;
+						}
 						break;
 					case 'or':
-						if ($this->filterOr($team, $value)) return true;
+						if ($this->filterOr($team, $value)) {
+							return true;
+						}
 						break;
 					default:
 						throw new \Exception('Unknown opperand type "'.$key.'". Expected "and" or "or".');
 						break;
 				}
-			}
-			elseif ($value instanceof TeamFilter) {
-				if (!$value->validate($team, $this->id, 'sum', $this)) return true;
-			}
-			else {
+			} elseif ($value instanceof TeamFilter) {
+				if (!$value->validate($team, $this->id, 'sum', $this)) {
+					return true;
+				}
+			} else {
 				throw new \Exception('Filer ['.$key.'] is not an instance of TeamFilter class');
 			}
 		}
@@ -220,16 +250,22 @@ 
 		return $t;
 	}
 	public function setType(string $type = R_R) {
-		if (in_array($type, groupTypes)) $this->type = $type;
-		else throw new \Exception('Unknown group type: '.$type);
+		if (in_array($type, groupTypes)) {
+			$this->type = $type;
+		} else {
+			throw new \Exception('Unknown group type: '.$type);
+		}
 		return $this;
 	}
 	public function getType() {
 		return $this->type;
 	}
 	public function setOrdering(string $ordering = POINTS) {
-		if (in_array($ordering, orderingTypes)) $this->ordering = $ordering;
-		else throw new \Exception('Unknown group ordering: '.$ordering);
+		if (in_array($ordering, orderingTypes)) {
+			$this->ordering = $ordering;
+		} else {
+			throw new \Exception('Unknown group ordering: '.$ordering);
+		}
 		return $this;
 	}
 	public function getOrdering() {
@@ -239,14 +275,20 @@ 
 		switch ($this->ordering) {
 			case POINTS:{
 				usort($this->teams, function($a, $b) {
-					if ($a->groupResults[$this->id]["points"] === $b->groupResults[$this->id]["points"] && $a->groupResults[$this->id]["score"] === $b->groupResults[$this->id]["score"]) return 0;
-					if ($a->groupResults[$this->id]["points"] === $b->groupResults[$this->id]["points"]) return ($a->groupResults[$this->id]["score"] > $b->groupResults[$this->id]["score"] ? -1 : 1);
+					if ($a->groupResults[$this->id]["points"] === $b->groupResults[$this->id]["points"] && $a->groupResults[$this->id]["score"] === $b->groupResults[$this->id]["score"]) {
+						return 0;
+					}
+					if ($a->groupResults[$this->id]["points"] === $b->groupResults[$this->id]["points"]) {
+						return ($a->groupResults[$this->id]["score"] > $b->groupResults[$this->id]["score"] ? -1 : 1);
+					}
 					return ($a->groupResults[$this->id]["points"] > $b->groupResults[$this->id]["points"] ? -1 : 1);
 				});
 				break;}
 			case SCORE:{
 				usort($this->teams, function($a, $b) {
-					if ($a->groupResults[$this->id]["score"] === $b->groupResults[$this->id]["score"]) return 0;
+					if ($a->groupResults[$this->id]["score"] === $b->groupResults[$this->id]["score"]) {
+						return 0;
+					}
 					return ($a->groupResults[$this->id]["score"] > $b->groupResults[$this->id]["score"] ? -1 : 1);
 				});
 				break;}
@@ -255,10 +297,14 @@ 
 	}
 	public function setInGame(int $inGame) {
 		if (gettype($inGame) === 'integer') {
-			if ($inGame === 2 || $inGame === 3 || $inGame === 4) $this->inGame = $inGame;
-			else throw new \Exception('Expected 2,3 or 4 as inGame '.$inGame.' given');
+			if ($inGame === 2 || $inGame === 3 || $inGame === 4) {
+				$this->inGame = $inGame;
+			} else {
+				throw new \Exception('Expected 2,3 or 4 as inGame '.$inGame.' given');
+			}
+		} else {
+			throw new \Exception('Expected integer as inGame '.gettype($inGame).' given');
 		}
-		else throw new \Exception('Expected integer as inGame '.gettype($inGame).' given');
 	}
 	public function getInGame() {
 		return $this->inGame;
@@ -286,16 +332,13 @@ 
 		foreach ($teams as $team) {
 			if ($team instanceOf Team) {
 				$this->progressed[] = $team->id;
-			}
-			elseif (gettype($team) === 'string' || gettype($team) === 'integer') {
+			} elseif (gettype($team) === 'string' || gettype($team) === 'integer') {
 				$this->progressed[] = $team;
-			}
-			elseif (gettype($team) === 'array') {
+			} elseif (gettype($team) === 'array') {
 				foreach ($team as $teamInner) {
 					if ($teamInner instanceOf Team) {
 						$this->progressed[] = $teamInner->id;
-					}
-					elseif (gettype($teamInner) === 'string' || gettype($teamInner) === 'integer') {
+					} elseif (gettype($teamInner) === 'string' || gettype($teamInner) === 'integer') {
 						$this->progressed[] = $teamInner;
 					}
 				}
@@ -349,12 +392,15 @@ 
 					while ($g > 0) {
 						foreach ($games as $key => $group) {
 							$this->games[] = array_shift($games[$key]);
-							if (count($games[$key]) === 0) unset($games[$key]);
+							if (count($games[$key]) === 0) {
+								unset($games[$key]);
+							}
 							$g--;
 						}
 					}
+				} else {
+					$this->games = $this->r_rGames();
 				}
-				else $this->games = $this->r_rGames();
 				break;
 		}
 		return $this->games;
@@ -366,8 +412,11 @@ 
 	}
 	public function addGame(Game ...$games){
 		foreach ($games as $game) {
-			if ($game instanceof Game) $this->games[] = $game;
-			else throw new \Exception('Trying to add game which is not instance of Game object.');
+			if ($game instanceof Game) {
+				$this->games[] = $game;
+			} else {
+				throw new \Exception('Trying to add game which is not instance of Game object.');
+			}
 		}
 		return $this;
 	}
@@ -395,21 +444,30 @@ 
 				$teams[$team->id] = 0;
 			}
 			foreach (end($this->games)->getTeams() as $team) {
-				if (!isset($teams[$team->id])) $teams[$team->id] = 4;
-				else $teams[$team->id] += 4;
+				if (!isset($teams[$team->id])) {
+					$teams[$team->id] = 4;
+				} else {
+					$teams[$team->id] += 4;
+				}
 			}
 			$g = prev($this->games);
 			if ($g instanceof Game) {
 				foreach ($g->getTeams() as $team) {
-					if (!isset($teams[$team->id])) $teams[$team->id] = 2;
-					else $teams[$team->id] += 2;
+					if (!isset($teams[$team->id])) {
+						$teams[$team->id] = 2;
+					} else {
+						$teams[$team->id] += 2;
+					}
 				}
 			}
 			$g = prev($this->games);
 			if ($g instanceof Game) {
 				foreach ($g->getTeams() as $team) {
-					if (!isset($teams[$team->id])) $teams[$team->id] = 1;
-					else $teams[$team->id]++;
+					if (!isset($teams[$team->id])) {
+						$teams[$team->id] = 1;
+					} else {
+						$teams[$team->id]++;
+					}
 				}
 			}
 
@@ -431,7 +489,9 @@ 
 					break;
 				}
 			}
-			if ($found) continue;
+			if ($found) {
+				continue;
+			}
 
 			// CYCLE 2
 			// ! TEAM WHICH PLAYED IN LAST TWO GAMES (NOT 6 or 7)
@@ -452,7 +512,9 @@ 
 					break;
 				}
 			}
-			if ($found) continue;
+			if ($found) {
+				continue;
+			}
 
 			// CYCLE 3
 			// ! TEAM WHICH PLAYED IN LAST THREE GAMES (NOT 7)
@@ -483,7 +545,9 @@ 
 					break;
 				}
 			}
-			if ($found) continue;
+			if ($found) {
+				continue;
+			}
 
 			// CYCLE 4
 			// ! TEAM WHICH PLAYED IN LAST THREE GAMES (NOT 7)
@@ -504,7 +568,9 @@ 
 					break;
 				}
 			}
-			if ($found) continue;
+			if ($found) {
+				continue;
+			}
 
 			// CYCLE 5
 			// TEAMS THAT DIDN'T PLAY IN LAST GAME WILL PLAY THIS GAME (< 4)
@@ -529,7 +595,9 @@ 
 					break;
 				}
 			}
-			if ($found) continue;
+			if ($found) {
+				continue;
+			}
 
 			// CYCLE 6
 			// FIRST AVAILABLE GAME
@@ -540,7 +608,9 @@ 
 	}
 	public function r_rGames(array $teams = []) {
 		$games = [];
-		if (count($teams) === 0) $teams = $this->teams;
+		if (count($teams) === 0) {
+			$teams = $this->teams;
+		}
 		switch ($this->inGame) {
 			case 2:
 				$games = circle_genGames2($teams, $this);
@@ -588,7 +658,9 @@ 
 			$game->setResults($results);
 		}
 		$return = $this->sortTeams($filters);
-		if (!$reset) return $return;
+		if (!$reset) {
+			return $return;
+		}
 		foreach ($this->getGames() as $game) {
 			$game->resetResults();
 		}
@@ -596,13 +668,17 @@ 
 	}
 	public function resetGames() {
 		foreach ($this->getGames() as $game) {
-			if (isset($game)) $game->resetResults();
+			if (isset($game)) {
+				$game->resetResults();
+			}
 		}
 		return $this;
 	}
 	public function isPlayed(){
 		foreach ($this->games as $game) {
-			if ((isset($game) || !$this->getSkip()) && !$game->isPlayed()) return false;
+			if ((isset($game) || !$this->getSkip()) && !$game->isPlayed()) {
+				return false;
+			}
 		}
 		return true;
 	}

src/classes/class_progression.php

Braces +13 added lines, -5 removed lines

@@ -29,7 +29,9 @@ 
 
 	public function addFilter(...$filters) {
 		foreach ($filters as $filter) {
-			if (!$filter instanceof TeamFilter) throw new \Exception('Trying to add filter which is not an instance of TeamFilter.');
+			if (!$filter instanceof TeamFilter) {
+				throw new \Exception('Trying to add filter which is not an instance of TeamFilter.');
+			}
 			$this->filters[] = $filter;
 		}
 		return $this;
@@ -37,8 +39,11 @@ 
 
 	public function progress() {
 		$teams = $this->from->sortTeams($this->filters);
-		if (count($this->filters) === 0 || $this->len !== null || $this->start !== 0) $next = array_splice($teams, $this->start, ($this->len === null ? count($teams) : $this->len));
-		else $next = $teams;
+		if (count($this->filters) === 0 || $this->len !== null || $this->start !== 0) {
+			$next = array_splice($teams, $this->start, ($this->len === null ? count($teams) : $this->len));
+		} else {
+			$next = $teams;
+		}
 		$this->from->addProgressed($next);
 		$this->to->addTeam($next);
 		return $this;
@@ -46,8 +51,11 @@ 
 
 	public function progressBlank(){
 		$teams = $this->from->isPlayed() ? $this->from->sortTeams($this->filters) : $this->from->simulate($this->filters);
-		if (count($this->filters) === 0 || $this->len !== null || $this->start !== 0) $next = array_splice($teams, $this->start, ($this->len === null ? count($teams) : $this->len));
-		else $next = $teams;
+		if (count($this->filters) === 0 || $this->len !== null || $this->start !== 0) {
+			$next = array_splice($teams, $this->start, ($this->len === null ? count($teams) : $this->len));
+		} else {
+			$next = $teams;
+		}
 		$this->from->addProgressed($next);
 		$i = 1;
 		foreach ($next as $team) {