Aller au menu - Aller au contenu

[Plan du site] Vous êtes ici --- > Le Site du Zéro > Les tutoriels > Non-Officiels > Site Web > PHP > Systèmes complets > Système de messagerie privée (MP) > Lecture des commentaires

Système de messagerie privée (MP)

Vous devez être inscrit pour pouvoir poster des messages

Page : 1  2  Suivante
Pseudo Commentaire
Page : 1  2  Suivante
Hors ligne Luigi_home # Posté le 30/10/2006 à 13:28:08 - Ce membre a mis la note : 14
I Love You Hilary Duff !
Avatar
Groupe : Membres
 Bon tuto mais il manque cruellement d'information. Tu nous présente des codes qui sont bien commentés mais pas trop d'explication sur les fonctions utilisés, pourquoi on les utilises, comment on les utilises etc. De plus, le script n'est pas sécurisé au maximum et n'est pas non plus optimisé.

Ex: Dans le code pour lire un MP, on peut voir une requête SQL qui demande de sélectionner le champ "destinataire". Si le destinataire doit normalement être pareil que $_SESSION['pseudo'], pourquoi le demander ? Pour vérifier si le destinataire est le bon, fais ceci:

Code : PHP
$retour = mysql_query("SELECT sujet, expediteur, timestamp, message FROM mp WHERE id='".$id_mp."' AND destinataire='".$_SESSION['pseudo']."'")or die(mysql_error());
        $donnees = mysql_fetch_assoc($retour);
// vérification pour pas qu'une autre personne que le destinataire puisse voir le message
        if(!empty ($donnees))
        {


Ça a l'air de rien mais ça optimise un peu parce que MySQL ne sélectionne que les champs utiles.

Mais bon, en général, le tuto est bien. Je lui donne un 14 sur 10
Souriez, Luigi_home vous aime :) .
 
Hors ligne Vizigr0u # Posté le 30/10/2006 à 15:19:20 - Ce membre a mis la note : 14
fooo !
Avatar
Groupe : Membres
 Moi ce qui me chiffonne c'est les types de champ SQL que tu utilises :euh:
C'est peut-être plus simple pour les débutants/zéros mais je trouverais ca plus logique au lieu des champs varchar de destinataire/expéditeur d'avoir des champs de type int qui contiennent l'id de ces derniers. Après vaut-il mieux stoquer plus d'infos dans la BDD et avoir une requête moins lourde...
Et sinon je trouverais l'emploi d'un champ timestamp beaucoup plus approprié pour cette utilisation, mais bon encore une fois, comme on s'adresse à des zéros qui pour la pluaprt connaissent les cours de matéo, je comprend.
Citation : tuto
Pour ceux qui s'attendaient à un tutoriel où on donne le code en entier en bas de page, vous vous trompez d'adresse. Je donne les codes par morceau avec des explications. Il faut quand même réfléchir un minimum :colere: :)
oui en plus les validateurs ne risquent pas de te le valider :p

Sinon le tuto est sympa. J'ai pas compris l'histoire du javascript qui redirige :euh: , tu devrais expliquer pourquoi tu fais ça.
Un ptit tuto qui mérite quand même une note sympa...
Image utilisateur


Une équipe, 4 membres, un projet : Project: HGF
www.fooo.fr
projet d'étudiants de l'EPITA
 
Hors ligne Gaga971 # Posté le 30/10/2006 à 16:03:36 - Ce membre a mis la note : 10
Programmeur indépendant
Avatar
Groupe : Membres
 Salut!

Moi je trouve ce tutorial pas super...
Tout dabord on n'utilise pas le pseudo du membre mais son id!
De plus la sécurité elle est pas top... Mais defois tu fais des trucs vraiment qui servent a rien... par exemple pour supprimer un element.

J'ai mis 10

 
Hors ligne Panzhor # Posté le 30/10/2006 à 20:59:27 - Ce membre a mis la note : 20
Groupe : Membres
 Personnelement, personne n'a voulu se lancer dans ce tuto.

