Forum PHP.pl

może chcę za dużo pisząc takie?

[PHP] pobierz, plaintext 
 
trait Tr {
 
public function getC() {
echo 'asd'; 
}
 
}
}
 
 
interface In {
 
public function getC();
 
}
 
 
class B extends A implements In {
 
 public function getC() { 
 echo 'asd'; 
 }
 
}
 
 
// a to wywala błąd
 
 
class B extends A implements In {
 
use Tr;
 
 
}
 
 
[PHP] pobierz, plaintext 

czyżby interface nie rozpoznaje metod wstrzykniętych przez trait?

Ten post edytował kayman 26.01.2015, 12:33:41

Nie uważasz, że informacja jaki błąd mogłaby być pomocna? I co to jest klasa A?

Ten post edytował Crozin 26.01.2015, 12:35:19

jak jest

[PHP] pobierz, plaintext 
 
class B extends A implements In {
 
use Tr;
 
}
 
[PHP] pobierz, plaintext 

wywala błąd o braku metody

a jak zdejmę interface

[PHP] pobierz, plaintext 
 
class B extends A {
 
use Tr;
 
}
 
[PHP] pobierz, plaintext 

kod działa

[PHP] pobierz, plaintext 
<?php
 
trait Tr {
    public function getC()
    {
        echo 'getC Trait'.PHP_EOL;
    }
}
 
interface In {
    public function getC();
}
 
class A {
 
}
 
class B extends A implements In
{
    public function getC()
    {
        echo 'getC Class B'.PHP_EOL;
    }
 
}
 
class C extends A implements In
{
    use Tr;
}
 
$b = new B();
$c = new C();
 
$b->getC();
$c->getC();
[PHP] pobierz, plaintext 

Ten post edytował Pyton_000 26.01.2015, 12:46:16

[PHP] pobierz, plaintext 
<?php
 
interface In {
	function getC();
}
 
trait Tr {
	public function getC() {
		return 'Tr.getC()'; 
	}
}
 
class A {
 
}
 
class B1 extends A implements In {
	public function getC() { 
		return 'B1.getC()'; 
	}
}
 
class B2 extends A implements In {
	use Tr;
}
 
class B3 extends A {
	use Tr;
}
 
$b1 = new B1();
$b2 = new B2();
$b3 = new B3();
 
var_dump($b1->getC());
var_dump($b2->getC());
var_dump($b3->getC());
var_dump($b1 instanceof In);
var_dump($b2 instanceof In);
var_dump($b3 instanceof In);
[PHP] pobierz, plaintext 

Jak widać wszystko w porządku.

EDIT: Pyton_000 mnie trochę wyprzedził.

Ten post edytował Crozin 26.01.2015, 12:47:17

a teraz już zgłupiałem, co ma metoda z parenta wspólnego z interface?

[PHP] pobierz, plaintext 
 
interface ModelInerface {
 
    public function __construct();        
 
    public function setData($id);
 
    public function __set($key, $value);
 
    public function __get($key);
 
    public function getAll($order = null);
 
    public function save();
 
    public function getData($id = null);
 
    public function delete($id);
 
}
 
trait ModelTraits {   
 
 
    public function setData($id) {
        parent::setData($this->table, $this->columns, $id);
    }    
 
    public function __set($key, $value) {
        $this->set($this->table, $key, $value);
    }
 
    public function __get($key) {
        return $this->get($this->table, $key);
    }
 
    public function getAll($order = null) {
        return parent::getAll($this->table, $this->columns, $order);
    }
 
    public function save() {
        parent::save($this->table);
    }
 
    public function getData($id = null) {
        if ($id)
            $this->setData($id);
        return parent::getData($this->table);
    }
 
    public function delete($id) {
        parent::delete($this->table, $id);
    }
 
}
 
 
class TableUsers extends MasterModel implements ModelInerface {
 
private $table = 'users';
private $columns = array('id', 'mail', 'pass', 'level', 'firstname', 'lastname', 'lang');
 
    public function __construct() {
        parent::__construct($this->table, $this->columns);
    }
 
use ModelTraits; 
 
// cut
 
}
 
class MasterModel {
 
    public $data;
 
    public function __construct($table, $columns) {
        foreach ($columns as $value) {
            $this->data[$table][$value] = null;
        }
    }
 
   public function setData($table, $columns, $id) {
        $query = new QueryBuider();
        $query->setColumns($columns);
        $query->setFrom($table);
        $query->addWhere('id', $id);
        $sql = \Framework\Libs\Sql::instance();
        $data = $sql->getSingleRow($query->getSelect());
        foreach ($columns as $value) {
            $this->data[$table][$value] = $data[$value];
        }
    }
 
// cut
 
}    
 
 
[PHP] pobierz, plaintext 

czyli to jest żle i trzeba by np. inaczej nazwać metodę w parencie

