CLASSE PAGINATION
testabc
Messages postés
33
Date d'inscription
jeudi 17 juillet 2008
Statut
Membre
Dernière intervention
23 juillet 2014
-
21 sept. 2009 à 10:23
tyranausor Messages postés 3545 Date d'inscription jeudi 6 août 2009 Statut Membre Dernière intervention 1 avril 2022 - 27 mars 2019 à 21:21
tyranausor Messages postés 3545 Date d'inscription jeudi 6 août 2009 Statut Membre Derniè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.
https://codes-sources.commentcamarche.net/source/50579-classe-pagination
https://codes-sources.commentcamarche.net/source/50579-classe-pagination
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