PHP Coding Standard

Logo: PHP
27. Regel 24: Code, der nicht benutzt wird, muss gelöscht werden

Jeder Programmierer wird früher oder später auf Code stoßen, der nicht mehr benutzt wird, aber nie gelöscht wurde. Besonders ärgerlich ist dann die Suche, ob und wofür der Code benutzt wird. Zum Beispiel eine IF-Struktur, deren Bedingung niemals TRUE wird.

Gültig
   // never TRUE
   if (isset($_GET['content'])
      and isset($_GET['navigation'])
      and isset($_POST['page']))
   {
      // 200 lines of code, which are never used
   }
   else
   {
      // ELSE is always used here
   }

Irgendwann nach langer Suche stellt man dann fest, dass es kein Formular mit dem POST-Parameter "page" gibt und demnach sind die 200 Zeilen Code überflüssig. Alles, was im Code überflüssig ist und nicht benutzt wird, muss gelöscht werden.

Auch wenn eine Funktion mit 300 Zeilen, in der fünften Zeile "return TRUE;" enthält und somit die Funktion immer vorzeitig verlassen wird, muss der überflüssige Code gelöscht werden.

Am besten man macht eine Sicherungskopie der Datei oder des ganzen Verzeichnisses und löscht dann den Code. Danach sollte man intensiv testen, um die Auswirkungen der Änderungen zu beobachten. Vollautomatisierte Tests, wie sie beim Refactoring eingesetzt werden, sind natürlich am besten geeignet.