Discussion :

[babyy]

Bonjour;

J'ai cette requête et j'ai créer toutes les vues qui lui concerne mais j'ai pas pu résolu cette partie :


Requête :

Code :

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

1
2
select s_acctbal, s_name, n_name, p_partkey, p_mfgr, s_address, s_phone, s_comment
 from part, supplier, partsupp, nation, region where p_partkey = ps_partkey and s_suppkey = ps_suppkey and p_size = 1 and p_type like '%NICKEL' and s_nationkey = n_nationkey and n_regionkey = r_regionkey and r_name = 'AMERICA' and ps_supplycost = ( select min(ps_supplycost) from partsupp, supplier, nation, region where p_partkey = ps_partkey  and s_suppkey = ps_suppkey and s_nationkey = n_nationkey and n_regionkey = r_regionkey  and r_name = 'AMERICA' ) order by s_acctbal desc, n_name, s_name, p_partkey;

Partie non résolu :

Code :

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

ps_supplycost = ( select min(ps_supplycost) from partsupp, supplier, nation, region where p_partkey = ps_partkey  and s_suppkey = ps_suppkey and s_nationkey = n_nationkey and n_regionkey = r_regionkey  and r_name = 'AMERICA' )

J'ai créer cette vue

Code :

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

Create view Q2_Project2 As select  min(ps_supplycost) from partsupp, supplier, nation, region ;

mais il reste ça cette écriture est il vrai :

Code :

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

Create view Q2_Filter4 As select * from  partsupp where  ps_supplycost = Q2_Project2 ;

Merci

[vttman]

Il y a 2 - 3 problèmes Babyy
- la forme ancienne d'écriture des jointures (chercher sur le net : sql, select, inner join)
- la vue Q2_Project2 est créée sans aucune jointure sur les tables
...
Donc plutôt que d'en disserter
Je propose de tester ceci

1) Essai (benh oui je ne ne connais pas le MCD)
- On vire la vue actuelle

Code :

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

drop view Q2_Project2;

- on la recrée avec les bonne jointures ? entre les tables

Code :

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

Create view Q2_Project2 As select min(ps_supplycost) as min_ps_supplycost from partsupp, supplier, nation, region where p_partkey = ps_partkey  and s_suppkey = ps_suppkey and s_nationkey = n_nationkey and n_regionkey = r_regionkey and r_name = 'AMERICA';

- on vérifie que le résultat est correct

Code :

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

select min_ps_supplycost from Q2_Project2

2) Pour mieux comprendre les vues
- On revire la vue actuelle

Code :

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

drop view Q2_Project2;

- on la recrée avec les bonne jointures ? entre les tables

Code :

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

Create view Q2_Project2 As select ps_supplycost,r_name from partsupp, supplier, nation, region where p_partkey = ps_partkey  and s_suppkey = ps_suppkey and s_nationkey = n_nationkey and n_regionkey = r_regionkey ;

- on vérifie que le résultat est correct

Code :

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

1
2
 
select min(ps_supplycost) from Q2_Project2 where r_name = 'AMERICA';

[escartefigue]

Il y a 2 - 3 problèmes Babyy
- la forme ancienne d'écriture des jointures (chercher sur le net : sql, select, inner join)
- la vue Q2_Project2 est créée sans aucune jointure sur les tables

Tout à fait et aussi, l'une des fonctions des vues est de garantir l'indépendance des données vis à vis des traitements.
Or, sur ce point, créer une vue avec "SELECT *" est une aberration car toute modification de la table aura un impact sur la requête !.
Une vue ne doit JAMAIS utiliser SELECT * une requête non plus du reste, hors requete jetable.

De plus, une requête doit être facile à comprendre pour permettre sa maintenance, il faut donc soigner la présentation, et utiliser des alias pour identifier clairement de quelle table vient telle colonne.
Et enfin, comme mentionné par VTTMan, mais non appliqué dans sa proposition, les jointures se codent avec l'opérateur JOIN.
Ce qui donnerait dans votre cas quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
Create view Q2_Project2 As 
 
       select min(ps_supplycost) as min_ps_supplycost 
       from       partsupp as PS
       inner join supplier as SU
          on SU.s_suppkey = PS.ps_partkey 
       inner join nation   as NA
          on NA.n_nationkey = SU.s_nationkey
       inner join region   as RG 
          on RG.r_regionkey = NA.n_regionkey
       where r_name = 'AMERICA';

Dernière remarque : il est très étrange que les clefs étrangères portent un nom différent de celui de la colonne de la table dont elle est issue. C'est très peu pratique !
Ah non encore une remarque : filtrer sur une chaine de caractères est dangereux et peu performant. Dangereux car attention aux majuscules/minuscules, problèmes de collation et autres erreurs de saisie et peu performant car les colonnes char/varchar sont rarement indexées, et si index il y a, il est plus coûteux qu'un index sur un code ou un identifiant

ATTENTION : j'ai repris les jointures telles qu'elles, sans savoir si elles sont utiles, d'ailleurs j'en doute. Sans description ni explication supplémentaire du demandeur, difficile de savoir quelles sont les tables vraiment utiles