C'est pas forcément facile de tout présenter surtout pour un tuto de Messagerie privée.
Je n'es encore jamais vu de tuto de ce genre ( MP ).

Alors comme c'est le seul à s'être lancer => 20
www.panzhor.cliranet.com/Univers-Games/
 
Hors ligne Arhenion # Posté le 30/10/2006 à 23:04:17 - Ce membre a mis la note : 20
Coding is more than a job :°)
Avatar
Groupe : Membres
 Salut,

Ton tuto est Génial.Tu est la premiere à étre lancée dans le sujet.Franchement je met 18. ;)
Je te suprime 2 points parce-que le script n'est optimisé.(tu fais des choses inutiles)
De plus, le javascript n'est pas expliqué.(il manque une accolade à la fin).
L'intelligence est l'aptitude à éviter de travailler tout en réalisant ce qui doit l'être. - Linus tolvard :-°

Ceux qui trouvent qu'installer Linux est compliqué n'ont pas essayé de sortir avec une fille.
 
Hors ligne Nicol@s # Posté le 30/10/2006 à 23:07:43 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Je suis un garçon....
Bref si l'erreur retournée est la table mp.membres n'existe pas c'est que tu n'as pas créé de table mp ^^ ou alors tu n'as pas changé le nom de la table dans le code.
Pour l'optimisation je sais et je le dit je crois (me rappelle plus ^^ ).
Et le javascript j'y connais rien alors je peux pas donner d'explications, voili voilou.
Tutoriel fini, validé et édité : Système de messagerie privée (MP). VALIDE !!!
Image utilisateur
 
Hors ligne Gaga971 # Posté le 31/10/2006 à 11:10:07 - Ce membre a mis la note : 10
Programmeur indépendant
Avatar
Groupe : Membres
 Tu devrais revoir la structure de presentation du tutorial ca compte quand tu es lecteur aussi ;)

 
Hors ligne Lynix # Posté le 31/10/2006 à 11:37:17 - Ce membre a mis la note : 12
Graphiste 3D
Avatar
Groupe : Membres
 mouais tu explique plutot bien mais ta redirection en javascript :colere: je t'aurai mis 16, je te met 12
Utopy Bientot :D
 
Hors ligne Lynix # Posté le 31/10/2006 à 11:37:39 - Ce membre a mis la note : 12
Graphiste 3D
Avatar
Groupe : Membres
 et j'avais oublié de voter ^^
Utopy Bientot :D
 
Hors ligne Luigi_home # Posté le 31/10/2006 à 13:18:11 - Ce membre a mis la note : 14
I Love You Hilary Duff !
Avatar
Groupe : Membres
 Citation : Panzhor
Personnelement, personne n'a voulu se lancer dans ce tuto.


Faux. Je suis lancé dans un tuto de MP depuis 2005, c'est juste qu'il n'a pas encore été validé :p
Souriez, Luigi_home vous aime :) .
 
Hors ligne Nicol@s # Posté le 31/10/2006 à 13:54:30 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Citation : Lynix

mouais tu explique plutot bien mais ta redirection en javascript :colere: je t'aurai mis 16, je te met 12

Je l'ai mit en JS parce qu'avec header ça n'aurait pas marché (headers already sent) parce qu'il y a du code HTML avant la redirection. Je sais qu'il existe ob_start() et ob_end_flush() (je crois) mais je n'ai jamais compris comment ça marche.

Citation : Gaga971
Tu devrais revoir la structure de presentation du tutorial ca compte quand tu es lecteur aussi ;)

Peux-tu me donner plus de précision s'il te plaît ?
Tutoriel fini, validé et édité : Système de messagerie privée (MP). VALIDE !!!
Image utilisateur
 
Hors ligne Gaga971 # Posté le 31/10/2006 à 15:22:45 - Ce membre a mis la note : 10
Programmeur indépendant
Avatar
Groupe : Membres
 En fait au lieu de nous étaler du code comme ceci tu devrais le présenter par block et le commenter par exemple,au lieu de mettre le code et une petite conclusion, parce-que a première vue je le trouve assez brouillon voilà tout.

