Hum... Je te trouve un peu dur. Comme je l'ai dit, ce n'est qu'un helper permettant de vérifier sa mise en page rapido. Pourquoi se prendre la tête à aller faire du texte aléatoire ? Je vois pas.
Le principe est bon, après le code beaucoup moins. Tu devrais générer un texte en fonction des paramètres et non retourner un truc plus ou moins fixe. Inspire toi donc des sites tels que http://lipsum.com/ ou http://cupcakeipsum.com/ qui, eux, propose des fonctionnalités bien plus utiles. À Toi de les égaler, voir de les dépasser ! Je met 4/10 parce que c'est un sujet assez peu traité sur phpcs, intéressant et utile, mais pas plus à cause de la qualité du code. Allez, épates nous avec une véritable génération de texte aléatoire :)
PS : Expliciter la visibilité des méthodes c'est bien, là tu ne l'as pas fait.
PPS : Le latin c'est un poil relou à force, générer du texte à thème c'est fun :p
- Messages postés
-
1309
- Date d'inscription
- samedi 31 janvier 2009
- Statut
- Membre
- Dernière intervention
- 5 juin 2013
12Le principe est bon, après le code beaucoup moins. Tu devrais générer un texte en fonction des paramètres et non retourner un truc plus ou moins fixe. Inspire toi donc des sites tels que http://lipsum.com/ ou http://cupcakeipsum.com/ qui, eux, propose des fonctionnalités bien plus utiles. À Toi de les égaler, voir de les dépasser ! Je met 4/10 parce que c'est un sujet assez peu traité sur phpcs, intéressant et utile, mais pas plus à cause de la qualité du code. Allez, épates nous avec une véritable génération de texte aléatoire :)
PS : Expliciter la visibilité des méthodes c'est bien, là tu ne l'as pas fait.
PPS : Le latin c'est un poil relou à force, générer du texte à thème c'est fun :p
très simple, mais bien pratique