Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [Klasa] Emailer
Forum PHP.pl > Inne > Oceny
PawelC
Hej 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.
  1. <?
  2.    
  3.    class emailer
  4.    {
  5.          
  6.           public $nadawca;
  7.           public $temat;
  8.           public $tresc;
  9.        
  10.          
  11.          
  12.            //Ustawienia nadawcy
  13.            public function nadawca($nadawca)
  14.            {
  15.                $this->nadawca=$nadawca;
  16.            }
  17.            
  18.            
  19.            
  20.            //Ustawienie tematu
  21.            public function setSubject($temat)
  22.            {
  23.                $this->temat=$temat;
  24.            }
  25.            
  26.            
  27.            
  28.            //Ustawienie treści wiadomości
  29.            public function setMessage($tresc)
  30.            {
  31.                $this->tresc=$tresc;
  32.            }
  33.            
  34.            
  35.            
  36.            
  37.            
  38.            //Wysyłanie maila, do odbiorców z pliku tekstowego
  39.            public function sendMail()
  40.            {    
  41.                $file=file('emaile.txt');
  42.                foreach($file as $mailer)
  43.                {
  44.                mail($mailer,$this->temat,$this->tresc,'From: ExPlOiT <'.$this->nadawca.'>');
  45.                }
  46.            }
  47.            
  48.            
  49.    }
  50.    
  51. $mail=new emailer;
  52. $mail->setSubject('Wiadomosc testowa');
  53. $mail->setMessage('Tresc testowa');
  54. $mail->nadawca('Exploit@wp.pl');
  55. $mail->sendMail();
  56.  
  57. ?>

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?
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
  1. <?php
  2. mail('adres@mail.1;adres@mail.2;adres@mail.3;adres@mail.4;', 'Temat', 'Treść');
  3. ?>
Słyszał?
Moli
Przecież to tutorial na najprostrzy newsletter wciśniety w klase smile.gif Strasznie tragiczne smile.gif
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 winksmiley.jpg
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.