Shoutbox Live Open Source

[holunda]

HI Folks

Versuche hier mal einen Thread aufzumachen in der wir/ich eine Shoutbox zusammen programmieren.

Das Prinzip dieses Threads ist einfach. Ein Anfangscode wird hier veröffentlicht. Wenn irgendwer was dazuproggen möchte schnappt er sich den vorhandenen Code und schreibt in den Thread was er genau dazuproggt. Damit sollten Codeüberschneidungen ausgeschlossen sein.

Ich fange hier mal mit der Datei [config.php] an:

PHP-Code:

### Start Installation ###
$server['host'] = "localhost";
$server['user'] = "root";
$server['pass'] = "";
$server['db_name'] = "";
$anzahl = 5;
### Emde Installation ###

function fill_box ($name,$email,$hp,$text){
    $date = date("d.m.Y H:i:s");
    // Wächter $name + $text
    if (!$name || !$text){
        echo "Sie müssen die Felder \"Name\" und \"Message\" ausfüllen!";
    }
    // Wächter $email + $hp
    if (!$email || !$hp){
        $email = "-";
        $hp = "-";
    }
    // Fill
    $anfrage = "INSERT INTO 'shoutout' (name,email,hp,text,date) VALUES ('$name','$email','$hp','$text')";
    @mysql_query ($anfrage) or die ("Konnte Daten nicht schreiben!<br>Grund: ".mysql_error());
}
function show_box(){
    $anfrage = "SELECT * FROM 'shoutout' ORDER BY 'date' DESC LIMIT 0,'$anzahl'";
    $erg = @mysql_query ($anfrage) or die ("Konnte Daten nicht holen!<br>Grund: ".mysql_error());
    while ($row = mysql_fetch_array($erg)){
        // echo html stuff
    }
}

// Connecte
@mysql_connect ($server['host'],$server['user'],$server['pass']) 
or die ("Datenbankserver konnte nicht erreicht werden<br>Grund: ".mysql_error());
// Selecte :)
@mysql_select_db ($server['db_name']) 
or die ("Datenbank nicht bereit<br>Grund: ".mysql_error()); 


mal sehen ob das was wird
mfg Alex

[Lars]

OOP-Styla:

PHP-Code:

<?
class shoutBox{
    var $connHandler = 0;
    
    function openConnection(){
        //### Start Installation 2 ###
        $server["host"] = "localhost";
        $server["user"] = "root";
        $server["pass"] = "";
        $server["db_name"] = "";
        $anzahl = 5;
        //### Emde Installation 2 ###
    
        $this->connHandler = @mysql_connect($server["host"],$server["user"],$server["pass"])
            or die("Datenbankserver konnte nicht erreicht werden<br>Grund: ".mysql_error());
        @mysql_select_db($server["db_name"],$this->connHandler)
            or die ("Datenbank nicht bereit<br>Grund: ".mysql_error());
    }

    function fillBox($name,$email,$hp,$text){
        $date = date("d.m.Y H:i:s");
        // Wächter $name + $text
        if (!$name || !$text){
            echo "Sie m&uuml;ssen die Felder \"Name\" und \"Message\" ausf&uuml;llen!";
        }
        // Wächter $email + $hp
        if (!$email) $email = "-";
        if (!$hp) $hp = "-";
        
        // Fill
        $query = "INSERT INTO 'shoutout' (name,email,hp,text,date) VALUES ('$name','$email','$hp','$text')";
        @mysql_query($query,$this->connHandler)
            or die ("Konnte Daten nicht schreiben!<br>Grund: ".mysql_error());
    }
    
    function showBox($queryString){
        $query = "SELECT * FROM 'shoutout' ORDER BY 'date' DESC LIMIT 0,'$anzahl'";
        $result = @mysql_query($query)
            or die ("Konnte Daten nicht holen!<br>Grund: ".mysql_error());
        
        while ($record = mysql_fetch_array($result)){
                // Hier sähe ich gerne einen hübschen Template-Parser - wer will?
        }
        @$mysql_free_result($query);
    }
    
    function closeConnection(){
        @mysql_close($this->connHandler);
    }
}
?>

Ist wie immer ungetestet, es befinden sich also vermutlich noch ein paar Klopper drin.

[Stuck Mojo]

Was mir sofort auffällt, ist dass der "Wächter" für $name + $text zwar "wacht", aber nicht richtig reagiert
und sobald man eine email angibt, aber keine hp, wird die email auf '-' gesetzt. Und umgekehrt.

Dann wird $date nicht mit in den INSERT-Query übergeben. Ich würd's sowieso via DATETIME-Feld und INSERT NOW() machen.

Achja... und dann sollte man vielleicht noch an register_globas = off denken.

[holunda]

wird erledigt ...

[Lars]

Original geschrieben von Stuck Mojo
Was mir sofort auffällt, ist dass der "Wächter" für $name + $text zwar "wacht", aber nicht richtig reagiert

Damit kann meine Version ja wohl kaum gemeint sein.

[Stuck Mojo]

Original geschrieben von Lars
Damit kann meine Version ja wohl kaum gemeint sein.
...

PHP-Code:

// Wächter $name + $text
        if (!$name || !$text){
            echo "Sie müssen die Felder \"Name\" und \"Message\" ausfüllen!";
        } 


Was soll das bewirken? Das der User sieht, dass er beim nächsten mal dran denken soll seinen Namen oder seinen Text einzugeben?

[Lars]

Huch, ich dachte es geht um
eMail und Homepätsch ...
aber ich habe dsa ja ah alles
eins zu eins von holunda
übernommen und nur geOOPt.



Jan = Affe

[Stuck Mojo]

LARS = pfff...
...ausserdem ist im Nachhinein ändern unfair
Puller dich mal nicht so schnell ein

[Lars]

Ich habe das gar nix im Nachinein geändert, das darfst Du mir ruhig glauben, mein Gutster.

Ausserdem ist das hier mein Beitrag Nummero 5.000 ... falls es jemanden interessiert.

[Stuck Mojo]

Original geschrieben von Lars
Ich habe das gar nix im Nachinein geändert, das darfst Du mir ruhig glauben, mein Gutster.

Ausserdem ist das hier mein Beitrag Nummero 5.000 ... falls es jemanden interessiert.

klaro... da stand vorher nur Affe ...du Affe ...nu' is aber auch gut, schliesslich soll hier mal ein dolles Script draus werden. Und keine Postingschinderei

Ach so, das "Affe" meinst Du, okidoki. Wann dürfen wir eigentlich mit Deinen Zugaben rechnen?

P.S. Ich bin kein Postingschinder, deswegen antworte ich jetzt nur noch "Beitrag in Beitrag".

[holunda]

Achja... und dann sollte man vielleicht noch an register_globas = off denken.

Wieso, was hat der bisherige Code mit Globals zu tun, meines Erachtens kommen da keine Fehler bezüglich on/off Globals vor, oder doch?

Alex

[Stuck Mojo]

achso... stimmt ja, das sind ja Funktions-Parameter... sorry...

[Lars]

Je nachdem, wie es mal eingesetzt werden soll, schon. Denn die Funktionsparamter können ja auch von einer anderen Seite stammen (ausgelagerte Funktionalität).

[mike]

// Hier sähe ich gerne einen hübschen Template-Parser - wer will?
<--/me

[Lars]

Leg' los, Opa.