Discussion :

[Hayabusa]

bijour

je fais une petite bibliothéque d'images, et mon problème se situe dans un formulaire pour éditer les images, leur description, ...

le code en dessous, c'est ce qui permet de mettre à jour la base, je vérifie que le visiteur a modifié la valeur de l'image (exemple : description, source, ) et si oui, bin alors je modifie cette image dans la base de donnée, enfin j'arrive pas à bien explique : visionner mon code ça sera à mon avis plus clair

Code :

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

1
2
3
4
5
6
7
8
9
10
11
else
				{
					$sql2 = 'UPDATE images SET ';
					if($_POST['nom'] != "") $sql2 .= 'nom="'.$_POST['nom'].'",';
					if($_POST['fichier'] != "") $sql2 .= 'chemin="'.$_POST['fichier'].'",';
					if($_POST['source'] != "") $sql2 .= 'source="'.$_POST['source'].'",';
					if($_POST['description'] != "") $sql2 .= 'description="'.$_POST['description'].'"';
					$sql2 .= ' WHERE id="'.$_POST['id_img'].'"';
					$res2 = mysql_query($sql2) or die(mysql_error());
					$message = 'Opération effectuée avec succès !';
				}

le problème si situe là :

Code :

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

if($_POST['nom'] != "") $sql2 .= 'nom="'.$_POST['nom'].'",';

ça dit en gros, si le nom n'est pas vide (docn que le visituer l'a modifié) alors change de nom dans la base de donnée...

plus précisément, le problème, c'est ça :

Code :

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

,

bin oui la virugule, à la fin

en fait, si on ajoute le nom, et rien que le nom, la requete va donner :

UPDATE images SET nom="nouveau_nom'", WHERE ...

et la virgule avant le WHERE, ça coince

donc, bin je sais pas comment faire, parce que si je ne met pas cette virgule, ça va coincer pour le suivant (fichier), parce qu'il y aura :

UPDATE images SET nom="nouveau_nom'" fichier="nouveau_fichier'" WHERE ...

et pas de virgule entre les duex, ça coince aussi...


bref, je sais pas commetn faire

est-ce que quelq'un a une idée ?

merci d'avance

[mamiberkof]

si j'étais à ta place , je ferai comme ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
else if($_POST['id_img']!="")
{
$trjVerifie= "id=$_POST['id_img']";					
$sql2 = 'UPDATE images SET $trjVerifie ';
if($_POST['nom'] != "") $sql2 .= ', nom="'.$_POST['nom'].'"';
if($_POST['fichier'] != "") $sql2 .= ', chemin="'.$_POST['fichier'].'"';
if($_POST['source'] != "") $sql2 .= ', source="'.$_POST['source'].'",';
if($_POST['description'] != "") $sql2 .= ', description="'.$_POST['description'].'"';
$sql2 .= ' WHERE id="'.$_POST['id_img'].'"';
$res2 = mysql_query($sql2) or die(mysql_error());
$message = 'Opération effectuée avec succès !';
}

explication : tu ajoutes une variable dont tu es sûr qu'elle soit toujours présente, et tu met le ',' au début de tes chaines .
voilà

A+