Discussion :

[Demirdiouff]

Bonjour,

Je me permets de vous contacter car je rencontre un petit soucis avec ma requête SQL... n'étant pas encore très à l'aise, j'ai du mal a faire des requêtes qui sont un peu plus complexes.
Toutes les requêtes que j'ai essayé de faire de mon côté ne fonctionnent pas... je m’emmêle un petit peu je pense avec les JOIN
Il paraît pourtant possible de faire ce que je souhaite avec la fonction SUM() et la fonction COUNT(), mais je n'arrive pas a tout mettre en place pour que cela fonctionne.

J'ai actuellement deux tables (customer(qui dispose d'un champ vendor_id) et vendor), dont je veux récupérer des informations sur les deux pour pouvoir les afficher dans un tableau.

Dans la première table (customer), je souhaite récupérer les informations suivantes :
- customer id

SUM() de :
- surface sol
- surface mur
- surface comble

dans la deuxième table (vendor), je souhaite récupérer les informations suivantes :
- vendor_id

COUNT() de :
- customer_id par vendor_id

- vendor_lastname
- vendor_firstname
- vendor_code

Je souhaiterai à la finalité, afficher tout ces éléments dans un tableau à 4 colonnes, avec Nom/PrénomVendor - NbContratsCustomer - surfaceTotal - codeVendor

Est-ce que vous pensez qu'il est possible dans un premier temps de faire une seule requête pour réunir toutes ces informations ? Et si oui, est-il judicieux de procéder ainsi ?


Savez-vous également dans quel cas cet erreur est affiché et pourquoi, et comment le résoudre ? Je n'ai pas réussi a trouver de réponse convaincant sur Google.

Code :

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

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '?' at line 1

Je vous remercie beaucoup pour votre aide, si vous pouvez m'éclaircir un petit peu...

[al1_24]

Sans connaître le code de la requête, il est difficile d'interpréter le message d'erreur.

[Demirdiouff]

Voici par exemple un morceau de code pour lequel ça m'affiche l'erreur :

Code :

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

SELECT COUNT(customeriso_id) FROM customersiso WHERE vendoriso_id=?

J'essayais de m'entraîner morceau par morceau avant de faire la requête entière pour voir comment ça réagit.

Lorsque je rentre ce bout de code dans Phpmyadmin, en remplaçant la valeur de '?' c'est fonctionnel.

[Michel.Priori]

Bonsoir,

Avant de se lancer dans les requêtes d'agrégation il est utile, dans un premier temps d'obtenir un tableau détaillé de ce qu'on agrégera plus tard.

1- tableau des surfaces

Code :

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

1
2
3
4
5
6
Nom/PrénomVendor    - codeVendor    - surface 
A/B                     1               1+3+7
A/B                     1               3+1+6
A/B                     1               2+7+1
B/C                     2               5+6+8
B/C                     2               1+2+8

2- tableau agrégé

Code :

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

1
2
3
Nom/PrénomVendor    - codeVendor    - surface_totale    - NbContratsCustomer    
A/B                     1               Sum(surface)        count(*)
B/C                     2               Sum(surface)        count(*)

Peux-tu faire la première requête ?

[Demirdiouff]

Bonjour,

Veuillez m'excuser pour le retard dans ma réponse... je n'ai pas eu accès à mon ordinateur durant tout le week-end étant en déplacement..

Je viens tout juste de me pencher sur la première requête demandée et la voici si je me suis pas trompé... :

Code :

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

1
2
 
SELECT c.customeriso_id, c.area_attic, c.area_wall, c.area_flooring, v.vendoriso_id, v.vendoriso_lastname, v.vendoriso_firstname, v.vendoriso_code FROM customersiso AS c LEFT JOIN vendorsiso AS v ON c.vendoriso_id = v.vendoriso_id

[escartefigue]

La requête publiée ne contient aucun agrégat, elle ne correspond pas à votre besoin.

Pour la demande initiale, voici une requête qui devrait convenir :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
select vd.vendoriso_id
     , vd.vendoriso_code
     , vd.vendoriso_firstname
     , count(cu.customeriso_id) as nb_cli
     , sum(cu.area_attic)   as sols
     , sum(cu.area_wall)   as murs
     , sum(cu.area_flooring)   as comb   
