Zmieniana obiektu w jego wnętrzu Opcje

[#luq]

Mam obiekt Person który odzwierciedla 2 tabelki w bazię.

user_name
userId | userName | userSurname | email (...)

user_memo
userId | userMemo

dlatego 2 tabelki, bo nie każda osoba musi mieć opis

Stworzyć go można tylko za pomocą metody statycznej get( $id ).
Mam metodę edit( $parmArr ), który edytuje strukturę w bazię jeśli jest to konieczne, np sprawdza czy osoba ma opis jeśli nie a ja chce go jej dodać tworzy nowy rekord w tabelcę user_memo

Ogólnie działanie

[PHP] pobierz, plaintext 
<?php
$p = Person::get( 1 );
$p->edit(
    'userName' => 'foo'
    'userMemo' => 'opis'
);
?>
[PHP] pobierz, plaintext 

i chcę teraz żeby po edycji obiekt został zaktualizowany. Myślałem nad prostym rozwiązaniem:

[PHP] pobierz, plaintext 
<?php
public function edit( $dataArr ){
    // jesli to potrzebne aktualizuje bazę i potem    
    $this = self::get( $this->getId() );
}
?>
[PHP] pobierz, plaintext 

Fatal error: Cannot re-assign $this

Ok, da się to jeszcze przejść po prostu zczytać dane podane w $dataArr i powpisywać je do właściwości, podobnie jak dla bazy robie. Ale co jeśli chce metode delete(), która ma usuwać odpowiadające rekordy w user_name oraz w user_memo oraz usunąć obiekt.

[PHP] pobierz, plaintext 
<?php
public function delete(){
      // wywal rekordy w bazie
      $this = NULL;
  }
?>
[PHP] pobierz, plaintext 

Nie zadziała.
Może totalnie źle myślę...

Edit

Oczywiście mógłbym zwracać obiekt tymi metodami ale zastanawiam się czy da się to zrobić wewnątrz.

Ten post edytował #luq 24.05.2009, 15:46:14

[vokiel]

Witaj w te piękne niedzielne popołudnie:)

Po pierwsze, to co mi się nasuwa, to czy dana persona może mieć klika opisów? Jeśli nie (jak to jest w przypadku np stopki na forum, opisu gg... etc) to uważam, że zaoszczędzenie miejsca wynikające z przeniesienia opisów do oddzielnej tabeli jest niewielkie, a nawet przy większej ilości osób z opisami żadne. Dodatkowo powoduje niepotrzebne utrudnienia.
Proponowałbym przenieść kolumne userMemo do tabeli user_name. W takiej strukturze wszelkie odczyty i modyfikację stają się ciut łatwiejsze.
A klasę operującą na tym można by zrobić mniej więcej tak:

[PHP] pobierz, plaintext 
<?php
$id = 1; // id osoby w bazie
$p = new Person($id);
echo $p->get('userMemo'); // "Mój stary opis"
$p->set('userMemo','Nowy opis');
// lub 
$fields = array ('userMemo'=>'Nowy opis','userSurname'=>'Kowalski');
$p->set($fields);
$p->save();
echo $p->get('userMemo'); // "Nowy opis"
?>
[PHP] pobierz, plaintext 

Ten post edytował vokiel 24.05.2009, 16:27:17

