DiceHand::getString() - Code Metrics - Inspection of "Scrutinizer run exclude controller tests" - Veddan12/mvc - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( da5731...6a4665 )

by Vedrana

created 2025-06-22 21:14 UTC

DiceHand::getString() A

↳ Parent: DiceHand

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	4
c	1
b	0
f	0
nc	2
nop	0
dl	0
loc	7
ccs	5
cts	5
cp	1
crap	2
rs	10

<?php

namespace App\Dice;

use App\Dice\Dice;


/**
 * Class DiceHand
 *
 * Represents a hand of Dice objects.
 */
class DiceHand
{
    /** @var Dice[] Array holding Dice objects in the hand */
    private $hand = [];

    /**
     * Add a Dice object to the hand.
     *
     * @param Dice $die
     */
    public function add(Dice $die): void
    {
        $this->hand[] = $die;
    }

    /**
     * Roll all dice in the hand.
     */
    public function roll(): void
    {
        foreach ($this->hand as $die) {
            $die->roll();
        }
    }

    /**
     * Get the number of dice in the hand.
     *
     * @return int The count of dice.
     */
    public function getNumberDices(): int
    {
        return count($this->hand);
    }

    /**
     * Get numeric values from all dice in the hand.
     *
    * @return int[]
    */
    public function getValues(): array
    {
        $values = [];
        foreach ($this->hand as $die) {
            $values[] = $die->getValue();
        }
        return $values;
    }

    /**
     * Get visual string representations from all dice in the hand.
     *
    * @return string[]
    */
    public function getString(): array
    {
        $values = [];
        foreach ($this->hand as $die) {
            $values[] = $die->getAsString();
        }
        return $values;
    }
}


1		<?php
2
3		namespace App\Dice;
4
5		use App\Dice\Dice;
6
7
8		/**
9		* Class DiceHand
10		*
11		* Represents a hand of Dice objects.
12		*/
13		class DiceHand
14		{
15		/** @var Dice[] Array holding Dice objects in the hand */
16		private $hand = [];
17
18		/**
19		* Add a Dice object to the hand.
20		*
21		* @param Dice $die
22		*/
23	5	public function add(Dice $die): void
24		{
25	5	$this->hand[] = $die;
26		}
27
28		/**
29		* Roll all dice in the hand.
30		*/
31	4	public function roll(): void
32		{
33	4	foreach ($this->hand as $die) {
34	4	$die->roll();
35		}
36		}
37
38		/**
39		* Get the number of dice in the hand.
40		*
41		* @return int The count of dice.
42		*/
43	2	public function getNumberDices(): int
44		{
45	2	return count($this->hand);
46		}
47
48		/**
49		* Get numeric values from all dice in the hand.
50		*
51		* @return int[]
52		*/
53	1	public function getValues(): array
54		{
55	1	$values = [];
56	1	foreach ($this->hand as $die) {
57	1	$values[] = $die->getValue();
58		}
59	1	return $values;
60		}
61
62		/**
63		* Get visual string representations from all dice in the hand.
64		*
65		* @return string[]
66		*/
67	3	public function getString(): array
68		{
69	3	$values = [];
70	3	foreach ($this->hand as $die) {
71	3	$values[] = $die->getAsString();
72		}
73	3	return $values;
74		}
75		}
76

Veddan12 / mvc

Push — main ( da5731...6a4665 )

DiceHand::getString() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like