erstelle das erste mal eine klasse :S

[dedlfix]

Hi!

class siteBuilder {
    protected $html; // warum sollte das public sein?

Warum nicht? Oder auch: warum protected und nicht private? Es gibt keine Verordnung, dass Eigenschaften nicht öffentlich sein dürfen. Ebenso gibt es keine, dass auf Eigenschaften nur über Zugriffsmethoden zugegriffen werden darf. Inwieweit das eine oder andere sinnvoll ist, hängt von der Aufgabenstellung ab, die uns jedoch nicht bekannt ist.

public function getHTML($template) {
        // hier fehlt eine Prüfung, ob $template ok ist!!

"ok" in welcher Hinsicht?

public $html = file_get_contents('template/'.$template);

Diese Zeile ist weiterhin fehlerhaft. $this->html = ...
venty: Hast du auch daran gedacht, dass der Dateizugriff aus vielerlei Gründen misslingen kann?

public function returnHTML() {
        return $this->html; // $this ist wichtig!!

Ja. Was ist allerdings, wenn returnHTML() aufgerufen wird bevor getHTML() ein Template besorgt hat. Außerdem wäre zu überlegen, wie sinnvoll es ist, den Zugriff so zu trennen. Aber das kann man nur unter Kenntnis der vorgesehenen Anwendungsfälle näher betrachten.

Lo!