Commentaires de RCA ArKanis sur tout CodeS-SourceS

404 commentaire(s) de RCA ArKanis sur des sources sur tout CodeS-SourceS

Le : 02/07/2008 02:27:00
Source : CORRECTEUR ORTHOGRAPHIQUE, MODÉRATION, JEUX ET AUTRES... :)

mets les fichiers essentiels (en gros : sans les sons, images, .exe, ...)
c'est obligatoire si tu veux garder la source ici.

Le : 30/06/2008 19:36:14
Source : WAUTOIDENTIFYGHOST V1.0 - UN AUTO AUTH ET AUTO GHOST MULTI SERVEUR EN DIALOG

Sans les fautes d'orthographe, ça serait largement mieux ^^

Je ne vois vraiment pas pourquoi tu me contactes en PV pour que je commente un code qui ne m'intéresse pas trop à la base. Si c'est pour avoir mon avis sur une source que je ne trouve pas très utile pour moi, j'vois pas l'intérêt :/

Le scripting mIRC ne m'intéresse plus, il me semble de l'avoir déjà dit.

Bon bah, je vais commencer vite fait le peu que j'ai effectivement envie de commenter :
on *:dialog:deswig:sclick:66: unload -rs AutoID/AutoID.txt
ça me fait très peu quand je vois ça. Qui te dit que ce fichier existe, qu'il est placé là et qu'il est chargé ?

    var %l = $lines(AutoID/nickname.txt)
    var %l > 0
    var %i = 0

tu t'expliques ? Surtout le > 0 en fait.

    while %i < %l {
      inc %i
      if (%nick == $gettok($read(AutoID/nickname.txt,%i),1-,46)) {
        ...
        return
      }
        ...
      return
    }

tu expliques là aussi à quoi sert la boucle si dans tous les cas tu l'arrêtes dès la première fois ?

"Veuillez mettre un pseudo que vous utilisez pas pour personne peut avoir (aléatoire)  par exp: ioijoijo, qgfdgf ou autres"
Pourquoi ça ne serait pas toi qui coderait ça ?

J'ai pas trop le temps de faire plus, mais j'suis sûr qu'il y a encore une tonne de choses à dire ;)

Le : 28/06/2008 13:41:59
Source : BIOTECH [V1.0]

faudrait quand même que tu expliques un minimum comment ta chose s'installe ...
que faut-il charger ? où mettre le mirc.exe ? bref, là on n'a aucune indication !

Le : 21/06/2008 10:53:52
Source : PACMAN ONLINE - 4 JOUEURS

vu que tu es dans la phase "correction des erreurs d'orthographe/grammaire", corrige moi ce titre en rajoutant un S à joueur ... "4 joueur", ça ne le fait vraiment pas
bon après, je passe les 20 fautes de la description :p

Le : 19/06/2008 17:43:58
Source : PACMAN ONLINE - 4 JOUEURS

Ah, ça fait plaisir de voir ce code finalement sorti :)
On y a passé tellement d'heures à le debugguer/tester que bon, s'il sortait pas ça aurait été vraiment dommage :(

Le : 22/05/2008 11:01:29
Source : SALON DU CONTROLE

oui IRCLAND. L'orthographe renvoit une image sur la qualité du code qui est derrière, et c'est ça que vous comprenez pas. S'il ne prend pas soin de son orthographe, il a de fortes chances de ne pas prendre soin de son code.

Un programme bourré de fautes d'orthographe ne peut PAS être un bon programme.

Depuis quand ça serait à une personne complètement extérieure au code de le corriger ?

Le : 21/05/2008 13:00:06
Source : SALON DU CONTROLE

Comme l'a dit PaDa, c'est blindé de fautes d'ortho ...
Quand je vois ça, j'ai même pas envie d'en savoir plus sur la source ou de savoir à quoi elle sert.

Le : 15/05/2008 14:44:41
Source : WANTISPAM V3.0

(au temps :p)
comme quoi, cette syntaxe est loin d'être la plus claire ! à éviter au maximum donc

Le : 15/05/2008 10:35:14
Source : WANTISPAM V3.0

PaDa, ce que tu dis est vrai s'il y avait des { }, mais ce n'est pas le cas
dans un IF, si tu n'as pas de { }, il ne fait que la 1ere action
//if (1 == 2) echo -a k1 | echo -a k2
affichera k2 dans tous les cas
si maintenant à la place du k1 tu as return, le k2 ne s'affichera que si la condition est fausse.

ce qu'il tenait à faire, c'est ça :
if (($2 isop $1) || ($readini(protect_user.ini,prot,$nick)) || ($wildtok($3-,#*,1,32) == $1) || (!$regex($3-,/(\#\w+\b|(?:[-_\.]\w+)?@\w+([-_\.]?\w+)?\.\w+\b|(www\.|http:\/\/)\w+([-_\.]?\w+)?\.\w{2,4})/i))) return
ban -k $1-2 2 4Spam Protection 1by WorldDMT 14 trouve moi en pv en cas d'erreur

et il s'est dit (truc stupide, certes), qu'il peut tout mettre en une ligne en rajoutant |. Ca fonctionne, mais c'est laid

Le : 14/05/2008 21:56:49
Source : WANTISPAM V3.0

le but, c'est de ne pas avoir de | (c'est super moche et ça ralenti inutilement le code !!!)
ne pas non plus abuser des lignes super longues
le regex est déjà assez long en lui même pour que tu puisses lui attribuer une ligne à lui tout seul

les fautes d'orthographe que je t'ai données sont bien entendu corrigées ... à toi de les trouver dans ta description qui fait partie de la source.

le unload peut lui aussi tenir en une ligne :
on *:unload:echo * Merci d'avoir utiliser WorldDMT Anti Spam (Wantispam V3.0)
(en corrigeant cette horrible faute au passage)

tu ne perds pas tellement en clarté en plus.