MESSAGES VIRTUELS

Messages postés
838
Date d'inscription
samedi 15 mai 1999
Statut
Modérateur
Dernière intervention
18 juillet 2009
- - Dernière réponse : flo39400
Messages postés
594
Date d'inscription
mardi 8 avril 2008
Statut
Membre
Dernière intervention
11 mai 2018
- 30 mai 2017 à 21:31
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/49888-messages-virtuels

Afficher la suite 
kankrelune
Messages postés
1305
Date d'inscription
mardi 9 novembre 2004
Statut
Membre
Dernière intervention
21 mai 2015
-
J'ai pas testé le code et je ne l'ai regardé que superficiellement mais...

function OuvertureFiche(){
Ouverture(i=e)
}

"i" et "e" sont indéfini ou alors ce sont des variable globales et dans ce cas c'est une mauvais idée de les appeler "i" et "e"

Sinon ça a l'air pas trop mal pensé mais n'ayant pas regardé le code en détail ma critique s'arrête là... par contre porter le code en POO serait à mon avis une bonne chose ça serait plus simple, clair et logique de travailler à partir d'un objet plutôt qu'a partir de multiples fonctions... .. .

@ tchaOo°
mickaelpfr
Messages postés
197
Date d'inscription
mardi 18 février 2003
Statut
Membre
Dernière intervention
29 avril 2009
2 -
De mon coté je viens de regarder le code en détail :

- code non nétoyé /!\ il reste au moins 3 fonctions vide !!
- beaucoup de style en dur dans le code
- pourquoi charger des xml ??????? (en plus directement en page ... )
- POO ? non toujours pas ??
- popup modale ?

voila, en attendant une version revue a la hausse ;)
grarestephane
Messages postés
34
Date d'inscription
mercredi 12 décembre 2007
Statut
Membre
Dernière intervention
6 décembre 2009
-
OK ok c'est sûrement pas parfait car à titre d'infos, je suis qu'un amateur et en plus j'y connais pas grand chose mais d'en pêche, que je cherchais un script dans le genre sur internet et que j'ai rien trouvé et que maintenant ben si quelqu'un cherche un truc dans ce genre, ben il trouvera. Donc c'est déjà bien. Ensuite chacun peut le modifier à sa guise et selon ce qu'il veut en faire. Moi ça me convient très bien comme il est car il fonctionne sous IE et que je ne suis que sur IE. D'ailleurs tout le monde devrait être que sur IE.
mickaelpfr
Messages postés
197
Date d'inscription
mardi 18 février 2003
Statut
Membre
Dernière intervention
29 avril 2009
2 -
"Moi ça me convient très bien comme il est car il fonctionne sous IE et que je ne suis que sur IE. D'ailleurs tout le monde devrait être que sur IE."

rien qu'avec ça tu viens de clore le débat !

bien le bonjour a toi :)
lakichemole
Messages postés
253
Date d'inscription
vendredi 13 juin 2003
Statut
Membre
Dernière intervention
18 mai 2009
-
Moi je met la moitié de la note car c'est vrai on en a pas vu beaucoup des sources comme celle là :) après je peux pas monter plus haut, pour els raison de mes cher conpatriotes (voir ci dessus) et vu le positionement ultra fermé de l'auteur "débrouillez vous avec ça MOI ça me va alors faites pas chi..." :).