OOP czy dobrze go stosuję ? Opcje

[Fanatyko]

Witam, wiem że w tym dziale jest już świeży temat o stosowaniu OOP, jednak chciałbym aby ktoś zweryfikował mój kod czy to co robię jest zgodne z założeniem OOP.

Jest to przykładowy game framework, dla treningu. Mam w nim 4 klasy User, UserManager, Character, CharacterManager, oto one :

[PHP] pobierz, plaintext 
<?php class User {
 
    function __construct() {
 
    }
 
    private $_login = null;
    private $_password = null;
    private $_email = null;
    private $_id = null;
    private $_logged = null;
 
 //  tutaj dalej sa standardowe settery i gettery
 
} ?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class UserManager {
 
    function __construct($user) {
        $this->_user = $user;
 
        try{
        $this->_pdo = new PDO('mysql:host=localhost;dbname=rsfc;', '', '');
        $this->_pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }catch(PDOException $error){
            echo 'Problem z baza :( :' . $error->getMessage();
        }
    }
 
    private $_user;
 
    public function Logon(){
 
        $db = $this->_pdo->prepare('SELECT * FROM user_account WHERE login = :login AND password = :password');
        $db->execute(array(':login' => $this->_user->getLogin(), ':password' => $this->_user->getPassword()));
 
        try{
 
             if($user = $db->fetch()){
                 echo "Zostales zalogowany";
                 $this->_user->setId($user[id]);
                 $this->_user->setLogged(1);
                 $this->_user->setSession();
             }
 
                else{
                        throw new Exception("Nie ma Cie w bazie ! Nie zostales zalogowany :(");
                    }
 
        }catch(Exception $error){
            echo $error->getMessage();
        }      
 
    }
 
 
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class Character {
 
    function __construct() {
 
    }
 
    private $_strenght;
    private $_speed;
    private $_dynamic;
    private $_condition;
    private $_pain;
    private $_endurance;
    private $_iq;
    private $_coordination;
    private $_technic;
    private $_flex;
    private $_life;
    private $_level;
    private $_experince;
    private $_needexperince;
    private $_money;
    private $_reputation;
    private $_movepoints;
 
    private $_id = null;
 
    //settery
 
    // tutaj tez standardowe settey i gettery
 
 
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class CharacterManager {
 
    function __construct($character) {
        $this->_character = $character;
 
        try{
        $this->_pdo = new PDO('mysql:host=localhost;dbname=rsfc;', '', '');
        $this->_pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }catch(PDOException $error){
            echo 'Problem z baza :( :' . $error->getMessage();
        }
 
 
    }
 
 
 
    private $_character;
    private $_pdo;
 
    public function render(){
        include './views/character.php';
    }
 
    public function upgradeStatistic($statistic, $how_much){
 
 
            $db = $this->_pdo->prepare('UPDATE user_character SET '.$statistic.' = '.$statistic.' + :how_much WHERE id = :id');
            $db->execute(array(':how_much' => $how_much, ':id' => $this->_character->getIdFromSession()));
 
    }
 
    public function downgradeStatistic($statistic, $how_much){
 
 
            $db = $this->_pdo->prepare('UPDATE user_character SET '.$statistic.' = '.$statistic.' - :how_much WHERE id = :id');
            $db->execute(array(':how_much' => $how_much, ':id' => $this->_character->getIdFromSession()));
 
    }
 
    public function setStatistics(){
        try{
 
            $db = $this->_pdo->prepare('SELECT * FROM user_character WHERE id = :id');
            $db->execute(array(':id' => $this->_character->getIdFromSession()));
 
            if($character = $db->fetch()){
                $this->_character->setCondition($character['condition']);
                $this->_character->setCoordination($character['coordination']);
                $this->_character->setDynamic($character['dynamic']);
                $this->_character->setEndurance($character['endurance']);
                $this->_character->setExperince($character['experince']);
                $this->_character->setFlex($character['flex']);
                $this->_character->setIq($character['iq']);
                $this->_character->setLevel($character['level']);
                $this->_character->setLife($character['life']);
                $this->_character->setMoney($character['money']);
                $this->_character->setMovePoints($character['move_points']);
                $this->_character->setNeedExperince($character['needexperince']);
                $this->_character->setPain($character['pain']);
                $this->_character->setReputation($character['reputation']);
                $this->_character->setSpeed($character['speed']);
                $this->_character->setStrenght($character['strenght']);
                $this->_character->setTechnic($character['technic']);
 
            }else{
                throw new Exception('Nie moge uaktualnic statystyk :(');
            }         
            }
            catch(Exception $error){
                echo 'Wystapil blad:' . $error->getMessage();
            }
        }
 
}
?>
[PHP] pobierz, plaintext 

Index.php

[PHP] pobierz, plaintext 
<?php
session_start();
 
require 'class/Character.php';
require 'class/CharacterManager.php';
require 'class/User.php';
require 'class/UserManager.php';
 
$User = new User;
$User->setLogin('');
$User->setPassword(md5(''));
 
$UserManager = new UserManager($User);
$UserManager->Logon();
 
try{
    if($User->getLogged() == 1){
 
        $Character = new Character;
        $Character->setIdFromSession();
        $CharacterManager = new CharacterManager($Character);
        $CharacterManager->upgradeStatistic('strenght', 5);
        $CharacterManager->setStatistics();
        echo $Character->getStrenght();
    }else{
        throw new Exception('Nie jestes zalogowany !');
    }
 
 
}catch(Exception $error){
    echo 'Wystapil blad:'.$error->getMessage();
 
}
 
 
 
?>
[PHP] pobierz, plaintext 

Pytania:

1. Głowne - czy ten kod jest zgodny z OOP ?
2. Czy metoda Logon() powinna być w klasie UserManager czy w osobnej np. Login
3. Czy tworzenie obiektu User a potem przekazywanie go do UserManager w celu obróbki itp jest prawidłowe ? Czy w klasie UserManager powinna być metoda tworząca obiekt User ? (to tyczy się również klas Character i CharacterManager)

Proszę o wytknięcie mi błedów i sposobów w jaki mogę je poprawić (IMG:style_emoticons/default/wink.gif)

[smentek]

Prawdę mówiąc nie wiem czy złapałem o co chodzi (IMG:style_emoticons/default/wink.gif)

Chodzi mi o to ze program mozna koncepcyjnie podzielić na:

- Cześć APLIKACYJNA, która jest "wspólna" dla większości systemów www. Do tego wchodzi np. autoryzacja. (twoje logowanie) obsługa sesji, jakaś implementacja MVC (lub prostszy mechanizm zamiast MVC). Routingu.

Tej części nie wymyślaj od nowa użyj jakiegoś frameworku.

- Część BIZNESOWĄ która jest unikalna dla danego systemu. Jeżeli piszesz system do obsługi pizzerii to to będą obiekty Kucharz, Klient, Pizza, Składnik, Pizzy, Kelner, etc. (z palca pisze).

Skup się na tej części biznesowej a pierwszą znajdziesz w gotowych rozwiązaniach. Obie warstwy powinny zajmować się różnymi rzeczami. To znaczy obiekt Kucharz nie powinien implementować logowania ani zapisu danych dobazy, nawet jeżeli koncepcyjnie użytkownik będący kucharzem może się logować.

Obiekt Kucharz może za to przyjmować w konstruktorze obiekty Składaniki Pizzy oraz posiadać metodę makePizza() która zwraca Pizze.

Poćwicz sobie tworzenie systemów składających się z małych obiektów które rozwiązują tego typu problemy.

Bez tej umiejętności będziesz jak papuga która klepie bezmyślnie kod według koncepcji wymyślonych przez innych, których to koncepcji nawet nie rozumie, ale sam nie będziesz potrafił zaprojektować nawet prostego systemu...