dead code detector
0.7.0
Extensão do Phpstan para encontrar código PHP não utilizado em seu projeto com facilidade!
composer require --dev shipmonk/dead-code-detectorUse o Instalador de Extensão Oficial ou apenas carregue as regras:
# phpstan.neon.dist
includes :
- vendor/shipmonk/dead-code-detector/rules.neon phpstan/phpstan-symfony com containerXmlPath deve ser usado#[AsEventListener] Atributo#[AsController] atributo#[AsCommand] atributo#[Required] atributo#[Route] atributosEventSubscriberInterface::getSubscribedEventsonKernelResponse , onKernelRequest , etc #[AsEntityListener] AtributoDoctrineORMEvents::* eventosDoctrineCommonEventSubscriber Methods#[PreFlush] , #[PostLoad] , ... testXxx@test , @before , @afterClass etc#[Test] , #[Before] , #[AfterClass] etc. handleXxx , renderXxx , actionXxx , injectXxx , createComponentXxxSmartObject Magic exige @property AnotaçõesTodas essas bibliotecas são autoeenabilitadas quando encontradas nas dependências do compositor. Se você deseja forçar ativar/desativar alguns deles, você pode:
# phpstan.neon.dist
parameters :
shipmonkDeadCode :
usageProviders :
phpunit :
enabled : true ReflectionClass é detectado como usado$reflection->getConstructor() , $reflection->getConstant('NAME') , $reflection->getMethods() , ... vendor não é relatado como mortoPsrLogLoggerInterface::log é considerado automaticamente usadoEsses provedores são ativados por padrão, mas você pode desativá -los, se necessário.
shipmonk.deadCode.memberUsageProvider e implementar ShipMonkPHPStanDeadCodeProviderMemberUsageProvider # phpstan.neon.dist
services :
-
class : AppApiOutputUsageProvider
tags :
- shipmonk.deadCode.memberUsageProvider Importante
A interface e a tag mudou em 0.7. Se você estiver usando o phpstan 1.x, eles foram usados de maneira diferente.
ShipMonkPHPStanDeadCodeProviderReflectionBasedMemberUsageProvider : use ReflectionMethod ;
use ShipMonk PHPStan DeadCode Provider ReflectionBasedMemberUsageProvider ;
class ApiOutputUsageProvider extends ReflectionBasedMemberUsageProvider
{
public function shouldMarkMethodAsUsed ( ReflectionMethod $ method ): bool
{
// all methods from our ApiOutput interface are called automatically (e.g. during serialization)
return $ method -> getDeclaringClass ()-> implementsInterface (ApiOutput::class);
}
}MemberUsageProvider : use ReflectionMethod ;
use ShipMonk PHPStan DeadCode Graph ClassMethodRef ;
use ShipMonk PHPStan DeadCode Graph ClassMethodUsage ;
use ShipMonk PHPStan DeadCode Provider MemberUsageProvider ;
use Symfony Component Serializer SerializerInterface ;
class DeserializationUsageProvider implements MemberUsageProvider
{
/**
* @return list<ClassMemberUsage>
*/
public function getUsages ( Node $ node , Scope $ scope ): array
{
if (! $ node instanceof MethodCall) {
return [];
}
if (
// our deserialization calls constructor
$ scope -> getType ( $ node -> var )-> getObjectClassNames () === [SerializerInterface::class] &&
$ node -> name -> toString () === ' deserialize '
) {
$ secondArgument = $ node -> getArgs ()[ 1 ]-> value ;
$ serializedClass = $ scope -> getType ( $ secondArgument )-> getConstantStrings ()[ 0 ];
// record the method it was called from (needed for proper transitive dead code elimination)
$ originRef = $ this -> getOriginMethodRef ( $ scope );
// record the hidden constructor call
$ constructorRef = new ClassMethodRef ( $ serializedClass -> getValue (), ' __construct ' , false );
return [ new ClassMethodUsage ( $ originRef , $ constructorRef )];
}
return [];
}
private function getOriginMethodRef ( Scope $ scope ): ? ClassMethodRef
{
return new ClassMethodRef (
$ scope -> getClassReflection ()-> getName (),
$ scope -> getFunction ()-> getName (),
false ,
);
}
} ------ ------------------------------------------------------------------------
Line src/App/Facade/UserFacade.php
------ ------------------------------------------------------------------------
26 Unused AppFacadeUserFacade::updateUserAddress
? shipmonk.deadMethod
Thus AppEntityUser::updateAddress is transitively also unused
Thus AppEntityAddress::setPostalCode is transitively also unused
Thus AppEntityAddress::setCountry is transitively also unused
Thus AppEntityAddress::setStreet is transitively also unused
Thus AppEntityAddress::MAX_STREET_CHARS is transitively also unused
------ ------------------------------------------------------------------------
phpstan.neon.dist : parameters :
shipmonkDeadCode :
reportTransitivelyDeadMethodAsSeparateError : true removeDeadCode : vendor/bin/phpstan analyse --error-format removeDeadCodeclass UserFacade
{
- public const TRANSITIVELY_DEAD = 1;
-
- public function deadMethod(): void
- {
- echo self::TRANSITIVELY_DEAD;
- }
}$unknown->method() ) marcando todos os métodos nomeados como method usadonew $unknown() marcará todos os construtores conforme usadophpstan.neon.dist :$unknown::CONSTANT ) parameters :
shipmonkDeadCode :
trackMixedAccess : false-vvv e verá alguns diagnósticos: Found 2 usages over unknown type:
• setCountry method, for example in AppEntityUser::updateAddress
• setStreet method, for example in AppEntityUser::updateAddress
__get , __set etc) nunca são relatados como mortos__construct , __clone parameters :
ignoreErrors :
- ' # ^Unused .*?::__construct$ # ' MemberUsageProvider personalizado: use ShipMonk PHPStan DeadCode Provider ReflectionBasedMemberUsageProvider ;
class IgnoreDeadInterfaceUsageProvider extends ReflectionBasedMemberUsageProvider
{
public function shouldMarkMethodAsUsed ( ReflectionMethod $ method ): bool
{
return $ method -> getDeclaringClass ()-> isInterface ();
}
}composer checkcomposer fix:cs