Tach!
Spätestens hier würde ich als Teamleiter laut aufschreien. Weißt Du, warum?
$calculator->printerDelegate = $printer; // Wir setzen den normalen Drucker als delegate ein
Weil es keine PHP-Syntax ist oder so was?
Es ist gültige PHP-Syntax. Vermutlich meint Tom, dass du öffentlichen Eigenschaften direkt Werte zuweist, statt über einen Setter zu gehen. Letzterer ist vor allem dann sinnvoll, wenn weitere Funktionalität beim Zuweisen gewünscht ist. Das kann zum Bespiel eine Prüfung des Arguments sein und sei es, dass man per Type-Hinting nur bestimmte Klassen zulässt. Ohne Prüfung ist es nur ein sinnloser Funktionsaufruf (den man vielleicht später noch erweitert oder auch YAGNI).
dedlfix.