Discussion :

[julien731]

Bonjour à tous,

Je n'ai pas de problème, juste besoin d'un conseil !

J'ai dans ma bdd des tables à ralonge (une vingtaine de champs). Lorsque j'enregistre une nouvelle ligne ou que j'en met une à jour avec PHP, j'ai donc une ligne très longue !

Ce qui donne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

mysql_query('INSERT INTO table VALUES("val1", "val2", "val3", "val4"....)');

Et c'est évidemment la même chose pour les mysql update.

Cette méthode me pose deux problèmes:

• les lignes sont longues et donc une source d'erreur
• très peu pratique lors de l'évolution d'une table

J'ai donc utilisé à certains moment une autre méthode. Au lieu de mettre la liste exhaustive de mes champs à chaque enregistrement, je donne à mes champs de formulaire (HTML) le même nom que le nom de mes champs en bdd et je fais une boucle pour le mysql_query.

Ce qui donne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
foreach($_POST as $key => $value) {
   mysql_query('UPDATE table SET $key="'.$value.'" WHERE id="1"');
}

Ou encore:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
foreach($_POST as $value) {
   mysql_query('INSERT INTO table VALUES($value)');
}

Cependant je ne suis pas sûr que cette solution soit très "propre" ni performante. J'aurais donc souhaité avoir vos avis, conseils et astuces concernant cette problématique.

Merci,
Julien

[julien731]

C'est en écrivant mon post que j'ai finalement pensé à construire ma chaîne en amont pour éviter les requêtes multiples:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
$string = '';
 
foreach($_POST as $key => $value) {
   $string.= $key'="'.$value.'"';
   $string.= ', ';
}
 
mysql_query('UPDATE table SET $string WHERE id="1"');

C'est mieux ?

[grunk]

Construire la chaine en amont est une solution, mais attention à ne pas utiliser les données issue de $_POST (ou tout autres sources) directement dans tes requêtes. Tu t'expose sinon au injection sql et tous les problème que cela implique.

les lignes sont longues et donc une source d'erreur

Rien ne t'empêche de sauter des lignes dans ta requête pour plus de lisibilité.

[gwharl]

Une méthode un peu plus avancée est d'utiliser un objet.

http://hachesse.developpez.com/objetphp/

[stealth35]

Une méthode un peu plus avancée est d'utiliser un objet.

http://hachesse.developpez.com/objetphp/

et de lui refiler un vieux tuto php 4, histoire de prendre de bonnes habitudes...