[Symfony 2.7.0]Doctrine query builder

Version imprimable

Bonjour,

J'ai cette relation :

entreprise one-to-many salarié one-to-many voiture

J'aimerais sélectionner tous les salariés qui n'ont pas de voiture mais je suis bloqué...

J'ai la contrainte de sélectionner aussi que les salariés appartenant à l'entreprise passer en paramètre et différents de celui passer en paramètre.

Voici ce que j'ai commencé à faire :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
    public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')
                  ->addSelect('entreprise')                  
                  ->leftJoin('entreprise.salaries', 'salaries')
                  ->where('salaries.voiture = :voiture')
                  ->setParameter('voiture', null );
 
        if($salarie){
            $q->andWhere('salarie.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

Je comprends que cela ne puisse pas marcher, mais je ne vois pas comment faire pour que ça fonctionne...
J'imagine qu'il y a peut être une sous requête à faire?

Merci pour votre aide.

quand on écrit une requête un peu complexe on procède par étape.

1.

Code:

1
2
3
4
5
 
$q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')
                  ->addSelect('entreprise')                  
                  ->leftJoin('entreprise.salaries', 'salaries')

j'otiens un résultat ?

oui --> 2

Code:

1
2
3
4
5
6
7
 
$q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')
                  ->addSelect('entreprise')                  
                  ->leftJoin('entreprise.salaries', 'salaries')
                  ->where('salaries.voiture = :voiture')
                  ->setParameter('voiture', null );

je n'ai pas de résultat donc il y a un problème avec voiture et null ??????

et on cherche ... ma base de donnée il y a bien un null dans les champs ? ....... etc.....

etc....

16/11/2015, 21h38
Billy12

Je crois qu'il est préférable de faire

Code:

->where('salaries.voiture IS NULL')

Citation:
Envoyé par dukoid
quand on écrit une requête un peu complexe on procède par étape.

1.
Code:

1 2 3 4 5 $q = $this->createQueryBuilder('salarie') ->leftJoin('salarie.entreprise', 'entreprise') ->addSelect('entreprise') ->leftJoin('entreprise.salaries', 'salaries')
j'otiens un résultat ?

oui --> 2
Code:

1 2 3 4 5 6 7 $q = $this->createQueryBuilder('salarie') ->leftJoin('salarie.entreprise', 'entreprise') ->addSelect('entreprise') ->leftJoin('entreprise.salaries', 'salaries') ->where('salaries.voiture = :voiture') ->setParameter('voiture', null );
je n'ai pas de résultat donc il y a un problème avec voiture et null ??????

et on cherche ... ma base de donnée il y a bien un null dans les champs ? ....... etc.....

etc....
Merci pour ta réponse.
C'est exactement la méthode que j'ai appliqué vois tu. Mais parfois on a besoin d'être guidé car on ne sais pas faire.
Je sais que ma requête ne peux pas fonctionner car il n'est sûrement pas possible de mettre
Code:

1 2 3 ->where('salaries.voiture = :voiture') ->setParameter('voiture', null );
étant donné qu'un salarié peut avoir plusieurs voiture. Malheureusement je ne connais pas le moyen de le faire.
Je cherche donc quelqu'un qui pourra me l'expliquer mais pas me faire un cours sur ma façon de déboguer mon code.
1 fois sur 2 on a droit à ce genre de réponse sur les forums. Peu importe la raison... ce n'est pas la réponse à la question.

17/11/2015, 12h08
pmithrandir

Est-ce que la méthode de billy fonctionne ?

je ne voit pas trop pkoi la tienne ne fonctionnerait pas a vrai dire.

pierre
17/11/2015, 12h52
dukoid

loin de moi l'idée de donner un cours à quiconque, je voulais identifier quel partie du code était erroné.

si tu relis ton message de départ tu ne donnes aucune erreur, tu ne donnes aucune indication sur la partie du code qui ne fonctionne pas. tu parles même de sous requête :weird:
si tu avais écris "la condition du null ne fonctionne pas, comment faire ?. puisque apparemment, tu connaissais cette information. dans ce cas tu aurais eu très certainement une réponse plus adapté voir aucune de ma part (parceque je ne sais pas)

à question précise, réponse précise....

et donc ça donne quoi la réponse de Billy ?

Merci à tous pour vos réponses.
En me relisant je viens de voir que j'avais oublié un s... en fait voiture est voitures

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')
                  ->addSelect('entreprise')                  
                  ->leftJoin('entreprise.salaries', 'salaries')
                  ->where('salaries.voitures = :voitures')
                  ->setParameter('voitures', null );
 
        if($salarie){
            $q->andWhere('salarie.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

La méthode de Billy12 ne solutionne pas mon problème, j'ai l'erreur :

Code:

1
2
 
Invalid Path Expression. StateFieldPathExpression or SingleValueAssociationField expected.

J'ai fait évolué ma requête car mon besoin a évolué, mais j'ai une nouvelle erreur :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')
                  ->addSelect('entreprise')                  
                  ->leftJoin('entreprise.salaries', 'salaries')
                  ->leftJoin('salaries.voitures', 'voitures')
                  ->where('voitures.dispo is NULL');
 
        if($salarie){
            $q->andWhere('salarie.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

Voici l'erreur :

Code:

1
2
 
A single-valued association path expression to an inverse side is not supported in DQL queries. Use an explicit join instead

J'ai modifié en mettant ceci :

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
 
public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')
                  ->addSelect('entreprise')                  
                  ->leftJoin('entreprise.salaries', 'salaries')
                  ->leftJoin('salaries.voitures', 'voitures')
                  ->leftJoin('voitures.dispo', 'dispo')
                  ->where('dispo is NULL');
 
        if($salarie){
            $q->andWhere('salarie.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

Je n'ai plus de message d'erreur, mais le résultat est faux : la requête me remonte aussi les cas où dispo n'est pas NULL...

17/11/2015, 15h50
Billy12

Et en enlevant le addSelect('entreprise') ?
17/11/2015, 15h53
Invité

Citation:

Envoyé par Billy12

Et en enlevant le addSelect('entreprise') ?

Pareil. Je croyais qu'il était obligatoire de mettre un addSelect après chaque leftJoin mais j'ai vu que non. A quoi ils servent alors?
17/11/2015, 15h55
Billy12

Si dispos est une table tu dois aussi modifier le where en dispo.id IS NULL
17/11/2015, 15h57
Invité

Citation:

Envoyé par Billy12

Si dispos est une table tu dois aussi modifier le where en dispo.id IS NULL

Dispo est bien une table, la relation entre dispo et voiture est one-to-one.
J'ai mis le dispo.id is NULL mais toujours le même résultat.

Je voudrais toutes les voitures qui n'ont pas de relation avec dispo...
17/11/2015, 16h09
Billy12

Pour t'aider tu peux regarder ce que te donne un dump($q->getDQL()) ? Et si cela correspond á ce que tu veux ?
17/11/2015, 16h16
Billy12

Déjà si tu pars de ton entity entreprise tu dois faire un createQueryBuilder dessus
Puis tu fais ton leftJoin de salaries
Puis de voiture ....
Désolé j'suis avec mon tel donc pour les copié collé c'est pas top
17/11/2015, 16h26
Invité

Citation:

Envoyé par Billy12

Déjà si tu pars de ton entity entreprise tu dois faire un createQueryBuilder dessus
Puis tu fais ton leftJoin de salaries
Puis de voiture ....
Désolé j'suis avec mon tel donc pour les copié collé c'est pas top

Merci pour ton aide.
Je ne pars pas de entreprise, je veux absolument récupérer les salariés sauf ceux qui ont des voitures avec dispo non null et sauf celui passé en paramètre de la méthode.
Du coup, est ce que je mets toujours entreprise dans le createQueryBuilder?
17/11/2015, 16h33
Invité

Citation:

Envoyé par Billy12

Pour t'aider tu peux regarder ce que te donne un dump($q->getDQL()) ? Et si cela correspond á ce que tu veux ?

Le dump me donne un résultat pas mal, mais je pense que le dispo.id is NULL n'est pas bon.
Là je suis en train de dire que je veux les voitures qui ont un dispo qui a un id NULL, or ne je pense pas que ça soit ça.
Il n'y a rien pour dire je veux les voitures qui n'ont pas de lien avec dispo ?
17/11/2015, 16h36
Billy12

Mais c'est quoi dans ton getRepository ?
Si c'est l'Entity Salaries (avec un S Normalement car c'est une collection ) tu fais un createQuertBuilder dessus en effet
Puis leftJoin de voiture

Et dispo c'est oneToOne ?
Si oui
Left joint dispo

Et un where dispo.id is null

Au final tu te fou d'entreprise c'est ça ?
17/11/2015, 16h37
Billy12

Et Si dispo est un champ de voiture
Tu fais un where voiture.dispo IS NUL

Édit : tu m'as répondu c'est oneToOne
Ce message n'est donc pas util
17/11/2015, 16h42
Invité

Dans mon getRepository c'est salarie.
dispo c'est one-to-one avec voiture.

Entreprise je m'en fous pas car je veux les salariés de l'entreprise en paramètre, pas les autres.
Dispo est une entité mais une voiture peut ne pas en avoir.
17/11/2015, 16h50
Billy12

Je crois que c'est depuis 2.6 où il faut mettre dispo.id is nul

C'est vrai qu'avant en 2.4
dispo is null
marchait

Je me trompe peut être ? Et j'ai eu ce PB peut être dans un autre contexte ...
Je reviens ce soir car là avec mon tel je fais pas de copié collé pour écrire le code dans les balises ...
17/11/2015, 18h46
dukoid
au lieu de IS NULL essaye :
Code:

1 2 ... IS EMPTY

avec $em->getRepository('ACMEMonBundle:Salarie')

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')               
                  ->leftJoin('salarie.voiture', 'voiture')
                  ->leftJoin('voiture.dispo', 'dispo')
                  ->where('dispo.id IS NULL')
 
        if($salarie){
            $q->andWhere('salarie.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

ça marche ça ?

17/11/2015, 20h23
Invité
Citation:
Envoyé par dukoid

au lieu de IS NULL essaye :

Code:

1 2 ... IS EMPTY
Merci dukoid pour ta réponse, mais j'ai une erreur car empty s'adresse uniquement aux collections et dispo n'est pas une collection.

Citation:

Envoyé par Billy12

avec $em->getRepository('ACMEMonBundle:Salarie')

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salarie')
                  ->leftJoin('salarie.entreprise', 'entreprise')               
                  ->leftJoin('salarie.voiture', 'voiture')
                  ->leftJoin('voiture.dispo', 'dispo')
                  ->where('dispo.id IS NULL')
 
        if($salarie){
            $q->andWhere('salarie.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

ça marche ça ?

J'ai essayé ton code, mais en fait tu as supprimé les s de voiture et salaries donc ça ne fonctionne plus.
Les relations sont les suivantes : entreprise one-to-many salarié one-to-many voiture one-to-one dispo
Donc salarie apparaît avec un s dans entreprise et pareil pour voiture dans salarie.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salaries')
                  ->leftJoin('salaries.entreprise', 'entreprise')               
                  ->leftJoin('salaries.voitures', 'voitures')
                  ->leftJoin('voitures.dispo', 'dispo')
                  ->where('dispo.id IS NULL')
 
        if($salarie){
            $q->andWhere('salaries.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

comme ça ? sinon met toi même les bonnes orthographes le principe est le même.

Citation:

Envoyé par Billy12

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public function queryfindByEntrepriseId($entreprise, $salarie = null)
    {
        $q = $this->createQueryBuilder('salaries')
                  ->leftJoin('salaries.entreprise', 'entreprise')               
                  ->leftJoin('salaries.voitures', 'voitures')
                  ->leftJoin('voitures.dispo', 'dispo')
                  ->where('dispo.id IS NULL')
 
        if($salarie){
            $q->andWhere('salaries.id != :salarie')
              ->andWhere('entreprise.id = :entreprise')
              ->setParameter('salarie',$salarie);
        }else{
            $q->andWhere('entreprise.id = :entreprise');               
        }
 
        $q->setParameter('entreprise',$entreprise);
 
        return $q;
 
    }

comme ça ? sinon met toi même les bonnes orthographes le principe est le même.

Merci bien Billy12!!! J'ai juste enlevé le id derrière dispo et les s à salaries et ça fonctionne :)