Advertisement
ozh

PSR

ozh
Jan 10th, 2014
74
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 2.14 KB | None | 0 0
  1. # PSR-0
  2. # https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-0.md
  3.  
  4. 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.
  5.  
  6. 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:
  7. * \Doctrine\Common\IsolatedClassLoader :non
  8. * \Doctrine\Common\Isolated_Class_Loader : à la rigueur.
  9. Le CamelCaseJeTrouveQueCestDeLaMerdeIllisible et là dessus je serai inflexible
  10.  
  11. # PSR-1
  12. # https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-coding-standard.md
  13.  
  14. 2.1 : ridicule. Long tag ou rien. `<?=` peut être complètement désactivé sur un serveur.
  15.  
  16. 2.2 : déjà le cas
  17.  
  18. 2.3 : tout à fait
  19.  
  20. 3 : cf remarque au dessus
  21.  
  22. 4.1 : déjà le cas
  23.  
  24. 4.2 : tant mieux parce que snake_case de toute façon
  25.  
  26. 4.3 : hors de question. snake_case là encore.
  27.  
  28. # PSR 2
  29. # https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md
  30.  
  31. 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.
  32.  
  33. 2.3 : déjà le cas
  34.  
  35. 2.4 : cas spécial détaillé sur github
  36.  
  37. 2.5 : déjà le cas
  38.  
  39. 3 : OK
  40.  
  41. 4.1 : OK
  42.  
  43. 4.2 : OK
  44.  
  45. 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.
  46. On reste sur l'usage des white space comme aujourd'hui :
  47. ```
  48. <?php
  49. namespace Vendor\Package;
  50.  
  51. class ClassName {
  52.  
  53. public function fooBarBaz( $arg1, &$arg2, $arg3 = [] ) {
  54. // method body
  55. }
  56.  
  57. }
  58. ```
  59.  
  60. 4.4 : OK + même remarque que 4.3
  61.  
  62. 4.5 : OK
  63.  
  64. 4.6 : Non, cf 4.3.
  65.  
  66. 5.1 : déjà le cas sauf que plus de white space, cf 4.3
  67.  
  68. 5.2 : idem 5.1
  69.  
  70. 5.3 : idem 5.1
  71.  
  72. 5.4 : idem 5.1
  73.  
  74. 5.5 : idem 5.1
  75.  
  76. 5.6 : idem 5.1
  77.  
  78. 6 : idem 5.1
  79.  
  80.  
  81. # PSR-3 : OSEF
  82.  
  83.  
  84. # 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