GolfLeague\Statistics\League\LeagueStatisticsEloquent

Complexity

Total Complexity

78

Size/Duplication

Total Lines	495
Duplicated Lines	72.73 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes	2
Bugs	1	Features	0

27 Methods

Rating	Name	Duplication	Size	Complexity
A	topFiveLowestScores()	0	4	1
A	topFiveLowestScoresByYear()	0	14	1
B	topFiveScoringAverageByYear()	0	32	4
A	mostSkinsByYear()	0	20	3
B	totalEagles()	25	25	5
B	netScoresByPlayer()	22	37	4
A	netScoresByPlayerTop()	0	5	1
B	netScoresByPlayerYear()	22	44	4
A	netScoresByPlayerTopYear()	0	5	1
A	netScoresLeague()	12	12	2
A	netScoresLeagueTop()	15	15	3
A	netScoresLeagueYear()	11	11	2
A	netScoresLeagueTopYear()	15	15	3
A	netCumulative()	11	11	2
A	netCumulativeTop()	15	15	3
A	netCumulativeByPlayer()	16	16	2
A	netCumulativeByPlayerTop()	16	16	2
A	netCumulativeYear()	11	11	2
A	netCumulativeTopYear()	15	15	3
A	netCumulativeByPlayerYear()	16	16	2
A	netCumulativeByPlayerTopYear()	16	16	2
A	topFiveNetScoresByYear()	0	10	1
B	totalBirdies()	26	26	5
B	totalPars()	24	24	5
B	totalBogeys()	24	24	5
B	totalDoubles()	24	24	5
B	totalOthers()	24	24	5

<?php namespace GolfLeague\Statistics\League;

use Illuminate\Support\Collection;
use \Round;
use \Player;
use \Skin;
use \Netwinner;
use \Holescore;
use \Match;
use Carbon\Carbon;

class LeagueStatisticsEloquent implements LeagueStatistics
{
    public function topFiveLowestScores()
    {
        return Round::with('player','course')->orderBy('score')->take(5)->get();
    }
    public function topFiveLowestScoresByYear($year)
    {
		$today = Carbon::today()->toDateString();
		$date1 = $year . '-01-01';
        $date2 = $year . '-12-31';
        return Round::with('player', 'course')
            ->where('match_id', '>', '0')
            ->where('date', '>=', $date1)
            ->where('date', '<=', $date2)
			->where('date', '<=', $today)
            ->orderBy('score')
            ->take(5)
            ->get();
    }
    public function topFiveScoringAverageByYear($year)
    {
        //Get players
        //For each player where match > 0 select scores and average them
        //Store in players array
        $date1 = $year . '-01-01';
        $date2 = $year . '-12-31';
        $players = Player::all();
        $average = array();
        $i = 0;
        foreach($players as $key => $player) {
            $rounds = Round::where('match_id', '>', '0')
                ->where('player_id', '=', $player->id)
				->where('score', '>', '30')
                ->where('date', '>=', $date1)
                ->where('date', '<=', $date2)
                ->get();
            $scores = array();
			foreach($rounds as $round) {
				$scores[] = $round->score;
			}
            if(count($scores) > 0) {
                $average[$i]['average'] = round((array_sum($scores) / count($scores)) ,2);
                $average[$i]['player_id'] = $player->id;
                $average[$i]['name'] = $player->name;
                $average[$i]['rounds'] = count($scores);
                $i++;
            }
        }
        array_multisort($average);
        return $average;
    }
    public function topFiveNetScoresByYear($year)
    {
		$date1 = $year . '-01-01';
		$date2 = $year . '-12-31';

		return Netwinner::with('player','match','match.course')
				->where('created_at', '>=', $date1)
				->where('created_at', '<=', $date2)
				->orderBy('score')->take(5)->get();
    }
    public function mostSkinsByYear($year)
    {
        $year = $year . '-01-01';
        $players = Player::all();
        $i = 0;
        $skinsCount = array();
        foreach($players as $key => $player) {
            $skins = Skin::with('player','level')
                ->where('player_id', '=', $player->id)
                ->where('created_at', '>', $year)
                ->get();
            if(count($skins) > 0) {
                $skinsCount[$i]['skins'] = $skins->count();
                $skinsCount[$i]['name'] = $player->name;
                $i++;
            }
        }
        array_multisort($skinsCount, SORT_DESC);
        return $skinsCount;
    }
    public function totalEagles($year){

This method seems to be duplicated in your project.

		$date1 = $year . '-01-01';
		$date2 = $year . '-12-31';
		$holescores = Holescore::with('round.player','hole')
			->where('created_at', '>=', $date1)
			->where('created_at', '<=', $date2)
			->get();
		$allEagles = array();
		foreach($holescores as $key => $holescore) {
			if($holescore['round']['match_id'] != null){
				if( ($holescore['hole']['par'] - $holescore['score']) === 2) {
					$allEagles[]= $holescore['round']['player']['name'];
				}
			}
		}
		$i =0;
		$newArray = array_count_values($allEagles);
		$eagles = array();
		foreach ($newArray as $key => $value) {
			$eagles[$i]['name'] = $key;
			$eagles[$i]['eagles'] =$value;
			$i++;
		}
		return $eagles;
	}
    public function totalBirdies($year)

This method seems to be duplicated in your project.

