dedlfix: PHPDocumentor Docu-Style, used tags

Beitrag lesen

echo $begrüßung;

gemäß dem würde es dann in etwa so aussehen?

* @copyright  GNU GPL
* @license    GNU GPL
* @version   My_XSLT 1.0
* @link       http://spinneimnetz.de
* @since      My_XSLT 1.0

Ob du diesen Kladderadatsch brauchst oder haben möchtest, musst du selbst entscheiden.

/**
* Short description for class
* Process XSLT
* Long description for class (if any)...

In der ersten Zeile steht eine Kurzbeschreibung. Dann folgt eine Leerzeile gefolgt von einer optionalen Langbeschreibung. Die Kurzbeschreibung wird in Übersichten angezeigt. Da sollte also was Aussagekräftiges drinstehen.

* @param  string $file_path_xml file_path_xml
* @param  string $file_path_xsl file_path_xsl
private function process()

Die Dokumentation passt nicht zur Signatur der Methode. Es ist auch kein func_*_args()-Aufruf in der Funktion zu sehen.

Hier noch ein Nicht-PHPDoc-Hinweis:

public function write($file_path_output)
{
  if (file_put_contents($file_path_output,$this->putout())) {
   return true;
  } else {
   return false;
  }
}
}

Wenn die Bedingung im if zutrifft, wird die Funktion aufgrund des return verlassen. Das return false; kann außerhalb des else notiert werden, da es ja in jedem Fall nur beim Nichtzutreffen ausgeführt wird. Obendrein ist hier gar kein if-else notwendig. return file_put_contents(...), zur Not nach boolean gecastet, ergibt das gleiche Ergebnis. Allerdings ist das im Spezialfall "alles i.O. es wurden aber nur 0 Bytes geschrieben" in keinem Fall richtig. return false !== file_put_contents(...); wäre dann genauer.

echo "$verabschiedung $name";