jChat BETA 0.1 - mysqli, ajax, php, html

Zitat von strohhalm

Zitat von »PixelFriend«

Der einzigste Downloadlink in deinem ersten Post ist im Spoiler der Updatescreen mit folgendem Link zum download: File-Upload.net - myChat-Update-0.0.1.zip
Mein Fehler, sorry.
Hab den Thread bearbeitet & hier ist der Link: File-Upload.net - myChat-0.1.zip

1. Danke für den Link.
2. Danke für deine Projekte und die Veröffentlichung.
3. Du kannst gut coden.

Zitat von Kurdt

Oh, du hast die Smilies von meinem Chat geklaut, du böser Junge!

Mein Chat: Willkommen zum Chat: Stelle anderen Mitgliedern Fragen und Entdecke die Community!

Bei dir werden pro Sekunde 3 Requests versendet. Wäre wohl ressourcenschonender, wenn du das alles über eine Art "pull" Request machst und du die Antwort dann zu JSON formatierst. Und das mit dem "vor 5 Sekunden.." stört etwas. Könnte man auf Minuten verlängern ( < 1 wäre dann vlt. "just now"). Sonst echt cool gemacht, das Template gefällt mir.

iExit, mir gefällt deine Struktur nicht so. Du erstellst anscheinend auch 2x eine MySQL Verbindung, was ich nicht verstehe. Und noch ein kleiner Tipp: Bei 'for' Schleifen bei $i < fun(..) wird fun() nach jedem Durchgang ausgeführt. Bei dir müsste also für jedes Leerzeichen der Array neugezählt werden. Am Besten in eine Variable stecken. Außerdem checkst du zuerst if($i > 0) und danach elseif($i <1).. Wenn $i > 0 ist, dann ist ja eigentlich schon der andere Fall klar. Ein "else" alleine wäre da genug.

Cheers
Steve Winfield

LIVEDEMO: http://62.141.37.141/

Die Livedemo (http://62.141.37.141) hat nun die neuste Version welche in wenigen Minuten veröffentlicht wird.

setInterval(function(){
    $("#chatcontent").load("Ajax.ChatInhalt.php");
}, 1);

http://www.w3schools.com/jsref/met_win_setinterval.asp

Zitat von strohhalm

Danke nochmals an Nova, hier die "gefixte" Version:
Sicherheitslücken sollten geschlossen sein, bei Funden einfach melden.
File-Upload.net - jChat.zip

Ehhhh, wtf? Bitte betreibe nie im Leben einen Webservice.

$id = htmlspecialchars($_GET["idtodelete"]);$delqry = $mySQLi->query("DELETE FROM ajax_chatMessages WHERE id = '".$id."'");

$user_query = $this->mySQL->query("SELECT * FROM chatUsers WHERE username = '".$_SESSION["username"]."' AND password = '".$_SESSION["password"]."' LIMIT 1");

Netter Chat, kann man allerdings noch ausbauen.
Ansonsten gefällt er mir im Großen & Ganzen.

Noch ein kleiner Tip von mir: Benutz am besten MySQLI in der OOPHP Version, das ist am sichersten und schließt MySQL Injections zu 99% aus.
Ansonsten verwendest du am besten mysql_real_escape_string zum parsen von externen Daten (PHP: mysql_real_escape_string - Manual)

Sieht aber ganz gut aus dein Projekt^^

Mit freundlichen Grüßen

Ich habe ein kleines Problem mit dem Chat.
Und zwar wenn man ein Passwort und Username beim Registrieren eingibt und auf Registrieren klickt kommt die Nachricht " Du hast nicht alle Felder ausgefüllt".
Danke für eure Hilfe

So soll es sein;
~closed