salut, je travail sur un site de stream
Ca fait 1 an que je n'ai pas fait de SQL ou PHP (je fais du réseau).

J'aimerai avoir un retour sur mon SQL, cela vous parait il logique ?

CREATE DATABASE STREAMING ; 

CREATE TABLE Serie (
	NomSerie varchar(50) NOT NULL,
	NombreSaisons INTEGER NOT NULL,
	PRIMARY KEY (NomSerie)
);

CREATE TABLE Saison (
	NumeroSaison INTEGER NOT NULL,
	NombreEpisodes INTEGER NOT NULL,
	PRIMARY KEY (NumeroSaison)
)

CREATE TABLE Episode (
	IdEpisode INTEGER NOT NULL,
	NumeroEpisode INTEGER NOT NULL,
	NomEpisode varchar(50) NOT NULL,
	DescriptionEpisode varchar(120) NOT NULL,
	NoteEpisode INTEGER NOT NULL,
	Lecteur1VF varchar(50) NOT NULL,
	Lecteur2VF varchar(50) NOT NULL,
	Lecteur3VF varchar(50) NOT NULL,
	LecteurVOSTFR varchar(50) NOT NULL,
	PRIMARY KEY (IdEpisode),
	FOREIGN KEY (NumeroSaison) REFERENCES Saison(NumeroSaison)
)

Sur le site j'ai simplement un menu

Saisons :
saison 1 -> EP1 EP2
saison 2 -> EP1 EP2

Un varchar en guise de clé primaire, surtout pour le nom de al série, ça pue à mon avis.

Le 18 juin 2016 à 21:54:45 GrandeGuerre a écrit :
Un varchar en guise de clé primaire, surtout pour le nom de al série, ça pue à mon avis.

en effet, c'est corrigé

CREATE TABLE Serie (
	IdSerie INTEGER NOT NULL,
	NomSerie varchar(50) NOT NULL,
	NombreSaisons INTEGER NOT NULL,
	PRIMARY KEY (IdSerie)
);

CREATE TABLE Saison (
	IdSaison INTEGER NOT NULL,
	NumeroSaison INTEGER NOT NULL,
	NombreEpisodes INTEGER NOT NULL,
	PRIMARY KEY (IdSaison)

J'ai maintenant des ID

Pourquoi faut il évité les varchar pour la clé primaire ?

je ne comprend pas bien le principe de clé étrangères
PhpMyAdmin m'indique une erreur

Salut, non ce n'est pas terrible imo.

- Utilise un identifiant (champs id) pour toutes tes tables et sers toi comme foreign key dans celles qui ont besoin de la relation.

En gros tu devrais avoir un schéma de la sorte (les x..x représente les relations, '1' pour une instance et 'n' pour plusieurs)
Serie (1 --> n) Saison (1 --> n) Episode
Donc chaque saison aurait un champs serie_id, et chaque episode un champs saison_id

- Débarasse toi du champs "nombreXX", c'est useless et contraignant, si tu veux savoir combien d'episodes possède ta saison, tu (qqchose du genre syntaximent)

 select count(episode.id) where episode.saison_id = :id_saison

- Autre avis, je trouve inutile de rajouter le nom de la table a ses champs (nomSerie pour la table serie etc.)

- Pense à faire une table pour tout ce qui dépend d'une row de la table. Par exemple un episode pourrait avoir plusieurs lien (et donc plusieurs "Lecteur" ? ) et un autre episode un autre nombre de liens. Donc plutot privilegier une table juste pour les liens, que chaque episode puisse en avoir autant qu'il veut, dans les formats qu'il souhaite (VF VOSTFR qualité aussi?.)

- Pour ton erreur sql, je ne sais pas, je m'y connais pas tellement que ca en sql pur en fait

Bon courage

Merci pour vos conseils
je vais chercher mes cours de SQL car je suis à la ramasse.

Me revoila
Changement de stratégie finalement je ne créer qu'une table.
Je ne vais pas me compliquer la vie alors que je suis un débutant

CREATE TABLE Episode (
	IdEpisode INTEGER NOT NULL AUTO_INCREMENT,
	Numero INTEGER NOT NULL,
	Nom varchar(50) NOT NULL,
	Description varchar(500) NOT NULL,
	Note INTEGER NOT NULL,
	Lecteur1VF varchar(400) NOT NULL,
	Lecteur2VF varchar(400) NOT NULL,
	Lecteur3VF varchar(400) NOT NULL,
	LecteurVOSTFR varchar(400) NOT NULL,
	Saison INTEGER NOT NULL,
	PRIMARY KEY (IdEpisode),
);

C'est suffisant pour les requêtes que je veut faire.
Maintenant je passe au PHP