Discussion :

[guy2004]

Bonjour,
j'ai un petit forum avec 2 tables, l'une pour les sujets, le titre et l'auteur qui se nomme "header" et l'autre pour le contenu qui se nomme "body".
Elles sont reliées par la clés "postid" qui est Primaire sur "header" .

Dans ma table "header" j'ai une colonne "parent" qui indique si ce sujet a un parent: elle a pour valeur le "postid" du message auquel elle repond ou "0" si c'est le 1er sujet.
J'ai une colonne "children" qui indique si ce message a des enfants en reponse.

Mon probleme est pour la suppression des messages, si je supprime une reponse je voudrais que tous ses descendants soient supprimés, si c'est le 1 er message je voudrai que tous ses descendants soient supprimés.
Pour le moment je n'arrive a supprimer que un par un...

Le "postid" est transmis dans l'URL. Voici comment je commence :

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
$postid = $_GET['postid'];
 
 if(!$postid) return false;
 
  $conn = db_connect();
 $query1 = "select from header where postid = $postid";
    $result1 = mysql_query($query1, $conn);
 $nombre_de_resultats = mysql_num_rows($result1);
 $ligne = mysql_fetch_array($result1); 
 
 parents = $ligne['parent'];
 children = $ligne['children'];
 
 if (!$result1)
     return false;

Ensuite pour le traitement je bloque avec toutes les conditions par rapport aux valeurs de "parent" et de "children".
Je sais pas si c'est bien clair mais merci à ceux qui pourront m'aider !

[guy2004]

personne ne voit ?

[Mr N.]

Il te manque quelque chose dans ton select

Sinon ben tu as tout pigé :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
(pseudo code)
function supprimer($id) {
   $children := "select children from ma_table where postid = $id"
   if ($children != 0) {
      supprimer($children);
   }
   delete from children where postid = $id
}
 
supprimer($_GET['postid']);

Grosso merdo c'est comme ça que j'aurais fait...

[khayyam90]

bien le bonjour,

si j'ai bien compris, chacun des messages connait son parent (s'il y en a un) et son fils (s'il y en a un).
donc, a partir du postid, il faudrait boucler sur tous ses fils de la maniere suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
si le header numero post_id a un pere, 
     recuperer l'id du pere
     dire au pere qu'il n'a plus de fils
FinSi
 
header_courant = post_id
Tant que header_courant a un fils,
     recuperer l'id de ce fils.
     supprimer le body correspondant au header courant.
     supprimer le header_courant. 
     header_courant = id du fils
Fin Tant que

tout ca me fait beaucoup penser a de la gestion de liste chainee (meme doublement chainee puisque chaque header connait son pere et son fils).

on aurait pu discuter (et troller ) pendant des semaines sur la maniere de gerer les tables. Elles sont deja fixees et c'est pas plus mal.

[Mr N.]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
si le header numero post_id a un pere, 
     recuperer l'id du pere
     dire au pere qu'il n'a plus de fils
FinSi
 
header_courant = post_id
Tant que header_courant a un fils,
     recuperer l'id de ce fils.
     supprimer le body correspondant au header courant.
     supprimer le header_courant. 
     header_courant = id du fils
Fin Tant que

+1, j'ai oublié de dire au pere qu'il a perdu son fils, le pauvre.
+1, en iteratif c'est plus performant qu'en récursif
-1, si il y a des contraintes d'integrité sur la base alors on ne peut supprimer le header courant car le fils référencerait alors un pere qui n'existe pas.

[guy2004]

oui c'est exactement ca que je cherche vous avez compris l'affaire.
Le probleme est qu'avec toutes ces conditions je suis un peu perdu dans mes requetes !

[Mr N.]

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
 
function supprimer($id) {
   $sql = "select children from header where postid = $id";
   $res = mysql_query($sql);
   $row = mysql_fetch_assoc($req);
   $children = $row['children'];
   if ($children != 0) {
      supprimer($children);
   }
   $sql = "delete from header where postid = $id";
   mysql_query($sql);
} 
 
$id = $_GET['postid'];
 
//On dit au parent qu'il vient de perdre son fils
$sql = "select parent from header where postid = $id";
$res = mysql_query($sql);
$row = mysql_fetch_assoc($req);
$parent = $row['parent'];
$sql = "update header set children = 0 where postid = $parent";
mysql_query($sql);
 
//On supprime le header et sa descendance
supprimer($id);

Je te laisse optimiser et faire les contrôle d'erreurs

[guy2004]

merci je vais etudier cela cet eprem et je vous tiens au courant.

[khayyam90]

dans le meme genre de remarque,

si il y a des contraintes d'integrité sur la base alors on ne peut supprimer $children car le pere référencerait alors un fils qui n'existe pas, c'est juste prendre le probleme dans l'autre sens.

il suffit donc de rajouter la suppression des liens de parente avant de supprimer les elements eux-memes. Evidemment ca n'est pas necessaire s'il n'y a pas de contraintes d'integrite a ce niveau.

[Mr N.]

vi c'est assez prise de tête finalement. Et si on changeait le schéma des tables ?

[guy2004]

c'est pas encore tout a fait ca, je m'explique :
Si il y a 1 parent et 2 children, si je supprime le 1er children cela devrait me supprimer son descendant, or cela ne me le supprime pas. Je me retrouve avec des descendant dont le "parent" vaut le "postid" du 1er enfant qui vient d'etre supprimé.
Je sais pas si c'est bien clair ?!

[guy2004]

Voici où j'en suis : j'arrive a supprimer uniquement le 1 er enfant des messages !
Si un message parent comporte 4 reponses, si je supprime le message parent cela me supprime le parent et le 1 er enfant, ce qui m'en laisse encore 2 dans ma base !
Voici mon code actuel :

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
 
function supprimer($id) { $conn = db_connect();
   $sql = "select children from header where postid = $id"; 
   $res = mysql_query($sql, $conn); 
   $row = mysql_fetch_assoc($res); 
   $children = $row['children']; 
   if ($children != 0) { 
   $sql = "delete from header where parent = $id"; 
      supprimer($children); 
	  mysql_query($sql, $conn); 
   } 
   $sql = "delete from header where postid = $id"; 
   mysql_query($sql, $conn); 
};
 
$id = $_GET['postid']; 
$conn = db_connect();
//On dit au parent qu'il vient de perdre son fils 
$sql = "select parent from header where postid = $id"; 
$res = mysql_query($sql, $conn); 
$row = mysql_fetch_assoc($res); 
$parent = $row['parent']; 
$sql = "update header set children = 0 where postid = $parent"; 
mysql_query($sql); 
 
//On supprime le header et sa descendance 
supprimer($id);

c'est surement une histoire de boucle mais comment faire ?