PHP CHAT (AVEC PHP5 + PAF)

Commentaire de kohntark le 02/05/2009 11:37:04

Salut,

Je n'ai fait que regarder la démo, et ça à l'air bien sympa.

quelques remarques "brutes" :
- sous IE8 le confirm salles=>joindre est bloqué
- le div qui permet d'afficher (par ex) le chat=>a propos ne permet pas de lire l'intégralité du texte (pas de scroll possible)
- le message "La commande pour rejoindre la salle '@dfdsfs' a pris trop de temps à s'exécuter" s'affiche alors qu'il serait plus parlant d'informer sur les caractères illégaux (j'imagine que c'est pour cette raison que ça "plante") De même un nom de salle avec espace ne donne pas le résultat escompté ("truc bidule" => créé une salle "truc")
- les cursors pourraient être méliorés ( entres autres sur les onglets des salles disponibles)
- certains caractères ne passent pas dans le chat, par exemple (<) : "si a <b" => "si a"
- si j'entre par exemple "<div style="width:200000px;height:4000000px">jklbnk</div>" ça fonctionne et ça rend le scrolling auto inopérant
- surligné, d'une manière ou d'une autre, le pseudo avec lequel l'utilisateur est connecté serait un plus
- il y a possibilité d'ouvrir plusieurs fenêtres du chat avec des pseudos différents qui occasionnent des bugs
- j'ai constaté également que certains messages n'apparaissent pas dans la fenêtre de celui qui les a entré, alors qu'ils apparaissent correctement chez les autres.
- ...


... voilà pour les quelques commentaires "en live"

Je regarderai le code à l'occasion.


Cordialement,


Kohntark-

Commentaire de Arto_8000 le 03/05/2009 00:12:53

pour le #1, je viens de tester sur IE8. En fait, le problème c'est que le navigateur a besoin (pour une raison assez obscure) d'avoir ta permission pour afficher le "prompt". Une petite bar apparait dans le haut, il suffit de dire oui je veux exécuter ce très dangereux "scripted window".

Commentaire de Arto_8000 le 03/05/2009 17:03:48

pour le #1, je peux absolument rien faire pour ça, parce qu'un prompt c'est un élément de base en javascript. D'autant plus que les utilisateurs de IE8 doivent être habitué d'avoir des messages aussi stupide qui leur soit demandé.

pour le #3, Le dernier fix a rajouté un message d'erreur quand on entre un nom de salle invalide.

pour le bogue de quand on quitte les fenêtres, j'avais initialement mis un message pour quand on quitte toutes les salles qui disaient que l'on ne pouvait pas quitter toutes les salles. Le problème est que après j'ai eu de la pollution dans mes objets et certains tests ne fonctionnaient plus bien.

pour la dernière remarque, il y a le mot clé "incorrect" dans le message. Je vais quand même rendre un peu plus clair les messages d'erreur.

Commentaire de kohntark le 05/05/2009 10:39:11

Je viens de télécharger et d'installer le chat pour test.
C'est "lourd", pas dans le sens péjoratif du terme, mais de celui du travail fourni.
Je te livre en vrac et en live certaines remarques et problèmes constatés lors du test. Je me place le plus souvent en tant que simple utilisateur de ta source, sans chercher dans le code (ce chat a clairement vocation a être utilisé sans grandes connaissances en programmation) :

