Thread: Mail Class
View Single Post
Old 02-03-2008, 01:28 AM   #7 (permalink)
Salathe
Moderateur
RegEx Guru PHP Guru Top Contributor Advanced Programmer 
 
Salathe's Avatar
 
Join Date: Apr 2007
Posts: 1,393
Thanks: 5
Salathe is on a distinguished road
Default

A nice start, and good to see you keeping the class simple and to the point. The actual number of lines could be whittled down with some rewriting (look where you're repeating yourself).

I'm not a fan of the whole 'reset' idea. Best, in my opinion, keeping two different email messages separate from each other. I'd perhaps also throw in a static method rather than having to create at least one instance of the class with a global variable: email::instance()->to('blah...
Salathe is offline  
Reply With Quote