1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
|
# Configurer son environnement
**TODO**
## Docker
L'image Docker contenant l'environnement de développement de FreshRSS est accessible via :
`$ docker pull marienfressinaud/freshrss`
Vous pouvez lire [le fichier README du dépôt dédié](https://github.com/FreshRSS/docker-freshrss).
# Architecture du projet
**TODO**
# Style de codage
Si vous désirez contribuer au code, il est important de respecter le style de codage suivant. Le code actuel ne le respecte pas entièrement mais il est de notre devoir à tous de le changer dès que l'occasion se présente.
Aucune nouvelle contribution ne respectant pas ces règles ne sera acceptée tant que les corrections nécessaires ne sont pas appliquées.
## Espaces, tabulations et autres caractères blancs
### Indentation
L'indentation du code doit être faite impérativement avec des tabulations.
### Alignement
Une fois l'indentation faite, il peut être nécessaire de faire un alignement pour simplifier la lecture. Dans ce cas, il faut utiliser les espaces.
```php
$resultat = une_fonction_avec_un_nom_long($param1, $param2,
$param3, $param4);
```
### Fin de ligne
Le caractère de fin de ligne doit être un saut de ligne (LF) qui est le caractère de fin de ligne des systèmes *NIX. Ce caractère ne doit pas être précédé par des caractères blanc.
Il est possible de vérifier la présence de caractères blancs en fin de ligne grâce à Git avec la commande suivante :
```bash
# commande à lancer avant l'ajout des fichiers dans l'index
git diff --check
# commande à lancer après l'ajout des fichiers dans l'index mais avant le commit
git diff --check --cached
```
### Fin de fichier
Chaque fichier doit se terminer par une ligne vide.
### Le cas de la virgule, du point et du point-virgule
Il n'y a pas d'espace avant ces caractères, il y en a un après.
### Le cas des opérateurs
Chaque opérateur est entouré d'espaces.
```php
if ($a == 10) {
// faire quelque chose
}
echo $a ? 1 : 0;
```
### Le cas des parenthèses
Il n'y a pas d'espaces entre des parenthèses. Il n'y a pas d'espaces avant une parenthèse ouvrante sauf si elle est précédée d'un mot-clé. Il n'y a pas d'espaces après une parenthèse fermante sauf si elle est suivie d'une accolade ouvrante.
```php
if ($a == 10) {
// faire quelque chose
}
if ((int)$a == 10) {
// faire quelque chose
}
```
### Le cas des fonctions chainées
Ce cas se présente le plus souvent en Javascript. Quand on a des fonctions chainées, des fonctions anonymes ainsi que des fonctions de rappels, il est très facile de se perdre. Dans ce cas là, on ajoute une indentation supplémentaire pour toute l'instruction et on revient au même niveau pour une instruction de même niveau.
```javascript
// Première instruction
shortcut.add(shortcuts.mark_read, function () {
//...
}, {
'disable_in_input': true
});
// Deuxième instruction
shortcut.add("shift+" + shortcuts.mark_read, function () {
//...
}, {
'disable_in_input': true
});
```
## Longueur des lignes
Les lignes ne doivent pas dépasser 80 caractères. Il est cependant autorisé exceptionnellement de dépasser cette limite s'il n'est pas possible de la respecter mais en aucun cas, les lignes ne doivent dépasser les 100 caractères.
Dans le cas des fonctions, les paramètres peuvent être déclarés sur plusieurs lignes.
```php
function ma_fonction($param_1, $param_2,
$param_3, $param_4) {
// faire quelque chose
}
```
## Nommage
L'ensemble des éléments du code (fonctions, classes, méthodes et variables) doivent être nommés de manière à décrire leur usage de façon concise.
### Fonctions et variables
Les fonctions et les variables doivent suivre la convention "snake case".
```php
// une fontion
function nom_de_la_fontion() {
// faire quelque chose
}
// une variable
$nom_de_la_variable;
```
### Méthodes
Les méthodes doivent suivre la convention "lower camel case".
```php
private function nomDeLaMethode() {
// faire quelque chose
}
```
### Classes
Les classes doivent suivre la convention "upper camel case".
```php
abstract class NomDeLaClasse {}
```
## Encodage
Les fichiers doivent être encodés en UTF-8.
## Compatibilité avec PHP 5.3
Il ne faut pas demander l'indice d'un tableau qui est retourné par une fonction ou une méthode. Il faut passer par une variable intermédiaire.
```php
// code compatible avec PHP 5.3
$ma_variable = fonction_qui_retourne_un_tableau();
echo $ma_variable[0];
// code incompatible avec PHP 5.3
echo fonction_qui_retourne_un_tableau()[0];
```
Il ne faut pas utiliser la déclaration raccourcie des tableaux.
```php
// code compatible avec PHP 5.3
$variable = array();
// code incompatible avec PHP 5.3
$variable = [];
```
## Divers
### Opérateurs
Les opérateurs doivent être en fin de ligne dans le cas de conditions sur plusieurs lignes.
```php
if ($a == 10 ||
$a == 20) {
// faire quelque chose
}
```
### Fin des fichiers
Si le fichier ne contient que du PHP, il ne doit pas comporter de balise fermante
### Tableaux
Lors de l'écriture de tableaux sur plusieurs lignes, tous les éléments doivent être suivis d'une virgule (même le dernier).
```php
$variable = array(
"valeur 1",
"valeur 2",
"valeur 3",
);
```
|
