Formulaire et gestion de requête SQL

Version imprimable

10/08/2018, 21h46
elkaktus

1 pièce(s) jointe(s)

Formulaire et gestion de requête SQL

Bonsoir à tous,

Je développe actuellement un système de suivi en PHP qui compte un nombre de véhicules par jour, semaine et mois. Pour récupérer les données, j'exécute trois requêtes, une par jour, une par semaine et une par mois (je crée trois tableaux que je remplis avec une boucle sur le résultat de ma requête).

Dans mon programme, j'ai une en-tête de "filtrage". En effet, je souhaite qu'en fonction des choix de l'utilisateur, j'exécute telle ou telle requête. Si l'utilisateur laisse tout les champs vide, on sort la requête globale. Si par exemple il choisit uniquement le champ Semaine et le champ Métier, on exécute la requête en fonction. Etc...

Pièce jointe 403087
(l'en tête en question).

Je ne vois pas trop comment faire ça, si quelqu'un a des pistes pour m'aiguiller...
Merci d'avance,

Bonne soirée
10/08/2018, 22h46
Manequin

Bonsoir, poste tes bouts de codes, tes requêtes et ta BD.

CORDIALEMENT.

Requetes SQL

Bonjour,

Donc voici les requêtes (au format SQLite) :

Pour les jours :

Code:

1
2
3
4
 $result = $database->query("SELECT * FROM retoucheurs where substr(\"Date/Heure de retouche (FD)\",1,4)||'-'||
            substr(\"Date/Heure de retouche (FD)\",6,2)||'-'||
            substr(\"Date/Heure de retouche (FD)\",9,2) 
            between DATE('". date("Y-d-m", strtotime($_SESSION['jours'][0])). "') and DATE('". date("Y-d-m", strtotime($_SESSION['jours'][6]))."') ORDER BY \"Date/Heure de retouche (FD)\" DESC");

Pour les semaines :

Code:

1
2
3
4
5
 $result = $database->query("select total(Contremarque) as Tot, 
    strftime('%Y-%m-%d', substr(\"Date/Heure de retouche (FD)\",1,4)||\"-\"||substr(\"Date/Heure de retouche (FD)\",6,2)
    ||\"-\"||substr(\"Date/Heure de retouche (FD)\",9,2)) as Week FROM retoucheurs 
    GROUP BY strftime('%W', substr(\"Date/Heure de retouche (FD)\",1,4)||\"-\"||substr(\"Date/Heure de retouche (FD)\",6,2)
    ||\"-\"||substr(\"Date/Heure de retouche (FD)\",9,2));");

Pour les mois :

Code:

1
2
3
4
5
$result = $database->query("select total(Contremarque) as Tot, 
    strftime('%Y-%m-%d', substr(\"Date/Heure de retouche (FD)\",1,4)||\"-\"||substr(\"Date/Heure de retouche (FD)\",6,2)
    ||\"-\"||substr(\"Date/Heure de retouche (FD)\",9,2)) as Month  FROM retoucheurs 
    GROUP BY strftime('%m', substr(\"Date/Heure de retouche (FD)\",1,4)||\"-\"||substr(\"Date/Heure de retouche (FD)\",6,2)
    ||\"-\"||substr(\"Date/Heure de retouche (FD)\",9,2));");

Pour chaque requête, je boucle ensuite sur ma variable $result avec une boucle while pour stocker le tout dans un tableau que j'affiche dans un graphique :

Code:

1
2
3
4
5
6
7
while($row=$result->fetchArray(SQLITE3_ASSOC)) {
        $valmois[intval(strftime('%m', strtotime($row['Month'])))] = $row['Tot'];
    }
for($i=0;$i<7;$i++) {
        $_SESSION['valsm'][$i] = $valmois[intval(strftime('%m', strtotime(date("Y-m-d", strtotime($moisJours[$j]->format('Y-m-d'), 1)))))]; 
        $j--;   
    }

Ce qui m'interesse le plus c'est de pouvoir dynamiquement modifier mes requêtes avec les valeurs choisies, ou lancer une requête par défaut si aucune valeur n'a été choisie.

Merci

11/08/2018, 08h49
Invité

Bonjour,

:recherch: "formulaire de recherche multi-critères"

Citation:

...Ce qui m'interesse le plus c'est de pouvoir dynamiquement modifier ...

On suppose donc "sans rechargement de page".

Dans ce cas : :recherch: "Ajax"

Bonjour, j'avais eu le même soucis et j'avais eu la solution en faisant ça.

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
 
<?php
 
	/*je récupère les critères sélectionnés*/
		extract($_POST);
 
	/* je crée le tableau choix pour stocker les différentes critères*/
		$choix = array();
 
	// si la variable est présente, 
//on lui affecte une place dans le tableau 'choix[]',
//qui nous servira ensuite à construire le WHERE de la requête.
	if (!empty($semaine)){
		$choix[] = "semaine LIKE '%$semaine%'";   
	}
 
	if (!empty($tournee)){
		$choix[] = "tournee LIKE '%$tournee%'";
	}
 
	if (!empty($metier)){
		$choix[] = "metier LIKE '%$metier%'";
	}
 
	if (!empty($retoucheur)){
		$choix[] = "retoucheur LIKE '%$retoucheur%'";
	}
 
	$first = true; /*on déclare la variable first pour tester la première condition*/
	$conditions = '';
 
	foreach($choix as $c){
    /* si c'est la premiere condition est vérifiée, on met where, sinon on met and*/
    if ($first){
        $first = false;
        $conditions .= " WHERE $c ";
    }
		else{
			$conditions .= " AND $c ";
		}
	}
 
	$sql = " SELECT * FROM ta_table $conditions ORDER BY id ";
 
	$res = mysql_query($sql) or die(mysql_error());
 
?>

si je ne suis pas pas dans la bonne voie, chèr(e) modérateur(trice) corrigez moi.

Cordialement.

11/08/2018, 09h03
elkaktus

Merci pour la réponse, et désolé la prochaine fois je ferais un peu plus de recherche.
Je vais potasser tout ça
11/08/2018, 09h12
Invité

1- Quand je mets le picto :recherch:, ce n'est pas un reproche : je te fournis les mots-clé pour ta recherche.

2- Les listes déroulantes restent des éléments de formulaire !

Code:

<option>ELE</option>

C'est correct, mais il vaut mieux toujours mettre une value :

Code:

<option value="ELE">ELE</option>

3- Dans ton cas, si tu veux utiliser Ajax, les balises <form> ne sont effectivement pas nécessaires, puisque les données seront récupérées et transmises via JavaScript.
Ajax se charge ensuite de les trnasmettre à un fichier PHP externe (indépendant), qui fera le traitement.

4- (EDIT) Je n'avais pas regardé tes requêtes : 8O
Appeler une colonne "Date/Heure de retouche (FD)" relève du masochisme ! :sm:
Les autres colonnes ont aussi des noms à coucher dehors ?
11/08/2018, 09h15
badaze

Je ne connais pas SQL Lite mais de ce que j’ai pu voir les dates sont stockées (entre auttres) comme du texte au format SSAA-MM-JJ HH:MM:SS ('2018-04-02 12:13:46')

Donc pourquoi faire tous ces substr ? S’ils sont nécessaires un substr de 1,10 devrait suffire et allègerait grandement la compréhension des requêtes.

@Manequin
ton code est bon, mais il peut être optimisé :

Remplace :

Code:

1
2
3
4
5
6
7
8
9
10
	foreach($choix as $c){
    /* si c'est la premiere condition est vérifiée, on met where, sinon on met and*/
    if ($first){
        $first = false;
        $conditions .= " WHERE $c ";
    }
		else{
			$conditions .= " AND $c ";
		}
	}

Par :

Code:

$conditions = (!empty($choix))? " WHERE " . implode( " AND ", $choix ) : "";

11/08/2018, 09h19
elkaktus

Citation:

3- Dans ton cas, si tu veux utiliser Ajax, les balises <form> ne sont effectivement pas nécessaires, puisque les données seront récupérées et transmises via JavaScript.
Ajax se charge ensuite de les trnasmettre à un fichier PHP externe (indépendant), qui fera le traitement.

Je ne connais pas du tout Ajax ni JavaScript du coup j'essaie de faire ça en PHP.

Citation:

Donc pourquoi faire tous ces substr ? S’ils sont nécessaires un substr de 1,10 devrait suffire et allègerait grandement la compréhension des requêtes.

Dans ma base les dates sont stockées en texte sous la forme YYYY/MM/DD et SQLite ne prend en charge que le format YYYY-MM-DD, du coup je dois substituer les '/' par des '-'. C'est pas le plus simple je l'admets...mais ça marche.

En tout cas merci à tous pour votre aide, particulièrement à jreaux62 et Manequin
11/08/2018, 09h23
Manequin

Citation:

Envoyé par jreaux62

Code:

$conditions = (!empty($choix))? " WHERE " . implode( " AND ", $choix ) : "";

Une préoccupation, quel le rôle d'implode?

Cordialement.
11/08/2018, 09h25
Invité

Citation:

Envoyé par elkaktus

...j'essaie de faire ça en PHP...

Alors, il y aura forcément rechargement de la page.

Mais le fichier de traitement PHP, dans les 2 cas, sera sensiblement le même.

Donc commence comme ça, et quand tu connaitras mieux JavaScript/Ajax, ce ne sera pas difficile à transposer.

Citation:

Envoyé par Manequin

Une préoccupation, quel le rôle d'implode?

Facile : :recherch: "PHP implode" :aie:

En résumé, ça fait la même chose que ton bout de script de 10 lignes,... mais en UNE SEULE ligne : concaténer les conditions dans le " WHERE.... AND ... AND ...".
11/08/2018, 09h32
Invité

Citation:

Envoyé par elkaktus

Dans ma base les dates sont stockées en texte sous la forme YYYY/MM/DD
et SQLite ne prend en charge que le format YYYY-MM-DD...

As-tu essayé avec SQLite REPLACE() function ?

Code:

REPLACE(\"Date/Heure de retouche (FD)\", \"/\",\"-\")
11/08/2018, 09h41
Manequin

Merci beaucoup @jreaux, t'es genial. Ça fonctionne hyper bien. ça marche .

Cordialement.
11/08/2018, 09h46
badaze

Une chose qu’on oublie souvent. Un mois ou une semaine appartiennent à une année et dans les requêtes je n’ai pas vu d’année dans la clause where.

@Manequin
:alerte: Rectification : ta requête N'EST PAS SÉCURISÉE (tu passes les variables directement dans la requête !
Il faut faire une REQUÊTE PRÉPARÉE !

PRINCIPE DE RECHERCHE MULTI-CRITERES :

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
<?php
// ---------------
// 1- Initialisation / Récupération des données du formulaire
if(...)
{
... 
}
// ---------------
// 2- Construction des bouts de requête, en fonction des criteres recus :
// on crée un array $sqlCritere pour stocker les morceaux de requêtes
$sqlCritere = array();
// on crée un array $sqlParams, destinés à être passé dans le execute()
$sqlParams = array();
// ---------------
// si la variable est présente, on lui affecte une place dans le tableau 'critere[]',
//qui nous servira ensuite à construire le WHERE de la requête.
if (!empty($critere1))
{
	$sqlCritere[] = " colonne1 LIKE ?";   
	$sqlParams[] = '%'.$critere1.'%';   
}
if (!empty($critere2))
{
	$sqlCritere[] = " colonne2 = ?";   
	$sqlParams[] = $critere2;   
}
...
// ---------------
// 3- Construction du WHERE ... AND ... :
$sqlWhereAnd = (!empty($choix))? " WHERE " . implode( " AND ", $choix ) : "";
// ---------------
// 4- Construction de la requete PREPAREE :
$sql = " SELECT * FROM ta_table " . $sqlWhereAnd . " ORDER BY id ";
 
$res = mysql_prepare( $sql ); // PREPARATION
$res = mysql_execute( $sqlParams ); // EXECUTION, en passant les paramètres
?>

@elkaktus
:alerte: idem : tes requêtes NE SONT PAS SÉCURISÉES (tu passes aussi les variables directement dans la requête !
Il faut faire des REQUÊTES PRÉPARÉES.

INSPIRE-TOI du PRINCIPE DE RECHERCHE MULTI-CRITERES ci-dessus.

11/08/2018, 17h01
Manequin

Oh la la, je le fait tout de suite. Merci.
Cordialement.
11/08/2018, 17h23
Invité
Remarque concernant empty() :
Citation:
empty :
Détermine si une variable est considérée comme vide.
Une variable est considérée comme vide si elle n'existe pas, ou si sa valeur équivaut à FALSE. La fonction empty() ne génère pas d'alerte si la variable n'existe pas.

Ce qui suit est considéré comme étant vide :

"" (une chaîne vide)
0 (0 en tant qu'entier)
0.0 (0 en tant que nombre à virgule flottante)
"0" (0 en tant que chaîne de caractères)
NULL
FALSE
array() (un tableau vide)
$var; (une variable déclarée, mais sans valeur)

!empty :
C'est donc l'inverse
Donc :
Code:

1 2 3 4 5 if (!empty($critere1)) { $sqlCritere[] = " colonne1 LIKE ?"; $sqlParams[] = '%'.$critere1.'%'; }
- si le critère peut prendre comme valeur 0 ou '' (chaine vide) ou false ou NULL, il faut alors utiliser isset(...) à la place de !empty(...) !
Sinon, on n'entre pas dans la condition, et le bout de requête correspondant n'est pas généré.