HttpCalls - Code Metrics - alexbonavila/LaravelMarketInformation - Measure and Improve Code Quality continuously with Scrutinizer

HttpCalls A
last analyzed 2016-06-15 07:08 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
wmc	7
c	2
b	0
f	0
lcom	1
cbo	1
dl	0
loc	79
rs	10

5 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	getTableFollow()	8	1
A	getExchangeHistory()	8	1
A	getCompanies()	10	1
A	discardRepeated()	13	3

<?php
/**
 * Created by PhpStorm.
 * User: alumne
 * Date: 31/05/16
 * Time: 14:04
 */

namespace App\Console\AuxiliaryClasses;

use App\InteractionMethodsMOD\GetMethods;

/**
 * Class HttpCalls
 * @package App\Console\AuxiliaryClasses
 */
class HttpCalls
{
    /**
     * @var GetMethods
     */
    protected $get_methods;

    /**
     * HttpCalls constructor.
     * @param GetMethods $get_methods
     */
    public function __construct(GetMethods $get_methods)
    {
        $this->get_methods=$get_methods;
    }

    /**
     * @param $symbol
     * @return mixed|\Psr\Http\Message\ResponseInterface
     */
    public function getTableFollow($symbol)
    {
        $data=$this->get_methods->stockQuote($symbol);

        sleep(4);

        return $data;
    }

    /**
     * @param $symbol
     * @return mixed|\Psr\Http\Message\ResponseInterface
     */
    public function getExchangeHistory($symbol)
    {
        $data=$this->get_methods->interactiveChart($symbol);

        sleep(4);

        return $data;
    }

    /**
     * @param $symbol
     * @return mixed|null|\Psr\Http\Message\ResponseInterface
     */
    public function getCompanies($symbol)
    {
        $data=$this->get_methods->companyLookup($symbol);

        $data=$this->discardRepeated($data,$symbol);

        sleep(4);

        return $data;
    }

    /**
     * @param $data
     * @param $symbol
     * @return $node

     * @return null
     */
    private function discardRepeated($data, $symbol)
    {
        for($i=0;$i<count($data);$i++){
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}

            $symbol_to_compare=(string) $data[$i]->Symbol;

            if($symbol_to_compare==$symbol){
                return $node=$data[$i];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            }

        }
        return null;
    }


}

1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: alumne
5			* Date: 31/05/16
6			* Time: 14:04
7			*/
8
9			namespace App\Console\AuxiliaryClasses;
10
11			use App\InteractionMethodsMOD\GetMethods;
12
13			/**
14			* Class HttpCalls
15			* @package App\Console\AuxiliaryClasses
16			*/
17			class HttpCalls
18			{
19			/**
20			* @var GetMethods
21			*/
22			protected $get_methods;
23
24			/**
25			* HttpCalls constructor.
26			* @param GetMethods $get_methods
27			*/
28			public function __construct(GetMethods $get_methods)
29			{
30			$this->get_methods=$get_methods;
31			}
32
33			/**
34			* @param $symbol
35			* @return mixed\|\Psr\Http\Message\ResponseInterface
36			*/
37			public function getTableFollow($symbol)
38			{
39			$data=$this->get_methods->stockQuote($symbol);
40
41			sleep(4);
42
43			return $data;
44			}
45
46			/**
47			* @param $symbol
48			* @return mixed\|\Psr\Http\Message\ResponseInterface
49			*/
50			public function getExchangeHistory($symbol)
51			{
52			$data=$this->get_methods->interactiveChart($symbol);
53
54			sleep(4);
55
56			return $data;
57			}
58
59			/**
60			* @param $symbol
61			* @return mixed\|null\|\Psr\Http\Message\ResponseInterface
62			*/
63			public function getCompanies($symbol)
64			{
65			$data=$this->get_methods->companyLookup($symbol);
66
67			$data=$this->discardRepeated($data,$symbol);
68
69			sleep(4);
70
71			return $data;
72			}
73
74			/**
75			* @param $data
76			* @param $symbol
77			* @return $node
			0 ignored issues – show Documentation introduced 2016-05-31 15:05 UTC by Report Bug Copy Issue Report The doc-type `$node` could not be parsed: Unknown type name "$node" at position 0. (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
78			* @return null
79			*/
80			private function discardRepeated($data, $symbol)
81			{
82			for($i=0;$i<count($data);$i++){
			0 ignored issues – show Performance Best Practice introduced 2016-05-31 15:05 UTC by Report Bug Copy Issue Report It seems like you are calling the size function `count()` as part of the test condition. You might want to compute the size beforehand, and not on each iteration. If the size of the collection does not change during the iteration, it is generally a good practice to compute it beforehand, and not on each iteration: for ($i=0; $i<count($array); $i++) { // calls count() on each iteration } // Better for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once } Loading history...
83
84			$symbol_to_compare=(string) $data[$i]->Symbol;
85
86			if($symbol_to_compare==$symbol){
87			return $node=$data[$i];
			0 ignored issues – show Unused Code introduced 2016-05-31 15:05 UTC by Report Bug Copy Issue Report `$node` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
88			}
89
90			}
91			return null;
92			}
93
94
95			}

alexbonavila / LaravelMarketInformation

HttpCalls A last analyzed 2016-06-15 07:08 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

5 Methods

Duplication Side-by-Side

Filter issues like

HttpCalls A
last analyzed 2016-06-15 07:08 UTC