Ajouter un tri par distance dans une requête de géolocalisation

Version imprimable

Bonjour,

J'ai un form qui comporte uniquement un champ Address, en résumé, le site géolocalise grâce a cette adresse et affiche les items mais sans tri de distance !

Voici ma requete dans mon repo :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 /** * @return Query */ public function findAllVisibleQuery(AdSearch $search): Query { $query = $this->findVisibleQuery(); if ($search->getLat() && $search->getLng() && $search->getDistance()) { $query = $query ->select('a') ->andWhere('(6353 * 2 * ASIN(SQRT( POWER(SIN((a.lat - :lat) * pi()/180 / 2), 2) +COS(a.lat * pi()/180) * COS(:lat * pi()/180) * POWER(SIN((a.lng - :lng) * pi()/180 / 2), 2) ))) <= :distance') ->setParameter('lng', $search->getLng()) ->setParameter('lat', $search->getLat()) ->setParameter('distance', $search->getDistance()); } private function findVisibleQuery(): QueryBuilder { return $this->createQueryBuilder('a') ->where('a.publishedAt != false'); }
La requête fonctionne très bien mais quand je rajoute :

Code:

->orderBy('a.distance', 'ASC')

J'ai le message d'erreur :

Citation:

[Semantical Error] line 0, col 253 near 'distance ASC': Error: Class App\Entity\Ad has no field or association named distance

Alors, j'ai bien compris le message il ne trouve pas de champ ou d'association nommée distance ok mais comment faire pour avoir un tri de la distance la plus petite a la distance la plus grande.

je vous remercie les amis :zoubi:

23/04/2020, 09h31
Invité

Bonjour,

Code:

->andWhere('(.......) AS distance <= :distance')

Merci pour la réponse mais la solution ne fonctionne pas, j'ai une erreur :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
    /**
     * @return Query
     */
    public function findAllVisibleQuery(AdSearch $search): Query
    {
        $query = $this->findVisibleQuery();
 
        if ($search->getLat() && $search->getLng() && $search->getDistance()) {
            $query = $query
                ->select('a')
                ->andWhere('(6353 * 2 * ASIN(SQRT( POWER(SIN((a.lat - :lat) *  pi()/180 / 2), 2) +COS(a.lat * pi()/180) * COS(:lat * pi()/180) * POWER(SIN((a.lng - :lng) * pi()/180 / 2), 2) ))) AS distance <= :distance')
                //->orderBy('a.distance', 'ASC')
                ->setParameter('lng', $search->getLng())
                ->setParameter('lat', $search->getLat())
                ->setParameter('distance', $search->getDistance());
        }
 
        return $query->getQuery();
    }

Erreur :

Citation:

[Syntax Error] line 0, col 227: Error: Expected =, <, <=, <>, >, >=, !=, got ')'

J'ai regardé attentivement mais je ne comprends pas :?

Il n'y a pas de As dans un prédicat en DQL.

Essaie plutôt :

Code:

1
2
->andWhere('(6353 * 2 * ASIN(SQRT( POWER(SIN((a.lat - :lat) *  pi()/180 / 2), 2) +COS(a.lat * pi()/180) * COS(:lat * pi()/180) * POWER(SIN((a.lng - :lng) * pi()/180 / 2), 2) ))) <= :distance')

26/04/2020, 21h56
bndd24

salut, ton code est identique a mon premier post :mrgreen:

il ne tri pas les items par ordre de distance.
26/04/2020, 23h05
Alexandre T
Non, mon code n'est pas identique. Regarde plus attentivement la fin de la ligne, tu verras une différence. J'ai retiré le "as distance" qui fait planter ta requête DQL.
Pour trier par la distance, tu dois malheureusement la recalculer avec DQL.
Tu peux pet-être le faire ainsi :
Code:

1 2 ->addOrder('(6353 * 2 * ASIN(SQRT( POWER(SIN((a.lat - :lat) * pi()/180 / 2), 2) +COS(a.lat * pi()/180) * COS(:lat * pi()/180) * POWER(SIN((a.lng - :lng) * pi()/180 / 2), 2) )))', 'asc');
(N'oublie pas de laisser le addWhere tel que je l'ai mis pour garder le filtre sur la distance maximale)
26/04/2020, 23h11
bndd24

si, si ton code est identique a mon premier post plus haut dans la conversation, regarde attentivement tu verras le premier post ;) après le second oui j'ai testé avec le As...

