Commentaires de MaX_62 sur tout CodeS-SourceS

317 commentaire(s) de MaX_62 sur des sources sur tout CodeS-SourceS

Le : 29/06/2007 11:09:00
Source : ASCII

Idem, je suis pas fan du dialog, une picture window aurait pu être une bonne alternative ;)

Mais bon, on va pas te faire tout recommencer ^^

Par contre j'ai pas bien pigé l'utilité, ça sert à faire quoi grosso modo ?

Le : 29/06/2007 11:04:44
Source : SOLVEUR DE SUDOKU

Un jeu d'échecs ? Super intéressant ça ! J'espère que tu pourras le sortir un jour ;-)

C'est un 1 vs 1 ou c'est contre l'ordinateur ? Parce que contre l'ordinateur ça te prendrait des dixaines de milliers de lignes pour faire un truc performant ^^

Perso je vais essayer de continuer le plus possible mon projet, et j'updaterai lorsqu'il y aura de gros changements !

Le : 08/07/2005 12:01:18
Source : STATISTIQUES PERSONNELLES

essaie d'optimiser ton code a certains endroits, des fois tu effectue la meme commande dans chaque condition, place une commande unique en dehors de tes conditions...
j'ai pas vu d'alias $b dans le code, ajoute le...
pour tous les conditions :
if $did(3).state = 1
tu peux mettre
if $did(3).state c'est suffisant
au lieu de créer ta table avec /hmake tu peux ajouter avec /hadd -m meme si la table n'existe pas...

au lieu de mettre plein d'évènements du genre OP, DEOP, VOICE, etc... récupère le tout avec RAWMODE, c'est plus pratique ;)

Voila..

Le : 04/07/2005 17:59:46
Source : 3 SCANNEURS DIFFÉRENTS

C'est vrai que c'est beaucoup mieux. Cependant, évites d'associer les boucles /goto aux variables globales... utilises une boucle /while avec des variables locales (/var) !
Les points devant /did sont inutiles...

Le : 04/07/2005 17:47:54
Source : POST-IT

Bof, bof, très basique et mal codé... quelques idée d'optimisations :
L'alias /post est très "bizarre", mets plutot ça :
alias post dialog -m post post
car la tu /set une variable inutilement

Je pense pas qu'une variable ait une très grande capacité, je pense qu'il vaut mieux tout stocker dans un fichier.txt (et après le recopier éventuellement avec le file handling)

Regroupe tes évènements de dialog :
on *:DIALOG:post:*:*:{
  if ($devent == init) { ... }
  elseif ($devent == sclick) {
    if ($did == 1) { ... }
    elseif ($did == 2) { ... }
  }
}

Les boucles avec /goto, beurk :( utilises les /while et les variables locales !

Je trouve que le dialog est pas très beau :s

Améliores tout ça car c'est vraiment pas terrible... bonne continuation

Le : 02/07/2005 08:07:03
Source : 3 SCANNEURS DIFFÉRENTS

franchement ton alias $tab sert pas a grand chose... tu n'as qu'a mettre :
$+(Pseudos,$chr(9),Infos) au lieu de $tab(Pseudos,Infos) ...
tu nous oblige a mettre mdx dans le répertoire Dlls, et tu supposes également que views.mdx est dans la racine du script ><

Sinon c'est très basique, et ça sert pas a grand chose je trouve... essaie d'intégrer quelques options, comme c'est dit plus haut ^

Le : 30/06/2005 08:10:34
Source : JEU DES IDENTIFIEURS

J'ai déjà eu l'occasion de tester, je trouve que c'est assez sympa a jouer :o)
Au niveau du code, quelques détails :
unset %nick
unset %pasjoueur
unset %partage
>>
unset %nick %pasjoueur %partage

T'aurais pu choisir des noms de variables un peu plus explicites, des variables comme %jeu peuvent tout a fait etre déjà occupées dans un script :(

Utiliser 3-4 hash tables pour une source de ce type, je trouve pas que ce soit super :s T'aurais pu tout regrouper dans la même

Tu peux mettre tes alias en local non ?

Globalement j'ai assez apprécié ce jeu même si il restait quelques bugs, j'espère que t'as tout corrigé maintenant, j'ai pas retesté :)

Le : 30/06/2005 08:02:57
Source : JEU : DÉMINEUR

J'avais commencé une source de ce type, faudra que je la termine un jour :o)
L'aspect visuel est pas très beau ni soigné, dommage pour une picwin... (genre le point dessiné pour une mine, c moche)

Faudrait pourvoir configurer la couleur des drapeaux (avec le clic droit) car ils sont tout le temps rouges, avec le fond rouge on les voit pas :D

Normalement au démineur, la première case sur laquelle on clique est forcément bonne, donc la grille est calculée après le premier clic...

J'ai vu que tes calculs sont relativement rapides par rapport a ce que j'ai pu faire, mais ça a quand même réussi a faire geler mIRC quelques secondes avec une grande grille et peu de mines...
Pense au hash tables, c'est plus rapide... :)

Ca m'a remotivé pour ma source, on devrait la voir ici bientot :)

Le : 28/06/2005 07:53:13
Source : WEB-BROWSER V1.0 BY DARKWYD

Moi j'aime bien... Tout fonctionne normalement chez moi, à l'exception de la recherche, il me dit de spécifier les options de recherche.
Quitte a utiliser les picwins, utilises les dans tout l'addon (c'est pas bien dur avec un gui tout fait...) ; ça t'éviteras d'utiliser MDX parce que 8 dlls pour un addon ça fait beaucoup...

Bel addon mais je note pas car c'est quand même un peu facile avec les GUI, et trop de dlls sont utilisées...

Le : 27/06/2005 18:20:07
Source : SNIPPET /NAMES

La variable globale ? %a, puisque tu l'as déclarée avec /set, et pas /var...
le temps d'éxécution ? bah oui, ton script est très, très lent...