HeatMap::getLevel() - Code Metrics - Inspection of "Objectified, cacheable Snapshots" - smtech/grading-analytics - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 505b4c...62d9c0 )

by Seth

created 2017-06-07 18:00 UTC

HeatMap::getLevel() D

↳ Parent: HeatMap

Complexity

Conditions	14
Paths	100

Size

Total Lines	113
Code Lines	79

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	14
eloc	79
nc	100
nop	2
dl	0
loc	113
rs	4.9516
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

namespace smtech\GradingAnalytics\HeatMap;

class HeatMap
{
    private static $levels = [];

    public static function getLevel($params, $smarty)

    {
        $key = false;
        $value = false;
        foreach ($params as $k => $v) {
            switch ($k) {
                case 'key':
                    $key = $v;
                    break;
                case 'value':
                    $value = $v;
                    break;
            }
        }
        if ($key === false || $value == false) {
            return "";
        }

        if (empty(self::$levels)) {
            self::$levels = [
                'average_grading_turn_around' => [
                    'warning' => [
                        new Level(3, 14, Level::$GREATER_THAN),
                        new Level(2, 7, Level::$GREATER_THAN)
                    ],
                    'highlight' => [
                        new Level (3, 3, Level::$LESS_THAN),
                        new Level (2, 7, Level::$LESS_THAN)
                    ]
                ],
                'average_assignment_lead_time' => [
                    'warning' => [
                        new Level(3, 1, Level::$LESS_THAN),
                        new Level(2, 2, Level::$LESS_THAN)
                    ],
                    'highlight' => [
                        new Level(3, 10, Level::$GREATER_THAN),
                        new Level(2, 7, Level::$GREATER_THAN)
                    ]
                ],
                'average_submissions_graded' => [
                    'warning' => [
                        new Level(3, 0.5, Level::$LESS_THAN),
                        new Level(2, 0.75, Level::$LESS_THAN)
                    ],
                    'highlight' => [
                        new Level(3, 1.0, Level::$GREATER_THAN_OR_EQUAL),
                        new Level(2, 0.9, Level::$GREATER_THAN)
                    ]
                ],
                'dateless_assignment_count' => [
                    'warning' => [
                        new Level(3, 20, Level::$GREATER_THAN),
                        new Level(2, 10, Level::$GREATER_THAN)
                    ],
                    'highlight' => [
                        new Level(3, 1, Level::$LESS_THAN),
                        new Level(2, 5, Level::$LESS_THAN)
                    ]
                ],
                'gradeable_assignment_count' => [
                    'warning' => [
                        new Level(3, 0, Level::$LESS_THAN_OR_EQUAL)
                    ]
                ],
                'graded_assignment_count' => [
                    'warning' => [
                        new Level(3, 0, Level::$LESS_THAN_OR_EQUAL)
                    ]
                ],
                'created_after_due_count' => [
                    'warning' => [
                        new Level(3, 10, Level::$GREATER_THAN),
                        new Level(2, 5, Level::$GREATER_THAN)
                    ],
                    'highlight' => [
                        new Level(3, 1, Level::$LESS_THAN),
                        new Level(2, 5, Level::$LESS_THAN)
                    ]
                ],
                'zero_point_assignment_count' => [
                    'warning' => [
                        new Level(3, 10, Level::$GREATER_THAN_OR_EQUAL),
                        new Level(2, 0, Level::$GREATER_THAN)
                    ]
                ]
            ];
        }

        foreach (self::$levels[$key] as $mode => $modeLevels) {
            foreach ($modeLevels as $level) {
                $match = false;
                switch ($level->comparison) {
                    case Level::$GREATER_THAN:
                        $match = $value > $level->value;
                        break;
                    case Level::$GREATER_THAN_OR_EQUAL:
                        $match = $value >= $level->value;
                        break;
                    case Level::$LESS_THAN:
                        $match = $value < $level->value;
                        break;
                    case Level::$LESS_THAN_OR_EQUAL:
                        $match = $value <= $level->value;
                        break;
                }
                if ($match) {
                    return " class=\"$mode level-{$level->level}\"";
                }
            }
        }
        return "";
    }
}


