[MySQL - PhP - Propel] Génération de requêtes personnalisés

**leya_jarhan** · 08/08/2012, 11h13

Bonjour,

Je me trouve confronté à un problème assez "conséquent". Je m'explique.

Technologies utilisées : Php, Symfony 1.0, MySQL, Propel

Les utilisateurs doivent pouvoir via une interface, générer automatiquement des requêtes personnalisées sur la base de donnée et de renvoyer un CSV avec les informations demandées via checkbox.

Exemple : Extraction personnalisée sur Individu

Doit me renvoyer une requête du genre
Civilite = Mme AND (Pays = France OR Pays = BELGIQUE)

Pour l'instant, je générais des requêtes via les criterias de Propel en regardant l'opérateur à n-1 pour ajouter un criteria addOr au lieu de addAnd.

Le problème c'est que pour cet exemple il me sort alors :
(Civilite = Mme AND Pays = France) OR Pays = BELGIQUE

Ce qui fait que je me retrouve avec des Mr dans ma requête.

On peut ajouter des critères de sélection via un bouton, ce qui rajoute une ligne avec un opérateur sur la ligne précédente.

Les opérateurs proposés : ET, OU, SAUF

Le problème c'est que je ne vois pas comment construire le schéma basique qui me permettrait de générer, par la suite, une requête personnalisée plus longue et plus complexe.
Je dois prendre en compte tout les schémas possible et je n'arrive pas à créer une fonction qui permette que toutes mes requêtes soient correctement organisé en fonction de mes opérateurs (ET, OU, SAUF).

Extrait de mon code de traitement :

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
	foreach($types as $key => $type){
		// Recherche de loperateur
		$operateur = 'addAnd';
 
		if ($key > 0 && $operateurs[$key-1] == 'OU') {
			$operateur = 'addOr';
		}
 
		$crit = 'EQUAL';
 
		if($key > 0 && $criterias[$key]!=''){
			if ($operateurs[$key-1] == 'SAUF') {
				$crit = 'NOT_EQUAL';
			}
		}else{
			if ($key > 0 && $operateurs[$key-1] == 'SAUF') {
				$crit = 'ISNOTNULL';
			}else{
				$crit = 'ISNULL';
			}
		}
 
 
 
		switch ($type) {
		//en fonction des types je génére des critérias exemple:
		    case "adresse":
		    case "nom":
		    case "prenom":
		          $typeContante = constant ('IndividuPeer::'.strtoupper ($type));
		          if($key>0){
                                if($criterias[$key]=='' && $operateurs[$key-1] == 'SAUF')
                                         $criterionTemp = $criteria->getNewCriterion ($typeContante, Criteria::ISNOTNULL);
                                else if($criterias[$key]=='' && $operateurs[$key-1] != 'SAUF')
                                         $criterionTemp = $criteria->getNewCriterion ($typeContante, Criteria::ISNULL);
                                else if($criterias[$key]!='' && $operateurs[$key-1] == 'SAUF')
                                      $criterionTemp = $criteria->getNewCriterion ($typeContante, '%'.$criterias[$key].'%', Criteria::NOT_LIKE);
                                else
                                      $criterionTemp = $criteria->getNewCriterion ($typeContante, '%'.$criterias[$key].'%', Criteria::LIKE);
                       }else{
                                 if($criterias[$key]=='')
                                        $criterionTemp = $criteria->getNewCriterion ($typeContante, Criteria::ISNULL);
                                 else
                                        $criterionTemp = $criteria->getNewCriterion ($typeContante, '%'.$criterias[$key].'%', Criteria::LIKE);
                        }
                        if ($key == 0) 
                            $criteria->addAscendingOrderByColumn($typeContante);
			    break;
                 }
 
                  if ($key > 0)
			$criterion->$operateur($criterionTemp);
		else{
                	$criterion = $criterionTemp;
		}
}

**Fred_34** · 08/08/2012, 11h54

Si j'étais toi, je ferai plutôt des critères de sélection "Colonne, opérateur, valeur", car tu ne vas pas aller loin avec tes OU,ET et SAUF.
Avec comme choix d'opérateur =,!=, <,>,<=,>= , et même MIN,MAX (dans ce cas pas de valeur)

Pour ton problème de "OU", à mon avis, le plus simple est de pouvoir ajouter un groupe de critère de sélection. Dans un groupe de critère tu ne fais que des ET et entre chaque groupe tu fais un OU.

