Destruktor wykonuje się za wcześnie. Opcje

[kazaa9]

Witam.

Mam dziwny problem z destruktorami i nie za bardzo wiem jak go rozwiązać. Przeglądałem trochę forum i manuala, ale nic w podobnym stylu nie było.

Oto opis mojego problemu: mam prosty systemik, w którym mam m. in. klasy DB (obsługa bazy danych, śle zapytania i przechowuje informacje o ewentualnych błędach) oraz Session (jak sama nazwa mówi - do obsługi sesji).

W klasie Session w destruktorze wywołuję metodę, która kończy mi daną sesję, czyli przy pomocy klasy DB robi UPDATE danych w bazie. Działa, jednak...no właśnie. Gdybym w klasie DB miał destruktor, w którym kończyłbym połączenie z bazą danych, to to zapytanie już wygeneruje błąd, czyli tak jakbym był rozłączony. Dodam jeszcze, że później pobieram sobie pewną metodą informacje o zapytaniach (klasa DB) i brakuje tego ostatniego zapytania kończącego sesję (teraz oczywiście rozważam wersję bez zakończonego połączenia).

Aha, jeszcze dodam (może to mieć jakiś związek - tak przypuszczam), że obiekty pobieram przy pomocy czegoś w stylu rejestru, tzw. ClassLoader'a.
Załączę odnośniki do kodów klas. Na logikę i angielski proszę na razie nie zwracać uwagi - poprawki wprowadzam na bieżąco (IMG:style_emoticons/default/winksmiley.jpg)
session.class.php
db.class.php
classloader.class.php

[LBO]

Przez lata programiści nauczyli się nie ufać destruktorom w PHP. Sprawiały mnóstwo problemów (wystarczy poczytać komentarze w odpowiednim rozdziale manuala). Istnieje obejście tego problemu, dosyć szeroko stosowane w wielu frameworkach - swój własny wirtualny system zarządzania obiektami.

[PHP] pobierz, plaintext 
class ApplicationManager
{
    protected $managers = array();
 
    public function registerManager(IManager $manager)
    {
        $this->managers[] = $manager;
    }
 
    public function start()
    {
        // number of registered managers
        $norm= count($this->managers);
        for($i = 0; $i < $norm; $i--)
        {
            $this->managers[$i]->start();
        }
    }
 
    public function stop()
    {
        // number of registered managers
        $norm= count($this->managers);
        for($i = $norm - 1; $i >= 0; $i--)
        {
            $this->managers[$i]->stop();
        }
    }
}
 
interface IManager
{
    public function start();
    public function stop();
}
 
class SessionManager implements IManager
{
    public function start()
    {
        // ustawiasz sobie sterownik sesji 
    }
 
    public function stop()
    {
       //zamykasz sesję
    }
}
 
 
class DatabaseManager implements IManager
{
    public function __construct($dsn)
    {
         // łączysz się z bazą
    }
 
    public function start() {}
 
    public function stop()
    {
       // zamykasz połączenie z bazą
    }
}
[PHP] pobierz, plaintext 

A wykorzystujesz to mniej więcej tak.

[PHP] pobierz, plaintext 
$application = new ApplicationManager();
 
// dalej pobierasz konfig aplikacji i tworzysz na jego podstawie potrzebne obiekty m.in. obiekty odpowiedzialne za sesje i połączenie z bazą danych.
 
$database = new Database($config['database']['dsn']);
 
$application->registerManager($database);
 
$session = new SessionManager();
 
$application->registerManager($session);
 
$application->start(); // wszystkie zarejestrowane obiekty wykonują własne metody start()
 
// etc
 
$application->stop(); // wszystkie zarejestrowane obiekty wykonują własne metody stop(). Zwróć uwagę, że robią to w odwrotnej kolejności niż zostały zarejestrowane. To jest ważne - pozwala zachować zależności.
 
[PHP] pobierz, plaintext 

Wszystko pisane z palca, więc jakby co poprawiać lub pytać (IMG:style_emoticons/default/smile.gif)

Pozdrawiam

Też racja.

Ale nawet nie o to mi chodziło. Chciałbym uzyskać odpowiedź na temat tego destruktora, dlaczego to nie działa. W sumie jakoś pracy szczególnie nie utrudnia, ale bywa w małym stopniu irytujące, gdyż (moim zdaniem) powinno wszystko działać.

O ile sam nie usuwasz obiektów robi to samo PHP przy zakończeniu skryptu - a wtedy nie możesz być 100%-owo pewny kolejności. Rozwiązanie powyżej.

Ten post edytował LBO 16.08.2009, 18:44:58