par
macgawel » 26 mars 2010, 13:48
Bon s'est bon je viens trouver pour faire le formulaire de remplacement, voici le code qui fonctionne très bien:
<?php
include("connexion.php");
mysql_select_db ('type') ;
$champ2=isset($_POST['champ2'])?$_POST['champ2']:"";
$champ3=isset($_POST['champ3'])?$_POST['champ3']:"";
$sql = "SELECT `type` FROM `type` WHERE 1";
$modification = mysql_query("UPDATE type SET type='$champ2' WHERE type='$champ3'");
mysql_close();
?>
Merci de votre aide
Juste quelques petites remarques sur le code :
1. Pense à sécuriser les données récupérées. Là, tu es vulnérable à une attaque par
injection...
2. Tu peux enlever la ligne où tu assignes $sql, vu que tu n'utilises pas cette variable.
3. Ca peut être intéressant de gérer les erreurs.
if ($modification == true) {
echo "modification faite";
}
else {
echo "erreur, la modification n'est pas faite";
}
Et si tu ne gères pas les erreurs, il n'y a pas d'intérêt à mettre le résultat de la requête dans une vairable...
[quote="vinvinsylvain"]Bon s'est bon je viens trouver pour faire le formulaire de remplacement, voici le code qui fonctionne très bien:
[php]<?php
include("connexion.php");
mysql_select_db ('type') ;
$champ2=isset($_POST['champ2'])?$_POST['champ2']:"";
$champ3=isset($_POST['champ3'])?$_POST['champ3']:"";
$sql = "SELECT `type` FROM `type` WHERE 1";
$modification = mysql_query("UPDATE type SET type='$champ2' WHERE type='$champ3'");
mysql_close();
?>[/php]
Merci de votre aide[/quote]
Juste quelques petites remarques sur le code :
1. Pense à sécuriser les données récupérées. Là, tu es vulnérable à une attaque par [url=http://fr.wikipedia.org/wiki/Injection_SQL]injection[/url]...
2. Tu peux enlever la ligne où tu assignes $sql, vu que tu n'utilises pas cette variable.
3. Ca peut être intéressant de gérer les erreurs. [php]if ($modification == true) {
echo "modification faite";
}
else {
echo "erreur, la modification n'est pas faite";
}[/php]Et si tu ne gères pas les erreurs, il n'y a pas d'intérêt à mettre le résultat de la requête dans une vairable...