N'oublions pas que les tutos des zeros s'adressent à des zeros :)

 
Hors ligne Nicol@s # Posté le 31/10/2006 à 18:23:34 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Citation : Gaga971
En fait au lieu de nous étaler du code comme ceci tu devrais le présenter par blocs et le commenter par exemple

Si tu regardes, tu verras que le code est présenté par blocs d'actions et il y a presque plus de lignes de commentaires que de lignes de codes (quant même pas mais bon, j'ai pas envie de compter :p ).
Tutoriel fini, validé et édité : Système de messagerie privée (MP). VALIDE !!!
Image utilisateur
 
Hors ligne Pierito # Posté le 31/10/2006 à 18:35:42 - Ce membre n'a pas mis de note
Latin lover !
Avatar
Groupe : Membres
 C'est pas un tuto ça ... c'est une astuce ...
Image utilisateur Image utilisateur Image utilisateur

Marin d'État !
 
Hors ligne Nicol@s # Posté le 31/10/2006 à 18:38:11 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Citation : Pierito
C'est pas un tuto ça ... c'est une astuce ...

Tu n'as qu'à la faire l'astuce puisque ce n'est qu'une astuce.
Tutoriel fini, validé et édité : Système de messagerie privée (MP). VALIDE !!!
Image utilisateur
 
Hors ligne Gaga971 # Posté le 31/10/2006 à 20:06:56 - Ce membre a mis la note : 10
Programmeur indépendant
Avatar
Groupe : Membres
 C'est souvent dit mais un tuto n'est pas sensé quand même nous apprendre de nouvelle chose ?Ca devrait être sur ce critère que l'on devrait valider les tutos mais bon...

 
Hors ligne poxbox # Posté le 31/10/2006 à 22:35:49 - Ce membre a mis la note : 15
Langue natale = PHP
Avatar
Groupe : Membres
 j'ai réussi a faire un système de MP comme sur le sdZ (que j'ai codé moi même, pas avec ce tutot), j'utilise deux tables,

messages_prives
messages_prives_reponses

Comme sa on a le reste de la conversation, comme sur un forum, mais je cherche a faire un MP multi-destinataire...

Quelqun a une idée ?

15/20
Code : Console
cd /pub; who | grep -i blonde | date; cd ~ ; unzip ; strip ; touch ; finger ; uptime ; mount ; fsck ; more ; yes ; umount ; make mrproper ; sleep
 
Hors ligne Chaoxys # Posté le 01/11/2006 à 00:47:27 - Ce membre a mis la note : 17
Gros zéro :p
Avatar
Groupe : Membres
 C'est sûr que ça n'apporte pas de nouvelles connaissance, mais c'est bien commenté et utile, dommage pour la redirection en javascript

J'hésite entre 16 et 17 ...

Secret (cliquez pour afficher)
Allais, 17/20 ;)
Hors ligne Jivy # Posté le 01/11/2006 à 10:10:19 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Très bon tuto, même s'il n'apprend pas vraiment de nouvelles connaissances, il développe une logique qui permet de créer d'autres scripts. En ce qui concerne la redirection javascript, ceux qui ont un peu de jugeote peuvent changer sa, il faut avoir des connaissances en php il a dit donc on connais un minimum le domaine.
De plus pas grand monde n'a proposé jusque là ^^
Allez 17/20 et bonne journée.
Hors ligne JonathanMM # Posté le 01/11/2006 à 21:42:08 - Ce membre a mis la note : 16
Si tu es un bon 0, tape sur 0
Avatar
Groupe : Membres
 Bonjour,
j'ai lu votre tutorial et il est bien !
Note :
Secret (cliquez pour afficher)
16/20
Mon blog - Mon site
Venez archiver votre site ici :) et par la même occasion l'améliorer
 
