aide pour suppression

**kOrt3x** · 18/08/2009, 11h30

Qu'est ce qu'il ne va pas dans mon code ?

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
28
29
<html>
    <head>
        <title>Suppresion du lien</title>
    </head>
    <body>
    Voulez vous vraiment supprimer le lien ?
    <br>
        <center>
            <form action="deletelien.php" method="post">
                <input type="submit" value="Oui" name="button" /></input>
                <input type="submit" value="Non" name="button" /></input>
                    <?
                    mysql_connect('localhost','login','pass');
                    mysql_select_db('base');
                    $numero_id = $_GET['ID'];
                    $type_de_reponse=$_POST['button'];
 
                        if ($type_de_reponse=="Oui"){
                            mysql_query("DELETE FROM liens_utiles WHERE ID='$numero_id'") .mysql_error();
                            echo 'liens supprimer avec succés !';
                        }elseif ($type_de_reponse=="Non"){
                            echo 'retour à l\'accueil';
                        }
                    mysql_close();
                    ?>
            </form>
        </center>
    </body>
</html>

**sabotage** · 18/08/2009, 11h39

Ton ID n'est pas défini dans le formulaire.

**Zwiter** · 18/08/2009, 11h41

Salut,
Il manque ID en parametre d'URL (GET).

Z.

[edit] Le temps de trouver l'erreur, on a été plus rapide :p

**metagoto** · 18/08/2009, 11h42

Tu es sûr que tu as un $_GET['ID'] ?

Aussi, pense à faire un mysql_real_escape_string() quand tu construits ta query avec WHERE ID='$numero_id'" parceque sinon, c'est une sérieuse trappe à l'sql injection

edit: grilled

**Tsoko** · 18/08/2009, 11h44

N'ouvre la connexion que si tu en as besoin (question de sécurité)
J'ai repris ton code:

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
28
29
30
31
32
33
34
35
36
 
<html>
    <head>
        <title>Suppresion du lien</title>
    </head>
    <body>
    Voulez vous vraiment supprimer le lien ?
    <br>
	<?php
	if($_POST['radio']){
		$numero_id = $_GET['ID'];
        $type_de_reponse = $_POST['radio'];
        if ($type_de_reponse == "oui"){
            mysql_connect('localhost','apicryptfaq','mortawindows');
			mysql_select_db('apicryptfaq');
			mysql_query("DELETE FROM liens_utiles WHERE ID='".$numero_id."'")or die(mysql_error());
			mysql_close();
            echo 'liens supprimer avec succés !';
        }else{
            echo 'retour à l\'accueil';
        }
	}
	else{
		?>
		<center>
            <form action="deletelien.php" method="post">
				<input type="radio" name="reponse" value="oui" />
				<input type="radio" name="reponse" value="non" checked />
                <input type="submit" value="Envoyer" />
            </form>
        </center>
		<?php
	}
	?>
    </body>
</html>

tu as aussi d'autres choses à mettre en place comme sécurisé des données

**kOrt3x** · 18/08/2009, 12h26

Envoyé par Tsoko

N'ouvre la connexion que si tu en as besoin (question de sécurité)
J'ai repris ton code:

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
28
29
30
31
32
33
34
35
36
 
<html>
    <head>
        <title>Suppresion du lien</title>
    </head>
    <body>
    Voulez vous vraiment supprimer le lien ?
    <br>
    <?php
    if($_POST['radio']){
        $numero_id = $_GET['ID'];
        $type_de_reponse = $_POST['radio'];
        if ($type_de_reponse == "oui"){
            mysql_connect('localhost','apicryptfaq','mortawindows');
            mysql_select_db('apicryptfaq');
            mysql_query("DELETE FROM liens_utiles WHERE ID='".$numero_id."'")or die(mysql_error());
            mysql_close();
            echo 'liens supprimer avec succés !';
        }else{
            echo 'retour à l\'accueil';
        }
    }
    else{
        ?>
        <center>
            <form action="deletelien.php" method="post">
                <input type="radio" name="reponse" value="oui" />
                <input type="radio" name="reponse" value="non" checked />
                <input type="submit" value="Envoyer" />
            </form>
        </center>
        <?php
    }
    ?>
    </body>
</html>

tu as aussi d'autres choses à mettre en place comme sécurisé des données

Merci pour tous ces conseils, mais ton code ne marche pas encore même.

**sabotage** · 18/08/2009, 12h42

Je me repete mais ID n'est toujours pas défini.

**Tsoko** · 18/08/2009, 13h49

Oui, je pars du principe que la valeur Id est déjà renseignée !!
Maintenant en partant du principe que ta table comporte une entrée ID renseignée à 1, tu peux contrôler comme suit:

Code :

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

<form action="deletelien.php?ID=1" method="post">

Là ça doit marcher

Mais je suppose bien évidemment que ta valeur ID est une reception de données d'une formulaire précédant celui-ci non??

Sinon effectivement, cela ne peut pas marcher car dans ce cas :
ID = ""; (vide) donc ne peut pas supprimer dans la table !!

Courage

aide pour suppression

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager