ORGANIGRAMME DYNAMIQUE

Commentaire de malalam le 02/10/2007 08:48:33 administrateur CS

Hello,

je n'ai pas testé, j'ai juste regardé un peu le code.
Il y a des trucs qui chagrinent quand même :-)
Côté PHP, déjà, vu le temps que tu as dû passer là-dessus, tu aurais gagné à le développer en objets. Et nous, nous aurions gagné en visibilité, lisibilité et surtout portabilité. Si c'est déjà dommage pour la structure de base, mais bon, tout le monde n'a pas les compétences pour monter une architecture orientée objet, ça l'est plus encore pour la base de données. Si quelqu'un n'utilise pas mysql, il va galérer pour basculer ton code sur son serveur de bdd; alors qu'il aurait été très facile d'utiliser une couche d'abstraction de bdd là-dessus.
Tes comparaisons sont laxistes, c'est dommage et parfois génant.
Il n'y a aucune gestion d'erreurs. Tu récupères et utilise des variables utilisateur sans vérifier leur existence ni leur validité.
Dans sa globalité, le code est fouilli, on ne s'y retrouve pas; et ce n'est quasiment que du séquentiel. Non vraiment, c'est le bordel ;-) Et pas que pour le PHP : un coup tu utilises des classes html, un coup l'attribut style. Un coup tu tentes des observateurs d'évènements via JS et Prototype, un coup tu balances des onmouseover et compagnie directement dans tes éléments html.
Et c'est l'autre côté de ma critique : tu utilises prototype (en fait, j'ai dans l'idée que tu utilises surtout scriptaculous et connais mal prototype), mais ne t'en sers qu'au quart, et encore. Tu mélanges les appels "prototypiens" via des Event.observe, des $('truc') etc, et des appels vieux javascript du genre document.forms['truc'].options...etc.
Ca fait vraiment peu soigneux...et en fait, on sent le code monté à l'arrache parce que -sans doute- tu as appris bcp de choses au fil du développement.

Un conseil : tu as dû te prendre la tête sur ce code, ça se sent. Et c'est bien, c'est courageux :-) Réécris le maintenant, complètement, en mettant en application ce que tu as appris et en soignant vraiment le fond : utilises prototype à fond et ne laisse plus de javascript envahir ton code html. Appliques-toi à faire de la POO, dans les 2 langages (php et js). Et tu auras de quoi être encore plus fier d'avoir monté une jolie application web qui change des forums et autres compteurs... ;-)

Commentaire de malalam le 02/10/2007 16:45:50 administrateur CS

"Je ne m'étendrai pas sur l'utilité de parfois mettre le style en ligne et d'autrefois de l'intégrer dans une feuille liée, c'est les bases du DHTML..."
Si, je veux bien que tu t'étendes ? Je fais aussi du "dhtml" par la force des choses, j'utilise souvent prototype, et je n'ai jamais l'utilité de mettre mes styles directement en tant qu'attributs des éléments, SAUF pour le display:none; si c'est nécessaire.
Quant aux attributs onclick, onmouseover et cie, les attributs d'évènements, c'est du js invasif. Il est plus propre, plus agréable et facile à lire, de les sortir de là et d'attaquer directement le DOM. Et ça facilite la vie, en plus.
Je te parle de 2 choses là : la réutilisation de ton code par d'autres : une css est plus facile à modifier que les éléments répartis dans une 10aine de pages.
Idem pour les évènements : il est plus simple de les modifier s'ils sont hors de ton document html.
Et cela rend le tout plus lisible et facilite donc l'intégration dans d'autres systèmes pour les utilisateurs éventuels de ton package.
Tu échanges, alors essaye de rendre la tâche agréable aux personnes intéressées par cet échange.
Mais c'est mon avis, évidemment...

Je testerai ton truc, plus tard.

Commentaire de kylekatarnls le 06/10/2007 11:25:30

Les fichiers Thumbs.db prennent de la place inutilement, il ne servent qu'à présenter le dossier sous windows.

Commentaire de vduffaut le 08/10/2007 21:54:31

Moi je trouve l'interface très réussie car originale, même si la mise en forme ne me séduit pas... C'est vrai que c'est original. Il reste certaines imperfections visuelles sous Firefox et IE7 ? Suis-je le seul à avoir ces lignes discontinues et décalées ?

Je n'ai en revanche pas regardé le code (oh !)

Commentaire de Billybobbonnet le 15/10/2007 13:52:14

attention mon antivirus a détecté un virus dans l'archive

Commentaire de Billybobbonnet le 18/10/2007 18:12:29

oui c'était un bug avec le java, désolé pour l'alerte.