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 

[ano]

Oczywiście, że tylko raz utworzyć połączenie (inaczej zabijesz sobie serwer, wydajność itp)
Dlatego saveArticle do przepisania ;-)
Poza tym czemu obiekt Article ma wiedzieć coś o Connection? Zrób dodatkową warstwę, tak aby on nic nie musiał wiedzieć o bazie. Dopiero ta pośrednia warstwa będzie Ci służyć do zapisu/odczytow artykułów z bazy.
Dzięki temu ten sam obiekt Article będziesz mógł np naturalnie raz pchać do mysql, a raz do zupełnie czegoś innego. (Jak wspominał @thek)