testabc
Messages postés33Date d'inscriptionjeudi 17 juillet 2008StatutMembreDernière intervention23 juillet 2014
-
21 sept. 2009 à 10:23
tyranausor
Messages postés3545Date d'inscriptionjeudi 6 août 2009StatutMembreDernière intervention 1 avril 2022
-
27 mars 2019 à 21:21
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
tyranausor
Messages postés3545Date d'inscriptionjeudi 6 août 2009StatutMembreDernière intervention 1 avril 20221 27 mars 2019 à 21:21
Bonjour, il manque quelques explication sur tes fonctions (au moins les documenter) et sur les variables. De plus, je ne comprends pas grand chose à ton exemple
cs_darkis
Messages postés1Date d'inscriptionsamedi 27 mai 2006StatutMembreDernière intervention12 octobre 2009 12 oct. 2009 à 16:00
J'ai mis un peu de temps avant de r?pondre et donner des explications d?sole,
Et pour voila pour pgassie pour etre honnete avec vous je suis la pour prendre et pour corriger mes fautes,
Merci pour tes conseils
pgassie
Messages postés12Date d'inscriptionmardi 28 octobre 2003StatutMembreDernière intervention13 décembre 2006 12 oct. 2009 à 02:32
Salut darkis,
Je n'ai pas l'habitude de me gausser.
Mais là, c'est tendre des verges pour se faire battre.
"phpcs.com" n'est pas un site pour spécialistes abscons ( ce n'est pas une injure ), mais de jeunes qui veulent apprendre par l'exemple.
Les cuistres n'y sont pas les bienvenus.
La seule chose à retenir de ce post est dans la présentation d'un code à l'usage de l'enseignant futile.
Cela te dérangerait-il beaucoup de donner une description courte de ce qu'est supposé faire ton code ?
Quelle en est l'entrée, qu'est-ce qu'on récupère en sortie ?
Lignes 12 à 22, tu déclares des attributs "public" rien que pour mettre dans le vent PHP4, la suite du code ne le justifie pas, et jusqu'à plus ample informé PHP5 considère un attribut comme public par défaut.
De la ligne 121 à 147, grâce à des sauts de ligne qui certes ne pèsent pas dans le fichier, mais qui ne rajoutent rien à la lecture ni sur l'écran ni sur papier, tu nous assassines de fonctions inutiles :
expliques-moi l'intérêt d'avoir "$page.nbPadges()" au lieu de "$page->nbPadges"
Si tu veux faire du POO (programmation orienté objet) pur et dur, pour chaque attribut X (private), tu fais un setX() et un getX(), si c'est utile. Si tu n'as pas de constructeur; au moins mets des valeurs par défaut (0,chaîne vide, array()).
Ta classe n'a qu'une seule fonction exploitable "generate(X,Y,Z)", vire ta classe; en ne gardant que la fonction "generate(X,Y,Z)", ça marche tout aussi bien pareil ; là dis donc!
Encore que...
On ne sait toujours pas à quoi ça sert !
Bravo les modérateurs qui mettent "initié" à ce code qui sert à... prochain épisode.
Bonjour chez toi.
testabc
Messages postés33Date d'inscriptionjeudi 17 juillet 2008StatutMembreDernière intervention23 juillet 2014 21 sept. 2009 à 18:52
le code est clair pour un dev
moi je suis graphiste mais j'apprends vite
sauf que les classes ca reste encore abstrait...
j'ai besoin d'exemple(s) concret pour comprendre !
echo 'merci'
cs_hornetbzz
Messages postés59Date d'inscriptionlundi 1 décembre 2008StatutMembreDernière intervention 3 janvier 2011 21 sept. 2009 à 15:44
Je n'ai pas testé mais le code est clair et compréhensible.
Dommage, je viens juste de passer la nuit à faire la même chose !
Justement, la comparaison me fait penser que tu pourrais gérer un incrément/décrément variable, dépendant du nbre total de pages. C'est plus sympa de naviguer par bloc de 50 pages s'il y en a 500 au total :-)
Et encore plus sympa, ça serait bien de balancer 2 autres classes, par exemple des objets et une mise en page html, sinon l'intérêt d'avoir une classe pagination seule est mineur.
Merci donc.
testabc
Messages postés33Date d'inscriptionjeudi 17 juillet 2008StatutMembreDernière intervention23 juillet 2014 21 sept. 2009 à 10:23
C'est exactement ce que je cherchais...
Cependant, vu mon faible niveau, j'aurais besoin pour comprendre
d'un exemple avec une requete sql
et un peu d'html pour le rendu.
Ensuite je pourrais te donner mes retours !
Merci darkis
27 mars 2019 à 21:21
12 oct. 2009 à 16:00
Et pour voila pour pgassie pour etre honnete avec vous je suis la pour prendre et pour corriger mes fautes,
Merci pour tes conseils
12 oct. 2009 à 02:32
Je n'ai pas l'habitude de me gausser.
Mais là, c'est tendre des verges pour se faire battre.
"phpcs.com" n'est pas un site pour spécialistes abscons ( ce n'est pas une injure ), mais de jeunes qui veulent apprendre par l'exemple.
Les cuistres n'y sont pas les bienvenus.
La seule chose à retenir de ce post est dans la présentation d'un code à l'usage de l'enseignant futile.
Cela te dérangerait-il beaucoup de donner une description courte de ce qu'est supposé faire ton code ?
Quelle en est l'entrée, qu'est-ce qu'on récupère en sortie ?
Lignes 12 à 22, tu déclares des attributs "public" rien que pour mettre dans le vent PHP4, la suite du code ne le justifie pas, et jusqu'à plus ample informé PHP5 considère un attribut comme public par défaut.
De la ligne 121 à 147, grâce à des sauts de ligne qui certes ne pèsent pas dans le fichier, mais qui ne rajoutent rien à la lecture ni sur l'écran ni sur papier, tu nous assassines de fonctions inutiles :
expliques-moi l'intérêt d'avoir "$page.nbPadges()" au lieu de "$page->nbPadges"
Si tu veux faire du POO (programmation orienté objet) pur et dur, pour chaque attribut X (private), tu fais un setX() et un getX(), si c'est utile. Si tu n'as pas de constructeur; au moins mets des valeurs par défaut (0,chaîne vide, array()).
Ta classe n'a qu'une seule fonction exploitable "generate(X,Y,Z)", vire ta classe; en ne gardant que la fonction "generate(X,Y,Z)", ça marche tout aussi bien pareil ; là dis donc!
Encore que...
On ne sait toujours pas à quoi ça sert !
Bravo les modérateurs qui mettent "initié" à ce code qui sert à... prochain épisode.
Bonjour chez toi.
21 sept. 2009 à 18:52
moi je suis graphiste mais j'apprends vite
sauf que les classes ca reste encore abstrait...
j'ai besoin d'exemple(s) concret pour comprendre !
echo 'merci'
21 sept. 2009 à 15:44
Dommage, je viens juste de passer la nuit à faire la même chose !
Justement, la comparaison me fait penser que tu pourrais gérer un incrément/décrément variable, dépendant du nbre total de pages. C'est plus sympa de naviguer par bloc de 50 pages s'il y en a 500 au total :-)
Et encore plus sympa, ça serait bien de balancer 2 autres classes, par exemple des objets et une mise en page html, sinon l'intérêt d'avoir une classe pagination seule est mineur.
Merci donc.
21 sept. 2009 à 10:23
Cependant, vu mon faible niveau, j'aurais besoin pour comprendre
d'un exemple avec une requete sql
et un peu d'html pour le rendu.
Ensuite je pourrais te donner mes retours !
Merci darkis