[PHP] pobierz, plaintext 
 
public function setData($id) {
        parent::setData($this->table, $this->columns, $id);
    }
 
//zmienić np tak
 
public function setData($id) {
        $this->setMasterData($this->table, $this->columns, $id);
    }
 
 
[PHP] pobierz, plaintext 

a to dobrze

[PHP] pobierz, plaintext 
 
public function __get($key) {
        return $this->get($this->table, $key);
}
 
[PHP] pobierz, plaintext 

Ten post edytował kayman 26.01.2015, 14:19:58

1. Wykorzystanie magicznego __get() jest raczej złym pomysłem. Dlaczego nie zrobisz zwykłej metody get(), getTableProperty() czy jakkolwiek inaczej nazwanej?
2. Nie wiem co dokładnie chcesz osiągnąć, więc ciężko napisać co byłoby dobrym rozwiązaniem.

chciałem np takie coś

[PHP] pobierz, plaintext 
 
$id = getNpZSessji();
 
$user = new TableUser();
 
$user->setData($id);
 
echo 'Witaj ' . $user->firstname . '. Twoje id to '. $user->id;
 
 
//lub 
 
$user = new TableUser();
 
$user->setData($id);
 
$user->firstame = 'Tomek';
 
$user->save(); // jest id -> update;
 
//albo
 
$user = new TableUser();
 
$user->firstame = 'Tomek';
//$user->lastname = costam etc
 
$user->save(); // brak id -> insert;
 
[PHP] pobierz, plaintext 

i analogicznie do wszystkich tabel w bazie danych takie mapowanie

do tego np takie konstrukcje incydentalne do poszczególnych tabel

[PHP] pobierz, plaintext 
 
$log = new TableLogs();
$log->addLog('zmiana wartości tu i tu');
 
// co jest równoznaczne z
 
$log = new TableLogs();
$log->addtime = mktime();
$log->user_id = $SESSION['user_id'];
$log->content = 'zmiana wartości tu i tu';
$log->save();
 
 
[PHP] pobierz, plaintext 

setData możesz spokojnie wywalić i dać w konstruktor.

W tym przypadku nie potrzebujesz ogólnie interfejsu.

Robisz sobie klasę bazową Model która zastępuje Ci trait i interface (możesz zrobić abstract żeby jej nie instancjować) a poszczególne modele extends Model i wsio. Nie ma co kombinować (IMG:style_emoticons/default/smile.gif)

imo nie -> bo setData jest dla rekordu który istnieje

ja widzę to tak

[PHP] pobierz, plaintext 
 
$page = new TablePages();
if(isset($_POST['id']) {
$page->setData($_POST['id']);
}
$page->url = $_POST['url']; 
$page->save(); // insert lub update
 
[PHP] pobierz, plaintext 

ihmo

[PHP] pobierz, plaintext 
$page = new Page(12);
 
if(!$page)
{
   // Insert
    $page->title = 'ala';
    $page->save();
}
 
$page->title = 'Klocek';
$page->save(); //Update
[PHP] pobierz, plaintext 

dzięki panowie, wiem mniej więcej jak to przebudować by było "do ręki" i nie powielać kodu

1. Takie podejście, czyli tzw. ActiveRecord, jest raczej słabym rozwiązaniem. W Google znajdziesz więcej na ten temat.
2. Temat dotyczy na dobra sprawę ORM-a. Pomyśl nad wykorzystaniem Doctrine.

Ten post edytował Crozin 27.01.2015, 10:41:55

Może też użyć lżejszego Eloquent z Laravela, ale nie proponowałem bo stwierdziłem że autor robi to dla sportu (IMG:style_emoticons/default/wink.gif)

nie mam nic przeciwko ORM z prawdziwego zdarzenia jak Doctrine, Propel, chciałam sprawę uprościć na własne potrzeby a przy okazji czegoś się nauczyć

mam wrażenie, że deklaracja klasy odpowiadającej za obsługę jakiejś tabeli w bazie nie wyszła najgorzej a przy wykorzystaniu trait będzie w miarę szybka do napisania i w miarę czytelna

W Twoim Przypadku Trait nie jest potrzebny Bo:
- i tak nigdzie go nie wykorzystasz więcej niż w tych Modelach
- Musisz dołączać Trait i Interface

Lepiej już:
- Zostawić interface
- Zrobić klasę DbModel implementującą interfejs
- Dziedziczyć po DbModel

A jeszcze lepiej to dziedziczyć po Fabryce Model dzięki czemu będziesz mógł w bardzo szybki sposób zmienić implementację z BD na File np.

Pyton masz racje tylko w ten sposób zaraz napiszę kombajn a nie uproszczę całość a w takim wypadku już chyba lepiej skorzystać z gotowego rozwiązania bo szkoda czasu

btw. inteface można by chyba wywalić stosując trait bo metody i tak są w trait te co potrzeba