Discussion :

[Squalthor]

Bonjour,

Voilà je viens poster car je me retrouve à court d'idées pour résoudre mon problème...

J'espère me trouver dans la bonne section du forum.

Je travail actuellement sur un forum et plus précisément sur la partie admin pour gèrer les sections du forum.

La partie sur laquelle je travaille en ce moment est l'ajout des sections du forum, cette partie comporteras un bouton "supprimer" ainsi que des boutons "monter" et "descendre".
Hors les choses se présentes assez mal car je rencontre un problème avec le bouton supprimer, mais afin que vous compreniez bien ce que je veux faire il me semble indispensable de détailler.

Je gère la position des section via les Id, Mysql se charge de les ressortir dans l'ordre, pour le positionnement je me servirais du même schéma en modifiant les id concernées.

Pour celà il m'est indispensable d'avoir des Id qui se suivent, hors quand j'efface une section je me retrouve avec un vide donc je suis en train de travailler sur cette fonction pour réorganiser les sections, cette fonction utilise Xajax mais ça c'est juste pour info.

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
 
function Effacer_section($id)
	{
	$Reponse = new xajaxResponse();
	query('DELETE FROM Sections_forum WHERE Id=\''.$id.'\'');
	$sql = query('SELECT Id FROM Sections_forum WHERE Id > \''.$id.'\' ORDER BY Id') or die(mysql_error());
	while($donnees = mysql_fetch_array($sql, MYSQL_ASSOC))
		{
		$n_id = $donnees['Id']-1;
		query('UPDATE Sections_forum SET Id=\''.$n_id.'\'') or die(mysql_error());
		}
	mysql_free_result($sql);
	$Reponse->addScript('xajax_Afficher_sections_forum();');
	return $Reponse->getXML();
	}

J'ai déjà écris cette fonction de plusieurs façons et je rencontre toujours le même problème:

J'ai l'impression que le critère de sélection ">" n'est pas pris en compte car la fonction modifie aussi bien l'Id 1 que la 3, alors qu'elle ne devrait modifier que la 3.

Bon je pense que mon niveau doit se voir via mon sujet mais je précise quand même que je suis débutant.

Enfin, quoi qu'il en soit je vous remercie.

[Djakisback]

Salut,
le problème vient peut-être du fait que ton id est un int et que tu le compares avec un string :
WHERE Id > \''.$id.'\'
au final tu as :
WHERE Id > '2'

Essaie d'afficher le résultat du select pour être sûr.
D'autre part tu n'as pas besoin de toutes ces requêtes, une seul suffit :

Code :

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

$sql = "UPDATE table SET Id = Id - 1 WHERE Id > ".$id;

[Edit] Je crois d'ailleurs qu'il y a une commande Sql qui permet de réorganiser les Id mais je ne sais plus laquelle.[/]
En fait je me trompe, ca permet de réordonner la table mais ca ne décale pas les ID : ALTER TABLE `items` ORDER BY `id` (en autre, de recalculer l'auto-increment)

Bye

[trotters213]

Essaie comme ceci :

Code :

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

"... WHERE Id > CAST('".$id."' AS UNSIGNED)"

Juste une question, pourquoi tu ne pars pas de la fin ? tu prend le dernier enregistrement et déplace à la la place du premier trou : ça serait plus simple non (et surtout plus rapide) ?

PS : utilise les doubles-quotes plutôt que les simples pour tes requêtes, c'est beaucoup plus lisible (mais plus lent je sais mais bon tu ne fais 10000 requêtes si )

[Squalthor]

Bonjour Et grand merci pour votre aide !

Voilà, j'ai utilisé vos conseils pour réecrire la fonction, ce qui donne ceci:

Code :

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

1
2
3
4
5
6
7
8
9
 
function Effacer_section($id)
	{
	$Reponse = new xajaxResponse();
	query('DELETE FROM Sections_forum WHERE Id=\''.$id.'\'');
	$sql = 'UPDATE Sections_forum SET Id = Id-1 WHERE Id > CAST(\''.$id.'\' AS UNSIGNED';
	$Reponse->addScript('xajax_Afficher_sections_forum();');
	return $Reponse->getXML();
	}

Mais malheureusement ça ne marche toujours pas. L'entrée à effacer est correctement effacée mais hélas les id devant être décrémentées ne le sont pas.

Comme tu peux le voir, Trotters213, j'adore optimiser, de plus je me suis habitué à ne pas utiliser les doubles quotes donc ça ne me gêne nullement.

Ta solution (d'utiliser simplement la dernière entrée) pourrait être plus pratique c'est vrai mais comme je me sert des Id pour classer les sections elle n'est pas applicable dans ce cas.

Je vous remercie, encore une fois je sais , pour votre aide.

[trotters213]

Tu as essayé de faire comme a dit Djakisback

Code :

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

query('UPDATE Sections_forum SET Id='.$id)

?

[Squalthor]

Merci beaucoup !

Maintenant ça marche, le seul dommage c'est que je n'ai pas vraiment compris pourquoi ça ne marchais pas.

Voici la fonction, ça pourrais servir à quelqu'un d'autre que moi:

Code :

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

1
2
3
4
5
6
7
8
9
 
function Effacer_section($id)
	{
	$Reponse = new xajaxResponse();
	query('DELETE FROM Sections_forum WHERE Id=\''.$id.'\'');
	query('UPDATE Sections_forum SET Id= Id - 1 WHERE Id > '.$id);
	$Reponse->addScript('xajax_Afficher_sections_forum();');
	return $Reponse->getXML();
	}

Encore une fois, quand je butais, vous avez pût m'aider.

Merci encore et bon courage à vous et à la prochaine.

(dommage que ce ne soit que quand j'ai des problèmes)

[trotters213]

ça ne marchait car tu mettait ton $id entre simples quotes : tu faisait ceci

Code :

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

query('UPDATE Sections_forum SET Id=\''.$n_id.'\'')

donc la requête était (pour un $id=2)

Code SQL :

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

UPDATE Sections_forum SET Id='2'

donc SQL prenait le 2 comme une chaine de caractères et non comme comme un entier.
En supprimant les simples quotes, tu as

Code SQL :

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

UPDATE Sections_forum SET Id=2

donc 2 est interprété comme un entier et plus une chaine comme la comparaison se fait correctement.

See you soon.

[Squalthor]

Ah, oui désolé je ne savais pas ça.

Merci, cette fois c'est compris et bien compris, je ne suis pas prêt d'oublier ça

Au plaisir et à la prochaine pour de nouvelles mésaventures...