Tworzenie obiektu wewnątrz innej klasy. Opcje

[yayco]

Witam,
zaczynam pisać pewną aplikację (prosty CMS do pisania artykułów), dodam iż celem jest tylko zdobycie wiedzy i doświadczenia w OOP.

Program: posiadam dwie klasy, jedna łaczy się z bazą danych, druga jest odpowiedzialna za dodawanie artykułow do bazy. Czy tworzenie obiektu oConnectionClass wewnątrz klasy ArticleClass to jest dobrym rozwiązaniem? Wiem, ze napewno można to zrobić inaczej ale to rozwiązanie tez działa.


Mam klasę do połączenia z bazą MySql:

[PHP] pobierz, plaintext 
class ConnectionClass {
    public $connection;
    protected $dns = 'mysql:host=localhost;dbname=projekt';
    protected $username = 'root';
    protected $password = '';
 
    public function openConnection()
    {
        try {
            $this->connection = new PDO($this->dns, $this->username, $this->password);
        }
        catch (PDOException $e) {
            echo $e->getMessage(); 
            file_put_contents('log/PDOErrors.txt', $e->getMessage() . "\n", FILE_APPEND | LOCK_EX);  
        }
    }
 
    public function closeConnection(){
        if($this->connection != null){
            $this->connection = null;
        }
    }
}
[PHP] pobierz, plaintext 

A tutaj mam klasę ArticleClass:

[PHP] pobierz, plaintext 
class ArticleClass {
    public $topic;
    public $content;
    public $author;
    public $posted;
    public $category;
    public $tags = array();
 
    public function __construct($topic,$content,$author,$category,$tags) {
        $this->topic = $topic;
        $this->content = $content;
        $this->author = $author;
        $this->posted = date ("Y-m-d H:i:s");   //format daty zgodny z MySQL DATETIME.
        $this->category = $category;
        $this->tags = $tags;
    }
 
    public function SaveArticle() {
        require 'class/ConnectionClass.php'; 
        $oConnectionClass = new ConnectionClass();
        $oConnectionClass->openConnection();
 
        $sql = 'INSERT INTO articles (article_topic,article_content,article_author,article_posted,article_categor
y,article_tags)
                VALUES (:article_topic,:article_content,:article_author,:article_posted,:article_ca
tegory,:article_tags)';
        $stmt = $oConnectionClass->connection->prepare($sql);
        $stmt->execute(array(
           ':article_topic' => $this->topic,
           ':article_content' => $this->content,
           ':article_author' => $this->author,
           ':article_posted' => $this->posted,
           ':article_category' => $this->category,
           ':article_tags' => $this->tags)
                );
        echo "Zapisano";
 
        $oConnectionClass->closeConnection();
   }
}
[PHP] pobierz, plaintext 

[thek]

Jak dla mnie to Connection powinno być abstrakcją. Nie skazuj się z góry na fakt, iż będziesz miał do czynienia z obiektami, które PDO obsłuży, bo się możesz przejechać. Przykład? Artykuły wręcz proszą się o połączenie ich z bazami noSQL lub silnikami pokroju Sphinx. Weź też pod uwagę, że możesz mieć inny cel zapisu niż baza. Sama klasa Article chcąc, nie chcąc, będzie korzystać z jakiegoś połączenia. Jeśli nie do zapisu, to do odczytu. Tylko, że IMHO powinno to połączenie być przekazywane jako obiekt dla odpowiednich metod lub w ostateczności jako parametr dla konstruktora (Dependancy Injection się kłania).