PhpClean

PhpClean - Phpstorm/Plugin

Анализ статического кода для Phpstorm и Intellij Idea.

Установка

Откройте IDE Перейти к Settings->Plugins->Marketplace Search для PhpClean . Нажмите кнопку install .

- Инспекции

- действия

Проверки

Назначен

Обнаружает операторов назначения и сравнения в одном утверждении.

 while ( false !== $ current = ldap_next_entry ( $ con , $ current )) {
  // ^^^ Hard to read this statements
  yield $ this -> getSingleEntry ( $ con , $ current );
}

ClassnameCollision

Классы с тем же именем в разных пространствах имен могут быть запутаны. (Отключено по умолчанию)

 namespace App {
  class User {}; // <- Class name collision with CliUser
}
namespace Cli {
  class User {}; // <- Class name collision with AppUser
}

Устаревшего

Вы можете установить некоторые теги PHPDOC в вашем проекте.

GlobalVariableUsage

Эта проверка обнаруживает использование глобальных переменных.

 echo $ _GET [ ' name ' ]; // <-- Global variable usage 

MethodCanBeprivate

Защищенные методы могут быть преобразованы в частные.

 final class User {
  protected function name () {} // <-- Method can be private
}

Methodshouldbefinal

Методы должны быть закрыты (метод или финал класса)

 class User {
 public function name (): string { // <-- Method should be final
   return '' ;
 }
}

Методвизость

Защищенные методы делают наши занятия более открытыми. Напишите только частные или публичные методы.

MISTPARAMETERTYPEDECLARATION

Всегда указывайте тип параметра. Это хорошая практика.

 class User {
 public function withName ( $ name ) {}  // <-- Missing parameter type
}

Пропал Returntype

Всегда указывайте тип результата функции.

 function phrase () { // <-- Missing return type
    return ' hi ' ;
}

ParentPropertyDeprected

Проверьте, устанавливается ли родительское свойство.

  class A {
    /** @deprecated */
    protected $ name ;
  }
  class B extends A {
    protected $ name ; // <-- Warn about deprecation
  }

Запрещено classsextend

Занятия, помеченные @final Doc, не должны быть расширены

 /**
 * @final
 */
 class User {};

 class Admin extends User {}; // <- Prohibited extentions of @final class User. 

PropertyAnnotation

Свойства, которые не инициализированы в конструкторе, должны быть аннотированы как нулевые.

 class User {
 /** @var string */ // <-- Property is not annotated correctly. Add null type
 private $ name ;
 public function getName () {  }
 public function setName ( string $ name ) {  }
}

PropertyCanBeprivate

Защищенные свойства могут быть преобразованы в частные.

 class User {
  protected $ user ; // <-- Property can be private
}

REDUNDANTDOCCOMMENTTAG

Типы, указанные в PHP, могут быть опущены в блоках PHPDOC

 /**
 * @return void // <-- Redundant PhpDoc tag
 */
 function show ( string $ message ): void {}

ToStringCall

Обнаружение автоматического литья типа

 class Hello {
    public function randomize (): self { /* ... */ return $ this ; }
    public function __toString () { return ' Hi ' ; }
}
echo ( new Hello ())-> randomize (); // <-- Deprecated __toString call 

VirualtyPecheck

Используйте Assert, чтобы проверить тип переменной вместо комментария DOC.

 /** @var User $user */ // <-- Use assert to check variable type
assert ( $ user instanceof User);

Действия

UsenamedConstructor

Замените new ClassName() выбранным названным конструктором.

 class Text {
 public function __construct ( string $ name ){ }
 public static fromName (string $ n ){}
}

Вызовать refactor this на название метода fromName , и все новые операторы с этим классом будут изменены

 new Text ( ' User ' ) // old code
 Text:: fromName ( ' User ' ) // new code