private methods that are never executed are dead code: unnecessary, inoperative code that should be removed. Cleaning out dead code
decreases the size of the maintained codebase, making it easier to understand the program and preventing bugs from being introduced.
public class Foo
{
private function Foo() {} // Compliant, private empty constructor intentionally used to prevent any direct instantiation of a class.
public static function doSomething()
{
$foo = new Foo();
...
}
private function unusedPrivateFunction() { // Noncompliant
}
}
public class Foo
{
private function Foo(){} // Compliant, private empty constructor intentionally used to prevent any direct instantiation of a class.
public static function doSomething()
{
$foo = new Foo();
}
}