mysql-PHP chat onlineanzeige

maxda · 29. November 2008, 13:32

Hallo,
ich habe einen chat mit PHP und MYSQL geschrieben. Das klappt auch prima, nur an der Online anzeige (wer die Chat-Seite gerade offen hat) harpert es!
Ich dachte es wäre am besten wenn ich es so mache:
Alle 5 Sekunden wird der eigene Name eingetragen (oder aktualisiert) und alle Namen, die älter als 6 Sekunden alt sind, werden gelöscht.
Hier ist der Code:

PHP-Quellcode

mysql_query("DELETE FROM online WHERE TIMESTAMPDIFF(SECOND, time, NOW()) > 6;",$connect);
//Alte Namen werden gelöscht
mysql_query("DELETE FROM online WHERE name='".mysql_real_escape_string($name)."';",$connect);
//Der eigene Name wird ausgetragen
mysql_query("INSERT INTO online (name,time) VALUES ('".$name."',NULL);", $connect);
//... und wieder eingetragen
//Dann werden die Nachrichten ausgelesen
//Hier werden alle Namen angezeigt
$sql_string = "SELECT * FROM online ORDER BY name ASC;";
$ergebnis = mysql_query($sql_string, $connect);
$rows = mysql_num_rows($ergebnis);
echo 'Online:<br>';
for ($i=0; $i<$rows; $i+=1)
{
echo mysql_result($ergebnis,$i,"name");
echo "<br>";
}

Alles anzeigen

Nur wenn ich das Fenster dann öffne, ist es manchmal so, dass der eigene Name oder der Name von jemandem, der eigentlich online ist, nicht angezeigt wird.

Woran könnte das liegen, wie könnte ich das verbessern oder gibt es eine bessere methode für eine Onlineanzeige?

Viele Grüße,
maxda

EDIT: Teil vom Code vergessen!

Windapple · 29. November 2008, 13:41

Du solltest erst mal beim eintragen nicht mit NULL sondern mit NOW() arbeiten. Sonst trägst du noch als Zeit 01.01.1970 0:00 ein und das ist definitiv länger her als jeweils 6 Sekunden.

Desweiteren verwendest du die Variable $rows, die wird aber im gezeigten Stück Code nirgendwo gesetzt. Wäre also durchaus denkbar, das $rows nicht mit der tatsächlichen Anzahl an Einträgen übereinstimmt. Das selbe gilt für den Query String, dessen Ergebnis in der Variable $ergebnis wohl gespeichert sein sollte. Auch dieser Codeteil ist nicht hier drin.

Du zeigst uns zwar etwas relevanten Code, aber nicht den ganzen. Probleme, die außerhalb des gezeigten Teiles existieren, kann dir keiner beheben.

maxda · 29. November 2008, 14:01

Ich mach das mit Timestamp?!

Oh ja das fehlt:

PHP-Quellcode

$sql_string = "SELECT * FROM online ORDER BY name ASC;";
$ergebnis = mysql_query($sql_string, $connect);
$rows = mysql_num_rows($ergebnis);