Witam!

Proszę szanowne grono o ocenę mojego czatu napisanego w OOP, dopiero się uczę, z chęcią przyjmę wszystkie uwagi (szczególnie krytyczne) dot. kodu a także co mogłoby być lepiej


config.php

[PHP] pobierz, plaintext 
 
<?php
 
//DANE DO POŁĄCZENIA Z BAZĄ
 
class Config{
 
    public static $db_host = 'localhost';
    public static $db_name = 'Czat';
    public static $db_user = 'root';
    public static $db_pass = 'root';
 
}
 
?>
 
 
 
[PHP] pobierz, plaintext 

database.php

[PHP] pobierz, plaintext 
 
<?php
 
require_once 'config.php';
 
abstract class database
{
 
    protected $pdo;
    public $error;
 
    public function __construct()
    {
      // POLACZENIE Z BAZA  
        try 
        {
             $this->pdo = new PDO('mysql:host='.Config::$db_host.';dbname='.Config::$db_name, Config::$db_user, Config::$db_pass);
             $this->pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }
       catch (PDOException $e) 
       {
            print "Error!: " . $e->getMessage() . "<br/>";
            die();
        }
 
    }
 
 
 
 
    protected function insert($author, $msg)
    {
 
            $stmt  = $this->pdo->prepare('INSERT INTO Czat (Author, Msg) VALUES (:auth, :msg)');
            $stmt->bindParam(':auth', $author);
            $stmt->bindParam(':msg', $msg);
 
           if($stmt->execute())
           {
               return TRUE;
           }
           else
           {
               return FALSE;
           }
     }
 
 
 
 
    protected function select()
    {
        $messages =  $this->pdo->query('SELECT * FROM Czat');
        return $messages;
    }
 
 
 
    public function __destruct() 
    {
        $this->pdo = NULL;
    }
 
 
}
 
 
?>
 
 
[PHP] pobierz, plaintext 

czat.php

[PHP] pobierz, plaintext 
 
<?php
 
require_once 'database.php';
 
class czat extends database
{
 
    public $error ='';
 
    public function __construct() {
        parent::__construct();
    }
 
 
    public function add_message($author, $msg)
    {
 
        if(!empty($author) && !empty($msg))
         {
           $escaped = $this->escape($author, $msg); // escapujemy
 
 
            if($this->insert($escaped[0], $escaped[1])){
                    $this->error = "DODANO";
            }
            else{
                    $this->error = "NIE DODANO";
            }
 
        }
 
 
    }
 
 
    public function get_messages()
   {
        return $this->select();
    }
 
 
    public function escape($val1, $val2)
    {
        $auth = addslashes($val1);
        $msg = addslashes($val2);
 
        return array($auth, $msg); // ZWRACAMY WYESCAPOWANA TABLICE
    }
 
 
}
 
 
?>
 
 
 
[PHP] pobierz, plaintext 

pozdrawiam!

jak ma być oop to rozdziel walidacje, na osobną klasę

Chat moim zdaniem nie powinien dziedziczyć z bazy a być oddzielną częścią i posiadać obiekt klasy jako zmienną.

w ktore miejsce najlepiej wrzucic obiekt bazy? jak sie wtedy odwolac do jego metod?

Bazę (PDO) przekazać jako parametr do konstruktora, a to co masz w klasie database przekopiuj do klasy chat
i w tedy

[PHP] pobierz, plaintext 
class chat {
protected $pdo;
 
public function __construct($pdo)
{
if ($pdo) {
throw new exception('nie pdo');
}
$this->pdo = $pdo;
}
[...]
}
[PHP] pobierz, plaintext 

@hind dobrze mówi, ale by jeszcze pójść o krok dalej powiem Ci jak działją fw.

W fw masz coś takiego jak klasa DI to taki rejestr wszystkich elementów składowych systemu, najłatwiejsza jego odmiana ma aż 2 metody set($name, $obiect) i get($name) i służy do przechowywania w sobie wszystkich obiektów typu baza danych, instancaj widoku, routingu, requestu, configu itd. i to ona jest przekazywana do kontrolerów (dla uproszczenia można uznać że u Ciebei to klasa chat) podczas konstrukcji klasy, a sama klasa chat ma zmienną $di i metody setDi() i getDi() dzięki który możesz odwołać się do obiektów zapisanych w niej.

Taki zabieg w przyszłośic ułatwi Ci kod, bo obiektów typu chat, form, controller jest wiele sposobem @hind musiał byś za każdym razem do nich wprowadzać wszystkie serwisty (db, request, routing itd), a tutaj wszystko wprowadzasz do Di, a samo Di przekazujesz zawsze dalej i ew w przyszłości dodajesz tylko do niego kolejne serwisy. ;-)

Coś tam działam sobie w wolnych chwilach w codeigniterze i przyznam, że sporo załapałem z obiektówki dzięki temu, ale nie bardzo doszukałem się informacji odnośnie architektury całego przedsięwzięcia.
Dzięki wielkie szanownym Panom za informację, będę walczył dalej.

A jako, ze już przeszliśmy na FW, to mam jeszcze małe pytanko, mam kilka podstron, których treść trzymam w bazie (strona oparta o codeigniter).

Stworzyłem sobie kontroler, który zajmuje się wyświetlaniem kolejnych podstron, przyjmuje 1 parametr z ID podstrony - wtedy url wyglada tak: xxx.pl/home/page/1 (2,3, itd)
Co myślicie o takim rozwiązaniu? Może jest jakieś lepsze?

Używasz PDO, używasz bindowania, a mimo to escapujesz dane i to na dodatek przy pomocy addslashes.... przecież to nie ma żadnego sensu...

Doczytałem trochę, faktycznie, wstyd

Ale chyba trzeba się troche powstydzić za swoje dokonania, żeby się nauczyć