GolfLeague\Statistics\Course\CourseStatisticsEloquent

Complexity

Total Complexity

8

Size/Duplication

Total Lines	74
Duplicated Lines	90.54 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	averageScore()	15	15	2
A	averageScoreByPlayer()	17	17	2
A	averageScoreByYear()	17	17	2
A	averageScoreByPlayerAndYear()	18	18	2

<?php namespace GolfLeague\Statistics\Course;

use Carbon\Carbon;
use \Round;

class CourseStatisticsEloquent implements CourseStatistics
{
    private $date1 = '-01-01';
    private $date2 = '-12-31';

    public function averageScore($courseId)

This method seems to be duplicated in your project.

    {
        $today = Carbon::today()->toDateString();
        $rounds = Round::where('course_id', '=', $courseId)
            ->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 averageScoreByPlayer($courseId, $playerId)

This method seems to be duplicated in your project.

    {
        $today = Carbon::today()->toDateString();
        $rounds = Round::where('course_id', '=', $courseId)
            ->where('date', '<', $today)
            ->where('player_id', '=', $playerId)
            ->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 averageScoreByYear($courseId, $year)

This method seems to be duplicated in your project.

    {
        $rounds = Round::where('course_id', '=', $courseId)
            ->where('date', '>=', $year . $this->date1)
            ->where('date', '<=', $year . $this->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 averageScoreByPlayerAndYear($courseId, $playerId, $year)

This method seems to be duplicated in your project.

    {
        $rounds = Round::where('course_id', '=', $courseId)
            ->where('player_id', '=', $playerId)
            ->where('date', '>=', $year . $this->date1)
            ->where('date', '<=', $year . $this->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;
        }
    }

}