Why is this an issue?
Private functions 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.
Noncompliant code example
public class Foo
{
private function Foo(){} //Compliant, private empty constructor intentionally used to prevent any direct instantiation of a class.
public static function doSomething():void
{
var foo:Foo = new Foo();
...
}
private function unusedPrivateFunction():void {...}
}
Compliant solution
public class Foo
{
private function Foo(){} //Compliant, private empty constructor intentionally used to prevent any direct instantiation of a class.
public static function doSomething():void
{
var foo:Foo = new Foo();
...
}
}
