Da spielt PHP gar keine Rolle.
Ohhh mein lieber Alex....und ob PHP eine Rolle spielt und zwar genau dann wenn du nach Jahren deinen eigenen Code abändern musst und nicht mehr genau verstehst was du da eigentlich gemacht hast. Oder natürlich wenn andere Coder ran müssen...dann ist das noch nerviger wenn nicht sauber gecoded wurde
Hier deshalb meine Anmerkungen (aus meinem persönlichen Coding-Style herausgesprochen wohl gemerkt
):
* Die Variablennamen sind nicht gut benannt. Z.B. gibt OOCategory::getCategoryById() ein Kategorieobjekt zurück und nicht ne Id. Deshalb wäre $category sinnvoller als $catID. getChildren() wäre $children oder $categoryChildren am sinnvolsten, nicht $childCatID.
* Auch die Gorßschreibung würd ich weglassen. Ist unüblich und sieht komisch aus.
* $var ist so ziemlich der schlimmste Variablename den man sich ausdenken kann, weil absolut nichts sagend über den Inhalt. Da bei dir $var ein einzelnes Kategorieobjekt darstellt sollte es auch so beannnt werden $cat/$category.
* $artName? Das sind doch die Meta Infos einer Kategorie?!
* Und die Klammern werden üblicherweise auch nicht so gesetzt.
* Desweiteren sollten immer prüfungen rein ob, die Objekt auch tatsächlich da sind.
* noch ein Kommentar was der ganze Codeblock eigentlich macht wäre noch das i Tüpfelchen
Hier das ganze durch den RexDude Code Beautifier gejagt
Code: Alles auswählen
// generate an image list based on parent category children
$currentCategoryId = $this->getValue("category_id");
$currentCategory = OOCategory::getCategoryById($currentCategoryId);
$parentCategory = $currentCategory->getParent();
if (is_object($parentCategory)) {
$parentCategoryChildren = $parentCategory->getChildren(true);
foreach ($parentCategoryChildren as $category) {
$name = $category->getValue("name");
$image = $category->getValue("art_file");
echo $name . '<br><img src="./files/' . $image.'" /><br>';
}
}