So erkennen Sie dynamisch deklarierte Felder auf Objekten mit Codesniffer in PHP

Dies ist kein Codesniffer- oder PHP-Storm-Problem. Und Sie können dieses Problem nicht mit Codesniffer oder IDE beheben. IDE, codesniffer, phpdocumentor usw. - das wird "statisch" analysiert. Und für die dynamische Analyse können Sie z.B. phpunit.

Wenn Sie das Vorhandensein einer Eigenschaft prüfen möchten, müssen Sie die Funktion property_exists () verwenden.

Oder Sie können die Reflektion für die Eigenschaft Ссылка

Wenn Sie nach "isset" suchen wollen, müssen Sie wissen:

Wenn Sie sich für diesen Fall der Überprüfung entscheiden, können Sie isset ()

Aber Sie sollten immer zuerst nach der Existenz der Eigenschaft suchen. Andernfalls können Sie ein undefiniertes Verhalten Ihres Codes haben.

  

Nach einem Refactoring

     

Es wäre gut, dies von vornherein zu verhindern.

Sie können diese Art von Refactoring-Fehlern nur durch Ausführen von Tests nach jedem Refactoring-Schritt erkennen. Dieser Fehler tritt auch auf, weil foo['blubb'] auf einen bestimmten Wert eingestellt ist und dies einen unerwünschten Effekt in einem anderen Test verursachen sollte - nicht nur im Test für die Setter-Logik.

  

Wir verwenden Tests, aber manchmal vermisst du Dinge ...

Ja, es ist ziemlich normal, dass die Abdeckung nicht hoch genug ist. Deshalb ist eine gute Testabdeckung der Ausgangspunkt für alle Refactorings.

Diese beiden Zeilen waren in Ihrem Deckungsbericht nicht "grün":

  

Auch, bitte überlege dir nicht, die magischen Methoden zu überschreiben. Ich möchte in keiner Klasse eine Eigenschaft / Zusammenfassung haben.

Sie haben es ausgeschlossen, aber das ist eine Möglichkeit, die Eigenschaften abzufangen: mit der magischen Funktion __set() (für nicht zugreifbare vars) oder property_exists() oder der Verwendung von Reflection* classes zu finden.

Nun, das ist zu spät, Sie wollen ein anderes Werkzeug, um den Fehler zu fangen, ok:

Das Tool müsste die PHP-Datei und ihre Eltern (aufgrund des Variablenbereichs) analysieren und $this->bla ohne vorherige public|private|protected -Variable (Klasseneigenschaft) -Deklaration finden. Dies zeigt nicht den genauen Fehlertyp an, nur dass auf "bla" ohne Deklaration zugegriffen wurde.

Es ist möglich, dies als CodeSniffer-Regel zu implementieren.

Sie können auch Ссылка oder Ссылка angeben ein Versuch. Und falls Sie PHP7 verwenden: Ссылка

tl; tr

Es ist kompliziert, den genauen Fehler und seinen Kontext zu ermitteln, ohne den zugrunde liegenden Code auszuführen, zu bewerten und zu analysieren. Denken Sie nur an "dynamische Variablennamen" und Sie wissen warum: Sie kennen den Namen der Eigenschaft nicht einmal, indem Sie sich den Quellcode anschauen, da dieser dynamisch während des Programmablaufs erstellt wird. Ein statischer Analysator würde das nicht verstehen.

Ein dynamischer Analysator muss alle Dinge verfolgen, hier $this-> greift zu und würde den Kontext berücksichtigen:! isset (x). Die Kontextauswertung kann viele häufige Programmierfehler finden. Am Ende können Sie einen Bericht erstellen: sagen, dass $ this- & gt; bla wurde nur einmal zugegriffen und das zeigt entweder, dass

• Eine dynamisch deklarierte Eigenschaft wurde eingeführt, aber nie wieder verwendet, mit dem Vorschlag, sie zu löschen oder als Klasseneigenschaft zu deklarieren
• ODER mit hinzugefügter Kontextevaluierung: dass und wenn auf diese Variable von innerhalb eines isset () zugegriffen wurde - dass auf einen nicht existierenden Schlüssel einer nicht deklarierten Eigenschaft zugegriffen wurde, ohne vorheriges set (), usw.