klasy i dziedziczenie Opcje

[yeti]

Witam, mam kłopot myślowy co do zagospodarowywania zasobów podczas tworzenia obiektów.

Wprowadzenie:
mamy serwis, w którym wykorzystywane są:
- 3 klasy podstawowe (połączenie z DB, obsługa zapytań, klasa serwisowa). Klasy te dzidziczą od siebie w podanej kolejności
- 1 klasa do obsługi prezentacji
- 2 klasy specjalizowane (obsługjące konkretne działy serwisu, np. newsy i forum)

Pytanie:
Czy sensowne jest rozwiązanie, w którym robimy dwie ścieżki dziedziczenia: od klasy News poprzez Prezentację i klasy podstawowe i od klasy Forum poprzez Prezentację i klasy podstawowe? Klasy News i Forum używane są rownocześnie tylko 2 razy, ale spowoduje to niepotrzebne dwukrotne wywołanie klasy Prezentacji i klas podstawowych.

Czy może jest sensowniejsze rozwiązanie?
Dodam, że funkcjonalność klas News i Forum zasadza się na użyciu metod z klas podstawowych.

[SHiP]

Ja może napisze jak ja to robie...

mam klase main która jest extends functions
klase functions oraz inne np user, admin, profil, stats,search itd które są extends do function.
jak mam jakąś funkcje dla wszystkich to wrzucam ją do klasy functions jeśli nie to do odpowiedniego pliku.
Później sprawdzam która klasa jest potrzebna i includuje jej pliki oraz tworze obiekt...

Nie wiem jak bym tak pare razy nie dziedziczył...

[M4chu]

Ja zamiast dziedziczenie poprostu przeslylalbym instancje potrzebnych klas do klasy News czy Forum, poprostu:

[PHP] pobierz, plaintext 
<?php
 
class News
{
	function __construct($db)
	{
		$this->db = $db;
	}
}
class Forum
{
	function __construct($db)
	{
		$this->db = $db;
	}
}
$db = new Db;
$news = new News($db);
$forum = new Forum($db);
?>
[PHP] pobierz, plaintext 

Najlepsze jest to, ze operujesz na tym samym obiekcie, a w nim masz uchwyt do polaczenia z baza, liczbe wykonanych zapytan itp. Zreszta gdyby np News dziedziczylo po Db, to w konstruktorze News musialbys podac parametry bazy danych - gdybys chcial dodac jeszcze obsluge bledow czy inputa to to juz nie jest zbyt elastyczne

[yeti]

Hmm, to ostatnie jest niezłe.
Popróbuję, dzięki

[bregovic]

[PHP] pobierz, plaintext 
<?php
 
class News
{
	function __construct($db)
	{
		$this->db = $db;
	}
}
class Forum
{
	function __construct($db)
	{
		$this->db = $db;
	}
}
$db = new Db;
$news = new News($db);
$forum = new Forum($db);
?>
[PHP] pobierz, plaintext 

zamiast przesylac referencje i zapisywac obiekt db w atrybucie (co zdaje sie zajmuje duzo pamieci) mozesz po prostu napisac w kazdej metodzie

[PHP] pobierz, plaintext 
<?php
classs Foo
{
	public function OhSoFoo()
	{
		global $db;
	}
}
?>
[PHP] pobierz, plaintext 

[yeti]

Można zdefiniować zmienną globalną, ale ja osobiście takich zmiennych nie lubię. IMHO są po prostu niebezpieczne.

[bregovic]

w jakim sensie niebezpieczne? chodzi ci o powtarzanie sie nazw zmiennych, czy o cos innego?

[yeti]

Ja zamiast dziedziczenie poprostu przeslylalbym instancje potrzebnych klas do klasy News czy Forum

Wyglądało nieźle, ale albo coś pomijam albo niestety nie działa

[Yarecki]

Wyglądało nieźle, ale albo coś pomijam albo niestety nie działa

To co napisał M4chu działa bez problemów w php5 gdzie obiekty zawsze przesyłane są przez uchwyt, a nie kopiowane. W php4 ten kod powinien wyglądać jakoś tak:

