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 

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).

Twoj post daje do myślenia. Pomysl z abstrakacja wydaje sie ze rozwiazal moj problem.

A mam jeszcze takie pytanie odnośnie otwieranie połaczenia i zakonczenie połaczenia z bazą.
Założmy ze bede korzystac z jednej bazy na jednym hoscie. Lepiej tworzyc i zamykać polaczenie w kazdej metodzie ktora zapisuje/odczytuje dane . Czy poprostu na poczatku strony np. index.php raz otworzyć polaczenie i raz zakonczyc gdzies na koncu.

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)

Ależ ja o tym napisałem ano. Zauważ, że obiekt Connection w sumie nie mówi nic o celu zapisu. To jakiś abstrakcyjny byt. Tak naprawdę wystarczy by klasa Article w tym wypadku miała metody get i set. Get z parametrem tablicowym szukało by pasującego do podanych kryteriów lub zwracało ewentualnie pusty obiekt klasy Article. Zależy czy chciałbyś połączyć get z ewentualną osobną metodą find czy nie. Set z tablicą zapisywało by takowy obiekt lub tworzyło nowy.Tu nie muszę wiedzieć czy celem operacji jest baza, xml czy cokolwiek. To bym w konfiguracji sobie zdefiniował jako driver, który dostarczyłby mi implementacji interfejsu. Oczywiście to tylko jedna z metod na szybko. Lepiej byłoby to jeszcze bardziej uniwersalnie zrobić.