Rolf B: current_page hat keine wirkunge,,,

Beitrag lesen

Hallo ursus,

gute Idee, die Navigationslinks als Widget in eine Klasse zu kapseln.

Ich hätte aber 2 Einwände:

  • Code wird mit echo ausgegeben statt in den HTML Kontext zu wechseln
  • Codewiederholung und Verletzung des Do One Thing Prinzips in addLinks, da gehört eine private-Methode checkAndStoreLink hin, die validiert und speichert

Ob man den Rahmen der Navigation von der Klasse ausgeben lässt, darüber könnte man auch diskutieren. Ich würde das nicht als Job dieser Klasse sehen, es erschwert Wiederverwendung.

Rolf

--
sumpsi - posui - clusi