Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- # PSR-0
- # https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-0.md
- On est pas vraiment concerné vu que YOURLS n'est pas une librairie qu'on va incorporer dans un projet pour l'utiliser, et ça n'est pas prêt de le devenir.
- Cette PSR peut cependant à terme s'appliquer à des classes écrites pour YOURLS, et je trouve la PSR acceptable avec plutôt du bon sens là dedans. A l'exception du CamelCase:
- * \Doctrine\Common\IsolatedClassLoader :non
- * \Doctrine\Common\Isolated_Class_Loader : à la rigueur.
- Le CamelCaseJeTrouveQueCestDeLaMerdeIllisible et là dessus je serai inflexible
- # PSR-1
- # https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-coding-standard.md
- 2.1 : ridicule. Long tag ou rien. `<?=` peut être complètement désactivé sur un serveur.
- 2.2 : déjà le cas
- 2.3 : tout à fait
- 3 : cf remarque au dessus
- 4.1 : déjà le cas
- 4.2 : tant mieux parce que snake_case de toute façon
- 4.3 : hors de question. snake_case là encore.
- # PSR 2
- # https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md
- 2.2 : rien à secouer du EOL (strictement aucun impact sur la manière dont est exécuté le code) + cf mon commentaire sur BitBucket. Déjà le cas pour la blank line et pas de closing tag à la fin.
- 2.3 : déjà le cas
- 2.4 : cas spécial détaillé sur github
- 2.5 : déjà le cas
- 3 : OK
- 4.1 : OK
- 4.2 : OK
- 4.3 : Non. Ca mélange des préconisations de code -- pour lesquelles je suis d'accord (déclarer la visibilité etc..) -- avec des préconisations de lisibilité qui ne me vont pas.
- On reste sur l'usage des white space comme aujourd'hui :
- ```
- <?php
- namespace Vendor\Package;
- class ClassName {
- public function fooBarBaz( $arg1, &$arg2, $arg3 = [] ) {
- // method body
- }
- }
- ```
- 4.4 : OK + même remarque que 4.3
- 4.5 : OK
- 4.6 : Non, cf 4.3.
- 5.1 : déjà le cas sauf que plus de white space, cf 4.3
- 5.2 : idem 5.1
- 5.3 : idem 5.1
- 5.4 : idem 5.1
- 5.5 : idem 5.1
- 5.6 : idem 5.1
- 6 : idem 5.1
- # PSR-3 : OSEF
- # PSR-4 : j'ai l'impression qu'on répète les mêmes choses.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement