Discussion :

[thierrybatlle]

Bonsoir,

J'ai écrit la requête SQL suivante :

Code :

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

1
2
3
4
 
SELECT produit.codeproduit, produit.libelle, stock.dlc, stock.dluo, stock.dlclimite, stock.dluolimite, stock.lieustockage, stock.poids, stock.quantite, tiers.codetiers, tiers.nomtiers
 FROM produit, stock, tiers
 WHERE codeproduit =  '143007'AND stock.codetiers=tiers.idtiers AND stock.codeproduit=produit.codeproduit

Mais j'ai des erreurs d'exécution que je n'arrive pas à localiser.

Merci bien pour votre aide.

[al1_24]

j'ai des erreurs d'exécution que je n'arrive pas à localiser.

Quelles sont ces erreurs ?

[thierrybatlle]

Bonjour,
Il me dit simplement que la requête n'a pu s'exécuter à cause d'une erreur. Je n'en sais pas plus.
Sûrement mes jointures.
Merci bien

[escartefigue]

Bonjour,
Si vous avez fidèlement recopié votre requete, alors il manque simplement un espace là où j'ai mis en couleur

Code :

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

...WHERE codeproduit =  '143007'AND stock.codetiers=tiers.idtiers AND stock.codeproduit=produit.codeproduit

Profitez de la correction pour séparer les critères de fitrage (where) de ceux de jointure (join on...)

[thierrybatlle]

Bonsoir,

Je me pose une question sur la meilleure pratique pour les jointures.

Est qu'il vaut mieux faire comme je fais là ?

Ou est qu'il vaut mieux passer par des "INNER JOIN" ?

A ce moment là comment dois-je réécrire ma requête.

Merci bien

[alassanediakite]

Salut

Bonjour,
Si vous avez fidèlement recopié votre requete, alors il manque simplement un espace là où j'ai mis en couleur

Code :

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

...WHERE codeproduit =  '143007'AND stock.codetiers=tiers.idtiers AND stock.codeproduit=produit.codeproduit

Je pense plutôt que le problème vient de

Code :

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

... WHERE codeproduit...

il faut préciser la table de la colonne "codeproduit"...

Code :

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

... WHERE produit.codeproduit...

ou

Code :

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

... WHERE stock.codeproduit...

Pour ce qui est des messages d'erreurs; PgAdmin a souvent la peine de les afficher (depuis la version 9.4). Je ne sais pas pourquoi c'est pas encore corrigé. Il faut voir dans le sous dossier pg_log du dossier data pour voir les erreurs.
@+

[escartefigue]

Est qu'il vaut mieux faire comme je fais là ?
Ou est qu'il vaut mieux passer par des "INNER JOIN" ?

Les deux syntaxes sont acceptées, mais l'utilisation explicite du mot clef JOIN (inner, outer, full) a le mérite de clarifier la requête :
Ce qui est du ressort du filtrage est dans la WHERE
Ce qui est du ressort de la jointure est dans le JOIN

Dans votre cas, la requete devient

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
SELECT pr.codeproduit, 
       pr.libelle, 
       st.dlc, 
       st.dluo, 
       st.dlclimite, 
       st.dluolimite, 
       st.lieustockage, 
       st.poids, 
       st.quantite, 
       ti.codetiers, 
       ti.nomtiers
FROM produit as PR
INNER JOIN stock as ST
   ON st.codeproduit=pr.codeproduit
INNER JOIN tiers as TI
   ON ti.idtiers = st.codetiers
WHERE PR.codeproduit =  '143007'

[thierrybatlle]

Bonjour,

Encore merci pour toutes ces informations.

[SQLpro]

Bonsoir,

J'ai écrit la requête SQL suivante :

Code :

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

1
2
3
4
 
SELECT produit.codeproduit, produit.libelle, stock.dlc, stock.dluo, stock.dlclimite, stock.dluolimite, stock.lieustockage, stock.poids, stock.quantite, tiers.codetiers, tiers.nomtiers
 FROM produit, stock, tiers
 WHERE codeproduit =  '143007'AND stock.codetiers=tiers.idtiers AND stock.codeproduit=produit.codeproduit

Mais j'ai des erreurs d'exécution que je n'arrive pas à localiser.

Merci bien pour votre aide.

La requête est syntaxiquement correcte mais votre syntaxe est obsolète depuis 33 ans ! En effet, les jointures se font avec l'opérateur JOIN !

A lire : http://sqlpro.developpez.com/cours/s...ointures/#LIII

A +