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)

[Fanatyko]

Im mniejsze czyli bardziej rozdrobnione obiekty tym lepszy system. Z drugiej strony jeżeli ma to być prosty system to nie ma sensu sztucznie go komplikować, tak że możesz rejestracje i logowanie zrobić w jednym obiekcie, tylko niech to nie będzie ten sam obiekt który reprezentuje użytkownika. Może 'UserAuthorisation'?

Brakowało mi zamiennika nazwy (IMG:style_emoticons/default/tongue.gif) Dzięki, po raz kolejny.

Jeżeli chodzi o sam obiekt Character. Nawrzucałeś do niego różnych zmiennych które ci przyszły do głowy a które nie mają jakiegokolwiek uzasadnienia. Obiekt Character to nie jest prawdziwy obiekt w rozumieniu OOP. Od tablicy różni go tylko forma zapisu.

W którymś tematcie o OOP było napisane, że całkiem rozsądne jest robienie takich kontenerów-obiektów zawierających dane. Są tam zmienne, settery i gettery. Czy jednak to jest zło i należy to zmienić ?

Przykładowo $_speed. Czym jest speed? Prędkość z którą porusza się byt w danym momencie czy maksymalna prędkość z którą może się poruszać? Prędkość to jest coś względnego. Względem czego liczysz prędkość? Jeżeli nie masz tego czegoś to może i sama speed powinna na razie zniknąć? Może prędkość nie powinna być zmienną tylko wynikać z relacji obiektów, tak jak w świecie rzeczywistym?

Prawdę mówiąc w kontekście tego co chcę zrobić, nie potrzebuję takich "szerszych horyzontów" bo $_speed ma po prostu być zwykłą statystyką, na której podstawie będzie można wyliczyć np. szanse na unik/trafienie.



To znowu ja (IMG:style_emoticons/default/biggrin.gif)

Zmiany:

Dodałem klasę UserAuthorisation dzięki której po przekezaniu obiektu User można się zalogować.

[PHP] pobierz, plaintext 
<?php
class UserAuthoristation {
 
    function __construct($user) {
        $this->_user = $user;
 
        $this->_db = new Database();
        $this->_db->connect();
    }
 
    private $user;
 
    public function Logon(){
 
        $q = $this->_db->_pdo->prepare('SELECT * FROM user_account WHERE login = :login AND password = :password');
        $q->execute(array(':login' => $this->_user->getLogin(), ':password' => $this->_user->getPassword()));
 
        try{
 
             if($user = $q->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 

Dodałem klase Database do obsługi bazy danych, za wiele w niej nie ma (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
 
class Database {
 
    function __construct() {
 
    }
 
    public function connect(){
        try{
        $this->_pdo = new PDO('mysql:host=localhost;dbname=;', '', '');
        $this->_pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }catch(PDOException $error){
            echo 'Problem z baza :( :' . $error->getMessage();
        }
    }
 
}
 
 
[PHP] pobierz, plaintext 

Zmiany w klasie UserManager, dodałem funkcję tworzącą użytkownika, wywalilem z niej funkcje logującą i dodałem wykorzystanie klasy Database.

[PHP] pobierz, plaintext 
class UserManager {
 
    function __construct($user) {
        $this->_user = $user;
        $this->_db = new Database;
        $this->_db->connect();
    }
 
    private $_user;
 
    public function create(){
        $q = $this->_db->_pdo->prepare('INSERT INTO user_account VALUES(null,:login,:nick,:password,:email)');
        $q->execute(array(':login' => $this->_user->getLogin(), ':password' => $this->_user->getPassword(), ':nick' => 'test', 'email' => 'test'));
 
        return "User zostal stworzony";
    }
 
 
}
 
[PHP] pobierz, plaintext 

W index.php dorzuciłem tworzenie usera.

[PHP] pobierz, plaintext 
 
<?php
session_start();
 
require 'class/Character.php';
require 'class/CharacterManager.php';
require 'class/User.php';
require 'class/UserManager.php';
require 'class/Database.php';
require 'class/UserAuthorisation.php';
 
//logowanie
 
$User = new User;
$User->setLogin('Fanatyko');
$User->setPassword(md5('beniamins9'));
 
$UserAuthorisation = new UserAuthoristation($User);
$UserAuthorisation->Logon();
 
// zwiekszanie statystyk jesli jest sie zalogowanym
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();
 
}
 
//tworzenie nowego usera
 
$User2 = new User;
$User2->setLogin('Test');
$User2->setPassword(md5('test'));
 
$UserManager2 = new UserManager($User2);
$UserManager2->create();
 
 
?>
[PHP] pobierz, plaintext 

Jest dobrze ?

Ten post edytował Fanatyko 29.08.2011, 21:48:05