- install => pas de pb
- appel de la page, connexion => affichage page : pb d'encodage, il y a des caractères exotiques un peu partout.
Ma base / php ... sont par défaut en utf8.
- un "actualiser" fait revenir à la page login, ce qui n'est pas une bonne chose à mon sens puisque de nombreux utilisateurs risquent d'utiliser cette fonctionnalité pour des raisons diverses ("je n'ai plus de réponses, le chat a dû planté, j'actualise !!")
- tes messages d'alertes sur les caractères non autorisés sont bien plus parlant. Cependant, pour ce qui est de quitter une salle, tu pourrai ajouter à "Vous devez avoir au moins une salle d'ouverte." cela : "pour vous déconnecter du chat utilisez 'chat>déconnexion'"
- lorsque l'on se déconnecte on ne peut pas utiliser le même pseudo après un temps suffisant pour que l'utilisateur se "décourage" et en choisissent un autre. De même il est possible de créer des centaines de pseudos (connexion=> déconnexion, choix d'un autre pseudo=> deconnexion, etc ... etc ...) ce qui remet en cause l'intégrité du script et peut le faire planter pour tous les utilisateurs.
La commande déconnexion devrait donc déconnecter immédiatement l'utilisateur est rendre son pseudo disponible.
- je reste convaincu que le prompt est une mauvaise idée et qu'il vaudrait mieux que tu procèdes de la même manière que l'affichage de la liste des salles disponibles. Ainsi il n'y aura pas la nécessité d'autoriser une action (IE), qui, comme je l'ai dit, ne sera pas effectué par de nombreuses personnes qui prendront peur au message d'alerte.
Par ailleurs la terminologie "joindre" n'est pas adaptée, je verrai plutôt "créer une nouvelle salle"
- concernant l'insertion de HTML :
lors de mon dernier message je t'avais alerté sur la possibilité (entres autres) de créer un div de height monstrueux qui posé pb en terme de lisibilité pour les autres utilisateurs (limite un deface) Ce n'est pas pour autant que l'insertion de HTML est une mauvaise chose, bien au contraire.
Pourquoi ne pas utiliser un script comme sur le forum CS ?
- je crois qu'implémenter un système anti-spam serait une bonne idée, pour interdire, par exemple, les appuis successifs sans messages sur la touche entrée


Tu penses peut être que je cherche la petite bête avec certaines de mes remarques, mais ce n'est pas le cas. La plupart des utilisateurs ne connaissent rien à la programmation au sens large du terme (paramètre incorrect, etc ..) Il leurs faut des messages clairs et des gestions d'erreurs efficaces ainsi que des limites pour les petits rigolos.

Je voulais regarder un peu le code et du coup, avec ces remarques purement utilisateur je n'ai pas eu le temps.
Ton script semble très intéressant en tous cas.

Cordialement,


Kohntark-

Commentaire de Arto_8000 le 07/05/2009 22:08:49

Afyn -> Le taux de rafraichissement est assez bas. Il est à par défaut, un rafraichissement au 6 secondes. Tu attends donc en moyenne 3 secondes pour ton message. Si tu veux l'augmenter, le paramètre se trouve à la ligne 9 de AppChat.js

Je vais surement le monter un peu surtout qu'avec le démo en ligne je vois que le traffic généré est assez petit et que le serveur peut en prendre beaucoup plus que ça.

kohntark -> Le problème d'encodage c'est avec l'affichage de la page ou du contenu qui est reçu par Ajax ?

Commentaire de kohntark le 08/05/2009 00:05:45

Je pensais l'avoir précisé, au temps pour moi. C'est avec l'affichage, le contenu envoyé / reçu par Ajax ne pose pas de pb.

Cordialement,


Kohntark-

Commentaire de Arto_8000 le 08/05/2009 00:25:10

Bon, j'ai fait une mise à jour du démo en ligne. J'ai modifié quelques petit truc pour règler l'encodage. Tu me diras si tu as toujours le problème avec le patch que j'ai fait.

Commentaire de Arto_8000 le 08/05/2009 18:06:57

Pour le charset, j'ai tout est passé en UTF-8, ça à l'air de bien fonctionner de mon côté.

Pour l'update que je viens de faire voici quelques détails :

- Avant les script js prennait le 2/3 de la bandwith du chat. J'ai rajouté une version compressé pour que ça utilise moins de bandwith.

- Le temps de rafraichissement est passé à 3 secondes (6 secondes avant)

- Et le problème de charset devrait être réglé (?)

Commentaire de Arto_8000 le 01/06/2009 17:23:00

Les messages privés sont déjà supportés. Pour initier une conversation, il faut faire /privmsg nomdelapersonne message. Un onglet supplémentaire avec comme nom ~nomdelapersonne va s'ouvrir et vous aller pouvoir avoir une conversation privée dans cet onglet.

Commentaire de gigamaster le 15/06/2009 22:08:49

P/ suivre !

Félicitations ^_^

Commentaire de dimoala1 le 02/08/2009 18:30:56

Marche pas :( quand tu apuy sur connecter il se passe rien , même sur son site ! j'ai testé sur deux pc et c pareil .

dommage

Commentaire de Arto_8000 le 02/08/2009 22:18:28

Il y a eu une mise à jour sur le serveur de la version de PHP qui est passé à 5.3.0, ce qui a causé un petit problème de compatibilité.

Le problème est corrigé et c'est maintenant compatible avec la version 5.3 de PHP.

Commentaire de Arto_8000 le 16/10/2009 14:50:16

As-tu installé les tables SQL et configuré le fichier configuration.ini avec tes paramètres de connexion ?