[Klasa] Emailer Opcje

[PawelC]

Hej
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ę.

[NoiseMc]

Po sugestie zapraszam tu:
http://framework.zend.com/manual/en/zend.mail.html

[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

[AxZx]

dziwny ten kod

najpierw metoda nadawca a później setSubject
jakie różnice? pl i en oraz konwencja z set

[Crozin]

Hmm... a o

[PHP] pobierz, plaintext 
<?php
mail('adres@mail.1;adres@mail.2;adres@mail.3;adres@mail.4;', 'Temat', 'Treść');
?>
[PHP] pobierz, plaintext 

Słyszał?

[Moli]

Przecież to tutorial na najprostrzy newsletter wciśniety w klase Strasznie tragiczne

[nospor]

na forum oceny oddaje sie do oceny ciut powazniejsze skrypty. to co tu pokazales to funkcja mail wsadzona w klase, ktora na dobrą sprawe nic nie robi

ps: tak, widzialem tego skomplikowanego foreach