[Klasa] Emailer Opcje

[PawelC]

Hej (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Napisałem klasę, która służy do wysyłania wiadomości email do kilku odbiorców na raz.
Zapraszam do komentowania jej i oczywiście sugestii.

[PHP] pobierz, plaintext 
<?
    
    class emailer
    {
           
           public $nadawca;
           public $temat;
           public $tresc;
        
           
           
            //Ustawienia nadawcy
            public function nadawca($nadawca)
            {
                $this->nadawca=$nadawca;
            }
            
            
            
            //Ustawienie tematu
            public function setSubject($temat)
            {
                $this->temat=$temat;
            }
            
            
            
            //Ustawienie treści wiadomości
            public function setMessage($tresc)
            {
                $this->tresc=$tresc;
            }
            
            
            
            
            
            //Wysyłanie maila, do odbiorców z pliku tekstowego
            public function sendMail()
            {    
                $file=file('emaile.txt');
                foreach($file as $mailer)
                {
                mail($mailer,$this->temat,$this->tresc,'From: ExPlOiT <'.$this->nadawca.'>');
                }
            }
            
            
    }
    
$mail=new emailer;
$mail->setSubject('Wiadomosc testowa');
$mail->setMessage('Tresc testowa');
$mail->nadawca('Exploit@wp.pl');
$mail->sendMail();
 
?>
[PHP] pobierz, plaintext 

Zdaję sobie sprawę, że pobieranie adresów email, można zrobić w osobnej metodzie, ale wybrałem taką a nie inną opcję.

[phpion]

Nie ma co oceniać. Kod na siłę napisany obiektowo, zupełnie bez większego sensu. Do tego mieszasz nazewnictwo polskie z angielskim. I na koniec pytanie zasadnicze: po co tworzysz setery skoro nie robią one nic konkretnego, a do tego operują na składowych publicznych? Kompletnie bez sensu.

PS: a co jeśli miałbym X plików z adresami e-mail i chciałbym dynamicznie wybierać konkretny plik?

Ten post edytował phpion 11.06.2009, 09:28:35