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ę... (IMG:http://forum.php.pl/style_emoticons/default/dry.gif)

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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) 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ę (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) 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. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

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

[vokiel]

No to teraz rozumiem troche bardziej (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)