Par exemple, si tu veux faire :
Civilite = Mme AND (Pays = France OR Pays = BELGIQUE)

Tu fais un premier groupe :
(
Civilite = Mme
AND
Pays = France
)
Et un second :
OR
(
Civilite = Mme
AND
Pays = BELGIQUE
)

Evidemment, si un utilisateur veux faire une requête avec des OU dans tous les sens, ça risque d'être laborieux!!!

**leya_jarhan** · 08/08/2012, 12h05

Je comprends ton point de vue avec la sélection "Colonne, opérateur, valeur"

Mais cela devra quand même avoir des opérateur de groupement après ... Pour lier les lignes les unes avec les autres.

Sachant que je n'ai pas besoin des >, <, MIN, MAX.... Mes spécifications sont claires... J'ai juste besoin de == et !=. Donc ET et OU et SAUF sont suffisant.
L'interface réalisée est conforme aux spécificités... Elle est simple d'utilisation et corresponds au besoin du client.

Le problème reste de lister des "règles" pour lire une ligne de donnée afin de l'interprêter en propel ou en SQL.

Exemple:
Civilite = Mme AND (Pays = France OR Pays = BELGIQUE)

SI op = ET
* alors ???
SI op = OU
* alors ???
SI op[n-1] = OU
* alors ???

etc...

EXEMPLE issu du cahier des charges

CIVILITE mme ET
www toto.fr OU
metier chanteur OU
telephone 0102030405 ET
ville LYON OU
civilite aucune

donne :
C1 = Mme ET (C2 contient 'onda.fr' OU C3 = chanteur OU C4 contient '0102030405') ET (C5 contient 'LYON' OU C6 est vide)

**Fred_34** · 08/08/2012, 12h58

Envoyé par leya_jarhan

Je comprends ton point de vue avec la sélection "Colonne, opérateur, valeur"

Mais cela devra quand même avoir des opérateur de groupement après ... Pour lier les lignes les unes avec les autres.

???

Code :

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

SELECT * FROM table WHERE a=1 AND (b=2 OR b=3)

donnera exactement le même résultat que

Code :

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

SELECT * FROM table WHERE (a=1 AND b=2) OR ( a=1 AND b=3)

Envoyé par leya_jarhan

Sachant que je n'ai pas besoin des >, <, MIN, MAX.... Mes spécifications sont claires... J'ai juste besoin de == et !=. Donc ET et OU et SAUF sont suffisant.

Qui peut le plus peut le moins. Pas la peine de tous les implémenter tout de suite, mais si un jour on te les demandes, tu ne seras pas obligé de tout casser...

Envoyé par leya_jarhan

L'interface réalisée est conforme aux spécificités... Elle est simple d'utilisation et corresponds au besoin du client.

Personnellement, je ne trouve pas ça trivial d'écrire :

Civilite Mme ET
Pays France SAUF

Je préfère :

Civilite = Mme
Pays != France

Envoyé par leya_jarhan

Le problème reste de lister des "règles" pour lire une ligne de donnée afin de l'interprêter en propel ou en SQL.

Exemple:
Civilite = Mme AND (Pays = France OR Pays = BELGIQUE)

SI op = ET
* alors ???
SI op = OU
* alors ???
SI op[n-1] = OU
* alors ???

etc...

Si tu restes avec tes OU,ET et SAUF, effectivement ça risque d’être compliqué...
Alors qu'avec la méthode que je t'ai proposé c'est assez simple de formater le "WHERE" d'une requête SQL:

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
37
$where="";
foreach($group) {
 
  if($firstGroup) {
    $firstGroup=false;
  }
  else {
    $where+=" OR ";
  }
 
  $where+="(";  
 
  foreach($criteredugroup) {
 
    $where+=$criteredugroup->type;
 
    $where+=" ";
    if($criteredugroup->criterias=="") {
 
      if($criteredugroup->operateur=="=") {
         $where+= "IS NULL";
      }
      else if($criteredugroup->operateur=="!=") {
         $where+= "IS NOT NULL";
      }
      else {
        //TODO
      }
    }
    else {
      $where+=$criteredugroup->operateur;
      $where+=" ";
      $where+=$criteredugroup->criterias;   
    }
  }
  $where+=")";  
}

**Fred_34** · 08/08/2012, 13h08

Envoyé par leya_jarhan

EXEMPLE issu du cahier des charges

