Discussion :

[okoweb]

Je souhaite créer une fonction qui gère les clauses ORDER BY et LIMIT. J'ai donc fait ceci :

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
 
function getSelection($identifiant_selection, $rang = true, $rand = false, $limit = false) {
	global $db;
	$orderby = '';
	if($rang == false and $rand == false) $orderby .= 'si.rang_item';
	if($rang == true and $rand == false) $orderby .= 'si.rang_item';
	if($rang == false and $rand == true) $orderby .= 'rand()';
	if($rang == true and $rand == true) $orderby .= 'rand(), si.rang_item';
	$sql = 'SELECT si.url_item as url, si.rang_item as rang FROM selections_items si
		INNER JOIN selections s ON s.id_selection = si.selection_id
		WHERE s.identifiant_selection = :identifiant_selection ';
	$sql .= 'ORDER BY '.$orderby;
	if($limit != false) $sql .= ' LIMIT '.$limit;
	$statement = $db->prepare($sql);
	$statement->execute(array(':identifiant_selection' => $identifiant_selection));	
	return $statement->fetchAll();
}
 
//ceci marche
getSelection(2);
 
//ceci ne marche pas
getSelection(2, $limit = 20);

En dehors de $identifiant_selection, les autres arguments semblent pas être pris en compte par la fonction.

Merci d'avance pour votre aide.

Meilleures salutations.

[okoweb]

Pour être plus simple, je souhaite une fonction qui me permette de régler sur mesure une requête SELECT, les clauses ORDER BY et LIMIT, et la fonction RAND() de MySQL.

Code :

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

1
2
3
4
 
function mafonction($rang = true, $rand = false, $limit = false){
    $sql = 'SELECT * FROM matable';
}

Merci d'avance...

[Benjamin Delespierre]

J'ai déjà fait ça dans Axiom; regarde cette classe, ça te donnera des idées: https://github.com/bdelespierre/php-...ject.class.php

[okoweb]

J'ai déjà fait ça dans Axiom; regarde cette classe, ça te donnera des idées: https://github.com/bdelespierre/php-...ject.class.php

J'ai regardé merci, mais la POO c'est pas mon truc.

[Benjamin Delespierre]

C'est pas beaucoup plus compliqué en procédural:

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
function generate_select_query ($connection, $type, $fields, $table, array $where = array(), array $group_by = array(), array $limit = array()) {
  $query = "SELECT " . implode(',', $fields) . " FROM {$table}";
 
  if (!emtpy($where)) {
    $pieces = array();
    foreach ($where as $key => $value) {
      $value = mysql_real_escape_string($value, $connection);
      $pieces[] = "$key=$value";
    }
    $query .= " WHERE " . implode(' AND ', $pieces);
  }
 
  if (!empty($group_by))
    $query .= " GROUP BY " . implode(',', $group_by);
 
  if (count($limit) == 1)
    $query .= " LIMIT {$limit[0]}";
  if (count($limit) == 2)
    $query .= " LIMIT {$limit[0]},{$limit[1]}";
 
  return $query;
}

[okoweb]

C'est pas beaucoup plus compliqué en procédural:

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
function generate_select_query ($connection, $type, $fields, $table, array $where = array(), array $group_by = array(), array $limit = array()) {
  $query = "SELECT " . implode(',', $fields) . " FROM {$table}";
 
  if (!emtpy($where)) {
    $pieces = array();
    foreach ($where as $key => $value) {
      $value = mysql_real_escape_string($value, $connection);
      $pieces[] = "$key=$value";
    }
    $query .= " WHERE " . implode(' AND ', $pieces);
  }
 
  if (!empty($group_by))
    $query .= " GROUP BY " . implode(',', $group_by);
 
  if (count($limit) == 1)
    $query .= " LIMIT {$limit[0]}";
  if (count($limit) == 2)
    $query .= " LIMIT {$limit[0]},{$limit[1]}";
 
  return $query;
}

On peut voir un exemple d'utilisation?
Merci d'avance...

[rawsrc]

Bonjour,

J'ai repris ton code et on devrait arriver au même résultat à ceci près c'est que tu devrais le sécuriser un peu et surtout ne pas présumer que ta requête se déroulera sans aucun problème. Vu que tu utilise une ressource (PDO), rien ne te garantit qu'elle est diponible ou que ta requête est montée sans erreur d'où l'exécution dans un try-catch{}

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
<?php
 
function getSelection($identifiant_selection, $rang = true, $rand = false, $limit = false) {
   global $db;
   $orderBy = array();
 
   if ($rand) {
      $orderBy[] = 'rand()';
   }
   if ($rang) {
      $orderBy[] = 'si.rang_item';
   }
   $sqlOby = ' ORDER BY ' . ((empty($orderBy)) ? 'si.rang_item' : implode(', ', $orderBy));
   $sqlLim = ($limit) ? ' LIMIT ' . $limit : NULL;
 
   $sql = <<<SQL
SELECT si.url_item as url, si.rang_item as rang
FROM selections_items si
   INNER JOIN selections s ON s.id_selection = si.selection_id
WHERE s.identifiant_selection = :identifiant_selection
$sqlOby
$sqlLim
SQL;
 
   try {
      $stmt = $db->prepare($sql);
      $exec = $stmt->execute(array(':identifiant_selection' => intval($identifiant_selection)));
      return ($exec) ? $stmt->fetchall() : array();
   }
   catch (Exception $e) {
      // traiter l'erreur ici
 
   }
}
 
