dead code detector
0.7.0
Расширение Phpstan, чтобы с легкостью найти неиспользованный PHP -код в вашем проекте!
composer require --dev shipmonk/dead-code-detectorИспользуйте официальный расширение-установщик или просто загрузите правила:
# phpstan.neon.dist
includes :
- vendor/shipmonk/dead-code-detector/rules.neon phpstan/phpstan-symfony с containerXmlPath должен использоваться#[AsEventListener] атрибут#[AsController] атрибут#[AsCommand] атрибут#[Required] атрибут#[Route] атрибутыEventSubscriberInterface::getSubscribedEventsonKernelResponse , onKernelRequest и т. Д. #[AsEntityListener] АтрибутDoctrineORMEvents::* СобытияDoctrineCommonEventSubscriber Методы#[PreFlush] , #[PostLoad] , ... testXxx@test , @before , @afterClass и т. Д.#[Test] , #[Before] , #[AfterClass] и т. Д. handleXxx , renderXxx , actionXxx , injectXxx , createComponentXxxSmartObject Magic требует аннотаций @propertyВсе эти библиотеки автономны, когда они найдены в зависимости ваших композиторов. Если вы хотите заставить включить/отключить некоторые из них, вы можете:
# phpstan.neon.dist
parameters :
shipmonkDeadCode :
usageProviders :
phpunit :
enabled : true ReflectionClass обнаруживается при использовании$reflection->getConstructor() , $reflection->getConstant('NAME') , $reflection->getMethods() , ... vendor , не сообщается как мертвыйPsrLogLoggerInterface::log автоматически рассматриваетсяЭти поставщики включены по умолчанию, но вы можете отключить их, если это необходимо.
shipmonk.deadCode.memberUsageProvider и реализовать ShipMonkPHPStanDeadCodeProviderMemberUsageProvider # phpstan.neon.dist
services :
-
class : AppApiOutputUsageProvider
tags :
- shipmonk.deadCode.memberUsageProvider Важный
Интерфейс и тег изменились в 0,7. Если вы используете Phpstan 1.x, они использовались по -разному.
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() ), отмечая все методы, названные method как используемыеnew $unknown() будет отмечать все конструкторы, которые используются, используемые как используемыеphpstan.neon.dist :$unknown::CONSTANT ) parameters :
shipmonkDeadCode :
trackMixedAccess : false-vvv , и вы увидите некоторую диагностику: Found 2 usages over unknown type:
• setCountry method, for example in AppEntityUser::updateAddress
• setStreet method, for example in AppEntityUser::updateAddress
__get , __set и т. Д.) никогда не сообщаются как мертвые__construct , __clone parameters :
ignoreErrors :
- ' # ^Unused .*?::__construct$ # ' MemberUsageProvider : use ShipMonk PHPStan DeadCode Provider ReflectionBasedMemberUsageProvider ;
class IgnoreDeadInterfaceUsageProvider extends ReflectionBasedMemberUsageProvider
{
public function shouldMarkMethodAsUsed ( ReflectionMethod $ method ): bool
{
return $ method -> getDeclaringClass ()-> isInterface ();
}
}composer checkcomposer fix:cs