	{
		$date1 = $year . '-01-01';
		$date2 = $year . '-12-31';
		$holescores = Holescore::with('round.player','hole')
			->where('created_at', '>=', $date1)
			->where('created_at', '<=', $date2)
			->get();
		$allBirdies = array();
		foreach($holescores as $key => $holescore) {
			if($holescore['round']['match_id'] != null){
				if( ($holescore['hole']['par'] - $holescore['score']) === 1) {
						$allBirdies[]= $holescore['round']['player']['name'];
				}
			}
		}
		$i =0;
		$newArray = array_count_values($allBirdies);
			$birds = array();
			foreach ($newArray as $key => $value) {
				$birds[$i]['name'] = $key;
				$birds[$i]['birds'] =$value;
				$i++;
			}
			return $birds;
	}
    public function totalPars($year)

This method seems to be duplicated in your project.

	{
		$year = $year . '-01-01';
		$holescores = Holescore::with('round.player','hole')
			->where('created_at', '>', $year)
			->get();
		$allPars = array();
		foreach($holescores as $key => $holescore) {
			if($holescore['round']['match_id'] != null){
				if( ($holescore['hole']['par'] - $holescore['score']) === 0) {
						$allPars[]= $holescore['round']['player']['name'];
				}
			}
		}
		$i =0;
		$newArray = array_count_values($allPars);
			$pars = array();
			foreach ($newArray as $key => $value) {
				$pars[$i]['name'] = $key;
				$pars[$i]['pars'] =$value;
				$i++;
			}
			return $pars;
	}
    public function totalBogeys($year)

This method seems to be duplicated in your project.

	{
		$year = $year . '-01-01';
		$holescores = Holescore::with('round.player','hole')
			->where('created_at', '>', $year)
			->get();
		$allBogeys = array();
		foreach($holescores as $key => $holescore) {
			if($holescore['round']['match_id'] != null){
				if( ($holescore['score'] - $holescore['hole']['par']) === 1) {
						$allBogeys[]= $holescore['round']['player']['name'];
				}
			}
		}
		$i = 0;
		$newArray = array_count_values($allBogeys);
		$bogeys = array();
			foreach ($newArray as $key => $value) {
				$bogeys[$i]['name'] = $key;
				$bogeys[$i]['bogeys'] =$value;
				$i++;
			}
		return $bogeys;
	}
    public function totalDoubles($year)

This method seems to be duplicated in your project.

	{
		$year = $year . '-01-01';
		$holescores = Holescore::with('round.player','hole')
			->where('created_at', '>', $year)
			->get();
		$allDoubles = array();
		foreach($holescores as $key => $holescore) {
			if($holescore['round']['match_id'] != null){
				if( ($holescore['score'] - $holescore['hole']['par']) === 2) {
						$allDoubles[]= $holescore['round']['player']['name'];
				}
			}
		}
		$i = 0;
		$newArray = array_count_values($allDoubles);
			$doubles = array();
			foreach ($newArray as $key => $value) {
				$doubles[$i]['name'] = $key;
				$doubles[$i]['doubles'] =$value;
				$i++;
			}
		return $doubles;
	}
    public function totalOthers($year)

This method seems to be duplicated in your project.