?>

[okoweb]

try-catch{}

Merci.
J'ai essayé d’alléger le 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
 
function mafonction($rang = true, $sort = false, $rand = false, $limit = false) {
   $orderBy = array();
   if ($rand) $orderBy[] = 'rand()';
   if ($rang) $orderBy[] = 'monchamp'. ($sort) ? '' : ' DESC';
   $sqlOby = ' ORDER BY ' . ((empty($orderBy)) ? 'monchamp' : implode(', ', $orderBy));
   $sqlLim = ($limit) ? ' LIMIT ' . $limit : NULL;
   $sql = <<<SQL
SELECT * FROM matable
$sqlOby
$sqlLim
SQL;
   echo $sql;
}
mafonction($limit = 12) //Resultat SELECT * FROM matable ORDER BY

Les paramètres ne passent pas.

[rawsrc]

Les paramètres ne passent pas.

Avec ton code c'est tout à fait normal : problème de précédence des opérateurs ici :

Code :

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

if ($rang) $orderBy[] = 'monchamp'. ($sort) ? '' : ' DESC';

Regardes ici
La concaténation est prioritaire sur l'opérateur ternaire.
Ton code revient à faire ceci :

Code :

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

if ($rang) $orderBy[] = ('monchamp' . ($sort)) ? '' : ' DESC';

Dans ce cas, la concaténation renverra TRUE et $orderBy[] sera rempli avec "".

Dès que tu as un doute, mets des parenthèses.

Code corrigé :

Code :

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

if ($rang) $orderBy[] = 'monchamp' . (($sort) ? '' : ' DESC');

Et je te conseillerai de prendre l'habitude de mettre des { } même pour une ligne c'est comme pour le parenthèses cela t'évitera des longues heures de débogage.

Code :

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

1
2
3
if ($rang) { 
   $orderBy[] = 'monchamp' . (($sort) ? '' : ' DESC');
}

[okoweb]

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
 
 
function mafonction($rang = true, $sort = false, $rand = false, $limit = false) {
   $orderBy = array();
   if ($rand) $orderBy[] = 'rand()';
   if ($rang) $orderBy[] = 'monchamp'. (($sort) ? ' ASC' : ' DESC');
   $sqlOby = ' ORDER BY ' . ((empty($orderBy)) ? 'monchamp' : implode(', ', $orderBy));
   $sqlLim = ($limit) ? ' LIMIT ' . $limit : NULL;
   $sql = <<<SQL
SELECT * FROM matable
$sqlOby
$sqlLim
SQL;
   echo $sql;
}
mafonction($sort = true, $limit = 12); //SELECT * FROM matable ORDER BY monchamp ASC

$sqlOby marche parfaitement maintenant mais $sqlLim ne marche toujours pas.

Merci d'avance...

[rawsrc]

$sqlLim ne marche toujours pas.

Ah bon, chez moi quand je fais :

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
<?php
 
function mafonction($rang = true, $sort = false, $rand = false, $limit = false) {
   $orderBy = array();
   if ($rand) $orderBy[] = 'rand()';
   if ($rang) $orderBy[] = 'monchamp'. (($sort) ? ' ASC' : ' DESC');
   $sqlOby = ' ORDER BY ' . ((empty($orderBy)) ? 'monchamp' : implode(', ', $orderBy));
   $sqlLim = ($limit) ? ' LIMIT ' . $limit : NULL;
   $sql = <<<SQL
SELECT * FROM matable
$sqlOby
$sqlLim
SQL;
   echo $sql;
}
mafonction(true, true, false, 12);
?>

Je récupère :

Code :

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

SELECT * FROM matable ORDER BY monchamp ASC LIMIT 12

Donc ça marche.

[okoweb]

Code :

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

1
2
 
mafonction(true, true, false, 12);

Si je comprends bien, à chaque appel de la fonction, il faut renseigner obligatoirement tous ses paramètres.

Merci d'avance...

[rawsrc]

Si je comprends bien, à chaque appel de la fonction, il faut renseigner obligatoirement tous ses paramètres.

Pas obligatoirement, tout dépend de l'ordre des paramètres avec des valeurs par défaut (dits paramètres optionnels).
Un paramètre sans valeur par défaut est obligatoire et un avec est optionnel.
On place généralement les paramètres obligatoires ou changeant souvent au début de déclaration et on termine par les paramètres optionnels.
Si ta déclaration avait été :

Code :

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

function mafonction($limit = false, $rang = true, $sort = false, $rand = false)

Ton appel aurait pu s'écrire tout simplement :

Code :

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

mafonction(12);

Les paramètres optionnels restant ($rang, $sort, $rand) auront leur valeur par défaut dans le corps de la fonction.
Maintenant si tu dois modifier la valeur d'un paramètre en fin de déclaration, tu n'as pas d'autre choix que de renseigner tous les paramètres (même optionnels) le précédant.

Une dernière chose, les paramètres nommés ne sont pas pris en charge par PHP. Avec cette déclaration :

Code :

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

function mafonction($rang = true, $sort = false, $rand = false, $limit = false)

Ton appel :

Code :

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

mafonction($sort = true, $limit = 12)

n'affecte pas du tout true à $sort et 12 à $limit. Cela revient à ceci :

Code :

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

1
2
3
4
5
mafonction(true, 12)
// $rang = true
// $sort = 12
// $rand = false (valeur par défaut)
// $limit = false (valeur par défaut)

En espérant avoir été clair.

[okoweb]

Merci à vous tous.