SCRIPTE DE MOT DE PASSE

Messages postés
10918
Date d'inscription
lundi 24 février 2003
Statut
Modérateur
Dernière intervention
2 mars 2010
- - Dernière réponse : Staify
Messages postés
10
Date d'inscription
dimanche 30 septembre 2007
Statut
Membre
Dernière intervention
16 avril 2008
- 16 avril 2008 à 16:01
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/46223-scripte-de-mot-de-passe

Afficher la suite 
malalam
Messages postés
10918
Date d'inscription
lundi 24 février 2003
Statut
Modérateur
Dernière intervention
2 mars 2010
17 -
Hello,

au moins, c'est plutôt propre pour quelqu'un qui a commencé le php hier. Le html est propre, les css aussi. Seul grief : le php en plein milieu de ton html.
Tu testes ton $_POST, c'est très bien.
Par contre, pourquoi faire ce htmlspecialchars() et ce htmlspecialchars_decode() au juste ? Je ne vois pas du tout la logique du truc. Tu voulais faire quoi ?
Puisque tu es débutant, tourne-toi vers les sessions si tu veux un peu avancer sur la protection de pages. Ca te sera utile.
cs_yoman64
Messages postés
593
Date d'inscription
samedi 19 janvier 2002
Statut
Membre
Dernière intervention
4 décembre 2008
-
Salut,
Comme dit malalam tes fonctions de specialchars html sont plutot inutile. Ensuite je te suggère de prendre l'habitude d'utiliser plutot que .
Ensuite je crois que cela a plus sa place sur codyx.org qu'ici, surtout que c'est assez inutile sachant que tu n'utilise aucune méthode pour garder l'authentification.

Mais le code est propre et tu semble te soucier des standards, ce qui est une excellente chose pour un débutant, on ne voit pas ça assez souvent :)
Staify
Messages postés
10
Date d'inscription
dimanche 30 septembre 2007
Statut
Membre
Dernière intervention
16 avril 2008
-
Bonjour,
J'ai ici dans les sources mi les codes de mes trop fichier, ils sont délimités par des commentaires dans leurs langages respectif.
J'ai utilisé la fonction htmlspecialchars ici car un ami à réussi à trouvé le mot de passe choisie avec un simple script d'apelle :
"echo $MDP;"
Alors que maintenant il ne peut plus :p . De plus c'était mon premier scripte, donc je me lance maintenant dans un mini-chat sans base de donné qui enregistrera le message et le pseudo dans un .txt.
Merci Yoman64, c'est une habitude que j'ai pris lorsque je fait du XML.
Malam je ne comprend pas ta phrase :

" Seul grief : le php en plein milieu de ton html. "
Merci encore de vos commentaire.
malalam
Messages postés
10918
Date d'inscription
lundi 24 février 2003
Statut
Modérateur
Dernière intervention
2 mars 2010
17 -
Pour ça, encore faut il pouvoir mettre ce script sur ton serveur...
et là, il n'a qu'à écrire echo htmspecialchars_decode($MDP);
je ne vois pas ce que ça change. Et cette fonction ne sert pas à ça. Sincèrement, ces deux fonctions sont inutiles dans ton code.

Il vaut mieux séparer PHP du html. Si tu mets les deux dans un seul fichier, tâche au moins de mettre le php avant le html, simplement. C'est bien plus lisible ainsi.
Staify
Messages postés
10
Date d'inscription
dimanche 30 septembre 2007
Statut
Membre
Dernière intervention
16 avril 2008
-
Bonsoir,
Malam voilà j'ai modifié mon code avec tes conseilles et j'aimerai dire que c'est la dernier modification car se script est super simple ! A part cela mon mini-chat sans base de donné qui fonction avec un fichier texte avance à grand pas !
Merci encore pour vos réactions !