Suppression d'enregistrement mySQL

**Cocalight** · 31/07/2006, 13h33

Bonjour, débutant PHP je me retrouve confronter a un probleme.
J'ai developpé un script permetant l'ajout d'un enregistrement dans une base mySQL qui fonctionne a merveille.

Je me suis alors mis au travail pour creer un script qui cette fois-ci supprime un enregistrement de la bdd.

VOici le code

Code :

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

1
2
3
4
5
<form action="validedelete.php" method="post">
      <span class="Style14">Effacer le ban numéro :</span>
        <input type="text" name="ban_id:"> 
      <input name="submit" type="submit" class="Style14" value="Effacer le ban">
</form>

Voici le traitement dans ma page validedelete.php

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?
/*
On se connecte et on choisi la base de données
*/
mysql_pconnect('localhost','user', '*****') or die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') or die("Impossible de sélectionner la base de données");
 
 
if (!isset($ban_id)) {
  echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
  $query_string = "DELETE FROM banlist WHERE ban_id=$id";
  $query=mysql_query($query_string);
  if ($query) {
    echo "<p>L'enregistrement a été effacé</p>";
  } else {
    echo "<p>Problème lors de la suppression</p>";
    echo "<p>" . mysql_error() . "</p>";
  }
}
?>

Les champs de la base de données sont

Code :

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

1
2
3
4
5
ban_id (clé primaire, auto_increment)
pseudo
steam_id
raison
admin

Je debute depuis peu en PHP. Je lis des tonnes de tutos mais tout n'est pas forcement clair ^^

Merci de détaillé la ou je me suis trompé

Cordialement

**guitou12** · 31/07/2006, 13h38

Tu as des erreurs ou ça fonctionne ?

**Cocalight** · 31/07/2006, 13h41

Non le formulaire de suppression ne fonctionne pas.

Il me dit

Code :

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

Aucun enregistrement à supprimer.

Je precise bien l'id de l'enregistrement a supprimer pourtant et lors de la soumission aucun message d'erreur .. pourtant l'enregistrement est toujours la.

**SnakemaN** · 31/07/2006, 13h44

Bonjour,

Deja premier conseils,
- recupere proprement la variable postée.sans utiliser de variable super globale
- Tu ne concatenne pas bien les variable dans ta requete

Code :

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

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
<?
/*
On se connecte et on choisi la base de données
*/
mysql_pconnect('localhost','user', '') or die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') or die("Impossible de sélectionner la base de données");

[B]if(isset($_POST['ban_id']))
$banId = $_POST['ban_id'];
else
$banId = NULL[/B]

 
if (banId == NULL) {
echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
$query_string = "DELETE FROM banlist WHERE ban_id=[B]'".$id."" [/B]";
$query=mysql_query($query_string)or die(mysql_error());
if ($query) {
echo "<p >L'enregistrement a été effacé</p>";
} else {
echo "<p>Problème lors de la suppression</p>";
echo "<p>" . mysql_error() . "</p>";
}
}
?>

Colorez votre code PHP sur les forums grâce à Developpez.com

Voila essayes deja ca, ca devrais marcher comme cela...

**Kerod** · 31/07/2006, 13h45

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
if (!isset($_POST['ban_id'])) {
  echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
  $query_string = "DELETE FROM banlist WHERE ban_id=".$_POST['ban_id']
;
  $query=mysql_query($query_string);
  if ($query) {
    echo "<p>L'enregistrement a été effacé</p>";
  } else {
    echo "<p>Problème lors de la suppression</p>";
    echo "<p>" . mysql_error() . "</p>";
  }
}

Comme ceci ?

**SnakemaN** · 31/07/2006, 13h49

Le BBcode c'est bien mais des fois...

**Cocalight** · 31/07/2006, 13h50

@SnakemaN

Je ne comprend pas ton code (newbie quand tu nous tiens) (je colorise mon code la prochaine fois)

@Kerod

TOujours le même résultat, a savoir

Aucun enregistrement a supprimer.
Tu peux voir le script a cette adresse => http://lfs-team.com/e107_files/banlist.php

**guitou12** · 31/07/2006, 13h55

Code :

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

<input name="ban_id:">

**Cocalight** · 31/07/2006, 14h03

Oui effectivement pas malin de ma part.

Maintenant j'ai cette erreur

Code :

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

1
2
3
Problème lors de la suppression
 
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Code :