1			<?php
2
3			namespace smtech\GradingAnalytics\HeatMap;
4
5			class HeatMap
6			{
7			private static $levels = [];
8
9			public static function getLevel($params, $smarty)
			1 ignored issue – show Unused Code introduced 2017-06-05 19:18 UTC by Report Bug Copy Issue Report The parameter `$smarty` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
10			{
11			$key = false;
12			$value = false;
13			foreach ($params as $k => $v) {
14			switch ($k) {
15			case 'key':
16			$key = $v;
17			break;
18			case 'value':
19			$value = $v;
20			break;
21			}
22			}
23			if ($key === false \|\| $value == false) {
24			return "";
25			}
26
27			if (empty(self::$levels)) {
28			self::$levels = [
29			'average_grading_turn_around' => [
30			'warning' => [
31			new Level(3, 14, Level::$GREATER_THAN),
32			new Level(2, 7, Level::$GREATER_THAN)
33			],
34			'highlight' => [
35			new Level (3, 3, Level::$LESS_THAN),
36			new Level (2, 7, Level::$LESS_THAN)
37			]
38			],
39			'average_assignment_lead_time' => [
40			'warning' => [
41			new Level(3, 1, Level::$LESS_THAN),
42			new Level(2, 2, Level::$LESS_THAN)
43			],
44			'highlight' => [
45			new Level(3, 10, Level::$GREATER_THAN),
46			new Level(2, 7, Level::$GREATER_THAN)
47			]
48			],
49			'average_submissions_graded' => [
50			'warning' => [
51			new Level(3, 0.5, Level::$LESS_THAN),
52			new Level(2, 0.75, Level::$LESS_THAN)
53			],
54			'highlight' => [
55			new Level(3, 1.0, Level::$GREATER_THAN_OR_EQUAL),
56			new Level(2, 0.9, Level::$GREATER_THAN)
57			]
58			],
59			'dateless_assignment_count' => [
60			'warning' => [
61			new Level(3, 20, Level::$GREATER_THAN),
62			new Level(2, 10, Level::$GREATER_THAN)
63			],
64			'highlight' => [
65			new Level(3, 1, Level::$LESS_THAN),
66			new Level(2, 5, Level::$LESS_THAN)
67			]
68			],
69			'gradeable_assignment_count' => [
70			'warning' => [
71			new Level(3, 0, Level::$LESS_THAN_OR_EQUAL)
72			]
73			],
74			'graded_assignment_count' => [
75			'warning' => [
76			new Level(3, 0, Level::$LESS_THAN_OR_EQUAL)
77			]
78			],
79			'created_after_due_count' => [
80			'warning' => [
81			new Level(3, 10, Level::$GREATER_THAN),
82			new Level(2, 5, Level::$GREATER_THAN)
83			],
84			'highlight' => [
85			new Level(3, 1, Level::$LESS_THAN),
86			new Level(2, 5, Level::$LESS_THAN)
87			]
88			],
89			'zero_point_assignment_count' => [
90			'warning' => [
91			new Level(3, 10, Level::$GREATER_THAN_OR_EQUAL),
92			new Level(2, 0, Level::$GREATER_THAN)
93			]
94			]
95			];
96			}
97
98			foreach (self::$levels[$key] as $mode => $modeLevels) {
99			foreach ($modeLevels as $level) {
100			$match = false;
101			switch ($level->comparison) {
102			case Level::$GREATER_THAN:
103			$match = $value > $level->value;
104			break;
105			case Level::$GREATER_THAN_OR_EQUAL:
106			$match = $value >= $level->value;
107			break;
108			case Level::$LESS_THAN:
109			$match = $value < $level->value;
110			break;
111			case Level::$LESS_THAN_OR_EQUAL:
112			$match = $value <= $level->value;
113			break;
114			}
115			if ($match) {
116			return " class=\"$mode level-{$level->level}\"";
117			}
118			}
119			}
120			return "";
121			}
122			}
123

smtech / grading-analytics

Push — master ( 505b4c...62d9c0 )

HeatMap::getLevel() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like