cs_garfield90
Messages postés388Date d'inscriptionlundi 7 juillet 2003StatutMembreDernière intervention10 février 2009 19 juil. 2006 à 12:07
Tout débutant devrait d'abord savoir qu'il existe un manuel en ligne, des livres très bien écrit , ainsi que des sites expliquant cela (vive les moteurs de recherches) mieux que ton tuto
- pas de session_start();
- pas de vérification d'existence des variables $_POST ( on demande comment on en arrive la)
- htmlentities !stripslashes> manuel pour savoir la différence
- si tu as plusieurs pseudos identiques, tu écrases les infos, pas cool
- tu pourrais expliquer les différents types d'ouverture de fichier a,r,w,x avec ou sans +
et pourquoi prendre l'1 plus que l'autre
- expliquer ce qu'est la gestion de droit et par extension les fonctions chmod et umask
- expliquer comment lire un tel fichier
- \n n'est pas un espace => c'est un code qui indique une nouvelle ligne et dire que cela dépend du systeme d'exploitation, tu peux aussi avoir \n\r ou l'inverse je sais plus (peut etre meme les 2)
- ce n'est pas un tuto mais plutot un code commenté car aucune pédagogie
- $_SESSION['connexion'] === $_SESSION['pseudo'] => intéret ?
- faille de sécurité a ne pas faire, laisser le passe visible ( hashé ou non ) dans un variable de session
-> choses intéressantes, faut bien chercher ;)
- tu fermes correctement l'accès au fichier via fclose
- tu utilises $_POST et non pas directement la variable
- utilisation de <?php et non pas de <?
PS : c'est pas le fichier qui doit être en CHMOD mais l'utilisateur qui doit avoir les bons droits ( en occurence écriture ) sur le dossier ou l'on mets le fichier et sur le fichier si on fait une modification.
En gros, je pense que c'est un source(tuto) a supprimé car il ne montre que des choses à ne pas faire et non l'inverse ;)
Ca reste que mon avis
cs_PaDa
Messages postés1802Date d'inscriptionmardi 15 juillet 2003StatutMembreDernière intervention22 septembre 20095 19 juil. 2006 à 11:23
Site de lame en même temps..
Idem pour l'orthographe sinon, yeurk
Bonne continuation
pastis51forever
Messages postés172Date d'inscriptiondimanche 29 juin 2003StatutMembreDernière intervention 8 septembre 20061 19 juil. 2006 à 09:38
Je pense que tu devrais éviter de faire de la promo de sites comme ca sur cs...
pastis51forever
Messages postés172Date d'inscriptiondimanche 29 juin 2003StatutMembreDernière intervention 8 septembre 20061 19 juil. 2006 à 09:35
Salut!
Pour que ton tuto soir un peu plus complet, tu devrais peut etre montrer comment ouvrir le fichier en lecture, et récupérer ces données (par exemple dans tes variables de session).
De plus, tu proposes de montrer comment enregistrer des variables de sessions, mais tu ne parles pas de session_start()... Ce qui sera certainement très dur à mettre en place.
Sinon, pour mélanger un peu mes deux commentaires précédents, tu pourrais aussi montrer comment on accède à ces variables de session... Une fois qu'on sait enregistrer dedans, c'est pas mal de pouvoir lire ces valeurs...
Enfin, l'intention est bonne...
Hello,
tu as dû aller un peu vite en mettant le niveau à Initié...
Sinon "membres_newsletter/$pseudo.txt" s'écrit 'membres_newsletter/'.$pseudo.'.txt', et on met $pseudo, pas "$pseudo".
Et fputs() est dépreciée, on utilise fwrite.
Pour finir, vu que c'est un tuto que tu nous as fait, tu peux revoir un peu l'orthographe STP? parce que là ça fait mal aux yeux ;^)
badr07
Messages postés53Date d'inscriptionmercredi 22 février 2006StatutMembreDernière intervention15 novembre 2006 19 juil. 2006 à 01:10
19 juil. 2006 à 12:07
- pas de session_start();
- pas de vérification d'existence des variables $_POST ( on demande comment on en arrive la)
- htmlentities !stripslashes> manuel pour savoir la différence
- si tu as plusieurs pseudos identiques, tu écrases les infos, pas cool
- tu pourrais expliquer les différents types d'ouverture de fichier a,r,w,x avec ou sans +
et pourquoi prendre l'1 plus que l'autre
- expliquer ce qu'est la gestion de droit et par extension les fonctions chmod et umask
- expliquer comment lire un tel fichier
- \n n'est pas un espace => c'est un code qui indique une nouvelle ligne et dire que cela dépend du systeme d'exploitation, tu peux aussi avoir \n\r ou l'inverse je sais plus (peut etre meme les 2)
- ce n'est pas un tuto mais plutot un code commenté car aucune pédagogie
- $_SESSION['connexion'] === $_SESSION['pseudo'] => intéret ?
- faille de sécurité a ne pas faire, laisser le passe visible ( hashé ou non ) dans un variable de session
-> choses intéressantes, faut bien chercher ;)
- tu fermes correctement l'accès au fichier via fclose
- tu utilises $_POST et non pas directement la variable
- utilisation de <?php et non pas de <?
PS : c'est pas le fichier qui doit être en CHMOD mais l'utilisateur qui doit avoir les bons droits ( en occurence écriture ) sur le dossier ou l'on mets le fichier et sur le fichier si on fait une modification.
En gros, je pense que c'est un source(tuto) a supprimé car il ne montre que des choses à ne pas faire et non l'inverse ;)
Ca reste que mon avis
19 juil. 2006 à 11:23
Idem pour l'orthographe sinon, yeurk
Bonne continuation
19 juil. 2006 à 09:38
19 juil. 2006 à 09:35
Pour que ton tuto soir un peu plus complet, tu devrais peut etre montrer comment ouvrir le fichier en lecture, et récupérer ces données (par exemple dans tes variables de session).
De plus, tu proposes de montrer comment enregistrer des variables de sessions, mais tu ne parles pas de session_start()... Ce qui sera certainement très dur à mettre en place.
Sinon, pour mélanger un peu mes deux commentaires précédents, tu pourrais aussi montrer comment on accède à ces variables de session... Une fois qu'on sait enregistrer dedans, c'est pas mal de pouvoir lire ces valeurs...
Enfin, l'intention est bonne...
19 juil. 2006 à 02:02
tu as dû aller un peu vite en mettant le niveau à Initié...
Sinon "membres_newsletter/$pseudo.txt" s'écrit 'membres_newsletter/'.$pseudo.'.txt', et on met $pseudo, pas "$pseudo".
Et fputs() est dépreciée, on utilise fwrite.
Pour finir, vu que c'est un tuto que tu nous as fait, tu peux revoir un peu l'orthographe STP? parce que là ça fait mal aux yeux ;^)
19 juil. 2006 à 01:10
...
Deconne!!!