StringChecker::checkStringArgument() - Code Metrics - Inspection of "Prevent Socket::getType from returning null" - mariano/disque-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#50)

by Dominic

created 2017-06-08 04:19 UTC

StringChecker::checkStringArgument() A

↳ Parent: StringChecker

Complexity

Conditions	3
Paths	2

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	6
ccs	4
cts	4
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	3
nc	2
nop	2
crap	3

<?php
namespace Disque\Command\Argument;

trait StringChecker
{
    use ArrayChecker;

    /**
     * This command, with all its arguments, ready to be sent to Disque
     *
     * @param array $arguments Arguments
     * @param int $numberOfElements Number of elements that must be present in $arguments
     * @throws InvalidCommandArgumentException
     */
    protected function checkStringArgument(array $arguments, $numberOfElements = 1)
    {
        if (!$this->checkFixedArray($arguments, $numberOfElements) || !is_string($arguments[0])) {
            throw new InvalidCommandArgumentException($this, $arguments);
        }
    }

    /**
     * This command, with all its arguments, ready to be sent to Disque
     *
     * @param array $arguments Arguments
     * @throws InvalidCommandArgumentException
     */
    protected function checkStringArguments(array $arguments)
    {
        if (empty($arguments)) {
            throw new InvalidCommandArgumentException($this, $arguments);
        }

        foreach ($arguments as $argument) {
            if (!is_string($argument) || $argument === '') {
                throw new InvalidCommandArgumentException($this, $arguments);
            }
        }
    }
}

1		<?php
2		namespace Disque\Command\Argument;
3
4		trait StringChecker
5		{
6		use ArrayChecker;
7
8		/**
9		* This command, with all its arguments, ready to be sent to Disque
10		*
11		* @param array $arguments Arguments
12		* @param int $numberOfElements Number of elements that must be present in $arguments
13		* @throws InvalidCommandArgumentException
14		*/
15	50	protected function checkStringArgument(array $arguments, $numberOfElements = 1)
16		{
17	50	if (!$this->checkFixedArray($arguments, $numberOfElements) \|\| !is_string($arguments[0])) {
18	36	throw new InvalidCommandArgumentException($this, $arguments);
19		}
20	14	}
21
22		/**
23		* This command, with all its arguments, ready to be sent to Disque
24		*
25		* @param array $arguments Arguments
26		* @throws InvalidCommandArgumentException
27		*/
28	53	protected function checkStringArguments(array $arguments)
29		{
30	53	if (empty($arguments)) {
31	7	throw new InvalidCommandArgumentException($this, $arguments);
32		}
33
34	46	foreach ($arguments as $argument) {
35	46	if (!is_string($argument) \|\| $argument === '') {
36	20	throw new InvalidCommandArgumentException($this, $arguments);
37		}
38	26	}
39		}
40		}

mariano / disque-php

Pull Request — master (#50)

StringChecker::checkStringArgument() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like