Analyzer::__construct() - Code Metrics - Inspection of "Add working example for class.jetpack.php" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — try/code-signature-diff ( a4f8e0...fdc01f )

unknown

created 2019-06-21 22:38 UTC

Analyzer::__construct() A

↳ Parent: Analyzer

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	5
rs	10
c	0
b	0
f	0

<?php

namespace Automattic\Jetpack\Analyzer;

use PhpParser\Error;
use PhpParser\NodeDumper;
use PhpParser\ParserFactory;
use PhpParser\Node;
use PhpParser\Node\Stmt\Function_;
use PhpParser\Node\Stmt\ClassMethod_;
use PhpParser\NodeTraverser;
use PhpParser\NodeVisitorAbstract;

// const STATE_NONE = 0;
// const STATE_CLASS_DECLARATION = 1;

const VIS_PUBLIC  = 0;
const VIS_PRIVATE = 1;

class Analyzer extends NodeVisitorAbstract {
	private $declarations;
	private $base_path;
	private $current_path;
	private $parser;

	function __construct( $base_path ) {
		$this->parser       = ( new ParserFactory() )->create( ParserFactory::PREFER_PHP7 );
		$this->declarations = array();
		$this->base_path    = $this->slashit( $base_path );
	}

	private function slashit( $path ) {
		$path .= ( substr( $path, -1 ) == '/' ? '' : '/' );
		return $path;
	}

	protected function add_declaration( $declaration ) {
		$this->declarations[] = $declaration;
	}

	private function print_declarations() {
		print_r( $this->declarations );
	}

	public function file( $file_path ) {
		$this->current_path = $file_path;
		$source             = file_get_contents( $file_path );
		try {
			$ast = $this->parser->parse( $source );
		} catch ( Error $error ) {

			echo "Parse error: {$error->getMessage()}\n";
			return;
		}

		// $dumper = new NodeDumper;
		// echo $dumper->dump($ast) . "\n";

		$traverser = new NodeTraverser();
		$traverser->addVisitor( $this );
		$ast = $traverser->traverse( $ast );
$myVar = 'Value';
$higher = false;

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

	public function enterNode( Node $node ) {
		// print_r($node);
		if ( $node instanceof Node\Stmt\Class_ ) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
			$this->add_declaration( new Class_Declaration( $this->current_path, $node->getLine(), $node->name->name ) );
		}
		if ( $node instanceof Node\Stmt\Property && $node->isPublic() ) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
			$this->add_declaration( new Class_Property_Declaration( $this->current_path, $node->getLine(), $node->props[0]->name->name ) );
		}
		if ( $node instanceof Node\Stmt\ClassMethod && $node->isPublic() ) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
			$method = new Class_Method_Declaration( $this->current_path, $node->getLine(), $node->name->name, $node->isStatic() );
			foreach ( $node->getParams() as $param ) {
				$method->add_param( $node->var->name, $node->default, $node->type, $node->byRef, $node->variadic );
			}
			$this->add_declaration( $method );
		}
	}
}

class Declaration {
	public $path;
	public $line;
	public $name;

	function __construct( $path, $line, $name ) {
		$this->path = $path;
		$this->line = $line;
		$this->name = $name;
	}
}

class Class_Declaration extends Declaration {
}

/**
 * We only log public class methods, whether they are static, and their parameters
 */
class Class_Method_Declaration extends Declaration {
	public $static;

	function __construct( $path, $line, $name, $static ) {
		$this->static = $static;
		$this->params = array();
class MyClass { }

$x = new MyClass();
$x->foo = true;
		parent::__construct( $path, $line, $name );
	}

	function add_param( $name, $default, $type, $byRef, $variadic ) {
		$this->params[] = (object) compact( 'name', 'default', 'type', 'byRef', 'variadic' );
	}
}

/**
 * We only log public class variables
 */
class Class_Property_Declaration extends Declaration {
}