	{
		$year = $year . '-01-01';
		$holescores = Holescore::with('round.player','hole')
			->where('created_at', '>', $year)
			->get();
		$allOthers = array();
		foreach($holescores as $key => $holescore) {
			if($holescore['round']['match_id'] != null){
				if( ($holescore['score'] - $holescore['hole']['par']) > 2) {
					$allOthers[]= $holescore['round']['player']['name'];
				}
			}
		}
		$i = 0;
		$newArray = array_count_values($allOthers);
		$others = array();
			foreach ($newArray as $key => $value) {
				$others[$i]['name'] = $key;
				$others[$i]['others'] =$value;
				$i++;
			}
		return $others;
	}

	public function netScoresByPlayer($playerId)
	{
		// Get players match scores
		$rounds = Round::with('player')
			->where('player_id', '=', $playerId)
			->whereNotNull('match_id')->get();

		//for each round get match handicap
		$rounds->map(function($round)

This code seems to be duplicated across your project.

		{
			$playerMatch = Player::find($round->player_id)->matches()->where('match_player.match_id','=', $round->match_id)->get();
			foreach($playerMatch as $player){
				$handicap = $player['pivot']['handicap'];
				$round->handicap = $handicap;
			}
		});
		//Calculate net score using match handicap
		$rounds->map(function($round)

This code seems to be duplicated across your project.

		{
			$net = ($round->score - round($round->handicap));
			return $round->netScore = ($net - $round->course->par);
		});

		// Sort by netScore
		$rounds->sort(function($a, $b)

This code seems to be duplicated across your project.

		{
			$a = $a->netScore;
			$b = $b->netScore;
			if ($a === $b) {
				return 0;
			}
			return ($a > $b) ? 1 : -1;
		});

		return $rounds;

	}
	public function netScoresByPlayerTop($playerId,$top)
	{
		$netScores = $this->netScoresByPlayer($playerId);
		return $netScores->take($top);
	}
	public function netScoresByPlayerYear($playerId, $year)
	{
		$date1 = $year . '-12-31';
		$date2 = $year . '-01-01';
		//$date2 = $date1 - 1;

40% of this comment could be valid code. Did you maybe forget this after debugging?

		//$date2 .= '-01-01';

43% of this comment could be valid code. Did you maybe forget this after debugging?

		// Get players match scores
		$rounds = Round::with('player')
			->where('player_id', '=', $playerId)
			->where('date', '<', $date1)
			->where('date','>', $date2)
			->whereNotNull('match_id')->get();

		//for each round get match handicap
		$rounds->map(function($round)

This code seems to be duplicated across your project.

		{
			$playerMatch = Player::find($round->player_id)->matches()->where('match_player.match_id','=', $round->match_id)->get();
			foreach($playerMatch as $player){
				$handicap = $player['pivot']['handicap'];
				$round->handicap = $handicap;
			}
		});
		//Calculate net score using match handicap
		$rounds->map(function($round)

This code seems to be duplicated across your project.

		{
			$net = ($round->score - round($round->handicap));
			return $round->netScore = ($net - $round->course->par);
		});

		// Sort by netScore
		$rounds->sort(function($a, $b)

This code seems to be duplicated across your project.

		{
			$a = $a->netScore;
			$b = $b->netScore;
			if ($a === $b) {
				return 0;
			}
			return ($a > $b) ? 1 : -1;
		});

		return $rounds;
	}
	public function netScoresByPlayerTopYear($playerId,$top, $year)
	{
		$netScores = $this->netScoresByPlayerYear($playerId, $year);
		return $netScores->take($top);
	}