CIVILITE mme ET
www toto.fr OU
metier chanteur OU
telephone 0102030405 ET
ville LYON OU
civilite aucune

donne :
C1 = Mme ET (C2 contient 'onda.fr' OU C3 = chanteur OU C4 contient '0102030405') ET (C5 contient 'LYON' OU C6 est vide)

Tu as des "=" et des "contient" comment vas tu pouvoir faire la différence en n'utilisant que ET, OU et SAUF ?

EDIT 1:
Et effectivement, si il faut faire cette requête avec des groupes c'est pas terrible :

C1 = Mme ET C2 contient 'onda.fr' ET C5 contient 'LYON'
OU
C1 = Mme ET C2 contient 'onda.fr' ET C6 est vide
OU
C1 = Mme ET C3 = chanteur ET C5 contient 'LYON'
OU
C1 = Mme ET C3 = chanteur ET C6 est vide
OU
C1 = Mme ET C4 contient '0102030405' ET C5 contient 'LYON'
OU
C1 = Mme ET C4 contient '0102030405' ET C6 est vide

EDIT 2:
Comment vas tu pouvoir faire ce genre de requête ?:
(C1 = Mme ET C2 contient 'toto.fr') OU (C1 = Mr ET C2 contient 'titi.fr')

**leya_jarhan** · 08/08/2012, 14h16

Envoyé par Fred_34

Tu as des "=" et des "contient" comment vas tu pouvoir faire la différence en n'utilisant que ET, OU et SAUF ?

Ce qui va faire la différence ce sera si le champ dans lequel il rentre les données sont des input ou des select. Les input donneront un LIKE ou un NOT LIKE à la place du == ou !=.

Envoyé par Fred_34

Comment vas tu pouvoir faire ce genre de requête ?:
(C1 = Mme ET C2 contient 'toto.fr') OU (C1 = Mr ET C2 contient 'titi.fr')

Tu prends le problème à l'envers... Les utilisateurs m'ont envoyé une spéc... Je ne fais que la suivre. Je ne peux leur proposer une interface différente de celle imposée car sinon cela va engendrer des coûts de notre côté.

Je cherche une solution pour le problème qui m'est apporté à savoir : une interface qui se présente comme définie précédement, avec gestion des lignes "une par une" où le OU est prioritaire et s'applique seulement à la ligne où il est placé et la suivante.

Reste à définir comment créer une liste de règles pour le codage et pour la lecture de ces lignes pour en générer une requête SQL.

**leya_jarhan** · 08/08/2012, 15h20

Que pensez - vous de ce genre de proposition ? (valable pour ET et OU)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
for($i=0;$i<count($tab);$i++){
    IF $tab['operateur'][$i] == 'OU'{
           if( $ma_requete_where != ' '){
                  $ma_requete_where .= $tab['operateur'][$i-1];
           }
           $ma_requete_where .= ' ( '+$tab['champ'][$i]+' = '+$tab['valeur'][$i]+' OR
                                             '$tab['champ'][$i+1]+' = '+$tab['valeur'][$i+1]+') ';
           $i++;
    }else{
         if( $ma_requete_where != ' '){
                 $ma_requete_where .= 'AND';
         }
         $ma_requete_where .= $tab['champ'][$i] + ' = ' + $tab['valeur'][$i];
}

**Fred_34** · 08/08/2012, 16h00

Envoyé par leya_jarhan

Tu prends le problème à l'envers... Les utilisateurs m'ont envoyé une spéc... Je ne fais que la suivre. Je ne peux leur proposer une interface différente de celle imposée car sinon cela va engendrer des coûts de notre côté.

Et dans ta spec, est-il expliqué comment tu dois gérer les SAUF ?

Que doit donner :
C1 Mme SAUF
C3 chanteur SAUF
Tu peux avoir ces deux résultats :
C1 != Mme ET C3 !=chanteur
C1 != Mme OU C3 !=chanteur

Il faudrait donc ajouter "SAUF ET" et "SAUF OU"...

Ce ne doit pas être très loin de la spec si tu ajoutes une colonne opérateur (avec uniquement = et !=) et que tu gardes ta colonne opérateur actuel mais avec uniquement le ET et le OU:

"Colonne, opérateur ( = / !=), valeur , liaison ( ET / OU / Rien)"

