Piszę sobie klasę rejestracji użytkownika. Natrafiłem na problem gdy chciałem przekazać obiekt klasy bazy danych do klasy rejestracji:

Kod:

DataBaseClass.php

[PHP] pobierz, plaintext 
<?php
class DataBase {
    private $host;
    private $user;
    private $password;
    private $database;
    public $connection;
 
    public function __construct($host, $user, $password)
    {
        $this->host = $host;
        $this->user = $user;
        $this->password = $password;
    }
 
    public function select_db($database)
    {
        $this->database = $database;
    }
 
    public function connect()
    {
        $this->connection = mysql_connect($this->host, $this->user, $this->password, $this->database);
    }
 
    public function query($query)
    {
        $this->query = mysql_query($query);
    }
 
    public function __destruct()
    {
        mysql_close($this->connection);
    }
}
?>
[PHP] pobierz, plaintext 

RegistrationClass.php

[PHP] pobierz, plaintext 
<?php
class Registration {
    public $connection;
    public $errors;
    public $User;
 
    public function __construct($User)
    {
        $this->User = $User;
    }
    public function set_connection($database)
    {
        $this->connection = $database;
    }
 
    public function check_user_register()
    {
        $query = $this->connection->query("SELECT name FROM users WHERE name = ".$this->User->login);
        $name = mysql_fetch_array($query);
        if($name[0] == $this->User->login){
            $this->errors[] = "Taki użytkownik już jest zarejestrowany";
            return false;
        }
    }
 
    public function register_user()
    {
        if($this->connection->$query = "INSERT INTO users (name, password) VALUES(".$this->User->login.", ".$this->User->password.")"){
            return true;
        }else{
            $this->errors[] = "Rejestracja nie powiodła się";
            return false;
        }
    }
}
?>
[PHP] pobierz, plaintext 

UserClass.php

[PHP] pobierz, plaintext 
<?php
class User {
    public $login;
    public $password;
    public $password2;
 
    public function __construct($login, $password, $password2)
    {
        $this->login = $login;
        $this->password = $password;
        $this->password2 = $password2;
    }
}
?>
[PHP] pobierz, plaintext 

Używanie:

[PHP] pobierz, plaintext 
<form action="Form.php" method="post">
    Login:<input type="text" name="login" /><br>
    Hasło:<input type="password" name="pass1" /><br>
    Potwierdź hasło: <input type="password" name="pass2" /><br>
    <input type="submit" name="submit" value="Wyślij">
</form>
 
<?php
 
if($_POST['submit'] == 'Wyślij'){
    include("./DataBaseClass.php");
    include("./ValidationClass.php");
    include("./UserClass.php");
    include("./RegistrationClass.php");
 
    if($_POST['login'] && $_POST['pass1']==$_POST['pass2']){
        $Database = new DataBase('localhost', 'root', 'pass');
        $Database->select_db('users');
        $Database->connect();
 
        $User = new User($_POST['login'], $_POST['pass'], $_POST['pass2']);
        
        $Registration = new Registration($User);
        $Registration->set_connection($Database);
        if($Registration->check_user_register()){
            $Registration->register_user();
        }
    }else{
         echo 'error';
    }
        
}
?>
[PHP] pobierz, plaintext 

Jestem nowy w OOP i pewnie to jest prosty problem, ale nie wiem jak sobie z nim poradzić. Przy okazji chciałbym się zapytać czy dobrze myślę i czy idę w dobrą stronę OOP.

Jako ze chodzi o bazę danych to najlepiej do tego celu wykorzystać "singleton". Informacji na jego temat znajdziesz mnóstwo.

Poradziłem sobie. Mam singletona. Działa. Dzięki.

Ale ponawiam drugie pytanie. Czy idę w dobrą stronę? Zamierzam dopisać do tego jeszcze klasę walidatującą i dodać kilka pól.

Myślę, że idziesz w dobrą stronę. Kilka uwag:

- jak już utworzyłeś klasę User, to swój system przebuduj tak aby odwołując się do tej klasy można by było czytać/pisać zmienne sesyjne. Czyli albo User sam zapisuje/odczytuje dane sesji, albo używa do tego osobnego obiektu (imo lepsze rozwiązanie). Bo teraz de facto ta klasa praktycznie jest tylko opakowaniem dla trzech zmiennych. Chyba że klasa User ma w Twoim przypadku pełnić rolę modelu, jeśli tak to taka funkcjonalność czytania/pisania danych do sesji w takiej klasie nie powinno się znaleźć (patrz punkt niżej).

- Klasę Registration powinieneś wg mnie przerobić na taką, która zarządza użytkownikami. Coś na wzór wzorca DAO, gdzie Registration (zmienić nazwę np. na UserManager) będzie operować na bazie danych (dodawać/usuwać/pobierać użytkowników). Wtedy klasa UserManager mogła by wpisać rekord do bazy danych, np. poprzez UserManager::insert($oUser) (w taki sposób "rejestrować" użytkowników) gdzie $oUser jest obiektem, który ma być wpisany do bazy danych (może to być np. obiekt Twojej klasy User, jeśli ta klasa nie ma pełnić rolę sesyjną, a rolę modelu).

- wymuszaj (gdzie to tylko możliwe) typy parametrów, czyli przykładowo metoda w Registration set_connection($db) powinna wyglądać set_connection(DataBase $db). Php to niestety pod względem wymuszania typów ułomny język, więc tym bardziej powinno się korzystać z tej biednej kontroli typów jaką daje nam php

- nie używaj (wogóle, nie ma wyjątków ) publicznych pól składowych - twórz metody dostępowe

Nierozumiem o co ci chodzi z tymi zmiennymi sesyjnymi i co to wogóle jest.

Klasa zarządzająca nie jest złym pomysłem, może to dopiszę kiedyś, narazie chcę zrobić dobrą rejestrację.

A wiesz co to tablica globalna $_SESSION? Przykładowo wywołujesz:

[PHP] pobierz, plaintext 
<?php
$user->setId($id);
$user->setEmail($email);
 
$user->getId();
?>
[PHP] pobierz, plaintext 

I ten kod zapisuje dane do sesji - o to mi chodziło gdy pisałem aby klasa User mogła czytać/pisać dane do sesji.

Może namieszałem ci trochę w głowie tymi wypocinami, zwłaszcza że najprawdopodobniej źle zidentyfikowałem do czego ma służyć twoja klasa User - bo zapewne ma być ona reprezentacją danych które mają być wpisane do bazy danych (rejestracja) oraz danych które mogą być wyciągnięte z tejże bazy danych. Więc dobrym pomysłem było by utworzenie klasy UserManager dla klasy User.

[PHP] pobierz, plaintext 
<?php
class UserManager{
  /** zwraca 1 obiekt User **/
  public static function getById($id){}
  /** zwraca tablicę obiektów User **/
  public static function getCollectionByIds(array $ids){}
  /** zapisuje obiekt User do bd **/
  public static function save(User $user){}
  //pozostałe metody np. remove, getAll itp.
}
 
//przykładowo rejestracja wyglądała by tak
$user = new User();
$user->setLogin($login);
//ustawienie pozostałych wartości...
 
UserManager::save($user);
//lub
$user->save();
?>
[PHP] pobierz, plaintext