AS HIDDEN distance

Hello,
en faite tu dois utiliser "AS HIDDEN distance"

j'ai écrit un tutorial autour de cette requette géo localisé 8-)

https://devindependant.com/apiplatfo...-plus-proches/

en plus, il faut utiliser un having, un simple where ne marchera pas sur les alias "as"

voici un peux se que ça donne
Code:

1 2 3 4 5 6 7 8 9 10 $queryBuilder->addSelect(" ( 6371 * acos( cos( radians(:latitude) ) * cos( radians( p.latitude ) ) * cos( radians( p.longitude ) - radians(:longitude) ) + sin( radians(:latitude) ) * sin( radians( p.latitude ) ) ) ) AS HIDDEN distance"); $queryBuilder->addOrderBy("distance", "ASC"); $queryBuilder->having("distance<=:distance"); $dataQueryParam['latitude'] = $this->getParam('latitude'); $dataQueryParam['longitude'] = $this->getParam('longitude'); $dataQueryParam['distance'] = $this->getParam('distance'); $queryBuilder->setParameters($dataQueryParam);
Un autre tutorial pour afficher les résultats avec angular

https://devindependant.com/apiplatform-gps-filtre-map/

adapte ce code à ton besoin

Salut, merci pour ta réponse mais malheureusement ton code ne fonctionne pas non plus. J'ai adapté bien-sur ton code mais j'ai une erreur :?

voici directement mon repo c'est plus simple je pense :

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
51
52
53
54
55
56
<?php
 
namespace App\Repository;
 
use App\Entity\Ad;
use Doctrine\ORM\Query;
use App\Entity\AdSearch;
use Doctrine\ORM\QueryBuilder;
use Doctrine\Persistence\ManagerRegistry;
use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
 
/**
 * @method Ad|null find($id, $lockMode = null, $lockVersion = null)
 * @method Ad|null findOneBy(array $criteria, array $orderBy = null)
 * @method Ad[]    findAll()
 * @method Ad[]    findBy(array $criteria, array $orderBy = null, $limit = null, $offset = null)
 */
class AdRepository extends ServiceEntityRepository
{
    public function __construct(ManagerRegistry $registry)
    {
        parent::__construct($registry, Ad::class);
    }
 
    /**
     * @return Query
     */
    public function findAllVisibleQuery(AdSearch $search): Query
    {
        $query = $this->findVisibleQuery();
 
        if ($search->getLat() && $search->getLng() && $search->getDistance()) {
            $query = $query
                ->select('a')
                ->addSelect('(6353 * 2 * ASIN(SQRT( POWER(SIN((a.lat - :lat) *  pi()/180 / 2), 2) +COS(a.lat * pi()/180) * COS(:lat * pi()/180) * POWER(SIN((a.lng - :lng) * pi()/180 / 2), 2) ))) AS HIDDEN distance') // MON CODE
                //->addSelect(' ( 6371 * acos( cos( radians(:latitude) ) * cos( radians( a.latitude ) ) * cos( radians( a.longitude ) - radians(:longitude) ) + sin( radians(:latitude) ) * sin( radians( a.latitude ) ) ) ) AS HIDDEN distance') // TON CODE
                ->andWhere('a.category = :category')
                ->having('distance<=:distance')
                ->setParameter('lng', $search->getLng())
                ->setParameter('lat', $search->getLat())
                ->setParameter('distance', $search->getDistance())
                ->setParameter('category', $search->getCategory())
                ;
        }
 
        return $query->getQuery();
    }
 
    private function findVisibleQuery(): QueryBuilder
    {
        return $this->createQueryBuilder('a')
            ->where('a.publishedAt != false')
            ;
    }
 
}

