GolfLeague\Statistics\Player\PlayerStatisticsEloquent

Complexity

Total Complexity

31

Size/Duplication

Total Lines	92
Duplicated Lines	17.39 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

26 Methods

Rating	Name	Duplication	Size	Complexity
A	matchesHandicap()	0	1	1
A	scoringAverage()	16	16	2
A	scoringAverageByYear()	0	22	2
A	scoringAverageMatchesByYear()	0	1	1
A	handicapRounds()	0	1	1
A	scoringAverageCourse()	0	1	1
A	scoringAverageCourseByYear()	0	1	1
A	scoringAverageCourseMatchesByYear()	0	1	1
A	totalEagles()	0	1	1
A	eaglesByYear()	0	1	1
A	eaglesMatchesByYear()	0	1	1
A	totalBirdies()	0	1	1
A	birdiesByYear()	0	17	4
A	birdiesMatchesByYear()	0	1	1
A	totalPars()	0	1	1
A	parsByYear()	0	1	1
A	parsMatchesByYear()	0	1	1
A	totalbogeys()	0	1	1
A	bogeysByYear()	0	1	1
A	bogeysMatchesByYear()	0	1	1
A	totalDoubles()	0	1	1
A	doublesByYear()	0	1	1
A	doublesMatchesByYear()	0	1	1
A	totalOthers()	0	1	1
A	othersByYear()	0	1	1
A	othersMatchesByYear()	0	1	1

<?php namespace GolfLeague\Statistics\Player;

use Carbon\Carbon;
use \Round;
use \Player;
use \Skin;
use \Netwinner;
use \Holescore;


class PlayerStatisticsEloquent implements PlayerStatistics
{
    private $date1 = '-01-01';

The property $date1 is not used and could be removed.

    private $date2 = '-12-31';

The property $date2 is not used and could be removed.

    public function matchesHandicap($playerId){}
    public function scoringAverage($playerId)

This method seems to be duplicated in your project.

    {
        $today = Carbon::today()->toDateString();
        $rounds = Round::where('player_id', '=', $playerId)
            ->where('date', '<', $today)
            ->get();
        $scores = $rounds->map(function($round)
        {
            return $round->score;
        });
        if($rounds->count() > 0) {
            return round(array_sum($scores->toArray())/($rounds->count()), 2);
        } else {
            return $rounds;
        }
    }
    public function scoringAverageByYear($playerId, $year)
    {
        $date1 = $year . '-01-01';
        $date2 = $year . '-12-31';


        $rounds = Round::where('player_id', '=', $playerId)
            ->where('date', '>=', $date1)
            ->where('date', '<=', $date2)
            ->get();
        $scores = $rounds->map(function($round)
        {
            return $round->score;
        });
        if($rounds->count() > 0) {
            return round(array_sum($scores->toArray())/($rounds->count()), 2);
        } else {
            return $rounds;
        }


    }
    public function scoringAverageMatchesByYear($year){}
    public function handicapRounds($playerId){}

    public function scoringAverageCourse($course){}
    public function scoringAverageCourseByYear($course, $year){}
    public function scoringAverageCourseMatchesByYear($course, $year){}

    public function totalEagles(){}
    public function eaglesByYear($year){}
    public function eaglesMatchesByYear($year){}

    public function totalBirdies($playerId){}
    public function birdiesByYear($playerId, $year)
    {
        $year = $year . '-01-01';
        $holescores = Holescore::with('round.player','hole')
            ->where('created_at', '>', $year)
            ->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']['id'];
                }
            }
        }
        return count(array_keys($allBirdies, $playerId));

    }
    public function birdiesMatchesByYear($year){}

    public function totalPars(){}
    public function parsByYear($year){}
    public function parsMatchesByYear($year){}

    public function totalbogeys(){}
    public function bogeysByYear($year){}
    public function bogeysMatchesByYear($year){}

    public function totalDoubles(){}
    public function doublesByYear($year){}
    public function doublesMatchesByYear($year){}

    public function totalOthers(){}
    public function othersByYear($year){}
    public function othersMatchesByYear($year){}

}