Discussion :

[ClarusAD]

Salut,
J'ai besoin d'aide pour mettre en place ma base de données
Voici ma base :

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
18
19
20
21
22
23
24
CREATE TABLE articles (
    id     INTEGER      PRIMARY KEY ASC AUTOINCREMENT
                        UNIQUE
                        NOT NULL,
    status VARCHAR (3)  NOT NULL
                        DEFAULT pub,
    date   INTEGER (12) NOT NULL,
    titre  VARCHAR (20) NOT NULL
                        DEFAULT ('Titre du billet'),
    corps  TEXT         NOT NULL
                        DEFAULT ('Corps du billet'),
    taga   INTEGER (2)  NOT NULL
                        DEFAULT (1) 
                        REFERENCES tags (id),
    tagb   INTEGER (2)  REFERENCES tags (id),
    tagc   INTEGER (2),
);
 
CREATE TABLE tags (
    id  INTEGER NOT NULL
                PRIMARY KEY AUTOINCREMENT
                UNIQUE,
    tag VARCHAR
);

Table des billet :

ID	STATUS	DATE	TITRE	CORPS	TAG A	TAG B
1	pub	201706071433	Titre du billet	Corps du billet.	2	3

et la table des types :

ID	tag
1	Non classée
2	transports
3	habitats
4	loisirs

Au final, j'aimerais avoir :

ID	STATUS	DATE	TITRE	CORPS	TAG A	TAG B
1	pub	201706071433	Titre du billet	Corps du billet.	transports	habitats

J'ai donc essaye avec cette requète :

Code :

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

1
2
3
4
5
SELECT * FROM "articles", "tags"
WHERE
articles.taga = tags.id
AND 
articles.tagb = tags.id

Évidemment ça me renvoie une erreur

Est-ce que quelqu'un pourrait m'aider ?
Merci.

[al1_24]

Évidemment ça me renvoie une erreur

Cette erreur est certainement accompagnée d'un message. Lequel ?

Parce que, syntaxiquement, il n'y a rien qui pose problème. Même si, fonctionnellement, elle ne retournera pas le résultat attendu.
Pour cela il faudrait joindre deux fois la table tags, une fois sur taga, une fois sur tagb.
D'ailleurs, si tagb est nullable, il faudra penser à utiliser une jointure externe. Sinon les lignes où tagb est null seront exclues du résultat de la requête.

J'ajouterai que la structure de la base pourrait être revue. En effet, les associations de tags à des articles devraient être placées dans une table... d'association.

Rappel : depuis 25 ans, les jointures s'écrivent avec l'opérateur JOIN.

[ClarusAD]

Merci pour cette piste, je vais donc revoir la structure de la base de données avec 3 tables.

[ClarusAD]

Il me vient une question : si j'ajoute une table d'associations, la présence de clés étrangères dans la table articles, devient-elle pertinante ?
Merci.

[ClarusAD]

Je me réponds à moi même : exit les clés étrangères puisque les colonnes tags disparaîssent (au profit d'une table assiciations).

[ClarusAD]

C'était donc ça la solution : une table d'associations entre un articles et ses catégories.
Et la requête qui va bien (par exemple pour récupérer l'article 1 avec toutes ses catégories) :

Code :

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

1
2
3
4
5
select * from articles,associations
where
associations.idart=1
and
articles.id=1

Ça me retourne :

ID	STATUS	DATE	TITRE	CORPS	IDART	IDTAG
1	pub	201706071433	titre du billet	corps du billet	1	2
1	pub	201706071433	titre du billet	corps du billet	1	3

[al1_24]

Ce serait tellement plus beau avec une opération de jointure

Code :

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

1
2
3
4
5
6
7
8
9
select  *
from    articles
    inner join
        associations
        on  associations.idart = articles.id
    inner join
        tags
        on  associations.idtag = tags.id
;

Au passage, il serait beaucoup plus judicieux de ne pas donner le même nom à tous les identifiants de table. La bonne pratique est qu'un nom de colonne, quelle que soit la table où il est utilisé, représente toujours la même information. Ainsi, l'identifiant d'article serait idart, aussi bien dans la table articles que dans la table associations.

[ClarusAD]

Effectivement : c'est déjà bien plus propre avec une jointure ^^
Quand aux bonnes pratiques, je comprends bien mais je m'intérroge sur un éventuel risque de confusion que donnerait ta requête ainsi modifiée :

Code :

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

1
2
3
4
5
6
7
8
select  *
from    articles
    inner join
        associations
        on  associations.idart = articles.idart
    inner join
        tags
        on  associations.idtag = tags.idtag

Tu vas me dire : "c'est une question d'habitude", ou bien ?