Ascer's OOP #1

[ascer]

Hi Leute,


so, hab mal eine kleine minimalistische Templateklasse erstellt, wäre das so im Sinne des Erfinders oder sind da noch Denkfehler drin?

PHP-Code:

class template {

    public $template; // Variable mit dem Originalinhalt der Templatedatei
    public $rdy_template; // Variable mit dem geänderten Template
    
    
    function loadTemplate( $url ){ // Funktion erstellen und Variable $url verwenden
        if( !$url OR empty($url) ){
            print "Es muss eine URL angegeben werden!";
            return;
        }
        $file = file_get_contents($url); // das Template laden
        if( !$file OR empty($file) ){ // falls $file leer ist nicht laden
            print "Templatedatei wurde nicht gefunden!";
            return;
        } else {
            $this -> template = $file;
        }
    }
    
    
    function editTemplate( $platzhalter, $wert ){ // Wert von $rdy_template gleichsetzen mit Wert von $template und Platzhalter durch echte Werte ersetzen
        $this -> rdy_template = str_replace( $platzhalter, $wert, $this -> rdy_template );
    }
    
    
    function resetTemplate(){ // $rdy_template auf den Wert von $template (Originaldatei) zurücksetzen
        $this -> rdy_template = $this -> template;
    }
    
    
    function parseTemplate(){ // Template ausgeben
        print $this -> rdy_template;
    }

} 


gruß,
Ascer

EDIT: der Kommentar von editTemplate() ist eigentlich einzeilig, nicht das ihr denkt das ich noch mehr "wie kraut und rüben" programmier, als ich das eh schon tu *g* da war wohl nur die Zeile hier im Codefenster nicht breit genug, bei mir im Editor ist der einzeilig...selbiges gilt für den Kommentar bei resetTemplate()...

[Socrates]

Hallo!
Mir sind gleich einige Sachen aufgefallen. Und zwar wieso zwei Variabeln? Eine reicht doch. Wieso sind diese Variabeln public? Die können privat sein, da du sie ja so oder so mit einer Methode ausgibst. Fehlerbehandlung macht man möglichst nicht mit print oder echo sondern mit Exceptions. Ließ dir dazu mal den dazu gehörigen Part im Manual durch. Des weiteren kannst du mal in diesem Beitrag schauen. Da gab es eine schöne Diskussion über TemplateEngines, eine Template-Klasse von mir und verschiedene Methoden soetwas zu verwirklichen. Ich hoffe, dass ich dir helfen konnte.
MfG, Andy

//EDIT: Du solltest auch die Methoden in ihrer Nutzbarkeit kennzeichnen. Also auch mit privat oder public.

[Area51]

PHP-Code:

function editTemplate( $platzhalter, $wert ){ // Wert von $rdy_template gleichsetzen mit Wert von $template und Platzhalter durch echte Werte ersetzen
        $this -> rdy_template = str_replace( $platzhalter, $wert, $this -> rdy_template );
    } 


gefällt mir nicht so. Es wäre besser, wenn man ein Array übergeben würde ;)

Weil sonst würde man ja

$lorem -> editTemplate ('Ipsum', 'dolor');
$lorem -> editTemplate ('sit', 'amet');

schreiben. Das wird mit der Länge immer unübersichtlicher ;)

[ascer]

ok, dankeschön!! sehr hilfreich auf jeden Fall :)

werd mich mal dranmachen und einiges umschreiben^^

[Socrates]

Also ich denke, dass das mit einem Array unübersichtlicher wäre. Ich finde den Teil eigentlich ganz ok. Sieht bei meiner Klasse ähnlich aus, jedoch gebe ich noch das Template in seiner momentanen Form via return zurück.
MfG, Andy

[Area51]

Wenn man das Array so gestalten würde:

PHP-Code:

$array = (
  'Var1' => 'Val1',
  'Var2' => 'Val2',
  'Var3' => 'Val3',
  'Var4' => 'Val4',
); 


finde ich nicht, dass es unübersichtlicher wird ;)