	public function netScoresLeague()

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$netScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$netScores->push($this->netScoresByPlayer($player->id));
		}
		return $netScores;

	}
	public function netScoresLeagueTop($number)

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$netScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$netScore = $this->netScoresByPlayerTop($player->id, $number);
			if(!$netScore->isEmpty()){
				$netScores->push($netScore);
			}
		}

		return $netScores;
	}
	public function netScoresLeagueYear($year)

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$netScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$netScores->push($this->netScoresByPlayerYear($player->id,$year));
		}
		return $netScores;
	}
	public function netScoresLeagueTopYear($top,$year)

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$netScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$netScore = $this->netScoresByPlayerTopYear($player->id, $top, $year);
			if(!$netScore->isEmpty()){
				$netScores->push($netScore);
			}
		}

		return $netScores;
	}

	public function netCumulative()

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$cumulativeNetScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$cumulativeNetScores->push($this->netCumulativeByPlayer($player->id));
		}
		return $cumulativeNetScores;
	}
	public function netCumulativeTop($top)

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$cumulativeNetScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$netScore = $this->netCumulativeByPlayerTop($player->id, $top);
			if($netScore->isEmpty() == false){

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

				$cumulativeNetScores->push($netScore);
			}
		}

		return $cumulativeNetScores;
	}
	public function netCumulativeByPlayer($playerId)

This method seems to be duplicated in your project.

	{
		$netScores = $this->netScoresByPlayer($playerId);
		$cumulativeNet= new \Illuminate\Support\Collection();

		if($netScores->count() > 0){
			$netScore = $netScores->first();

			$totalNetScore = $netScores->sum('netScore');
			$cumulativeNet->put('player', $netScore->player->name);
			$cumulativeNet->put('score', $totalNetScore);
			$cumulativeNet->put('rounds', $netScores->count());
		}

		return $cumulativeNet;
	}
	public function netCumulativeByPlayerTop($playerId, $top)

This method seems to be duplicated in your project.

	{
		$netScores = $this->netScoresByPlayerTop($playerId, $top);
		$cumulativeNet= new \Illuminate\Support\Collection();

		if($netScores->count() > 0){
			$netScore = $netScores->first();

			$totalNetScore = $netScores->sum('netScore');
			$cumulativeNet->put('player', $netScore->player->name);
			$cumulativeNet->put('score', $totalNetScore);
			$cumulativeNet->put('rounds', $netScores->count());
		}

		return $cumulativeNet;
	}
	public function netCumulativeYear($year)

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$cumulativeNetScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$cumulativeNetScores->push($this->netCumulativeByPlayerYear($player->id,$year));
		}
		return $cumulativeNetScores;
	}
	public function netCumulativeTopYear($top,$year)

This method seems to be duplicated in your project.

	{
		//get players
		$players = Player::all();

		$cumulativeNetScores = new \Illuminate\Support\Collection();
		foreach($players as $player){
			$netScore = $this->netCumulativeByPlayerTopYear($player->id, $top, $year);
			if($netScore->isEmpty() == false){

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

				$cumulativeNetScores->push($netScore);
			}
		}

		return $cumulativeNetScores;
	}
	public function netCumulativeByPlayerYear($playerId,$year)

This method seems to be duplicated in your project.

	{
		$netScores = $this->netScoresByPlayerYear($playerId, $year);
		$cumulativeNet= new \Illuminate\Support\Collection();

		if($netScores->count() > 0){
			$netScore = $netScores->first();

			$totalNetScore = $netScores->sum('netScore');
			$cumulativeNet->put('player', $netScore->player->name);
			$cumulativeNet->put('score', $totalNetScore);
			$cumulativeNet->put('rounds', $netScores->count());
		}

		return $cumulativeNet;
	}
	public function netCumulativeByPlayerTopYear($playerId,$top,$year)

This method seems to be duplicated in your project.

	{
		$netScores = $this->netScoresByPlayerTopYear($playerId, $top, $year);
		$cumulativeNet= new \Illuminate\Support\Collection();

		if($netScores->count() > 0){
			$netScore = $netScores->first();

			$totalNetScore = $netScores->sum('netScore');
			$cumulativeNet->put('player', $netScore->player->name);
			$cumulativeNet->put('score', $totalNetScore);
			$cumulativeNet->put('rounds', $netScores->count());
		}

		return $cumulativeNet;
	}


}