[PHP] pobierz, plaintext 
<?php
class News
{
	var $objDb = NULL;
 
	/**
	 * Konstruktor
	 */
	function News( &objDb )
	{
		$this->objDb = &objDb;
	}
}
 
$objDb = new Db;
$objNews = new News( $objDb );
?>
[PHP] pobierz, plaintext 

[chfast]

zamiast przesylac referencje i zapisywac obiekt db w atrybucie (co zdaje sie zajmuje duzo pamieci) mozesz po prostu napisac w kazdej metodzie

[PHP] pobierz, plaintext 
<?php
classs Foo
{
	public function OhSoFoo()
	{
		global $db;
	}
}
?>
[PHP] pobierz, plaintext 

Tu się nie zgodze. Odniesienie się do zmiennej zapisanej wewnątrz obiektu jest kilkakrotnie szybsze niż odnoszenie się do zmiennych globalnych. A leśli chodzi o zasoby to chyba oba rozwiązania zjadają podobną ilość.

[M4chu]

A leśli chodzi o zasoby to chyba oba rozwiązania zjadają podobną ilość.

Wiec jak to lepiej zrobic?
Wiem, ze mozna jeszcze kozystac ze statycznej zmiennej i funkcji np (php5)

[PHP] pobierz, plaintext 
<?php
 
class Foo
{
	public static $instance;
	static function getInstance()
	{
		if(self::$instance == null)
		{
			 self::$instance = new Foo;
			 return self::$instance;
		}
		return self::$instance;
	}
}
Foo::getInstance()->method();
 
?>
[PHP] pobierz, plaintext 

Ale jak to sie ma do wydajnosci?

[rogrog]

M4chu - to co piszesz to singletony a więc rozwiązanie szeroko stosowane właśnie w tekich przypadkach

generalnie można używać albo zmiennych globalnych albo singletonów, ale przy programowaniu obiektowym używanie zmiennych globalnych jest po prostu mało eleganckie

[yeti]

w jakim sensie niebezpieczne? chodzi ci o powtarzanie sie nazw zmiennych, czy o cos innego?

Chodzi mi o to, że w pewnym sensie tracisz kontrolę nad zmienna. Wolę wiedzieć gdzie zmienna się kończy, a gdzie zaczyna. Korzystam z tablic globalnych i to mi wystarcza. A poza tym, jak to napisano powyżej, to nie jest zbyt eleganckie

A oto i rozwiązanie problemu:

[PHP] pobierz, plaintext 
<?php
 
   class Db {
 
	  function dbTest(){
		 echo('class Db, function testDb<br />');
	  }
 
   }
   
 
   class Sql {
 
	  var $db = NULL;
   
	  function sqlTest(){
		 echo('class Sql, function sqlTest<br />');
	  }
 
	  function sqlDbTest(){
		 echo('class Sql, function sqlDbTest<br />');
		 $this -> db -> dbTest();
	  }
 
	  function Sql(&$db){
		 $this -> db = &$db;
	  }
   }
 
   $db = new Db();
   $db -> dbTest();
   
   $sql = new Sql($db);
   $sql -> sqlTest();
   $sql -> sqlDbTest();
?>
[PHP] pobierz, plaintext 

Ten post edytował yeti 24.09.2004, 20:34:36

[ShaXbee]

[PHP] pobierz, plaintext 
<?php
class News
{
    var $objDb = NULL;
 
    /**
     * Konstruktor
     */
    function News( &objDb )
    {
        $this->objDb = &objDb;
    }
}
 
$objDb = new Db;
$objNews = new News( $objDb );
?>
[PHP] pobierz, plaintext 

Testowalem to rozwiazanie.
Zamiast &objDb nalezy wstawic & $objDb, ponieważ interpreter PHP4 zwraca błąd parsowania.

[yeti]

Testowalem to rozwiazanie.
Zamiast &objDb nalezy wstawic & $objDb, ponieważ interpreter PHP4 zwraca błąd parsowania.

hehe, a co ja napisałem powyżej