<?
mysql_pconnect('localhost','user', '') or die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') or die("Impossible de sélectionner la base de données");

if (!isset($_POST['ban_id'])) {
  echo "Aucun enregistrement à supprimer.";
} else {
  $query_string = "DELETE FROM banlist WHERE ban_id=".$_POST['ban_id']
;
  $query=mysql_query($query_string);
  if ($query) {
      echo "L'enregistrement a été effacé";
  } else {
      echo "Problème lors de la suppression";
      echo "" . mysql_error() . "";
  }
}
?>

Colorez votre code PHP sur les forums grâce à Developpez.com

**Kerod** · 31/07/2006, 14h11

Ca vient de ton champ post il doit etre vide..Vérifies bien qu'il y a un contenu

**SnakemaN** · 31/07/2006, 14h12

Code :

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

1
2
 
$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

Envoyé par fanta agrum

Je ne comprend pas ton code (newbie quand tu nous tiens) (je colorise mon code la prochaine fois)

c'est normal c'est le BBcode qui fait des siennes, c'est normal ce n'est qu' un bébé (<- Ultimate Joke Inside

)

**guitou12** · 31/07/2006, 14h13

C'est laborieux tout ça, révise un peu ton html

Code :

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

<form action="validedelete.php" method="$_POST">

--> method="post"

EDIT: tu avais juste sur ta 1ère version pourquoi avoir changé ?

**Kerod** · 31/07/2006, 14h13

Envoyé par SnakemaN

Code :

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

1
2
 
$query_string = "DELETE FROM banlist WHERE ban_id="'.$_POST['ban_id'].'" ";

Ce serait mieux si tu faisais le contraire

Code :

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

1
2
 
$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

**SnakemaN** · 31/07/2006, 14h14

Envoyé par Kerod

Ce serait mieux si tu faisais le contraire

Code :

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

1
2
 
$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

Qui qui me parle ??

**Kerod** · 31/07/2006, 14h18

$query_string = "DELETE FROM banlist WHERE ban_id="'.$_POST['ban_id'].'" ";

Les quotes doivent être à l'intérieur...Pas d'accord ??

**guitou12** · 31/07/2006, 14h18

T'as une balise </form> de trop qui traine en fin de page !!!

**guitou12** · 31/07/2006, 14h20

Ce qu'à écrit Snakeman est juste mais ce qu'à cité Kerod est faux.... qui des 2 a raison ?

Il me semble que

Code :

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

$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

est la bonne réponse, ceci dit si son ID est un int c'est un peu inutile !

**SnakemaN** · 31/07/2006, 14h23

Et puis je ne sais pas le niveau de securité voulu, mais je mettrai un petit htmlentities()
Au cas ou...

**Cocalight** · 31/07/2006, 14h25

@guitou : Je m'en sort comme je peut dsl

@snakeman : htmlentities() connais pas du tout (j'ai déja du mal avec un simple formulaire

si tu me donne la marche a suivre

Donc je récapitule mon code formulaire et traitement :

Formulaire :

Code :

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

1
2
3
4
5
6
7
<form action="validedelete.php" method="post">
  <p><span class="Style14">Effacer le ban numéro :</span>
      <input type="text" name='ban_id'>   
 
    <input name="submit" type="submit" class="Style14" value="Effacer le ban">
  </p>
</form>

traitement (validedelete.php) :

<?
/* On se connecte et on choisi la base de données */
mysql_pconnect('localhost','Coyote', '*****')
or die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') or die("Impossible de sélectionner la base de données");
if(isset($_POST['ban_id']))
$banId = $_POST['ban_id'];
else $banId = NULL
if (banId == NULL)
{ echo "Aucun enregistrement à supprimer.";
} else {
$query_string = "DELETE FROM banlist WHERE ban_id="'.$_POST['ban_id'].'" ";
$query=mysql_query($query_string)or die(mysql_error());
if ($query) { echo "L'enregistrement a été effacé";
} else {
echo "Problème lors de la suppression";
echo "" . mysql_error() . "";
}
}
?>

Colorez votre code PHP sur les forums grâce à Developpez.com

**guitou12** · 31/07/2006, 14h28

C'est sur que si tu te relis pas....

Code :

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

1
2
3
4
if ($banId == NULL)
{ echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

Suppression d'enregistrement mySQL

MySQL

Discussions similaires

Partager

Partager