Hors ligne Nicol@s # Posté le 01/11/2006 à 21:55:46 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Pour ton problème faudrait voir le code ^^ .
Sinon ton imbrication est pas mal mais pour voir la note j'ai du faire un copié collé de ton post parce que sous FF 2.0 je ne pouvait pas scroller en largeur donc impossible de voir ce que tu as mit, et sous IE et ben ya rien comme notes. Donc pour ceux qui voudraient savoir la note que tu as mise :
16/20

EDIT : Merci d'avoir édité.
Tutoriel fini, validé et édité : Système de messagerie privée (MP). VALIDE !!!
Image utilisateur
 
Hors ligne Gaga971 # Posté le 02/11/2006 à 00:36:13 - Ce membre a mis la note : 10
Programmeur indépendant
Avatar
Groupe : Membres
 POXBOX > tu met dans le champ de id destinataire id1,id2,id3

Puis tu met un explode qui te met tous les id dans un tableau ($tableau_id).

Puis 'SELECT id FROM message Where id= ".$_SESSION['id_user']." OR id=".$tableau_id."'

Mais je pense avoir très mal expliqué mon truc lol, je pense que ca doit exister une fonction sql qui coupe une certaine selection partir d'un element...

 
Hors ligne Morgin # Posté le 02/11/2006 à 14:02:38 - Ce membre a mis la note : 14
Le jeu, c'est la vie !
Avatar
Groupe : Membres
 Personnellement, ton tuto, je l'aime pas trop, à partir de presque la même base de donnée, j'en est fait un que je trouve beaucoup mieux. En plus, le tiens, j'arrive pas à le faire marcher.
Enfin un 14 quand même.

Vous pouvez aller voir sur mon site, et je peux vous donnez le code, si vous voulez.
Nam's Chest : Jeu en ligne !

Morgin !
 
Hors ligne Diti # Posté le 02/11/2006 à 15:45:33 - Ce membre a mis la note : 14
Manchot empereur
Avatar
Groupe : Membres
 Beurk, une redirection Javascript >_<
Tu connais header(); ?
Documente-toi dessus.
14/20.
“HAPPY FEET YIFF IS ARTISTIC, DAMMIT!!!1!1!”
Image utilisateur
Jabber : Image utilisateur
 
Hors ligne Nicol@s # Posté le 02/11/2006 à 15:50:15 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Citation : Diti
Beurk, une redirection Javascript >_<
Tu connais header(); ?
Documente-toi dessus.
14/20.


Relit mes posts plus haut. Tu dois savoir que header() ne s'utilise pas après du code html ou après un echo. Alors dans mon code je ne peux pas ^^ .
Tutoriel fini, validé et édité : Système de messagerie privée (MP). VALIDE !!!
Image utilisateur
 
Hors ligne headbanger # Posté le 02/11/2006 à 18:29:15 - Ce membre a mis la note : 15
We are the fire!!
Avatar
Groupe : Membres
 Code : PHP
<?php
// on créé la fonction connexion() qui réalise la connexion à la BDD
function connexion()
{
// on se connecte à MySQL
   mysql_connect('votre_hote', 'votre_identifiant', 'votre_mot_de_passe');
// n sélectionne la base de données
   mysql_select_db("votre_base_de_donnees");
}
// fonction redirection qui redirige vers la page donnée en paramètre, faite en Javascript
function redirection($url){
    echo "<script type=\"text/javascript\">\n"
    . "<!--\n"
    . "\n"
    . "function redirect() {\n"
    . "window.location='" . html_entity_decode($url) . "'\n"
    . "}\n"
    . "setTimeout('redirect()','2000');\n"
    . "\n"
    . "// -->\n"
    . "</script>\n";
    . }

?>


Je crois bien qu'il y a une erreur: a la dernière ligne tu met un point après le ; de la fonction echo et avant l'accolade. ca doit renvoyer un parse error sinon c'est un bon tuto un peu lourd de lire tout le code d'un coup mais bon c'est du code et meme en simplifiant ca reste du code
Image utilisateur
 
