PHP przykładow Klasa ocena

Question

Napisałem taką klasę zarządzającą wpisami na blog, to jest oczywiście wstępny przykład. I chciałem się dowiedzieć czy w dobrą stronę zmierzam pisząc klasę w ten sposób.

class BlogPost
{
    public $id;
    public $title;
    public $contents;
    public $author;
    public $created_at;

    function __construct($id = null, $title = null, $contents = null, $author = null, $created_at = null)
    {
        if(!empty($id))
        {
            $this->id = $id;
        }
        if(!empty($title))
        {
            $this->title = $title;
        }
        if(!empty($contents))
        {
            $this->contents = $contents;
        }
        if(!empty($author))
        {
            $this->author = $author;
        }
        if(!empty($created_at))
        {
            $this->created_at = $created_at;
        }
    }

    public function setTitle($title)
    {
        if(!empty($title))
        {
            $this->title = $title;
        }
    }

    public function setContents($contents)
    {
        if(!empty($contents))
        {
            $this->contents = $contents;
        }
    }

    public function setAuthor($author)
    {
        if(!empty($author))
        {
            $this->author = $author;
        }
    }

    public function setCreatedAt($created_at)
    {
        if(!empty($created_at))
        {
            $this->created_at = $created_at;
        }
    }

    public function save()
    {
        echo "<p>Id: " . $this->id . "</p>"; 
        echo "<p>Title: " . $this->title . "</p>";
        echo "<p>Contents: " . $this->contents . "</p>";
        echo "<p>Author: " . $this->author . "</p>";
        echo "<p>Created at: " . $this->created_at . "</p>";

        echo "<hr> Ok, Zapisano post!";
    }

    public function delete($id)
    {
        echo "Usunięto wpis!";
    }
}

 

Answer 1

1. ogolnie wywalil bym funkcje save() i delete(), zgodnie ze sztuka klasa powinna miec jedna odpowiedzialnosc czyli przechowywanie informacji o poscie

2.  wydzielil bym kod walidujacy do osobnej metody prywatnej

<?php

class BlogPost {
    public setAttribute($v) {
        if ($this->isValid($v)) {
            $this->v = $v;
        }
    }
    
   private function isValid($v) {
       return !empty($v);
   }
}

@OFFTOP
do wypisywania powinna byc oddzielna kasa np.

<?php
$bp = new BlogPost(args ...);

BlogPostPrinter::toHtml($bp);
BlogPostPrinter::toXml($bp);
BlogPostPrinter::toPlainText($bp);

?>

unikasz dzieki temu rozrostu klasy o kod ktory do niej nie powinien nalezec

Comments

Przypuszczam że zabiegi na tym poście będzie się opierał o bazę danych MySQL, tak wiec metodę zapisu i usunięcia itp. trzeba wyrzucić do nowej klasy która będzie obsługiwała zapytanai sql