Avis sur mon script
Résolu/Fermé
Mephistole
Messages postés
55
Date d'inscription
lundi 28 mars 2016
Statut
Membre
Dernière intervention
20 avril 2020
-
Modifié le 1 mai 2018 à 09:56
Mephistole Messages postés 55 Date d'inscription lundi 28 mars 2016 Statut Membre Dernière intervention 20 avril 2020 - 11 mai 2018 à 12:49
Mephistole Messages postés 55 Date d'inscription lundi 28 mars 2016 Statut Membre Dernière intervention 20 avril 2020 - 11 mai 2018 à 12:49
A voir également:
- Avis sur mon script
- Script vidéo youtube - Guide
- Ghost script - Télécharger - Polices de caractères
- Script bat - Guide
- Microsoft activation script - Accueil - Windows
- Executeur de script - Télécharger - Édition & Programmation
2 réponses
yg_be
Messages postés
23338
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
19 novembre 2024
Ambassadeur
1 551
Modifié le 1 mai 2018 à 10:55
Modifié le 1 mai 2018 à 10:55
bonjour,
il est préférable de :
- mettre session_start() en début de page php
- de ne pas enregistrer les mots de passe, mais de stocker un hash du mot de passe. tu peux utiliser password_hash pour créer le hash, et password_verify pour vérifier que le mot de passe est correct.
par ailleurs, il me semble que ce script mélange erronément deux actions:
- inscription d'un nouvel utilisateur
- connection d'un utilisateur existant
il est préférable de :
- mettre session_start() en début de page php
- de ne pas enregistrer les mots de passe, mais de stocker un hash du mot de passe. tu peux utiliser password_hash pour créer le hash, et password_verify pour vérifier que le mot de passe est correct.
par ailleurs, il me semble que ce script mélange erronément deux actions:
- inscription d'un nouvel utilisateur
- connection d'un utilisateur existant
jordane45
Messages postés
38296
Date d'inscription
mercredi 22 octobre 2003
Statut
Modérateur
Dernière intervention
19 novembre 2024
4 704
2 mai 2018 à 20:57
2 mai 2018 à 20:57
Bonjour,
En ce qui me concerne.. je ne comprend pas que tu ne fasses pas la vérification dans la class user ?
Class User dans laquelle je placerait un constructeur avec le login et le password.
Ensuite, il est tout de même conseillé de récupérer PROPREMENT les variables AVANT de les utiliser.
Je t'invite à lire ceci : https://forums.commentcamarche.net/forum/affich-37584947-php-gestion-des-erreurs-debogage-et-ecriture-du-code
Et pour finir .... l'instruction session_start est à placer au tout début de ton fichier PHP. Ne jamais le mettre en plein milieu.
En ce qui me concerne.. je ne comprend pas que tu ne fasses pas la vérification dans la class user ?
Class User dans laquelle je placerait un constructeur avec le login et le password.
Ensuite, il est tout de même conseillé de récupérer PROPREMENT les variables AVANT de les utiliser.
Je t'invite à lire ceci : https://forums.commentcamarche.net/forum/affich-37584947-php-gestion-des-erreurs-debogage-et-ecriture-du-code
Et pour finir .... l'instruction session_start est à placer au tout début de ton fichier PHP. Ne jamais le mettre en plein milieu.
Mephistole
Messages postés
55
Date d'inscription
lundi 28 mars 2016
Statut
Membre
Dernière intervention
20 avril 2020
1
11 mai 2018 à 12:49
11 mai 2018 à 12:49
Merci de ton avis, je vais en tenir compte.
2 mai 2018 à 15:29
Tu veux dire quoi quand tu dis que le script mélange 2 actions ?
2 mai 2018 à 20:10
si il ne s'agit pas d'enregistrer un nouvel utilisateur, je ne vois pas l'interet de vérifier la syntaxe de l'adresse email.
que fait getHashByMail()?
(aussi discuté ici: https://openclassrooms.com/fr/forum/sujet/avis-sur-mon-script?page=1#message-92346501)
2 mai 2018 à 20:38
Le nom, le prénom et le mail sont récupérés pour être utilisé plus tard.