Deux requêtes DELETE sur une même page Résolu

Question

Bonjour, Je voudrais savoir si il est possible de faire deux requètes SQL dans une même page, pour supprimer, dans deux tables différentes soit un enregistrement, soit un autre, au lieu de faire deux pages de suppression. Afin que chaque membre connecté puisse supprimer soit ses messages envoyé, soit ses messages reçu. J'ai bien sur deux liens pour la suppression. Je vous remercie. echo '

'; echo '

'; $PARAM = array ( 'IDMESS' => isset($_REQUEST['idmess']) ? $_REQUEST['idmess'] : NULL, ); $sql = "DELETE FROM cdc_destinataires WHERE cdc_destinataires.iddest='".$_SESSION['idpers']."' AND cdc_destinataires.idmess = ".$PARAM['IDMESS'].""; try { $requete = $bdd->prepare($sql); $requete->execute(); } catch(Exception $e) { // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); exit(0); // en cas d'erreur.. arrête le script ! } $sql2 = "DELETE FROM cdc_messages WHERE cdc_messages.idexp='".$_SESSION['idpers']."' AND cdc_messages.idmess = ".$PARAM['IDMESS'].""; try { $requete2 = $bdd->prepare($sql2); $requete2->execute(); } catch(Exception $e) { // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); exit(0); // en cas d'erreur.. arrête le script ! } $success = "Ton message à été supprimé."; echo '
'; echo $success

jordane45 · Answer

Bonjour,

Bien sûr que c'est possible.
C'est d'ailleurs ce que tu as écris dans ton code ... deux requêtes dans le même fichier.
Je ne comprend pas où se trouve le souci... 
Par contre ... au lieu d'utiliser les variables de session... pourquoi ne pas te servir des variables GET que tu passes dans tes URL ?
Tu dispose de idexp  et de iddest ... pourquoi ne t'en sers tu pas ?

LaChaux78 · Answer

Bonjour,
En fait tel quel cela supprime le message envoyé et également le message reçu qui corresponds au même idmess.
Oui lorsque par exemple j'envoie un message groupé, étant donné que je fait partie du groupe je reçois également le message donc dans message envoyé j'ai le message et également dans message reçu, de ce fait je voulais uniquement supprimer le message reçu.
C'est quelque peu compliqué peux tu me donner une explication pour simplifier cela.
Merci.

yg_be · Answer

bonsoir, 
je suggère d'ajouter iddest et idexp dans $PARAM.
ensuite tu pourras, en fonction duquel des deux est non null, faire l'action que tu souhaites.

LaChaux78 · Answer

Bonjour,
j'ai rajouté comme tu me l'a suggéré cela me retourne cette erreur.
Merci.

Erreur ! SQLSTATE[42000]: Syntax error or access violation: 1064 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 4

LaChaux78 · Answer

$PARAM = array ( 'IDMESS' => isset($_REQUEST['idmess']) ? $_REQUEST['idmess'] : NULL, 'IDEXP' => isset($_REQUEST['idexp']) ? $_REQUEST['idexp'] : '', 'IDDEST' => isset($_REQUEST['iddest']) ? $_REQUEST['iddest'] : '', ); $sql = "DELETE FROM cdc_destinataires WHERE cdc_destinataires.iddest='".$_SESSION['idpers']."' AND cdc_destinataires.idmess = ".$PARAM['IDMESS']." AND cdc_destinataires.iddest = ".$PARAM['IDDEST'].""; try { $requete = $bdd->prepare($sql); $requete->execute(); } catch(Exception $e) { // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); exit(0); // en cas d'erreur.. arrête le script ! } $sql2 = "DELETE FROM cdc_messages WHERE cdc_messages.idexp='".$_SESSION['idpers']."' AND cdc_messages.idmess = ".$PARAM['IDMESS']." AND cdc_messages.idexp = ".$PARAM['IDEXP'].""; try { $requete2 = $bdd->prepare($sql2); $requete2->execute(); } catch(Exception $e) { // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); exit(0); // en cas d'erreur.. arrête le script ! } $success = "Ton message à été supprimé."; echo '
'; echo $success;

jordane45 · Answer

Bonsoir, Déjà.. sachant que nous avons déjà bossé ensemble sur d'autres de tes scripts... il serait bien que tu t'en inspire... Ton code actuel devrait ressembler à un truc du genre : prepare($sql); $requete->execute($datas); }catch(Exception $e){ // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); echo " Les datas : " ; print_r($aadh,$mois,$jour); exit(0); // en cas d'erreur.. arrête le script ! } return $requete; } /** *Suppression du message reçu */ function deleteMessage_recu($idmess,$iddest){ $sql = "DELETE FROM cdc_destinataires WHERE idmess = :idmess AND iddest = :iddest"; $datas = array(':idmess'=>$idmess,':iddest'=>$iddest); return executeQuery($sql,$datas); } /** *Suppression du message expédié */ function deleteMessage_exp($idmess,$idexp){ $sql = "DELETE FROM cdc_message WHERE idmess = :idmess AND idexp = :idexp"; $datas = array(':idmess'=>$idmess,':idexp'=>$idexp); return executeQuery($sql,$datas); } //-----------------------------------------------------------------------------// // Récupération PROPRE des variables AVANT de les utiliser ! //-----------------------------------------------------------------------------// $idmess = !empty($_GET['idmess']) ? $_GET['idmess'] : NULL; $idexp = !empty($_GET['idexp']) ? $_GET['idexp'] : NULL; $iddest = !empty($_GET['iddest']) ? $_GET['iddest'] : NULL; //début du script : if($idmess && $iddest){ if(deleteMessage_recu($idmess,$iddest)){ echo "Ton message à été supprimé."; } }else{ echo "Erreur lors de la suppression !"; } if($idmess && $idexp){ if(deleteMessage_exp($idmess,$idexp)){ echo "Ton message à été supprimé."; } }else{ echo "Erreur lors de la suppression !"; } Mais bon... la deuxième requête pose problème .... Si la personne qui à reçu le message veut le conserver.. celui-ci sera quand même supprimé.... EDIT : Requêtes corrigées

LaChaux78 · Answer

Bonjour, J'ai essayé de faire comme ça mais je ne suis pas de sur que j'ai bien et tout compris. Ca à l'air de fonctionner mais je ferai d'autres essais demain, là il se fait tard, je te remercie. $PARAM = array ( 'IDMESS' => isset($_REQUEST['idmess']) ? $_REQUEST['idmess'] : NULL, 'IDEXP' => isset($_REQUEST['idexp']) ? $_REQUEST['idexp'] : NULL, 'IDDEST' => isset($_REQUEST['iddest']) ? $_REQUEST['iddest'] : NULL ); if ($PARAM['IDEXP'] == !NULL) { $sql = "DELETE FROM cdc_destinataires WHERE cdc_destinataires.iddest = '".$_SESSION['idpers']."' AND cdc_destinataires.idmess = ".$PARAM['IDMESS'].""; try { $requete = $bdd->prepare($sql); $requete->execute(); } catch(Exception $e) { // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); exit(0); // en cas d'erreur.. arrête le script ! } } if ($PARAM['IDDEST'] == !NULL) { $sql2 = "DELETE FROM cdc_messages WHERE cdc_messages.idexp = '".$_SESSION['idpers']."' AND cdc_messages.idmess = ".$PARAM['IDMESS'].""; try { $requete2 = $bdd->prepare($sql2); $requete2->execute(); } catch(Exception $e) { // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); exit(0); // en cas d'erreur.. arrête le script ! } } $success = "Ton message à été supprimé."; echo '
'; echo $success; et le lien de suppression echo '

';

LaChaux78 · Answer

Bonjour,
Toutes mes excuses je ne l'avais pas vu.
Merci

LaChaux78 · Answer

j'ai cette erreur....

Erreur lors de la suppression ! Erreur ! SQLSTATE[42000]: Syntax error or access violation: 1064 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 'D WHERE D.idmess = '22' AND D.idexp = '2'' at line 1

LaChaux78 · Answer

Bonjour,
Oui comme tu l'a dit.
Si la personne qui à reçu le message veut le conserver.. celui-ci sera quand même supprimé....
C'est justement cela que je voulais éviter en fait.
Merci

LaChaux78 · Answer

Voici mes deux tables

LaChaux78 · Answer

Je te remercie.

LaChaux78 · Answer

Bonjour,
En fait je vais changer la structure des tables de ma messagerie afin de pouvoir faire des liens pour la suppression des messages envoyés et reçus.
Merci.

LaChaux78 · Answer

Bonjour,
Merci pour vos retours
J'ai essayé de faire cela en fait...Je n'ai pas testé pour l'instant vous en pensez quoi je trouve ça peut-être cohérent non ??
Merci pour vos avis.
1 table expediteur
idmess
idexp
1 table destinataire
idmess
iddest
lumess
1 table message
idmess
date_envoie
titre
message

LaChaux78 · Answer

Bonjour,
En fait là avec une requète DELETE chaque destinataire peut se supprimer son iddest et son idmess et idem pour les expéditeurs comme cela les messages sont conservés dans la table cdc_messages enfin je cois je n'ai pas encore fait la page supprimer.

LaChaux78 · Answer

je ne comprends pas ce que tu veux dirre... par exemple si je fait cette requète cela me supprime l'expéditeur d'un message données non? n'oublions pas je suis débutant donc pas sur de moi merci. isset($_REQUEST['idmess']) ? $_REQUEST['idmess'] : NULL, 'IDEXP' => isset($_REQUEST['idexp']) ? $_REQUEST['idexp'] : NULL, ); $sql2 = "DELETE FROM cdc_expediteurs WHERE cdc_expediteurs.idexp = '".$_SESSION['idpers']."' AND cdc_expediteurs.idmess = ".$PARAM['IDMESS'].""; try { $requete2 = $bdd->prepare($sql2); $requete2->execute(); } catch(Exception $e) { // en cas d'erreur : echo " Erreur ! ".$e->getMessage(); exit(0); // en cas d'erreur.. arrête le script ! } $success = "Ton message à été supprimé."; echo '
'; echo $success; } else { redir("index.php"); } ?>

LaChaux78 · Answer

oui je préfére que le destinataire ou l'expéditeur si ils veulent supprimer un message ils n'ont plus acces aux messages ni qui leur a envoyé. Car de toute manière si ils ne voient plus le message quel importance de savoir qui leur a envoyé de toute façon ils n'ont plus accès au contenu, bref je verrai cela comme ça.

LaChaux78 · Answer

oui je n'avait pas pensé à tout ça, alors je vais reprendre ton post précédent à savoir....
j'ai 1 table destinataire
1 table expéditeur
1 table message

je rajoute 1 champ à la table cdc_destinataires => hidedest
je rajoute 1 champ à la table cdc expediteurs => hideexp

tu peux m'aider à partir de là s'il te plait.
Quel type dois je mettre pour ces champs.
Je te remercie.

LaChaux78 · Answer

ok merci je vais regarder tout ça.

LaChaux78 · Answer

lorsque je rentre le type BOOLEAN dans ma BDD cela mets tinyint es ce normal, je ne connait pas bien ce type de champ.

LaChaux78 · Answer

ok je ne savais pas merci

LaChaux78 · Answer

Bonjour, Voilà pour terminer il me faudrait ta connaissance pour faire fonctionner correctement les deux requètes UPDATE dans le même fichier. Si je cré deux fichiers différent avec chacune des requètes cela marche bien sinon lorsque je supprime un message de la table cdc_destinataire cela me supprime également le message de la table cdc_expediteurs. Merci isset($_REQUEST['idmess']) ? $_REQUEST['idmess'] : NULL, ); $sql = "UPDATE cdc_expediteurs SET hideexp = TRUE WHERE cdc_expediteurs.idexp ='".$_SESSION['idpers']."' AND cdc_expediteurs.idmess = ".$PARAM['IDMESS'].""; $requete = $bdd->prepare($sql); $requete->execute(); $sql = "UPDATE cdc_destinataires SET hidedest = TRUE WHERE cdc_destinataires.iddest ='".$_SESSION['idpers']."' AND cdc_destinataires.idmess = ".$PARAM['IDMESS'].""; $requete = $bdd->prepare($sql); $requete->execute(); $success = "Ton message à été supprimé."; echo '
'; echo $success; } else { redir("index.php"); } ?>

LaChaux78 · Answer

Merci pour tout j'ai finalement réussi à faire ce que je souhaitais.
Merci

Deux requêtes DELETE sur une même page

23 réponses

Votre réponse