GroupHomework::cacheStatistics() - Code Metrics - Inspection of "release: 0.17.0 Characinae" - ZsgsDesign/NOJ - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#724)

by John

created 2021-10-08 06:52 UTC

GroupHomework::cacheStatistics() F

↳ Parent: GroupHomework

Complexity

Conditions	10
Paths	933

Size

Total Lines	73
Code Lines	49

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	1

Metric	Value
cc	10
eloc	49
nc	933
nop	0
dl	0
loc	73
rs	3.9871
c	2
b	1
f	1

How to fix Long Method Complexity

<?php

namespace App\Models\Eloquent;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Carbon\Carbon;
use DateTimeInterface;
use Exception;
use Cache;

class GroupHomework extends Model
{
    use HasFactory;

    public function group()
    {
        return $this->belongsTo('App\Models\Eloquent\Group', 'group_id', 'gid');
    }

    public function problems()
    {
        return $this->hasMany('App\Models\Eloquent\GroupHomeworkProblem');
    }

    protected function serializeDate(DateTimeInterface $date)
    {
        return $date->format('Y-m-d H:i:s');
    }

    public function getStatisticsAttribute()
    {
        $cachedStatistics = Cache::tags(['homework', 'statistics'])->get($this->id);

        if (blank($cachedStatistics)) {
            $cachedStatistics = $this->cacheStatistics();
        }

        if ($cachedStatistics === false) {
            return null;
        }

        $cachedStatistics['timestamp'] = Carbon::createFromTimestamp($cachedStatistics['timestamp']);

        return $cachedStatistics;
    }

    public function cacheStatistics()
    {
        try {
            $statistics = [];
            $homeworkProblems = $this->problems->sortBy('order_index');
            $users = $this->group->members()->where('role', '>=', 1)->get();
            $userIDArr = $users->pluck('uid');
            $defaultVerdict = [];

            foreach ($homeworkProblems as $homeworkProblem) {
                $statistics['problems'][] = [
                    'pid' => $homeworkProblem->problem_id,
                    'readable_name' => $homeworkProblem->problem->readable_name,
                ];
                $defaultVerdict[$homeworkProblem->problem_id] = [
                    "icon" => "checkbox-blank-circle-outline",
                    "color" => "wemd-grey-text"
                ];
            }

            foreach ($users as $user) {
                $statistics['data'][$user->uid] = [
                    'name' => $user->user->name,
                    'nick_name' => blank($user->nick_name) ? null : $user->nick_name,
                    'solved' => 0,
                    'attempted' => 0,
                    'verdict' => $defaultVerdict
                ];
            }

            $endedAt = Carbon::parse($this->ended_at);

            foreach ($homeworkProblems as $homeworkProblem) {
                $userProbIDArr = [];

                foreach ($homeworkProblem->problem->users_latest_submission($userIDArr->diff($userProbIDArr), null, $endedAt, ['Accepted'])->get() as $acceptedRecord) {
                    $statistics['data'][$acceptedRecord['uid']]['verdict'][$homeworkProblem->problem_id] = [
                        "icon" => "checkbox-blank-circle",
                        "color" => $acceptedRecord['color']
                    ];
                    $statistics['data'][$acceptedRecord['uid']]['solved']++;
                    $userProbIDArr[] = $acceptedRecord['uid'];
                }

                foreach ($homeworkProblem->problem->users_latest_submission($userIDArr->diff($userProbIDArr), null, $endedAt, ['Partially Accepted'])->get() as $acceptedRecord) {
                    $statistics['data'][$acceptedRecord['uid']]['verdict'][$homeworkProblem->problem_id] = [
                        "icon" => "cisco-webex",
                        "color" => $acceptedRecord['color']
                    ];
                    $statistics['data'][$acceptedRecord['uid']]['attempted']++;
                    $userProbIDArr[] = $acceptedRecord['uid'];
                }

                foreach ($homeworkProblem->problem->users_latest_submission($userIDArr->diff($userProbIDArr), null, $endedAt)->get() as $acceptedRecord) {
                    $statistics['data'][$acceptedRecord['uid']]['verdict'][$homeworkProblem->problem_id] = [
                        "icon" => "cisco-webex",
                        "color" => $acceptedRecord['color']
                    ];
                    $statistics['data'][$acceptedRecord['uid']]['attempted']++;
                    $userProbIDArr[] = $acceptedRecord['uid'];
                }
            }

            usort($statistics['data'], function ($a, $b) {
                return $b["solved"] == $a["solved"] ? $b["attempted"] <=> $a["attempted"] : $b["solved"] <=> $a["solved"];
            });

            $statistics['timestamp'] = time();
            Cache::tags(['homework', 'statistics'])->put($this->id, $statistics, 360);
            return $statistics;
        } catch (Exception $e) {
            Log::alert($e);

            return false;
        }
    }
}


1			<?php
2
3			namespace App\Models\Eloquent;
4
5			use Illuminate\Database\Eloquent\Factories\HasFactory;
6			use Illuminate\Database\Eloquent\Model;
7			use Carbon\Carbon;
8			use DateTimeInterface;
9			use Exception;
10			use Cache;
11
12			class GroupHomework extends Model
13			{
14			use HasFactory;
15
16			public function group()
17			{
18			return $this->belongsTo('App\Models\Eloquent\Group', 'group_id', 'gid');
19			}
20
21			public function problems()
22			{
23			return $this->hasMany('App\Models\Eloquent\GroupHomeworkProblem');
24			}
25
26			protected function serializeDate(DateTimeInterface $date)
27			{
28			return $date->format('Y-m-d H:i:s');
29			}
30
31			public function getStatisticsAttribute()
32			{
33			$cachedStatistics = Cache::tags(['homework', 'statistics'])->get($this->id);
34
35			if (blank($cachedStatistics)) {
36			$cachedStatistics = $this->cacheStatistics();
37			}
38
39			if ($cachedStatistics === false) {
40			return null;
41			}
42
43			$cachedStatistics['timestamp'] = Carbon::createFromTimestamp($cachedStatistics['timestamp']);
44
45			return $cachedStatistics;
46			}
47
48			public function cacheStatistics()
49			{
50			try {
51			$statistics = [];
52			$homeworkProblems = $this->problems->sortBy('order_index');
53			$users = $this->group->members()->where('role', '>=', 1)->get();
54			$userIDArr = $users->pluck('uid');
55			$defaultVerdict = [];
56
57			foreach ($homeworkProblems as $homeworkProblem) {
58			$statistics['problems'][] = [
59			'pid' => $homeworkProblem->problem_id,
60			'readable_name' => $homeworkProblem->problem->readable_name,
61			];
62			$defaultVerdict[$homeworkProblem->problem_id] = [
63			"icon" => "checkbox-blank-circle-outline",
64			"color" => "wemd-grey-text"
65			];
66			}
67
68			foreach ($users as $user) {
69			$statistics['data'][$user->uid] = [
70			'name' => $user->user->name,
71			'nick_name' => blank($user->nick_name) ? null : $user->nick_name,
72			'solved' => 0,
73			'attempted' => 0,
74			'verdict' => $defaultVerdict
75			];
76			}
77
78			$endedAt = Carbon::parse($this->ended_at);
79
80			foreach ($homeworkProblems as $homeworkProblem) {
81			$userProbIDArr = [];
82
83			foreach ($homeworkProblem->problem->users_latest_submission($userIDArr->diff($userProbIDArr), null, $endedAt, ['Accepted'])->get() as $acceptedRecord) {
84			$statistics['data'][$acceptedRecord['uid']]['verdict'][$homeworkProblem->problem_id] = [
85			"icon" => "checkbox-blank-circle",
86			"color" => $acceptedRecord['color']
87			];
88			$statistics['data'][$acceptedRecord['uid']]['solved']++;
89			$userProbIDArr[] = $acceptedRecord['uid'];
90			}
91
92			foreach ($homeworkProblem->problem->users_latest_submission($userIDArr->diff($userProbIDArr), null, $endedAt, ['Partially Accepted'])->get() as $acceptedRecord) {
93			$statistics['data'][$acceptedRecord['uid']]['verdict'][$homeworkProblem->problem_id] = [
94			"icon" => "cisco-webex",
95			"color" => $acceptedRecord['color']
96			];
97			$statistics['data'][$acceptedRecord['uid']]['attempted']++;
98			$userProbIDArr[] = $acceptedRecord['uid'];
99			}
100
101			foreach ($homeworkProblem->problem->users_latest_submission($userIDArr->diff($userProbIDArr), null, $endedAt)->get() as $acceptedRecord) {
102			$statistics['data'][$acceptedRecord['uid']]['verdict'][$homeworkProblem->problem_id] = [
103			"icon" => "cisco-webex",
104			"color" => $acceptedRecord['color']
105			];
106			$statistics['data'][$acceptedRecord['uid']]['attempted']++;
107			$userProbIDArr[] = $acceptedRecord['uid'];
108			}
109			}
110
111			usort($statistics['data'], function ($a, $b) {
112			return $b["solved"] == $a["solved"] ? $b["attempted"] <=> $a["attempted"] : $b["solved"] <=> $a["solved"];
113			});
114
115			$statistics['timestamp'] = time();
116			Cache::tags(['homework', 'statistics'])->put($this->id, $statistics, 360);
117			return $statistics;
118			} catch (Exception $e) {
119			Log::alert($e);
			0 ignored issues – show Bug introduced 2021-10-08 07:02 UTC by Report Bug Copy Issue Report The type `App\Models\Eloquent\Log` was not found. Did you mean `Log`? If so, make sure to prefix the type with `\`. Loading history...
120			return false;
121			}
122			}
123			}
124

ZsgsDesign / NOJ

Pull Request — master (#724)

GroupHomework::cacheStatistics() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like