DataTransformer - Code Metrics - Inspection of "update docs" - php-ai/php-ml - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 95bfc8...365a9b )

by Arkadiusz

created 2016-05-07 21:53 UTC

DataTransformer A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Metric	Value
wmc	15
lcom	1
cbo	0
dl	0
loc	95
rs	10

5 Methods

Rating	Name	Size	Complexity
A	trainingSet()	13	4
A	testSet()	13	3
A	predictions()	12	3
A	numericLabels()	13	3
A	sampleRow()	9	2

<?php

declare (strict_types = 1);

namespace Phpml\SupportVectorMachine;

class DataTransformer
{
    /**
     * @param array $samples
     * @param array $labels
     * @param bool  $targets
     *
     * @return string
     */
    public static function trainingSet(array $samples, array $labels, bool $targets = false): string
    {
        $set = '';
        if (!$targets) {
            $numericLabels = self::numericLabels($labels);
        }

        foreach ($labels as $index => $label) {
            $set .= sprintf('%s %s %s', ($targets ? $label : $numericLabels[$label]), self::sampleRow($samples[$index]), PHP_EOL);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
        }

        return $set;
    }

    /**
     * @param array $samples
     *
     * @return string
     */
    public static function testSet(array $samples): string
    {
        if (!is_array($samples[0])) {
            $samples = [$samples];
        }

        $set = '';
        foreach ($samples as $sample) {
            $set .= sprintf('0 %s %s', self::sampleRow($sample), PHP_EOL);
        }

        return $set;
    }

    /**
     * @param string $rawPredictions
     * @param array  $labels
     *
     * @return array
     */
    public static function predictions(string $rawPredictions, array $labels): array
    {
        $numericLabels = self::numericLabels($labels);
        $results = [];
        foreach (explode(PHP_EOL, $rawPredictions) as $result) {
            if (strlen($result) > 0) {
                $results[] = array_search($result, $numericLabels);
            }
        }

        return $results;
    }

    /**
     * @param array $labels
     *
     * @return array
     */
    public static function numericLabels(array $labels): array
    {
        $numericLabels = [];
        foreach ($labels as $label) {
            if (isset($numericLabels[$label])) {
                continue;
            }

            $numericLabels[$label] = count($numericLabels);
        }

        return $numericLabels;
    }

    /**
     * @param array $sample
     *
     * @return string
     */
    private static function sampleRow(array $sample): string
    {
        $row = [];
        foreach ($sample as $index => $feature) {
            $row[] = sprintf('%s:%s', $index + 1, $feature);
        }

        return implode(' ', $row);
    }
}


1			<?php
2
3			declare (strict_types = 1);
4
5			namespace Phpml\SupportVectorMachine;
6
7			class DataTransformer
8			{
9			/**
10			* @param array $samples
11			* @param array $labels
12			* @param bool $targets
13			*
14			* @return string
15			*/
16			public static function trainingSet(array $samples, array $labels, bool $targets = false): string
17			{
18			$set = '';
19			if (!$targets) {
20			$numericLabels = self::numericLabels($labels);
21			}
22
23			foreach ($labels as $index => $label) {
24			$set .= sprintf('%s %s %s', ($targets ? $label : $numericLabels[$label]), self::sampleRow($samples[$index]), PHP_EOL);
			0 ignored issues – show Bug introduced 2016-05-07 21:56 UTC by Report Bug Copy Issue Report The variable `$numericLabels` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
25			}
26
27			return $set;
28			}
29
30			/**
31			* @param array $samples
32			*
33			* @return string
34			*/
35			public static function testSet(array $samples): string
36			{
37			if (!is_array($samples[0])) {
38			$samples = [$samples];
39			}
40
41			$set = '';
42			foreach ($samples as $sample) {
43			$set .= sprintf('0 %s %s', self::sampleRow($sample), PHP_EOL);
44			}
45
46			return $set;
47			}
48
49			/**
50			* @param string $rawPredictions
51			* @param array $labels
52			*
53			* @return array
54			*/
55			public static function predictions(string $rawPredictions, array $labels): array
56			{
57			$numericLabels = self::numericLabels($labels);
58			$results = [];
59			foreach (explode(PHP_EOL, $rawPredictions) as $result) {
60			if (strlen($result) > 0) {
61			$results[] = array_search($result, $numericLabels);
62			}
63			}
64
65			return $results;
66			}
67
68			/**
69			* @param array $labels
70			*
71			* @return array
72			*/
73			public static function numericLabels(array $labels): array
74			{
75			$numericLabels = [];
76			foreach ($labels as $label) {
77			if (isset($numericLabels[$label])) {
78			continue;
79			}
80
81			$numericLabels[$label] = count($numericLabels);
82			}
83
84			return $numericLabels;
85			}
86
87			/**
88			* @param array $sample
89			*
90			* @return string
91			*/
92			private static function sampleRow(array $sample): string
93			{
94			$row = [];
95			foreach ($sample as $index => $feature) {
96			$row[] = sprintf('%s:%s', $index + 1, $feature);
97			}
98
99			return implode(' ', $row);
100			}
101			}
102

php-ai / php-ml

Push — develop ( 95bfc8...365a9b )

DataTransformer A

Complexity

Size/Duplication

Coupling/Cohesion

5 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like