widocznosc obiektu wewnątrz klasy Opcje

[Black-Berry]

Czy we wnątrz klasy mogę użyć zewnętrzny obiekt bez konieczności używania deklaracji public ?

[PHP] pobierz, plaintext 
<?php
	class c_page_creator_main
	{
		function c_page_creator_main()
		{
			global $db; //wewnątrz chciałbym użyć ten obiekt bez ciągłego deklorowania go jako public
			$db->query( "SELECT * FROM tabela" );
 
	}
?>
[PHP] pobierz, plaintext 

[webdice]

Może dziedziczenie?

[Black-Berry]

nie ma innego sposobu ?

[webdice]

[PHP] pobierz, plaintext 
<?php
	class c_page_creator_main
	{
		function c_page_creator_main()
		{
			$db = new myDB ();
			$db->query( "SELECT * FROM tabela" );
 
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował webdicepl 13.02.2008, 01:24:17

[Black-Berry]

no tak ale wtedy za kazdym razem bede musiał tworzyc nowy obiekt a to mi ma ułatwić sprawę (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Chodzi o to ze to są odrębne klasy i chciałbym zachoać odrębnoać ale w kazdej linijce pisanie global troche mi się nie widzi.

[webdice]

Możesz jeszcze przekazywać obiekt do metody.

[LonelyKnight]

Czy we wnątrz klasy mogę użyć zewnętrzny obiekt bez konieczności używania deklaracji public ?

[PHP] pobierz, plaintext 
<?php
	class c_page_creator_main
	{
		function c_page_creator_main()
		{
			global $db; //wewnątrz chciałbym użyć ten obiekt bez ciągłego deklorowania go jako public
			$db->query( "SELECT * FROM tabela" );
 
	}
?>
[PHP] pobierz, plaintext 

A co złego w tym:

[PHP] pobierz, plaintext 
<?php
class c_page_creator_main
	{
				public $objDB;
 
		function c_page_creator_main()
		{
			$this->objDB = Database::getInstance(); // do bazy danych powinieneś użyć singletona 
			$this->objDB ->query( "SELECT * FROM tabela" );
 
			 } 
		 }
?>
[PHP] pobierz, plaintext 

[Black-Berry]

@LonelyKnight MAsz u mnie piwo:)

[webdice]

Dodam że singleton jest porównywany z globalami tylko w ładniejszym opakowaniu.

[em1X]

po prostu napisz sobie jakiś obiekt podstawowy, który wszystkie inne klasy będą rozszerzać.

[PHP] pobierz, plaintext 
<?php
class Object {
 
   public function & getDB() {
	  // zwraca obiekt mysql
   }
 
   // i tak dalej
 
}
?>
[PHP] pobierz, plaintext 

i w następnych klasach nie będzie problemów:

[PHP] pobierz, plaintext 
<?php
class myClass extends Object {
 
   public function display()
   {
	  $db = $this->getDB();
 
	  print $db->result('select field from table where id = 1');
   }
 
}
?>
[PHP] pobierz, plaintext 

Ten post edytował em1X 13.02.2008, 12:13:07

[Black-Berry]

@webdicepl Tak sobie myślę... że skoro różnica ma polegać tylko na opakowaniu to już chyba zostanę przy "globalach" i przy kazdym wywołaniu $db dopiszę sobie to global. Nie chcę za dużo komplikować.

[webdice]

Tak jak Ci mówiłem, tu najlepszym rozwiązaniem będzie dziedziczenie. I tak w metodach używasz funkcji z klasy DB, więc będzie ona nierozłączną częścią Twoich klas.

[em1X]

@webdicepl Tak sobie myślę... że skoro różnica ma polegać tylko na opakowaniu to już chyba zostanę przy "globalach" i przy kazdym wywołaniu $db dopiszę sobie to global. Nie chcę za dużo komplikować.

Używanie globali jest bardzo nieeleganckie, bardzo niepraktyczne i świadczy o niskim poziomie programisty (IMG:http://forum.php.pl/style_emoticons/default/wstydnis.gif) . PHP ma całkowicie usunąć w przyszłości funkcję global z języka.

[Black-Berry]

Rozumiem, W takim razie zrobię tak jak sugerujesz. Obiekt główny i dziedziczące po nim wszystkie inne. To lepszy sposób niż SINGLETON ? Jak uważacie. Zależy mi na dobrym profesjonalnym i przyszłościowym rozwiązaniu, tak żebym mógł pozniej do sterownika bazy dopisać np cache.

[em1X]

Zauważ, że jeżeli będziesz chciał wprowadzić jakieś modyfikacje do sterownika bazy danych to zrobisz to zupełnie niezależnie od systemu.
Myślę, że rozsądnie jest użyć Singletona w przypadku obsługi bazy danych. Masz wtedy pełną kontrolę nad tym połączeniem z bazą. Ale też (w przypadku innych klas) z nim nie przesadzaj, ostatnio jakaś taka dziwna moda jest na walenie tego Singletona w każdą klasę, gdzie popadnie, a przecież przeważnie ten Singleton wcale nie jest w niej potrzebny.

A więc z głową wszystko, z głową (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

ps. siedzisz dalej w PHP4 ?

Ten post edytował em1X 13.02.2008, 16:49:51

[Black-Berry]

Tak, jakoś tak wyszło, że zaczynałem w PHP4 i się go trzymam. Wiem, że wychodzi już z użycia i nie ma sensu tego ciągnąć, a co do pełnej obiektowości to trochę ona do mnie nie przemawia. Wiem, ze teraz moda na obiekty i wogóle ale kiedy to wszystko się zaczynało to klasa znaczyła grupę obiektów i nikt nie budował całej klasy dla tylko jednego obiektu. Moim zdaniem to trochę nadużycie ale pewnie nie idę z duchem czasu... ale nie o tym:

Przyszło mi do głowy jeszcze jedno rozwiązanie. Co o nim myślicie?

[PHP] pobierz, plaintext 
<?php
class c_my_sql_database_driver()
{
	function query( $query )
	{
		//ciało funkci
	}
}
 
$object_my_sql_database_driver = new c_my_sql_database_driver();
 
function db_query($query)
{
	// ta funkcja będzie mogła w przyszłości przełaczać na inne drivery bazy za pomocą
 switch();
	// więc dodatkowo storzę pomost między driverem bazy a klasami.
	global $object_my_sql_database_driver;
	$object_my_sql_database_driver->query($query);
}
 
 
class c_page_creator()
{
	function body_drop()
	{
		 db_query( 'select coś tam' );
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Black-Berry 13.02.2008, 21:04:50

[em1X]

1) Bez sensu.... i skończ z tym globalem w końcu (IMG:http://forum.php.pl/style_emoticons/default/sciana.gif)

2) Napisałem Ci jak poprawniej to powinno wyglądać:
http://forum.php.pl/index.php?showtopic=87...rt=#entry441695

Możesz sobie tylko zamienić metodę getDB() na np. query(), która będzie wykonywać zapytania SQL.

3) Poczytaj jak wyglądają klasy w PHP5. Stwarzają masę nowych możliwości. Ludzie jeżdzą samochodami, a Ty dalej razem z Flinstonami na kwadratowych kołach jedziesz (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

4) przykład:

[PHP] pobierz, plaintext 
<?php
abstract class Object {
 
	public function getDB()
	{
		return DB::getInstance();
	}
 
}
 
class DB {
 
	/**
	 * Pola
	 */
	private static $instance = null;
 
	/**
	 * Metody
	 */
	private function __construct() {
		$this->connect();
	}
 
	public function __destruct()
	{
		$this->disconnect();
	}
 
	public function getInstance()
	{
		$class = __CLASS__;
		return self::$instance == null ? self::$instance = new $class : self::$instance;
	}
 
	public function connect()
	{
		// polacz sie z baza danych
	}
 
	public function disconnect()
	{
		// rozlacz sie z baza danych
	}
 
	public function query($qry)
	{
		return mysql_query($qry);
	}
 
}
 
class TwojaKlasa
	extends Object
{
	public function display()
	{
		return $this->getDB()->query('select pole from tabela where pole_id = 1');
	}
}
 
/**
 * Przyklad
 */
 
$object = new TwojaKlasa();
 
print $object->display();
?>
[PHP] pobierz, plaintext 

Ten post edytował em1X 14.02.2008, 12:23:06

[Black-Berry]

(IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Dobra przekonałes mnie. Postęp musi być (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) Dzis w nocy zabieram się do analizy tego wszystkiego. Dzięki.


4 godziny później...
Zrobiłem to według schematu. Wszystko działa jak należy:) Jesteś dziś moim bogiem (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) Jeszcze raz wielkie dzięki!

i jeszcze...
Tak apropo globalsów (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Czy takie coś wchodzi w grę?

[PHP] pobierz, plaintext 
<?php
$this->connect_id = @mysql_connect( $GLOBALS["db_host"], $GLOBALS["db_user"], $GLOBALS["db_password"] );
?>
[PHP] pobierz, plaintext 

Ten post edytował Black-Berry 14.02.2008, 01:16:02

[Tubis]

Lepiej by było jakby te db_host, db_user i password były stałymi. Nie miałbyś problemy z ich dostępnąścią w klasach, bo stałe mają widoczność globalną.

[Cysiaczek]

Może tak jednak nie globale i stałe, a jakiś obiekt konfiguracji?

[Black-Berry]

Taki obiekt konfiguracyjny były również singletonem prawda? Podobnie jak driver bazy danych ?

[Cysiaczek]

To już od Ciebie zależy. Może być sigletonem, ale nie musi

[Cotter]

Ja w tym celu tworzę sobie obiekt globalny w tablicy GLOBALS i w celu skrócenia kodu odwołuję się do niego przez funkcję (nie będącą w żadnej klasie):

[PHP] pobierz, plaintext 
<?php
$GLOBALS['sql'] = new Sql();
 
function sql()
{
	return $GLOBALS['sql'];
}
 
//i teraz można już używać ten obiekt wszędzie tak:
sql()->query(...);
?>
[PHP] pobierz, plaintext 

Jeżeli zależy nam na singletonie to można oczywiście zastosować zwykły singleton i dopisać sobie taką funkcję:

[PHP] pobierz, plaintext 
<?php
function sql()
{
	return Sql::getInstance();
}
?>
[PHP] pobierz, plaintext 

Idea polega na tym aby skrócić kod odwołania. Prościej i szybciej jest używać sql()->query() niż Sql::getInstance()->query();

Jeżeli chodzi o tworzenie obiektu nadrzędnego, po którym będą dziedziczyły wszystkie inne to nie polecam tego rozwiązania. W pewnym sensie zmusza ono programistę do dziedziczenia po tej klasie co może czasami okazać się niewygodne.

[Black-Berry]

no to teraz mi już namieszałeś totalnie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Znacznie bardziej podoba mi się Twoje rozwiązanie, ale poprzednicy twierdzili zupełnie inaczej.

Przy okazji nasuwa mi się pytanie. Skoro znacznik global ma zostać usunięty z PHP w przyszłości to jak to się ma do tablicy $GLOBALS? tez ma zostać usunięta ?

[em1X]

No pewnie, że namieszał.

Jeżeli chcesz przechowywać dane konfiguracyjne to zapoznaj się ze wzorcem Registry.

Jeżeli chcesz skrócić sobie notację, możesz utworzyć metodę-osłonkę w swojej klasie:

[PHP] pobierz, plaintext 
<?php
class X {
   public function query($sql)
   {
		 // przykład
		 return $this->getDatabase()->query($sql);
   }
}
?>
[PHP] pobierz, plaintext 

Ten post edytował em1X 17.02.2008, 12:49:50

[Black-Berry]

@em1X tego typu osłonka sprawdza się świetnie. Wielkie dzięki za sugestie. @Cotter Za pomysł ze skróceniem notacji też dziękuję. Wydaje mi się jednak, że obiekt po którym dziedziczą wszystkie inne obiekty to dobry pomysł bo mogę sobie podpinać wtyczki do całego systemu w bardzo wygodny sposób. O to co udało mi się sklecić. Proszę o komentarze.

[PHP] pobierz, plaintext 
<?php
abstract class edc_object
	{   
		/* Pobieranie instancji singletona */
		public function db_driver(){ 
			return mysql_db_driver::get_instance();
		}
 
		/* Pomost dla query */
		public function query($sql){ 
			return $this->db_driver()->query($sql);
		}
 
		/* Pomost dla result */
		public function fetch_row($result=""){ 
			return $this->db_driver()->fetch_row($result);
		}
 
		/* Tutaj zbiór przydatnych funkcji też jako singleton. Nie wiem czy to dobry sposób. */
		public function db_functions(){ 
			return db_functions::get_instance(); 
		}
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował Black-Berry 17.02.2008, 15:38:05

[em1X]

1) dużo lepiej niż było na początku (IMG:http://forum.php.pl/style_emoticons/default/czarodziej.gif)
Trzeba pisać, pisać i pisać.. wszystko opiera się na doświadczeniu.

2) proponuję kupić sobie tą książkę. Na pewno Ci się przyda, bo wszystko u czym tutaj pobieżnie pisaliśmy, w tej książce jest opisane od podstaw, zaczynając od wzorców projektowych i ciekawych metod programowania obiektowego.

[Black-Berry]

boje się że mi życia na naukę braknie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Dzięki wielkie. Zaopatrzę się w nią bo doszedłem chyba do etapu w którym trzeba zmienić swój punkt widzenia o programowaniu w PHP.

Pozdrawiam.

Ten post edytował Black-Berry 17.02.2008, 17:01:24