Ton "where" est alors assez simple à écrire :

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
      $insideOR=false;
    $ma_requete_where='';
    FOR($i=0;$i<count($tab);$i++){
 
      IF $tab['operateur'][$i] == 'OU'{
        if(!$insideOR) {
          $ma_requete_where.= '(';
          $insideOR=true;
        }
        $ma_requete_where.= $tab['champ'][$i];
        $ma_requete_where.= $tab['newOperateur'][$i];
        $ma_requete_where.= $tab['valeur'][$i];
        $ma_requete_where.= " OR ";
      }
      ELSE IF $tab['operateur'][$i] == 'ET'{
        $ma_requete_where.= $tab['champ'][$i];
        $ma_requete_where.= $tab['newOperateur'][$i];
        $ma_requete_where.= $tab['valeur'][$i];
 
        if( $insideOR) {
          $ma_requete_where.=')';
          $insideOR=false;
        }
        $ma_requete_where.= " AND ";
      }
      else {// C'est forcement le dernier
        $ma_requete_where.= $tab['champ'][$i];
        $ma_requete_where.= $tab['newOperateur'][$i];
        $ma_requete_where.= $tab['valeur'][$i];
        if( $insideOR) {
          $ma_requete_where.=')';
          $insideOR=false;
        }
      }
    }

Si tu remplaces "$tab['newOperateur'][$i]" par "'='" c'est plus ou moins ta proposition corrigée.

**leya_jarhan** · 08/08/2012, 16h04

Envoyé par Fred_34

Ce ne doit pas être très loin de la spec si tu ajoutes une colonne opérateur (avec uniquement = et !=) et que tu gardes ta colonne opérateur actuel mais avec uniquement le ET et le OU:

L'interphace est déjà intégrée et non modifiable (ou alors minimalement). Rajouter une colonne n'est pas minimal... Reste à parser ce qu'il y a à l'écran.

Je comprends que ta solution soit bonne, mais je me doit de réaliser ceci avec les contraintes qui m'ont été imposées.

**Fred_34** · 08/08/2012, 16h52

J'ai ajouté le SAUF ( Mais le SAUF OU est impossible) :

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
 
    $insideOR=false;
    $ma_requete_where='';
    FOR($i=0;$i<count($tab);$i++){
 
      IF($tab['operateur'][$i] == 'OU'){
        IF(!$insideOR) {
          $ma_requete_where.= '(';
          $insideOR=true;
        }
        $ma_requete_where.= $tab['champ'][$i];
        $ma_requete_where.= ' = ';
        $ma_requete_where.= $tab['valeur'][$i];
        $ma_requete_where.= " OR ";
      }
      ELSE IF ($tab['operateur'][$i] == 'ET'){
        $ma_requete_where.= $tab['champ'][$i];
        $ma_requete_where.= ' = ';
        $ma_requete_where.= $tab['valeur'][$i];
 
        IF( $insideOR) {
          $ma_requete_where.=')';
          $insideOR=false;
        }
        $ma_requete_where.= " AND ";
      }
      ELSE IF ($tab['operateur'][$i] == 'SAUF'){
        $ma_requete_where.= $tab['champ'][$i];
        $ma_requete_where.= ' <> ';
        $ma_requete_where.= $tab['valeur'][$i];
 
        IF( $insideOR) {
          $ma_requete_where.=')';
          $insideOR=false;
        }
        IF(($i+1)<count($tab)) {
          $ma_requete_where.= " AND ";
        }
      }
      ELSE {
        $ma_requete_where.= $tab['champ'][$i];
        $ma_requete_where.= ' = ';
        $ma_requete_where.= $tab['valeur'][$i];
        if( $insideOR) {
          $ma_requete_where.=')';
          $insideOR=false;
        }
 
        IF(($i+1)<count($tab)) {
          //ERREUR
        }
      }
    }

**leya_jarhan** · 08/08/2012, 16h54

Après test la base n'est pas bonne quand j'ai plus de deux OU se suivants. Il fait abstraction du dernier....

Donc à revoir...

**Fred_34** · 08/08/2012, 17h20

Envoyé par leya_jarhan

Après test la base n'est pas bonne quand j'ai plus de deux OU se suivants. Il fait abstraction du dernier....

Donc à revoir...

Grrrr...

Tu peux faire un print_r de $tab et un echo de $ma_requete_where ?

**leya_jarhan** · 09/08/2012, 08h52

en gros j'ai séparé en 3 tab

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
 
