[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 

[phpion]

Ogólnie idziesz w dobrą stronę ale poczytaj o tym, co to jest polimorfizm i jak można go wykorzystać. Konkretnie chodzi mi o metody:
- GetMailsFromFile
- GetMailsFromDb
Generalnie nie powinno się tak robić (tworzyć osobnych metod do pobierania danych z różnych źródeł). Zdecydowanie lepiej utworzyć obiekt typu Loader, który by pobierał dane w speclajny sposób, coś na wzór:

[PHP] pobierz, plaintext 
<?php
abstract class Loader {
    abstract public function load();
}
 
class Loader_File extends Loader {
    public function load() {
        // wczytaj adresy z pliku
    }
}
 
class Loader_Database extends Loader {
    public function load() {
        // wczytaj adresy z bazy danych
    }
}
?>
[PHP] pobierz, plaintext 

Klasa Newsletter mogłaby wyglądać mniej-więcej tak:

[PHP] pobierz, plaintext 
<?php
class Newsletter {
    private $loader;
    
    public function setLoader(Loader $loader) {
        $this->loader = $loader;
    }
    
    public function getMails() {
        return $this->loader->load();
    }
}
?>
[PHP] pobierz, plaintext 

oraz wykorzystanie:

[PHP] pobierz, plaintext 
<?php
$obj = new Newsletter();
$obj->setLoader(new Loader_File());
print_r($obj->getMails());
?>
[PHP] pobierz, plaintext