Discussion :

[enterprise]

Bonjour,

Je demande votre aide car je suis bloqué, je ne sais as faire...
Je dispose d'une table 'users' comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
CREATE TABLE users
(
    id_users INT PRIMARY KEY AUTO_INCREMENT,
    firstname VARCHAR(100),
    lastname VARCHAR(100),
    login VARCHAR(255),
    email VARCHAR(100),
    password VARCHAR(255),
    manager INT,
    id_role INT,
    first_connex INT
);

La colonne manager correspond à un id_users. Quand je créer un utilisateur, je dois lui attribué un manager, et je stock donc dans manager la valeur de id_users du manager.
Je cherche à faire un "select firstname, lastname, manager" et je veux que pour "manager" ça m'affiche le "login" du manager et non son "id_users"

Pour vous donner un exemple, voici ce que j'ai:


voici ce que je veux obtenir avec ma requête:


En vous remerciant.

[CinePhil]

1) Stocker les mots de passe en clair dans la BDD est une mauvaise idée !
En plus vos mots de passe sont crackables par un enfant !

2) Oui, on peut faire une auto-jointure. Avez-vous essayé ?
Si oui, quelle requête avez-vous essayée ?

[enterprise]

Bonjour cinephil!

1) Vous vous doutez bien que vu les noms présent dans ma BDD je suis en période de tests j'ai besoin que les MDP soient en clair pour les vérifier, car ils sont auto-générer.
Enfin, cela non ne sont pas auto-générer, mais les nouveaux oui!

2) Voici un des trucs que j'ai testé, mais je n'ai pas le résultat voulu:

Code :

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

1
2
SELECT u.lastname, u.firstname, u.manager FROM users
INNER JOIN users u on u.manager=u.id_users=u.login

[CinePhil]

Comme vous utilisez deux fois la même table, il faut donner deux alias différents :

Code :

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

1
2
3
SELECT u1.lastname, u1.firstname, u2.login AS login_manager 
FROM users u1
INNER JOIN users u2 on u1.manager = u2.id_users

[enterprise]

Merci beaucoup ça me donne le résultat souhaité!!!

Je ne savais pas qu'on pouvais donné deux alias différent pour une même table!

Et quand je vois cette requête je me dis que j'ai été idiot de mettre u.manager au lieu de u.login...

[CinePhil]

Les deux utilités des alias sont :
1) Utiliser deux fois la même table dans la même requête
2) Simplifier l'écriture des requêtes multi-tables pour ne pas écrire le nom de la table en entier dans toute la requête. Ça simplifie aussi la lecture.

[enterprise]

Et bien merci beaucoup je ne connaissais pas!

Aujourd'hui grâce à toi j'ai appris.

[CinePhil]

Encore quelques remarques sur votre table...

1) Écrivez le nom de vos tables et colonnes au singulier.
L'usage d'un pluriel peut prêter à confusion et faire penser à une mauvaise conception.
Votre table user (sans s, donc) est issue de l'entité type user du modèle conceptuel de données (méthode Merise) ou de votre diagramme de classes (langage UML). Ce MCD (le meilleur modèle pour concevoir une base de données) est lui-même issu de règles de gestion de ce type :
Un user est encadré par un seul manager et un manager peut encadrer plusieurs users.

Vous voyez que la règle de gestion décrit l'association entre UN user et UN manager au singulier. De plus, chaque ligne de votre table ne contiendra qu'un seul user.

=> table user et colonne id_user

2) Vos colonnes sont probablement formatées de manière trop grande.
Je n'aimerais pas porter un nom et un prénom de 100 caractères et je ne me choisirais sûrement pas un login ni un mot de passe aussi long !

3) Puisque vous allez chiffrer les mots de passe, déterminez la longueur de la colonne à la longueur du chiffrage. Du coup, vous pouvez utiliser le type CHAR plutôt que VARCHAR puisque tous les mots de passe auront la même longueur chiffrée.

4) N'oubliez pas de programmer vos clés étrangères. Vous en avez ici deux : manager et id_role.
Ceci est valable bien entendu si votre table n'est pas en MyISAM, ce que j'ose espérer !

Au passage, j'appellerais la colonne plutôt id_manager.

5) Éviter l'usage du bonhomme NULL !
Il nuit aux performances et est sujet à interprétations car il ne veut rien dire en lui-même sinon une absence de valeur.

6) N'oubliez pas non plus d'indexer votre table... et les autres, évidemment !

[enterprise]

Merci pour tous ces conseils!
C'est vrai que ma base n'est pas très propre, je prends note et je vais tout redimensionner et renommer lorsque je passerais en prod.

4)Normalement j'ai programmé toute mes clés étrangères, mais je comptes quand même vérifier.

5) J'ai laissé Null car sinon il aurait fallu que je l'initialise à 0 par défaut et je ne savais pas comment faire...
D'ailleurs dans une autre table, j'ai une colonne 'commentaire' qui est aussi initialisé à NULL.... et je ne vois pas quoi mettre d'autre car c'est une varchar

6)Ensuite pour l'index, oui c'est prévu

[CinePhil]

J'ai laissé Null car sinon il aurait fallu que je l'initialise à 0 par défaut et je ne savais pas comment faire...

Code :

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

first_connex INT NOT NULL DEFAULT 0

D'ailleurs dans une autre table, j'ai une colonne 'commentaire' qui est aussi initialisé à NULL.... et je ne vois pas quoi mettre d'autre car c'est une varchar

Code :

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

commentaire NOT NULL DEFAULT ''

À noter que sans précision de la valeur par défaut, MySQL en a une pour chaque type de colonne :
- 0 pour les colonnes numériques ;
- '' (chaîne vide) pour les colonnes textuelles ;
- et l'horrible '0000-00-00' pour les colonnes de date.

[enterprise]

Merci!

[CinePhil]

J'ai mis à jour mon commentaire précédent, probablement après votre merci.

[enterprise]

Effectivement!

On ne m'avait jamais appris tout ça, comme quoi il faut pratiquer pour s'améliorer!
Merci beaucoup