[klasa] mailing Opcje

[piotrooo89]

dziękuje za wszelkie porady, uwagi. zabieram się do poprawek.

proszę również o dalsze komentarze.

Odłożyłem na chwile stronicowanie, potrzeba chwili wymogła na mnie zrobienie newslettera. Nie chce zakładać kolejnego tematu więc pokaże go tu. Jest to najprostsze wysyłanie maila, tymczasowo bez nagłówków. Powiedzcie mi czy idę w dobrą stronę.

newsletter-class.php

[PHP] pobierz, plaintext 
<?php
class Newsletter
{
    public function GetMailsFromFile($file, $separator)
    {
        $string = file_get_contents($file);
        $mails_file = explode($separator, $string);
        return $mails_file;
    }
    public function GetMailsFromDb($sql)
    {
        $sql_query = mysql_query($sql);
        while ($rows = mysql_fetch_row($sql_query))
        {
            $mails_db[] = $rows[0];
        }
        return $mails_db;
    }
    public function Send($where, $subject, $message)
    {
        foreach($where as $mails)
        {
            mail($mails, $subject, $message);
        }
    }
}
?>
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
<?php
require_once ('newsletter-class.php');
require_once ('mysql.php');
 
$topic = 'Test';
$msg = 'Wiadomość testowa';
 
$sql = 'SELECT mail FROM emails';
 
$news = new Newsletter();
$file = $news -> GetMailsFromFile('mail.txt', ';');
$db = $news -> GetMailsFromDb($sql);
// z pliku
$news -> Send($file, $topic, $msg);
//lub lub bazy
$news -> Send($db, $topic, $msg);
?>
[PHP] pobierz, plaintext 

[skowron-line]

Według mnie mógłbyś dać więcej metod typu

Kod

setMessage( $msg )
{
$this->msg = stripslashes( $msg );
}
addAddress( $address )
{
//walidacja adresu
}

Albo jeszcze sprawdzać czy wiadomość nie jest pusta.

[PHP] pobierz, plaintext 
<?php
public function addHeaders($value)
   {
       $this->headers[] = $value;
   }
public function Send($where, $subject, $message)
   {
       foreach($where as $mails)
       {
           mail($mails, $subject, $message, $his->headers);
       }
   }
?>
[PHP] pobierz, plaintext 

Po co wstawiasz do metody kolejny parametr jak możesz je wyciągać wewnątrz klasy.
Tak samo jak message i where i subject i miałbyś

[PHP] pobierz, plaintext 
<?php
$news->send();
?>
[PHP] pobierz, plaintext