avec ton code ou avec le mien, j'ai la même erreur :

Citation:

Cannot count query that uses a HAVING clause. Use the output walkers for pagination

Je galère grave a faire un tri sur une requête, j'y crois pas !! je suis dessus depuis une semaine, ras le bol :mur:

Autre question : Je vois que ton calcul est différent, pourquoi ? comment savoir lequel choisir ? tu l'as trouvé ou ton calcul ? je te remercie pour ton aide.

27/04/2020, 20h50
phpiste

Citation:

Envoyé par bndd24

tu l'as trouvé ou ton calcul ? je te remercie pour ton aide.

https://stackoverflow.com/questions/...m-sql-database

ton dql doit génrer un sql pareil

Code:

SELECT id, ( 3959 * acos( cos( radians(37) ) * cos( radians( lat ) ) * cos( radians( lng ) - radians(-122) ) + sin( radians(37) ) * sin( radians( lat ) ) ) ) AS distance FROM markers HAVING distance < 25 ORDER BY distance LIMIT 0 , 20;
27/04/2020, 20h54
phpiste

Citation:

Envoyé par bndd24

Cannot count query that uses a HAVING clause. Use the output walkers for pagination

lorsque tu fait un dump(return $query->getQuery()->getResult(); y'a un résultat ??

apparemment tu utilise cette requête après dans un paginator qui ajoute un count à la requête ...
27/04/2020, 20h55
bndd24

Oui je sais qu'il doit générer une requête sql mais je ne suis pas plus avancé.

Citation:

Envoyé par phpiste

apparement tu utilise cette requette après dans un paginator qui ajoute un count à la requette ...

oui exact j'utilise : https://github.com/KnpLabs/KnpPaginatorBundle

voici la méthode de mon contrôleur :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
    /**
     * @Route("/", name="ad_index")
     * @return Response
     */
    public function index(Request $request, PaginatorInterface $paginator): Response
    {
        $search = new AdSearch();
        $form = $this->createForm(AdSearchType::class, $search);
        $form->handleRequest($request);
 
        $ads = $paginator->paginate(
            $this->repository->findAllVisibleQuery($search),
            $request->query->getInt('page', 1),
            9
        );
 
        return $this->render('ad/search.html.twig', [
            'ads'   => $ads,
            'form'  => $form->createView()
        ]);
    }

essai avec:

Code:

1
2
3
4
5
6
7
        $pagination = $paginator->paginate(
            $this->repository->findAllVisibleQuery($search),
            $request->query->getInt('page', 1),
            9
            array('wrap-queries'=>true) // Cannot count query that uses a HAVING clause. Use the output walkers for pagination
        );

Je viens d'enlever totalement la pagination afin de faire des tests plus précis.

Je n'ai plus aucune erreur mais je n'ai aucun tri dans mes items.

voici mon repo

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
    /**
     * @return Ad[]
     */
    public function findAllVisibleQuery(AdSearch $search): array
    {
        if ($search->getLat() && $search->getLng() && $search->getDistance()) {
            return $this->findVisibleQuery()
                ->select('a')
                ->addSelect('(6353 * 2 * ASIN(SQRT( POWER(SIN((a.lat - :lat) *  pi()/180 / 2), 2) +COS(a.lat * pi()/180) * COS(:lat * pi()/180) * POWER(SIN((a.lng - :lng) * pi()/180 / 2), 2) ))) AS HIDDEN distance')
                ->andWhere('a.category = :category')
                ->having('distance<=:distance')
                ->setParameter('lng', $search->getLng())
                ->setParameter('lat', $search->getLat())
                ->setParameter('distance', $search->getDistance())
                ->setParameter('category', $search->getCategory())
                ->getQuery()
                ->getResult();
        }
    }

tu as une idée ?

29/04/2020, 02h11
phpiste

->addOrderBy("distance", "ASC");
02/05/2020, 23h31
bndd24

ça marche je te remercie. :ccool: