BOT IRC

Commentaire de krust le 02/11/2006 12:20:06

Pour un premier code c'est plustot pas mal. J'ai pas tout regardé avec attention mais si je peux me permettre quelques commentaires :

- Utilise le moins possible les variables globales, essaie plustot de guarder des locales que tu passes en argument (Socket, buffers). Ca rendera tes fonctions réutilisables en plus par la suite et si jamais tu passes au multi server tu devras pas tout recoder :)
- Dans read_funct : Pourquoi le void* (Question anodine)?;
   : Dans ta boucle, quand tu fais ton recv, il faut attendre d'avoir une commande entière avant d'essayer de l'interpreter. Je m'explique, si jamais tu reçois une commande coupé en plusieurs packet, ou deux commandes en un packet, ta fonction ne fonctionnera pas (En fait tout dépend de l'état de ton buffer socket). Le mieux c'est de lire byte par byte jusqu'à arriver à un \r\n qui indique la fin de la commande puis après l'interpreter !


Ciao !

Krust.

Commentaire de keikoz le 14/11/2006 04:47:23

Merci bien pour vos commentaires.

Concernant les variables globales, au début je voulais n'en utiliser aucune (on m'avait dit que c'était mal) mais je galérais pas mal par rapport au fait que j'utilise les threads pour avoir à la fois la possibilité d'écrire dans le socket et d'y lire (sinon, ça bloque, soit sur la lecture, soit sur l'écriture).
Précisément concernant les threads? Existe-t-il un moyen de s'en passer, d'autres méthodes qui permettent que le bot soit à la fois à l'écoute de messages provenant de sockets mais aussi d'un input à écrire dans le socket?).

Commentaire de krust le 14/11/2006 21:27:52

Je me souvient plus du code exactement mais c'est possible de faire un bot monothread. D'un autre coté, il envoyait que des messages lorsqu'il en recevait l'ordre par irc (si vous me suivez toujours).

Mais il est vrai que 2 threads semblent la solution la plus pratique dans ton cas.