[#luq]

Tak, osoba może mieć jeden opis. W sumie masz rację z tym przeniesieniem userMemo do user_name, przy łączeniu tych 2 tabelek idzie to na pewno dłużej niż jakby było w jednej tabelce Było by łatwiej. W sumie można by robić to tak jak to pokazałeś, tyle że ja mam statyczną metode get ponieważ są jeszcze metody

[PHP] pobierz, plaintext 
<?php
Person::getAll();
Person::getSite( $site );
?>
[PHP] pobierz, plaintext 

Ale mniejsza z tym.

Z editem jeszcze dałbym rade zaktualizować obiekt, tylko co z delete? Można by to zrobić tak.

[PHP] pobierz, plaintext 
<?php
public function edit( $dataArr ){
    // jesli to potrzebne aktualizuje bazę i potem    
    return self::get( $this->getId() );
}
 
public function delete(){
      // wywal rekordy w bazie
      return NULL;
 }
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$p = Person::get( 1 );
$p = $p->delete();
?>
[PHP] pobierz, plaintext 

Tylko jeśli przez pomyłkę napiszemy:

[PHP] pobierz, plaintext 
<?php
$p = Person::get( 1 );
  $p->delete();
?>
[PHP] pobierz, plaintext 

To już mamy dupkę Obiekt istnieje natomiast w bazie już go nie ma. Dlatego zastanawiam się czy można w jakiś sposób wewnątrz obiektu go usunąć, żeby takie błędy nie mały możliwości wystąpić.

[vokiel]

Trochę nie rozumiem struktury klas. Masz tą klasę Person, która pobiera dane, tak? Np ::get, ::getAll, getSite(), czyli pobiera poszczególne pola z tabeli dla danego usr?
Jeśli tak, to bym proponował jedną metodę get($what), potem drugą set($what, $newValue)
Te metody, które piszesz (edit, delete) są metodami klasy Person?

A może zrobić tak:

[PHP] pobierz, plaintext 
<?php
class Person{
 
    public function __construct($id){}
    public function get($what){}
    public function set($what, $newValue){}
    public function delete(){}
    public function __destruct(){
        $this->delete();
    }
}
//Wykorzystanie
$p = new Pesron(1);
$p->get('userMemo');
unset($p); // obiekt zniszczony, z bazy usunięty
?>
[PHP] pobierz, plaintext 

[#luq]

Np ::get, ::getAll, getSite(), czyli pobiera poszczególne pola z tabeli dla danego usr?

Nie, nie, nie.
::get() pobiera wszystkie dane jednego usera i tworzy z tego obiekt
::getAll() pobiera i tworzy obiekty wszystkich userów z bazy
::getSite() pobiera stronę userów przykładowo gdy podamy parametr 2 dostaniemy userów od 21 do 40 (ORDER BY un.userId LIMIT (...)) oczywiście zależne jest to od wartości stałej ile ma być userów na stronie. W tym wypadku stała jest ustawiona na 20.

Ogólnie ta klasa reprezentuje jednego konkretnego człowieka, właściwie tylko przechowuje we właściwościach odpowiednie pola z bazy. Ten obiekt będzie wrzucony do innego obiektu, jak zresztą masa innych.
Na stronie będzie jedynie używana tylko w takiej formie.

[PHP] pobierz, plaintext 
<?php
$id = // pobrane skądś tam
Person::get( $id );
echo Person->getName();
?>
[PHP] pobierz, plaintext 

Natomiast w panelu admina, musi być możliwośc przeglądania wszystkich, podzielenia tego na strony, edycji, usuwania i dodawania nowych. I tutaj będzie to wykorzystywane w taki sposób

[PHP] pobierz, plaintext 
<?php
$persons = Person::getSite( $actSite );
$cntPer = count( $persons );
for( $i = 0; $i < $cntPer; $i++ ){
    echo $persons[$i]->getName().'<br />';
    //...
}
?>
[PHP] pobierz, plaintext 

Tak naprawdę to co aktualnie mam mi wystarcza. Przy usuwaniu kieruje się na adres /admin/deleteuser/3

[PHP] pobierz, plaintext 
<?php
$person = Person::get( 3 );
$person->delete();
//przenosi mnie spowrotem, wyświetla komunikat że wywalono user
?>
[PHP] pobierz, plaintext 

Wyszysko działa, tyle, że pomyślałem, że fajnie byłoby usuwać te obiekt, któego po delecie tak naprawdę już nie ma. Zostaje mi obiekt który już nie ma odzwieciedlenia w bazie. Myślę, że to było by to bardziej zgodne z OOP, logiką itd.

Jak widać destruktor nie wchodzi w rachubę. Wyświetle usera, parser pojedzie do końca kodu i buum usuwa mi wszystkie obiekty, tym samym wywala mi je z bazy.

Nie wiem, taki model widaje mi się rozsądny. Może się myle? Z OOP nie jestem jeszcze tak bardzo obeznany

Ten post edytował #luq 24.05.2009, 18:42:02

[vokiel]

No to teraz rozumiem troche bardziej

Nie widzę tu możliwości usunięcia obiektu poza unset(), poza tym nie ma takiej potrzeby.
Żeby zmodyfikować jakiegoś usr musisz wpiewr go pobrać, stworzyć obiekt. Zatem jak zrobisz:

[PHP] pobierz, plaintext 
<?php
$person = Person::get( 3 );
$person->delete();
unset($person);
//to już teraz 
$person = Person::get( 3 ); //się nie powiedzie
// ani nie zrobisz
$person->edit();
?>
[PHP] pobierz, plaintext 

Myślę, że chodziło Ci o automatyzację tego unset, juz w metodzie delete, tak? Chyba pozostaje Ci samo unset, albo przeprojektowanie klas. Tak, że miałbyś klase Usr, i UsrsAdm, do której byś przekazywał obiekt tej pierwszej, na wzór:

[PHP] pobierz, plaintext 
<?php
$ua = new UsrAdm(new Usr(1));
$ua->delete(1);
?>
[PHP] pobierz, plaintext 

Ten post edytował vokiel 24.05.2009, 22:02:14

[#luq]

Myślę, że chodziło Ci o automatyzację tego unset, juz w metodzie delete, tak?

Dokładnie.

Dzięki za zainteresowanie. Twoja odpowiedź mnie satysfakcjonuje, nie da się tak jak ja to wymyśliłem