PDO prosty czat - proszę o ocenę Opcje

[sajegib]

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!

[hind]

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 

Ten post edytował hind 24.04.2013, 06:57:50