Newsklasse bewerten

holunda · 19.01.2003, 19:56

HI Leute!

MUss mich in OOP ein wenig fit machen.
Wie schaut die KLasse aus? Ist was falsch bzw. unsinnig?
Was kann man besser machen, was erweitern?

PHP-Code:


			
class News{

    

    var $conn = 0; // Verbindungshandler

    var $news_tpl = "news.tpl"; // News Template

    var $table = "news"; // Newstabelle

    var $limit = 5; // Anzahl sichtbare News

    

    function openConnection(){

        $server['host'] = "localhost"; // DB Host

        $server['user'] = "***"; // DB User

        $server['pass'] = "***"; // DB Pass

        $server['db'] = "***"; // DB Name

        $this->conn = mysql_connect($server['host'],$server['user'],$server['pass'])

            or die ("Konnte Verbindung nicht herstellen<br>Fehler: ".mysql_error());

        mysql_select_db($server['db'])

            or die ("Konnte DB ".$server['db']." nicht ansteuern<br>Fehler: ".mysql_error());

    }

    function newNews($name,$titel,$autor,$text,$linktext,$link){

        $sql = "INSERT INTO ".$this->table." (name,titel,autor,zeit,text,linktext,link) VALUES ('$name','$titel','$autor',NOW(),'$text','$linktext','$link')";

        mysql_query($sql,$this->conn)

            or die ("Konnte Daten nicht eintragen<br>Fehler: ".mysql_error());

    }

    function showNews(){

        $template = $this->news_tpl;

        $sql = "SELECT * FROM ".$this->table." ORDER BY 'id' DESC LIMIT 0,".$this->limit;

        $result = mysql_query($sql,$this->conn)

            or die ("Konnte News nicht laden<br>Fehler: ".mysql_error());

        while ($record = mysql_fetch_array ($result)){

            $newsstr = implode ('',file($template));

            $newsstr = str_replace('{Autor}',$record['autor'],$newsstr);

            $newsstr = str_replace('{Titel}',$record['titel'],$newsstr);

            $newsstr = str_replace('{Zeit}',$record['zeit'],$newsstr);

            $newsstr = str_replace('{Text}',$record['text'],$newsstr);

            $newsstr = str_replace('{Link}',"<a href=\"".$record['link']."\">".$record['linktext']."</a>",$newsstr);

        }

        @mysql_free_result($sql);

        return $newsstr;

    }

    function closeConnection(){

        mysql_close($this->conn)

            or die ("Konnte Verbindung nicht schließen<br>Fehler: ".mysql_error());

    }

}

aja, und könnte mir irgendwer nochmal die Vorteile von OOP erklären, ich meine, Funktionen kann ich auch wo anders einfügen, muss ja nicht gleich eine KLasse sein oder? Was sind also die genauen Vorteile?

danke Alexander

holunda · 20.01.2003, 14:04

keiner? Wäre wichtig muss nämlich heute noch damit weiterarbeiten ...

Alex

papo · 20.01.2003, 16:15

Hallöchen,

was mich stört ist die Verwendung von die(); in der Klasse;
Die Einsetzbarkeit wird dadurch ein wenig reduziert. Wenn ich das ganze in eigene Projekte einbinden möchte, und nur durch einen Fehler bei den News der Rest nicht mehr ausgeführt wird fände ich das lästig.
Ich würde hier
1. Die Ausgabe der php-eigenen Fehlermeldungen unterdrücken, mit @ vor den betreffenden Funktionen.
2. Evtl. Eine Klasseneigene Variable setzen die den Verbindungsstatus ($connected = 0 oder 1) enthält.
3. Die Ausgabe der News von dieser Variablen abhängig machen z.B. mit if ($connected).
4. Eine eigene Fehlermeldung ausgeben wenn die Verbindung fehlschlägt.

Zu OOp:
..) Bessere Wiederverwendbarkeit des Scripts.
..) Jede Klasse bekommt ihren eigenen Namensraum.
..) Aus dem selben Grund hat man es leichter bei der Namensgebung
..) Man gibt sich automatisch mehr Mühe bei der Programmierung (und das stimmt wirklich

..) Neue Objekte / Klassen können von bestehenden Abgeleitet werden (Vererbung)...

Generell hängt es aber eher vom zu lösenden Problem ab, ob es sich lohnt Objektorientiert zu Programmieren.

Gruß, Paul