[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 

[piotrooo89]

1. jak będę posiadał 60k maili w bazie czy pliku (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ... To myślisz że wszystko gładko wyśle za jednym zamachem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ?

tak tak już o tym pomyślałem. dodam odpowiednie funkcje.

2. Może warto byłoby zaopatrzyć maila w nagłówki i autoryzację smtp. przy obecnym stanie większość serwerów oznaczy to flagą SPAM (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

nagłóweczki już są choć narazie chce zrobić ogólny zarys klasy i zapytać innych o zdanie.

@piotrooo89 a dlaczego wymyślasz koło od nowa jest klasa phpMailer ma wszystko czego tobie potrzeba.

tak jak napisał ~phpion:

Może dlatego, że się chłopak uczy?

i chyba bardziej cieszy coś własnego (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

a wracając do tematu jak oceniacie? co zmienić, dodać?

aktualnie wygląda to tak:

newsletter.Class.php

[PHP] pobierz, plaintext 
<?php
abstract class Loader 
{
    abstract public function load();
    public $errors = 
        array(
        'no_file' => 'Plik nie istnieje',
        'wrong_separator' => 'Nie podano separatora',
        'wrong_sql' => 'Brak zapytania SQL',
        'wrong_sql_col' => 'Podaj kolumne z adresami e-mail',
        'wrong_sql_tab' => 'Podaj tabele SQL'
        );
}
class Loader_File extends Loader 
{
    public function __construct($file, $separator)
    {
        if (empty($separator))
            throw new Exception($this->errors['wrong_separator']);
 
        $this->file = $file;
        $this->sep = $separator;
    }
    public function load()    
    {
        if (!file_exists($this->file))
            throw new Exception($this->errors['no_file']);
            
            $string = file_get_contents($this->file);
            $mails_file = explode($this->sep, $string);
            return $mails_file;
    }
}
class Loader_Database extends Loader 
{
    public function __construct($col, $tab)
    {
        if (empty($col) && empty($tab))
            throw new Exception($this->errors['wrong_sql']);
        elseif (empty($col))
            throw new Exception($this->errors['wrong_sql_col']);
        elseif (empty($tab))
            throw new Exception($this->errors['wrong_sql_tab']);
            
        $this->col = $col;
        $this->tab = $tab;
    }
    public function load() 
    {
        $sql_query = mysql_query("SELECT `".$this->col."` FROM `".$this->tab."`");
        while ($rows = mysql_fetch_row($sql_query))
        {
            $mails_db[] = $rows[0];
        }
        return $mails_db;
    }
}
class Newsletter
{
    private $loader;
 
    public function setLoader(Loader $loader) 
    {
        $this->loader = $loader;
    }
    public function getMails() 
    {
        return $this->loader->load();
    }
    public function Send($where, $subject, $message, $head)
    {
        foreach($where as $mails)
        {
            mail($mails, $subject, $message, $head);
        }
    }
}
?>
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
<?php
require_once ('newsletter.Class.php');
require_once ('mysql.php');
 
$topic = 'Test';
$msg = 'Wiadomość testowa';
$headers .= "From: Olaszewski <piotroo89@gmail.com>r\n"; 
$headers .= "Reply-To: Olaszewski <piotroo89@gmail.com>r\n";
$headers .= "Return-Path: Olaszewski <piotroo89@gmail.com>r\n";
$headers .= 'X-Mailer: PHP/' . phpversion() ."r\n";
$headers .= 'MIME-Version: 1.0'. "\n"; 
$headers .= 'Content-type: text/html; charset=utf-8' . "r\n"; 
$headers .= "Content-transfer-encoding: utf-8r\n"; 
 
$column_sql = 'mail';
$table_sql = 'emails';
 
$f = 'mail.txt';
 
try
{
    $news = new Newsletter();
    $file = new Loader_File($f, ';');
    $db = new Loader_Database($column_sql, $table_sql);
    $news -> setLoader($db);
 
    print_r($news->getMails());
 
    $news -> Send($news->getMails(), $topic, $msg, $headers);
}
catch(Exception $e)
{
    $error = ($e->getMessage());
    echo $error;
}
?>
[PHP] pobierz, plaintext