from vendorsiso as VD
left join customersiso as CU
  on CU.vendoriso_id= VD.vendoriso_id
group by vd.vendoriso_id
       , vd.vendoriso_code
       , vd.vendoriso_firstname

La jointure OUTER permet de récupérer les éléments pour les éventuels vendeurs sans client (nouveaux vendeurs...), à remplacer par une jointure INNER le cas échéant.

Notez qu'une requête mise en forme est quand même plus facile à lire qu'une requête écrite sur une seule ligne

[Michel.Priori]

bonsoir,

Code :

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

1
2
 
SELECT c.customeriso_id, c.area_attic, c.area_wall, c.area_flooring, v.vendoriso_id, v.vendoriso_lastname, v.vendoriso_firstname, v.vendoriso_code FROM customersiso AS c LEFT JOIN vendorsiso AS v ON c.vendoriso_id = v.vendoriso_id

Le code SQL n'est sensible ni aux retours lignes ni aux espaces ou tabulations surnuméraire.
De plus la clause SELE
En mettant ça en application votre requête devient plus lisible :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
SELECT 
      v.vendoriso_id                /* vendor_id */
    , v.vendoriso_lastname
    , v.vendoriso_firstname
    , v.vendoriso_code 
    , c.customeriso_id              /* customer id */
    , c.area_attic                  /* surface sol */
    , c.area_wall                   /* surface mur */
    , c.area_flooring               /* surface comble */
FROM customersiso AS c 
    LEFT JOIN vendorsiso AS v 
        ON c.vendoriso_id = v.vendoriso_id

Ajoutons des calculs "lignes" c'est à dire des opérations entre les colonnes.
Les symboles +-*/ sont pour les opérations primaires
Les fonctions permettent d'obtenir un résultat plus complexe (et dépendant de la fonction elle même). la syntaxe globale est MA_FONCTION(Param1, Param2, ...) où MA_FONCTION doit être remplacé par le nom de celle ci (CONCAT, ISNULL, ...) suivi d'une paire de parenthèse () dans lesquelles on trouve les arguments (représentés ici par Param1, Param2, ...)
La fonction CONCAT() permet "d'additionner" du texte.
La fonction ISNULL(Param1, Param2) remplace Param1 par Param2 seulement si Param1 est NULL
Note : Null est différent de 0 - c'est différent de ne pas savoir combien on doit (NULL) que de dire qu'on doit 0 - Du coup Null agit comme un élément absorbant dans la mathématique SQL : 100*NULL renvoie NULL (comprendre 100 fois "ne sais pas" = "ne sais toujours pas")

En mettant ça en application votre requête devient :

Code :

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

1
2
3
4
5
6
7
8
SELECT 
      v.vendoriso_id                /* vendor_id */
    , concat( v.vendoriso_lastname , ' ' ,  v.vendoriso_firstname, '(' , v.vendoriso_code ,')' ) as vendor
    , c.customeriso_id              /* customer id */
    , isnull(c.area_attic,0) + isnull(c.area_wall, 0) + isnull(c.area_flooring,0) as surface
FROM customersiso AS c 
    LEFT JOIN vendorsiso AS v 
        ON c.vendoriso_id = v.vendoriso_id

Il ne vous reste plus qu'à regarder le résultat et comprendre quelles sont les colonnes dont les valeurs sont à regrouper et quelles sont les valeurs à sommer.
@+

[Demirdiouff]

J'y vois beaucoup plus clair, et j'ai adapté la requête en fonctions des besoins maintenant
Un petit détail : dans la dernière requête, "isnull" n'a pas l'air de fonctionner, son équivalent semble être "ifnull".

Je vous remercie beaucoup en tout cas pour votre aide apportée !

[escartefigue]

plutôt que d'utiliser ISNULL, utilisez COALESCE plus riche fonctionnellement (plus de 2 colonnes ou valeurs possibles) et qui fait partie de la norme SQL