JException::__construct() - Code Metrics - Inspection of "Merge remote-tracking branch 'refs/remotes/origin/..." - XaBerr/JATE - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Branch — master (8beb08)

by Federico

created 2018-02-15 12:44 UTC

JException::__construct() B

↳ Parent: JException

Complexity

Conditions	9
Paths	16

Size

Total Lines	11
Code Lines	10

Duplication

Lines	8
Ratio	72.73 %

Importance

Changes

Metric	Value
cc	9
eloc	10
nc	16
nop	4
dl	8
loc	11
rs	7.756
c	0
b	0
f	0

<?php
  class JException extends Exception {
namespace YourVendor;

class YourClass { }
    static protected $escapeString = "|jate|";
    public function __construct($_message, $_level = 2, $_code = 0, Exception $_previous = null) {
      parent::__construct($_message, $_code, $_previous);
      if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["file"]))

        $this->file  = debug_backtrace()[$_level]["file"];
      if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["line"]))

        $this->line  = debug_backtrace()[$_level]["line"];
      if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["function"]))

        $this->function = debug_backtrace()[$_level]["function"];
class MyClass { }

$x = new MyClass();
$x->foo = true;
      if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["class"]))

        $this->class = debug_backtrace()[$_level]["class"];
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }
    public function __toString() {
      return JException::encode(JException::decode($this->message));
    }
    static public function decode( $_message ) {

      $messageSplitted = explode(JException::$escapeString, $_message);
      if(count($messageSplitted) == 3)
        return $messageSplitted[1];
      else
        return $_message;
    }
    static public function encode( $_message ) {

      $escape = JException::$escapeString;
      return "$escape$_message$escape";

    }
  }
?>



1		<?php
2		class JException extends Exception {
		0 ignored issues – show Coding Style Compatibility introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
3		static protected $escapeString = "\|jate\|";
4		public function __construct($_message, $_level = 2, $_code = 0, Exception $_previous = null) {
5		parent::__construct($_message, $_code, $_previous);
6	View Code Duplication	if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["file"]))
		0 ignored issues – show Duplication introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
7		$this->file = debug_backtrace()[$_level]["file"];
8	View Code Duplication	if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["line"]))
		0 ignored issues – show Duplication introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
9		$this->line = debug_backtrace()[$_level]["line"];
10	View Code Duplication	if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["function"]))
		0 ignored issues – show Duplication introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
11		$this->function = debug_backtrace()[$_level]["function"];
		0 ignored issues – show Bug introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report The property `function` 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...
12	View Code Duplication	if(isset(debug_backtrace()[$_level]) && isset(debug_backtrace()[$_level]["class"]))
		0 ignored issues – show Duplication introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
13		$this->class = debug_backtrace()[$_level]["class"];
		0 ignored issues – show Bug introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report The property `class` 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...
14		}
15		public function __toString() {
16		return JException::encode(JException::decode($this->message));
17		}
18		static public function decode( $_message ) {
		0 ignored issues – show Coding Style introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report As per PSR2, the `static` declaration should come after the visibility declaration. Loading history...
19		$messageSplitted = explode(JException::$escapeString, $_message);
20		if(count($messageSplitted) == 3)
21		return $messageSplitted[1];
22		else
23		return $_message;
24		}
25		static public function encode( $_message ) {
		0 ignored issues – show Coding Style introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report As per PSR2, the `static` declaration should come after the visibility declaration. Loading history...
26		$escape = JException::$escapeString;
27		return "$escape$_message$escape";
28
29		}
30		}
31		?>
		0 ignored issues – show Best Practice introduced 2018-02-15 12:48 UTC by Report Bug Copy Issue Report It is not recommended to use PHP's closing tag `?>` in files other than templates. Using a closing tag in PHP files that only contain PHP code is not recommended as you might accidentally add whitespace after the closing tag which would then be output by PHP. This can cause severe problems, for example headers cannot be sent anymore. A simple precaution is to leave off the closing tag as it is not required, and it also has no negative effects whatsoever. Loading history...
32

XaBerr / JATE

Branch — master (8beb08)

JException::__construct() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like