[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]

~phpion dzięki za przykład, uczę się dopiero i nie bardzo mi to wychodzi. zrobiłem coś takiego:

[PHP] pobierz, plaintext 
<?php
abstract class Loader 
{
    abstract public function load();
}
class Loader_File extends Loader 
{
    public function load() 
    {
        $string = file_get_contents($file);
        $mails_file = explode($separator, $string);
        print_r($mails_file);
    }
}
class Loader_Database extends Loader 
{
    public function load() 
    {
        $sql_query = mysql_query($sql);
        while ($rows = mysql_fetch_row($sql_query))
        {
            $mails_db[] = $rows[0];
        }
        print_r($mails_db);
    }
}
class Newsletter
{
    private $loader;
 
    public function setLoader(Loader $loader) 
    {
        $this->loader = $loader;
    }
 
    public function getMails() 
    {
        return $this->loader->load();
    }
}
?>
[PHP] pobierz, plaintext 

ale jest źle. dlaczego? ponieważ nie wiem gdzie mam podać nazwę pliku/query do bazy.

~Cypherq
wiem o nazwach narazie testuje wszytko, później będę wyjątki łapał. narazie chce to uruchomić.

#EDIT

jak podam na sztywno nazwy pliku/query działa.

#EDIT2
zrobiłem tak, powiedzcie co o tym sądzicie:

newsletter-class.php

[PHP] pobierz, plaintext 
<?php
abstract class Loader 
{
    abstract public function load();
}
class Loader_File extends Loader 
{
    public function __construct($file, $separator)
    {
        $this->file = $file;
        $this->sep = $separator;
    }
    public function load() 
    {
        $string = file_get_contents($this->file);
        $mails_file = explode($this->sep, $string);
        return $mails_file;
    }
}
class Loader_Database extends Loader 
{
    public function __construct($sql)
    {
        $this->sql = $sql;
    }
    public function load() 
    {
        $sql_query = mysql_query($this->sql);
        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)
   {
       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';
$f = 'mail.txt';
 
$news = new Newsletter();
$file = new Loader_File($f, ';');
$db = new Loader_Database($sql);
 
$news -> setLoader($file);
 
print_r($news -> getMails());
 
$news -> Send($news->getMails(), $topic, $msg);
?>
[PHP] pobierz, plaintext 

Ten post edytował piotrooo89 10.04.2009, 20:03:48