Renumérotation d'une liste dans une BDD à partir de PHP

Version imprimable

11/05/2016, 11h06
tbecker

Renumérotation d'une liste dans une BDD à partir de PHP

Bonjour,

après de nombreuses recherches je n'arrive pas à trouver mon bonheur.

J'ai créé un site avec une partie administrateur dans laquelle je peux écrire des articles pour qu'ils soient visibles dans la zone publique.

Je souhaite créer un bouton qui me permet de remonter l'article dans la liste afin de le positionner selon mon souhait.

Pour cela j'ai créé une colonne affichage dans ma BDD. La valeur 0 correspond à un article non visible, puis ensuite ils sont affichés dans l'ordre de la numérotation.

Je souhaite :
- Etre capable de renuméroter cette liste au besoin avec une série de chiffres qui se suivent, un trou peu arriver lorsque je supprime définitivement un article, mais je n'ai rien trouvé qui permet de le faire si ce n'est agir dans mysql directement.
- Etre capable de faire remonter un article, c'est à dire diminuer son numéro d'affichage pour le substituer à l'article précédent qui prendra la numérotation de l'article déplacé.

Pour ces deux demandes, je sèche.

Merci de votre aide
11/05/2016, 12h48
Dendrite

2 ou 3 exemples valent mieux que de longues explications...
ET aussi... ton code please...

Salut,

quelque chose dans ce genre :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
$data = range('a', 'z');  // [0 => 'a', 1 => 'b', ..., 25 => 'z']
 
// supprime la lettre a et réordonne les clés
unset($data[0]);
$data = array_values($data);  // maintenant 0 => b
 
// monter d'un cran la place de la lettre m
$i = array_search('m', $data, true); // recherche de l'index de la lettre
if (($i !== false) && ($i > 0))
{
    list($data[$i], $data[$i - 1]) = [$data[$i - 1], $data[$i]];    // swap des valeurs
}

11/05/2016, 21h09
tbecker

@Dendrite

Désolé, je n'ai pas de code pour cette partie là car je sèche complètement
11/05/2016, 21h13
tbecker

@rawsrc

Merci pour cet exemple.

Si je comprends bien, j'importe ma liste dans un tableau en php, puis je traite en php selon l'exemple donné et enfin je réinvente dans la base à l'issue.
11/05/2016, 23h51
rawsrc

Pas tout à fait, je t'ai donné la logique que tu dois adapter à la base de données

Perso, je ne m'occuperais pas des problèmes de trous dans la numérotation, un moteur de base de données s'en tamponne (du moment que ton affichage tient la route)

Pour le changement d’ordonnancement en base de données, il va te falloir gérer cette problématique soit avec 2 requêtes soit avec une procédure stockée.
12/05/2016, 10h30
andry.aime

Citation:

Envoyé par rawsrc

Pour le changement d’ordonnancement en base de données, il va te falloir gérer cette problématique soit avec 2 requêtes soit avec une procédure stockée.

S'il ne s'agit que deux valeurs:

Code:

UPDATE table SET colonne = IF(colonne='valeur1','valeur2','valeur1') WHERE colonne IN ('valeur1','valeur2');

A+.

Bonjour à tous,
suite à tous vos conseils je pense être arrivé à quelque chose qui tient la route.

Voici les éléments

Sur la page où j'ai mes articles :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
<?php
...
    if ($result->affichage != recherche_min())
	{
	echo('<li><a href=permuter_articles.php?pos_origine='.$result->affichage.'&type_permutation=up&page='.$result->lieu_affichage.'>remonter dans la liste</a></li>');
	}
    if ($result->affichage != recherche_max())
	{
	echo('<li><a href=permuter_articles.php?pos_origine='.$result->affichage.'&type_permutation=down&page='.$result->lieu_affichage.'>baisser dans la liste</a></li>');
	}
...
?>

Les fonctions nécessaires :

Code:

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
 
<?php
 
    function recherche_max(){
    include('cnx_bdd.php');
 
    $sql_max = "SELECT MAX(affichage) AS affichage_max FROM post_page_accueil WHERE `lieu_affichage`='type' AND affichage!=0";
    $res_max = mysql_query($sql_max) or die('Erreur SQL !<br>'. $sql_max.'<br>'.mysql_error());
    $data_max = mysql_fetch_array($res_max);
 
    return $data_max[0];
	mysql_close();
    }
 
    function recherche_min(){
    include('cnx_bdd.php');
 
    $sql_min = "SELECT MIN(affichage) AS affichage_min FROM post_page_accueil WHERE `lieu_affichage`='type' AND affichage!=0";
    $res_min = mysql_query($sql_min) or die('Erreur SQL !<br>'. $sql_min.'<br>'.mysql_error());
    $data_min = mysql_fetch_array($res_min);
 
    return $data_min[0];
	mysql_close();
    }
