Как использовать функцию UnusedMethod Псалма?
Я пытаюсь использовать инструмент статического анализа псалма для PHP. Насколько я понимаю, этот инструмент может рассказать мне о неиспользуемых методах в моей кодовой базе. Тем не менее, если я создаю простой тестовый файл
#File: src/test.php
<?php
class A {
private function foo() : void {}
}
new A();
а потом беги psalm
$ ./vendor/bin/psalm --find-dead-code src/test.php
Scanning files...
Analyzing files...
------------------------------
No errors found!
------------------------------
Checks took 0.16 seconds and used 32.694MB of memory
Psalm was able to infer types for 100% of the codebase
или же psalter
,
$ ./vendor/bin/psalter --find-unused-code --dry-run --issues=UnusedMethod src/test.php
Scanning files...
Analyzing files...
------------------------------
No errors found!
------------------------------
Checks took 0.05 seconds and used 29.214MB of memory
Psalm was able to infer types for 100% of the codebase
ошибок не найдено.
Почему псалом не находит неиспользуемый метод foo
? Нужна ли дополнительная конфигурация? Или я неправильно понимаю, что делает этот инструмент? мой psalm.xml
файл ниже.
<?xml version="1.0"?>
<psalm
totallyTyped="false"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns="https://getpsalm.org/schema/config"
xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
>
<projectFiles>
<directory name="src" />
<ignoreFiles>
<directory name="vendor" />
</ignoreFiles>
</projectFiles>
<issueHandlers>
<LessSpecificReturnType errorLevel="info" />
<!-- level 3 issues - slightly lazy code writing, but provably low false-negatives -->
<DeprecatedMethod errorLevel="info" />
<DeprecatedProperty errorLevel="info" />
<DeprecatedClass errorLevel="info" />
<DeprecatedConstant errorLevel="info" />
<DeprecatedInterface errorLevel="info" />
<DeprecatedTrait errorLevel="info" />
<InternalMethod errorLevel="info" />
<InternalProperty errorLevel="info" />
<InternalClass errorLevel="info" />
<MissingClosureReturnType errorLevel="info" />
<MissingReturnType errorLevel="info" />
<MissingPropertyType errorLevel="info" />
<InvalidDocblock errorLevel="info" />
<MisplacedRequiredParam errorLevel="info" />
<PropertyNotSetInConstructor errorLevel="info" />
<MissingConstructor errorLevel="info" />
<MissingClosureParamType errorLevel="info" />
<MissingParamType errorLevel="info" />
<RedundantCondition errorLevel="info" />
<DocblockTypeContradiction errorLevel="info" />
<RedundantConditionGivenDocblockType errorLevel="info" />
<UnresolvableInclude errorLevel="info" />
<RawObjectIteration errorLevel="info" />
<InvalidStringClass errorLevel="info" />
<UnusedMethod errorLevel="info" />
</issueHandlers>
</psalm>
2 ответа
Создатель псалма здесь - обнаружение мертвого кода обнаруживает неиспользуемые классы и методы только при анализе всего проекта - например, ./vendor/bin/psalm --find-dead-code
, опуская src/test.php
,
Хотя частные методы и свойства являются особым случаем (их неиспользование можно вывести без проверки всего проекта), для открытых / защищенных методов и свойств все должно быть использовано.
Согласно документации, вы захотите использовать --find-dead-code
аргумент psalm
:
./vendor/bin/psalm --find-dead-code foo.php
Выход:
Scanning files...
Analyzing files...
ERROR: UnusedVariable - foo.php:6:1 - Variable $a is never referenced
$a = new A();
------------------------------
1 errors found
------------------------------
Checks took 0.27 seconds and used 67.096MB of memory
Psalm was able to infer types for 100% of the codebase