$types = array(
     0 => 'civilite',
     1 => 'codepostal',
     2 => 'departement_id',
     3 => 'ville',
     4 => 'prenom',
     5 => 'prenom',
);
 
$valeurs = array(
     0 => 'Mr',
     1 => '69',
     2 => 45,
     3 => 'lyon',
     4 => 'albert',
     5 => 'paul',
);
 
$operateurs = array(
     0 => 'ET',
     1 => 'OU',
     2 => 'ET',
     3 => 'OU',
     4 => 'OU',
     5 => 'ET',
);

Le résultat : (il ne prend pas en compte le OU pour la ligne 2)
civilite=Mr ET (codepostal=69 OR departement_id=45) AND ville=lyon ET (prenom=albert OR prenom=paul)

**Fred_34** · 09/08/2012, 10h40

Dans $operateurs il ne devrait pas y avoir " 5 => 'ET'," (C'est le dernier, donc vide sauf si "SAUF").

Et je ne comprends pas pourquoi dans ton where il y a des ET :

civilite=Mr ET (codepostal=69 OR departement_id=45) AND ville=lyon ET (prenom=albert OR prenom=paul)

Tu as surement fait quelques modifications. Post les.

**leya_jarhan** · 09/08/2012, 11h14

Bon entre temps j'ai trouvé une solution récurcive qui a l'air de fonctionner... Je suis encore dans les tests.

Dans criteria j'ai mes "Civilité = Mme" ou "Prénom like '%marie%'".

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
public function recursive($i, $criterias, $operateurs){
  	if($i == (count($operateurs)-1) && ($i>0 && $operateurs[$i-1] == 'ET')){
  		return $criterias[$i].' ';
  	}else if($i == (count($operateurs)-1) && ($i>0 && $operateurs[$i-1] == 'OU' || $operateurs[$i-1] == 'SAUF')){
  		return $criterias[$i].' )';
  	}else if($i == (count($operateurs)-1) && $i==0){
  		return $criterias[$i].' ';
  	}else if($i == 0 && $operateurs[$i] == 'OU'){
  		return ' ('.$criterias[$i].' OR '.$this->recursive($i+1, $criterias, $operateurs);
  	}else if($i == 0 && $operateurs[$i] == 'SAUF'){
  		return ' ('.$criterias[$i].' AND '.$this->recursive($i+1, $criterias, $operateurs);
  	}else if($i == 0 && $operateurs[$i] == 'ET'){
  		return ' '.$criterias[$i].' AND '.$this->recursive($i+1, $criterias, $operateurs);
  	}else{
  		if($operateurs[$i-1] == 'ET' && $operateurs[$i] == 'ET'){
  			return ' '.$criterias[$i].' AND '.$this->recursive($i+1, $criterias, $operateurs);
  		}else if($operateurs[$i-1] == 'ET' && $operateurs[$i] == 'OU'){
  			return ' ('.$criterias[$i].' OR '.$this->recursive($i+1, $criterias, $operateurs);
  		}else if($operateurs[$i-1] == 'OU' && $operateurs[$i] == 'SAUF'){
  			return ' ('.$criterias[$i].' OR '.$this->recursive($i+1, $criterias, $operateurs);
  		}else if($operateurs[$i-1] == 'ET' && $operateurs[$i] == 'SAUF'){
  			return ' ('.$criterias[$i].' AND '.$this->recursive($i+1, $criterias, $operateurs);
  		}else if($operateurs[$i-1] == 'SAUF' && $operateurs[$i] == 'OU'){
  			return ' '.$criterias[$i].') OR '.$this->recursive($i+1, $criterias, $operateurs);
  		}else if($operateurs[$i-1] == 'SAUF' && $operateurs[$i] == 'ET'){
  			return ' '.$criterias[$i].') AND '.$this->recursive($i+1, $criterias, $operateurs);
  		}else if($operateurs[$i-1] == 'OU' && $operateurs[$i] == 'ET'){
  			return ' '.$criterias[$i].') AND '.$this->recursive($i+1, $criterias, $operateurs);
  		}else if($operateurs[$i-1] == 'OU' && $operateurs[$i] == 'OU'){
  			return ' '.$criterias[$i].' OR '.$this->recursive($i+1, $criterias, $operateurs);
  		}
  	}
  }

[MySQL - PhP - Propel] Génération de requêtes personnalisés

Requêtes MySQL

Discussions similaires

Partager

Partager