?>

permuter_articles.php :

Code:

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
<?php
 
    // declaration du tableau
    $tab_position = array();
 
    // recuperation des donnees de la page
   $position_origine=$_GET['pos_origine'];
   $type_permutation=$_GET['type_permutation'];
   $location=$_GET['page'];
 
    //connexion serveur et selection base
    include('cnx_bdd.php');
 
    //affectation des valeurs d affichage dans un tableau
    $sql ='SELECT affichage FROM table WHERE lieu_affichage="type" AND affichage!=0 ORDER BY affichage';
    $req = mysql_query( $sql, $cnx ) ;
    $i=1;
    while ($donnee = mysql_fetch_array($req))
	{
      $tab_position[$i] = $donnee[0];
      $i++;
    }
 
    // recherche de la position d'origine
    $k=1;
    while ($position_origine!=$tab_position[$k])
	{
	  $k++;
	}
 
    // inversion des valeurs dans la BDD
    if($type_permutation=='up') // remonter dans la liste
	  {
		$sql_inversion='UPDATE post_page_accueil SET affichage = IF(affichage='.$tab_position[$k-1].','.$tab_position[$k].','.$tab_position[$k-1].') WHERE affichage IN ('.$tab_position[$k-1].','.$tab_position[$k].')';
		$req_inversion = mysql_query( $sql_inversion, $cnx ) ;	
	  }
	  else //descendre dans la liste
	  {
		$sql_inversion='UPDATE post_page_accueil SET affichage = IF(affichage='.$tab_position[$k].','.$tab_position[$k+1].','.$tab_position[$k].') WHERE affichage IN ('.$tab_position[$k].','.$tab_position[$k+1].')';
		$req_inversion = mysql_query( $sql_inversion, $cnx ) ;		
	  }
	mysql_close();
?>
   // retour sur la page d'origine 
   <script language="JavaScript">
		 var retour_page = '<?php echo $location ?>';
		 var adresse_redirection = "window.location='adresse de la page"+retour_page+"'";
		 setTimeout(adresse_redirection,0); // delai en millisecondes
    </script>

Je suis dans l'attente de vos remarques éventuelles au niveau sécurité notamment

Toutefois déjà merci de votre aide ;)

18/05/2016, 17h24
Celira
Remarques en vrac :
- L'API Mysql est obsolète et supprimée en PHP7. Il va donc falloir remplacer toutes tes fonction mysql_xxx, soit en utilisant mysqli, soit avec PDO. :fleche: Comprendre PDO
- Tu n'as pas besoin d'ouvrir et de fermer la connexion à la base à chaque requête. L'ouvrir une fois en début de script et la fermer à la fin est suffisant.
- Pour faire une redirection immédiate, utilise plutôt header :fleche: :faq: Comment faire une redirection ?
- Dans tes fonctions, tu as des choses après les return. Or le return fait sortir de la fonction. Donc tout ce qui se trouve après ne sera pas exécuté.
18/05/2016, 21h42
tbecker

Bonsoir,

Citation:

L'API Mysql est obsolète et supprimée en PHP7...

-> Je l'admet il va falloir que je me mette à jour sur les procédure :oops:

Citation:

- Pour faire une redirection immédiate ...

-> J'ai essayé avec header() mais je galère sur ce sujet malgré les exemples et explication, ça plante :arf:

Citation:

- Tu n'as pas besoin d'ouvrir et de fermer la connexion à la base à chaque requête. L'ouvrir une fois en début de script et la fermer à la fin est suffisant.
- Dans tes fonctions, tu as des choses après les return ...

->Merci des conseils je vais faire le nécessaire :P

J'annonce le sujet résolu, encore merci à tous :merci::ccool:

Salut,

tu peux factoriser :

Code:

1
2
3
4
5
6
$html = '<li><a href=permuter_articles.php?pos_origine=%s&type_permutation=%s&page=%d>%s dans la liste</a></li>';
 
if ($result->affichage != recherche_min())
    printf($html, $result->affichage, 'up', $result->lieu_affichage, 'remonter');
elseif ($result->affichage != recherche_max())
    printf($html, $result->affichage, 'down', $result->lieu_affichage, 'baisser');