CLASS E-MAIL

Signaler
Messages postés
488
Date d'inscription
samedi 5 avril 2003
Statut
Membre
Dernière intervention
31 mars 2009
-
cs_depression
Messages postés
100
Date d'inscription
mardi 7 novembre 2000
Statut
Membre
Dernière intervention
13 juillet 2009
-
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/46503-class-e-mail

cs_depression
Messages postés
100
Date d'inscription
mardi 7 novembre 2000
Statut
Membre
Dernière intervention
13 juillet 2009

waterw72 > Euh, tu es à mon avis un peu plus loin que simple débutant.

Ensuite, je pense que tu devrais quand même garder à l'esprit l'objectif de faire quelque chose de vraiment nouveau, qui innove. C'est avec un côté visionnaire que l'on évolue dans le bon sens.

Allez, bonne prog!
spipod
Messages postés
23
Date d'inscription
mercredi 2 février 2005
Statut
Membre
Dernière intervention
17 juillet 2009

Salut,

Je trouve que c'est dommage de faire tout ceci, et, utiliser finalement imap_mail. Tu devrais faire une connexion IP vers un serveur SMTP pour envoyer ton mail de façon à devenir indépendant et gagner en intérêt pour le source.

As-tu essayé File_Get_Content (en mode binaire) à la place de FRead ?
(gain en temps de compil et exécution)

Autre petite remarque : BodyToArray ne me semble pas trop optimisé, il y a du redondant (perte de temps de compil).

Dommage pour la stratégie de nommage aléatoire.

Mais le pire de tout, c'est le manque de gestion d'erreur.

A améliorer, mais c'est un bon début.
cs_yoman64
Messages postés
592
Date d'inscription
samedi 19 janvier 2002
Statut
Membre
Dernière intervention
4 décembre 2008

Salut,

Ça m'a l'air bien sympatique tout ça. J'aurais juste quelques toutes petites remarques:
1. Comme dit depression, tu manque de constance. des fois tu utilises des doubles quotes, d'autre fois des simples. Des fois les variables sont en français, d'autre fois en anglais. Parfois tu mets des attributs de portée à tes fonction (private,public, statis) et parfois non.

2. for( $i=0; $i< sizeof($this->sendTo); $i++ ) ou for ( $i=0;$i<count($partie->parts); $i++) => En générale, on évite le plus possible de mettre un appel de fonction qui retourne toujours la même valeure dans une boucle, ça fait X appel inutile. Préfère faire: for( $i=0, $iLEN=sizeof($this->sendTo); $i< $iLEN; $i++ )

3. # if( ! file_exists( $filename) ) {
# echo "Class Email, method attach : file $filename can't be found"; exit;
# }

Évite les echos, préfères les exceptions comme ça on peut mieu controller l'affichage.

4. Tu pourrais pas ajouter un petit exemple ? :)

Je te laisse 7/10

@+ et bonne continuation
waterw72
Messages postés
63
Date d'inscription
jeudi 30 septembre 2004
Statut
Membre
Dernière intervention
18 juillet 2008

=> DEPRESSION.
pour la seconde critique : Etant un débutant, je publie mes sources dans le but d'améliorer ma syntaxe et mon code en général. Mon objectif n'est donc pas de fournir aux autres des outils qui, souvent, existent déjà.