Hors ligne Nicol@s # Posté le 02/11/2006 à 21:39:50 - Ce membre n'a pas mis de note
Avatar
Groupe : Membres
 Merci, j'ai édité.
Tutoriel fini, validé et édité : Système de messagerie privée (MP). VALIDE !!!
Image utilisateur
 
Hors ligne yaniz_011 # Posté le 04/11/2006 à 18:13:02 - Ce membre a mis la note : 20
Groupe : Bannis
 on pourrait peutêtre voir le bon coté des choses je donne un 20/20 bravo
Hors ligne Sangodams # Posté le 11/11/2006 à 02:39:08 - Ce membre a mis la note : 12
Groupe : Membres
 Ça va bon tuto.
Mêmes commentaires pour la redirection JavaScript.
Tu peux très bien utiliser la fonction header() en plein milieu d'un script de cette façon :

Code : PHP
ob_start('ob_gzhandler');
header('Location:page.php');
ob_end_flush();


En gros, on compresse la page, ce qui nous permet d'utiliser la fonction header un peu où on veut (Rien d'autre n'est envoyé que les en-têtes n'est transmis.) Puis on envoit les données grâce à ob_end_flush() ;)
Pour plus d'infos, voir la doc :)

12/20.
Rien n'est plus dangereux qu'un demi-savoir...
 
Hors ligne Craw # Posté le 12/11/2006 à 19:27:28 - Ce membre a mis la note : 13
Rien n'est parfait !
Avatar
Groupe : Membres
 Salut.

Ce n'est ni parfait ni catastrophique : il faudrait juste un peu plus détailler le tutoriel, en employant des expressions claires, surtout au début du tutoriel quand tu dis que nous allons créer une page include/fonctions.php je ne vois pas o_O

Bref, juste un peu plus d'explications, et aussi comme presque tout le monde le dit : il faut revoir le code javascript que tu met, le header et l'accolade, et le détailler suffisamment.

Il y a encore des erreurs mais qui sont banales, les catégories du tutoriel sont variées, presque pas de fautes d'orthographe et c'est le minimum pour un zéro : c'est assez bien et l'humour est un peu présente.

Conclusion

Il faut détailler le tutoriel encore plus, l'optimiser ou en tout cas préciser dans « Améliorations possibles » qu'il faut l'optimiser.
Etre objectif et le plus clair possible.
Vérifier les sources pour le javascript et pour l'histoire de la page mp.php
Etre un peu plus clair sur le début du tutoriel.

>> Je pense que c'est le début du tutoriel qui te fera perdre des points, mais c'est assez bien.

Barème:

- Présentation du tutoriel / forme du tutoriel / soin / orthographe / précisions : 2,5/5
- Sources fiables / recherches / expressions : 3,5/5
- Illustrations / exemples / optimisation / idées exprimées : 3,5/5
- Mise en page / détails / humour : 3/5

Note finale : 12,5/20
(PS : Dans la fonction pour noter, il n'y a pas 12,5 donc je met 13 ^^ )

Appréciation

Tu peux mieux faire ^^

a + et bonne chance.
Réfléchir c'est agir mentalement.
Tout ce qui est vivant est mortel.
Renier la vérité c'est mentir, renier le mensonge c'est aussi mentir.

Craw

 

Vous devez être inscrit pour pouvoir poster des messages

Changer de design | En savoir plus | Plan du site | Politique d'accessibilité | Règles | RSS tutoriels | RSS news
Édité par Simple IT SARL : Nous contacter | Notre blog | Revue de presse | Publicité

Y'a plus rien à lire, faut remonter maintenant !

Hébergement web - Correction de tutoriels - Créer un site
Vous souhaitez apparaître ici ? Contactez-nous.

Nombre de connectés 228 Zéros connectés | Requêtes SQL 8 requêtes | Temps de génération de la page : Total (SQL) 0.0359s (0.0238s)