Simple question sur l'indentation du code

Version imprimable

Bonjour.

Je me suis fais la réflexion comme quoi j'ai des lignes de code assez longues notamment au niveau des requêtes SQL et des array, et que ça ne rendait pas toujours mon code bien lisible.
J'ai lu quelque part, que l'on ne devait pas dépasser une certaine longueur pour une ligne de code (ce qui semble assez logique dans le fond pour que ça reste lisible).
D'où ma question : j'ai une requête SQL assez longue. Pour plus de lisibilité est-ce correcte de la découper comme ceci ? (même si ça ajoute plus de ligne au final):
Code:

1 2 3 4 5 6 7 8 9 $query = 'INSERT INTO DISSOLUTIONS'; $query. = '(nom, prenom, pseudo, adresse, numero_tel, adresse_mail, nombre_enfant, age, sexe, taille, poids, couleur_cheveux)'; $query. = 'VALUES(:nom, :prenom, :pseudo, :adresse, :numero_tel, :adresse_mail, :nombre_enfant, :age, :sexe,: taille, :poids, :couleur_cheveux)'; $requete = $pdo->prepare($query); //au lieu de $requete = $pdo->prepare('INSERT INTO BLEBLE (nom, prenom, pseudo, adresse, numero_tel, adresse_mail, nombre_enfant, age, sexe, taille, poids, couleur_cheveux) VALUES(:nom, :prenom, :pseudo, :adresse, :numero_tel, :adresse_mail, :nombre_enfant, :age, :sexe, :taille, :poids, :couleur_cheveux)');
Pareil pour un array :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 $return = $requeteDisso -> execute(array( ':nom'=>$personne['nom'], ':prenom'=>$personne['prenom'], ':pseudo'=>$personne['pseudo'], ':adresse'=>$personne['adresse'], ':numero_tel'=>$personne['numero_tel'], ':adresse_mail'=>$personne['adresse_mail'], ':nombre_enfant'=>$personne['nombre_enfant'], ':age'=>$personne['age'], ... )); //au lieu de $return = $requeteDisso -> execute(array(':nom'=>$personne['nom'], ':prenom'=>$personne['prenom'], ':pseudo'=>$personne['pseudo'], ':adresse'=>$personne['adresse'], ':numero_tel'=>$personne['numero_tel'], ':adresse_mail'=>$personne['adresse_mail'], ':nombre_enfant'=>$personne['nombre_enfant'], ':age'=>$personne['age'],...));
Désolé si la question est un peu bête mais je voudrais juste savoir si cette pratique est correct dans le cas d'une requête ou d'un tableau.

Merci.

Bonjour,

Pourquoi ne pas faire de simple retour à la ligne:

Code:

1
2
3
$query = 'INSERT INTO DISSOLUTIONS
   (nom, prenom, pseudo, adresse, numero_tel, adresse_mail, nombre_enfant, age, sexe, taille, poids, couleur_cheveux)
   VALUES(:nom, :prenom, :pseudo, :adresse, :numero_tel, :adresse_mail, :nombre_enfant, :age, :sexe,: taille, :poids, :couleur_cheveux)';

14/12/2015, 16h03
flavors

Je ne savais même pas que cela était possible.
Je pensais que le seul moyen était de passer par une concaténation.

Sinon cette indentation est bien celle à adopter ?
14/12/2015, 16h21
Kreepz

Personnellement je fais toujours comme ça pour mes requêtes SQL, et n'ai jamais eu de retour là dessus (pour le moment :roll: )
14/12/2015, 16h31
flavors

D'accord.
Et merci pour l'info concernant la concaténation.

Pour le deuxième, je separerais mieux le tableau :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
$return = $requeteDisso -> execute(
	array(
	   ':nom'=>$personne['nom'], 
	   ':prenom'=>$personne['prenom'], 
	   ':pseudo'=>$personne['pseudo'], 
	   ':adresse'=>$personne['adresse'], 
	   ':numero_tel'=>$personne['numero_tel'], 
	   ':adresse_mail'=>$personne['adresse_mail'],
	   ':nombre_enfant'=>$personne['nombre_enfant'], 
	   ':age'=>$personne['age']
	)
);

Salut,

il existe des tonnes de manières en php pour arriver à un résultat lisible :
Code:

1 2 3 4 5 6 $query = <<<'sql' INSERT INTO DISSOLUTIONS (nom, prenom, pseudo, adresse, numero_tel, adresse_mail, nombre_enfant, age, sexe, taille, poids, couleur_cheveux) VALUES (:nom, :prenom, :pseudo, :adresse, :numero_tel, :adresse_mail, :nombre_enfant, :age, :sexe, :taille, :poids, :couleur_cheveux) sql;
Regarde du côté du formatage des chaines de caractères par ici (notations heredoc et nowdoc)

Ensuite tu peux aussi réduire le risque d'erreur de frappe en minimisant au maximum la partie dupliquée :
Code:

1 2 $fld = ['nom', 'prenom', 'pseudo', 'adresse', 'numero_tel', 'adresse_mail', 'nombre_enfant', 'age', 'sexe', 'taille', 'poids', 'couleur_cheveux']; $sql = 'INSERT INTO DISSOLUTIONS ('.implode(', ', $fld).') VALUES ('.implode(', ', preg_filter('/^/', ':', $fld)).')';
Sans compter les bons conseils des collègues... Bref tu as l'embarras du choix on peut dire :)

15/12/2015, 08h28
flavors

Merci beaucoup pour vos conseils.
Au début la mise en forme du code n'était pas vraiment ma priorité, mais la plus il s’alourdit, plus j'ai du mal à m'y retrouver.
Je vais enfin pouvoir m'y retrouver à nouveau!

Bonne journée