1			<?php
2
3			namespace Automattic\Jetpack\Analyzer;
4
5			use PhpParser\Error;
6			use PhpParser\NodeDumper;
7			use PhpParser\ParserFactory;
8			use PhpParser\Node;
9			use PhpParser\Node\Stmt\Function_;
10			use PhpParser\Node\Stmt\ClassMethod_;
11			use PhpParser\NodeTraverser;
12			use PhpParser\NodeVisitorAbstract;
13
14			// const STATE_NONE = 0;
15			// const STATE_CLASS_DECLARATION = 1;
16
17			const VIS_PUBLIC = 0;
18			const VIS_PRIVATE = 1;
19
20			class Analyzer extends NodeVisitorAbstract {
21			private $declarations;
22			private $base_path;
23			private $current_path;
24			private $parser;
25
26			function __construct( $base_path ) {
27			$this->parser = ( new ParserFactory() )->create( ParserFactory::PREFER_PHP7 );
28			$this->declarations = array();
29			$this->base_path = $this->slashit( $base_path );
30			}
31
32			private function slashit( $path ) {
33			$path .= ( substr( $path, -1 ) == '/' ? '' : '/' );
34			return $path;
35			}
36
37			protected function add_declaration( $declaration ) {
38			$this->declarations[] = $declaration;
39			}
40
41			private function print_declarations() {
42			print_r( $this->declarations );
43			}
44
45			public function file( $file_path ) {
46			$this->current_path = $file_path;
47			$source = file_get_contents( $file_path );
48			try {
49			$ast = $this->parser->parse( $source );
50			} catch ( Error $error ) {
			0 ignored issues – show Bug introduced 2019-06-22 03:13 UTC by Report Bug Copy Issue Report The class `PhpParser\Error` does not exist. Did you forget a USE statement, or did you not list all dependencies? Scrutinizer analyzes your `composer.json`/`composer.lock` file if available to determine the classes, and functions that are defined by your dependencies. It seems like the listed class was neither found in your dependencies, nor was it found in the analyzed files in your repository. If you are using some other form of dependency management, you might want to disable this analysis. Loading history...
51			echo "Parse error: {$error->getMessage()}\n";
52			return;
53			}
54
55			// $dumper = new NodeDumper;
56			// echo $dumper->dump($ast) . "\n";
57
58			$traverser = new NodeTraverser();
59			$traverser->addVisitor( $this );
60			$ast = $traverser->traverse( $ast );
			0 ignored issues – show Unused Code introduced 2019-06-22 03:13 UTC by Report Bug Copy Issue Report `$ast` 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...
61			$this->print_declarations();
62			return;
63			}
64
65			public function enterNode( Node $node ) {
66			// print_r($node);
67			if ( $node instanceof Node\Stmt\Class_ ) {
			0 ignored issues – show Bug introduced 2019-06-22 03:13 UTC by Report Bug Copy Issue Report The class `PhpParser\Node\Stmt\Class_` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
68			$this->add_declaration( new Class_Declaration( $this->current_path, $node->getLine(), $node->name->name ) );
69			}
70			if ( $node instanceof Node\Stmt\Property && $node->isPublic() ) {
			0 ignored issues – show Bug introduced 2019-06-22 03:13 UTC by Report Bug Copy Issue Report The class `PhpParser\Node\Stmt\Property` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
71			$this->add_declaration( new Class_Property_Declaration( $this->current_path, $node->getLine(), $node->props[0]->name->name ) );
72			}
73			if ( $node instanceof Node\Stmt\ClassMethod && $node->isPublic() ) {
			0 ignored issues – show Bug introduced 2019-06-22 03:13 UTC by Report Bug Copy Issue Report The class `PhpParser\Node\Stmt\ClassMethod` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
74			$method = new Class_Method_Declaration( $this->current_path, $node->getLine(), $node->name->name, $node->isStatic() );
75			foreach ( $node->getParams() as $param ) {
76			$method->add_param( $node->var->name, $node->default, $node->type, $node->byRef, $node->variadic );
77			}
78			$this->add_declaration( $method );
79			}
80			}
81			}
82
83			class Declaration {
84			public $path;
85			public $line;
86			public $name;
87
88			function __construct( $path, $line, $name ) {
89			$this->path = $path;
90			$this->line = $line;
91			$this->name = $name;
92			}
93			}
94
95			class Class_Declaration extends Declaration {
96			}
97
98			/**
99			* We only log public class methods, whether they are static, and their parameters
100			*/
101			class Class_Method_Declaration extends Declaration {
102			public $static;
103
104			function __construct( $path, $line, $name, $static ) {
105			$this->static = $static;
106			$this->params = array();
			0 ignored issues – show Bug introduced 2019-06-22 03:13 UTC by Report Bug Copy Issue Report The property `params` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
107			parent::__construct( $path, $line, $name );
108			}
109
110			function add_param( $name, $default, $type, $byRef, $variadic ) {
111			$this->params[] = (object) compact( 'name', 'default', 'type', 'byRef', 'variadic' );
112			}
113			}
114
115			/**
116			* We only log public class variables
117			*/
118			class Class_Property_Declaration extends Declaration {
119			}
120

Automattic / jetpack

Push — try/code-signature-diff ( a4f8e0...fdc01f )

Analyzer::__construct() A

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like