Przekazanie klasy do innej klasy, Przekazanie klasy do innej klasy Opcje

[vadergb]

Dotychczas piszac skrypty np. chcac przekazac dostepne rzeczy z klasy obslugi bazy danych robilem to na zasadzie przekazania jej do nastepnej.

cos typu:
$klasa = news baza();
$klasa2 = news baza($klasa);

i potem np w klasa2:

[PHP] pobierz, plaintext 
class klasa2 {
	private $mysqli;
	public function __construct(mysqli $mysqli) {
		$this->mysqli = $mysqli;
	}
	public function findAllNews() {
 
		$result = $this->mysqli->query ( "SELECT * FROM my_news where title='DEU' limit 2" );
		$result_arr = array ();
		while ( $data = $result->fetch_object () ) {
			$result_arr [] = $data;
		}
		return $result_arr;
 
	}
}
[PHP] pobierz, plaintext 

Czy jest to najwydajniejszy sposob?

Czy jest mozliwosc zrobienia klasy publicznej nie uzywajac singletona?

Przy wiekszej ilosci klas do przekazania robi sie balagan:

np. trzeba przekazac klasy: obsluga bazy danych, obsluga $_POST (itp.), Klasa obslugujaca Sesje, Klasa ACL - i jeszcze kilka by sie znalazlo...
Te klasy powinny byc tak na dobra sprawe dostepne prawie wszedzie w kazdej innej klasi - Czy jest jakis inny sposob na przekazanie jej do innej klasy innym sposobem niz singleton?

[LBO]

Przy wiekszej ilosci klas do przekazania robi sie balagan:

Nie jeżeli umieścisz je w jednym obiekcie. wtedy przekazujesz tylko jego.

[vadergb]

Nie jeżeli umieścisz je w jednym obiekcie. wtedy przekazujesz tylko jego.

[PHP] pobierz, plaintext 
$klasa1=new baza();
$klasa3=new session();
$klasa3=new acl();
$klasa4=new laczklasy($klasa1, $klasa2, $klasa3);
$klasa5=new controller($klasa4);
[PHP] pobierz, plaintext 

hmm czyli chodzi o cos takiego?
klasa laczklasy - laczy te klasy w jedna i przekazuje kontrollerowi?

To jest najwydajniejszy sposob(nie chodzi o szybkosc tworzenia aplikacji - wiadomo singleton ulatwia).?

[Fifi209]

Najprościej to zrobić klasę statyczną, która będzie "pojemnikiem" na Twoje obiekty. Wtedy nic nie musisz mieć w konstruktorze jako parametr.

[vadergb]

Najprościej to zrobić klasę statyczną, która będzie "pojemnikiem" na Twoje obiekty. Wtedy nic nie musisz mieć w konstruktorze jako parametr.

Zrobilem cos takiego:

[PHP] pobierz, plaintext 
class join {
	static $vars = array ();
	public function __set( $index,  $value) {
		join::$vars [$index] = $value;
	}
}
[PHP] pobierz, plaintext 

Uzywam:

[PHP] pobierz, plaintext 
		$mysqli = new mysqli ( "localhost", "root", "pass", "nazwabazy" );
		$httprequest = new HTTPRequest ( );
		$template = new template ( );
		$join = new join();
		$join->template=$template;
		$join->httprequest=$httprequest;
		$join->mysqli=$mysqli;
[PHP] pobierz, plaintext 

I teraz gdy uzyje:

[PHP] pobierz, plaintext 
print_r(join::$vars);
[PHP] pobierz, plaintext 

To wypisuje my wszystkie objekty gdzie chce. Mozna zrobic podzial do jakiej klasy jakie obiekty maja przychodzic.
To dobra droga?

Czy chodzi o cos innego?

[Fifi209]

Bardziej o coś takiego:

[PHP] pobierz, plaintext 
class globals {
 
static private $vars;
 
static public function _get($name) {
if (isset(self::$vars[$name])) {
return self::$vars[$name];
}else{
return false;
}
}
 
static public function _set($name, $value) {
return self::$vars[$name] = $value;
}
}
[PHP] pobierz, plaintext 

i potem używasz w innej klasie:

[PHP] pobierz, plaintext 
globals::_set('zmienna', 'test');
[PHP] pobierz, plaintext 

Ten post edytował fifi209 17.08.2009, 14:28:59

[vadergb]

Bardziej o coś takiego:

[PHP] pobierz, plaintext 
class globals {
 
static private $vars;
 
static public function _get($name) {
if (isset(self::$vars[$name])) {
return self::$vars[$name];
}else{
return false;
}
}
 
static public function _set($name, $value) {
return self::$vars[$name] = $value;
}
}
[PHP] pobierz, plaintext 

i potem używasz w innej klasie:

[PHP] pobierz, plaintext 
globals::_set('zmienna', 'test');
[PHP] pobierz, plaintext 

hmm ale mowimy o zapisywaniu w tej klasie objektow nie zmiennych.
zrobimy np

[PHP] pobierz, plaintext 
globals::_set('obiekt1', $object);
[PHP] pobierz, plaintext 

a potem:

[PHP] pobierz, plaintext 
$obiekt1=globals::_get('obiekt1');
[PHP] pobierz, plaintext 

i w $obiekt1 mamy juz obiekt ktory chcemy.

Ale jest sens pchac w to nowa klase - przez to chyba skrypt traci na wydajnosci niz jakbysmy robili to za pomoca tradycyjnej metody:

[PHP] pobierz, plaintext 
$obiekt1=new obiekt();
$obiekt2=new obiekt1($obiekt1);
[PHP] pobierz, plaintext 

Zalezy mi na zeby jak najbardziej to bylo wydajne. To nie ma byc pod budowe xxx stron skrypt - tylko pod budowe okreslonej strony ktora docelowa bedzie odwiedzac sporo grupa ludzi.

pozdrawiam

[Fifi209]

hmm ale mowimy o zapisywaniu w tej klasie objektow nie zmiennych.
zrobimy np

[PHP] pobierz, plaintext 
globals::_set('obiekt1', $object);
[PHP] pobierz, plaintext 

No obiektów, a to że pokazałem jak zapisać zmienną to inna sprawa... Obiekt też się tam zapisze.

Ale jest sens pchac w to nowa klase - przez to chyba skrypt traci na wydajnosci niz jakbysmy robili to za pomoca tradycyjnej metody:

[PHP] pobierz, plaintext 
$obiekt1=new obiekt();
$obiekt2=new obiekt1($obiekt1);
[PHP] pobierz, plaintext 

Zalezy mi na zeby jak najbardziej to bylo wydajne. To nie ma byc pod budowe xxx stron skrypt - tylko pod budowe okreslonej strony ktora docelowa bedzie odwiedzac sporo grupa ludzi.

pozdrawiam

To zastanów się co chcesz. Powiedziałem Ci jak to prościej zrobić, ew. możesz singletona wrzucić.

[marcio]

Sam loader mozna zrobic tak jak pisza wyzej za pomoca singletona ale to tez na wszystkie obiekty klas nie moze byc singleton dlatego ja uzywam wlasny import klas:

[PHP] pobierz, plaintext 
 
<?php
 
class Imports {
 
 
 $cfg = Loader::load('Cfg');
 
 
  private $PathImports = array(
  
                 $cfg -> PathControllers,
 			 $cfg -> PathLibs,
 			 $cfg -> PathModels,
 			 $cfg -> PathModules.$cfg -> PathControllers,
 			 $cfg -> PathModules.$cfg -> PathModels,
 			 $cfg -> PathModules.$cfg -> PathViews,
 			 $cfg -> PathPlugins.$cfg -> PathControllers,
 			 $cfg -> PathPlugins.$cfg -> PathModels,
 			 $cfg -> PathPlugins.$cfg -> PathViews,
 			 $cfg -> PathConfig
 			 
  
                      );
 
 public  function __get($class) {
 
  foreach($this -> PathImports as $imports) {
  
   if(file_exists($imports.$class.'.php')) {
  
    require_once($imports.$class.'.php');
 
   }
   
   else if(file_exists($imports.$class.'_Model'.'.php')) {
   
     require_once($imports.$class.'_Model'.'.php');
   
   }
 
  }
 
 if(class_exists($class) || class_exists($class.'_Model'))
 
  $obj = new $class();
 
  
    return $obj;
 
 }
 
}
 
?>
 
 
 
[PHP] pobierz, plaintext 

Gdzie glowny plik rozruchowy czyli index.php go includuje a glowny kontroler dziedziczy po tej klasie i mam dostep do kazdej klasy robiac:

[PHP] pobierz, plaintext 
 
$this -> Klasa -> Metoda();
 
 
 
//ewentualnie jesli klasa nie ma dziedziczyc po Klasie imports, lub mozna przekazac obiekt do jakiejs funkcji lub cos 
 
 
 
$import = new Imports;
 
$import -> Klasa -> metoda();
 
[PHP] pobierz, plaintext 

Mnie to bardzo pasuje.

[vadergb]

hmm a jakby zrobil cos w stylu:

[PHP] pobierz, plaintext 
$obiekt1=new obiekt();
$obiekt2=new obiekt1($obiekt1);
$obiekt3=new obiekt2($obiekt2);
 
[PHP] pobierz, plaintext 

Klasy:

[PHP] pobierz, plaintext 
class obiekt{
	public function __construct(klasa $obiekt1) {
 
	}
	public function show() {
		echo 'a';
	}
...
}
class obiekt1{
	private $obiekt1;
	public function __construct(klasa $obiekt1) {
		$this->obiekt1= $obiekt1;
 
	}
...
 
}
class obiekt2{
	private $obiekt2;
	public function __construct(klasa $obiekt2) {
		$this->obiekt2= $obiekt2;
		$this->obiekt2->obiekt1->show();
	}
...
 
}
[PHP] pobierz, plaintext 

tylko ze potem wywolac cos z obiektu w obiekcie 2 to trzeba robic : $this->obiekt2->obiekt1->show(); co wydluza kod. Moze daloby sie go jakos skrocic?


Moze ze cos takiego:
register class

[PHP] pobierz, plaintext 
<?php
 
Class Registry {
 
 private $vars = array();
 
 
 public function __set($index, $value)
 {
	$this->vars[$index] = $value;
 }
 
 public function __get($index)
 {
	return $this->vars[$index];
 }
 
 
}
 
?>
[PHP] pobierz, plaintext 

index i 3 klasy

[PHP] pobierz, plaintext 
class obiekt {
		public $mysqli;
		protected $registry;
 
		function __construct( $registry) {
			$this->registry = $registry;
 
		}
 
	}
	class obiekt1 {
		protected $registry;
 
		function __construct( $registry) {
			$this->registry = $registry;
 
		}
		public function show() {
			echo 'a';
		}
 
	}
	class obiekt2 {
		protected $registry;
		public function __construct( $registry) {
			$this->registry = $registry;
 
		}
	}
 
	$registry = new registry ( );
	$registry->obiekt = new obiekt ( $registry );
	$registry->obiekt1 = new obiekt1 ( $registry );
 
	$registry->obiekt2 = new obiekt2 ( $registry );
	$registry->obiekt1->show();
[PHP] pobierz, plaintext 

Dzieki temu nie musimy robic:$registry->obiekt2->obiekt1->show(); itd.
odwolujemy sie tylko $register->nazwaklasy->funkcja();

Wydaje sie pomysl z register chyba najbardziej optymalny i najlepszy. nie musimy nigdzie martwic sie o klasy. Wszedzie je mamy. Teraz pytanie czy jest on wydajny?



Ten post edytował vadergb 18.08.2009, 14:05:37

[marcio]

1 sposob jest do bani.



Drugi jesli dziala jest ok i czemu nie mialby byc wydajny choc wedlug mnie i tak zabardzo masz to przekombinowane.

[vadergb]

1 sposob jest do bani.



Drugi jesli dziala jest ok i czemu nie mialby byc wydajny choc wedlug mnie i tak zabardzo masz to przekombinowane.

Hmm dlaczego 1 sposob akurat jest do bani?

2 sposob ma duzo wspolnego z http://forum.php.pl/index.php?showtopic=122586&hl=
wiec nie wiem czy to jest dobre ... slyszalem ze singletona powinno sie omijac...

[Whisller]

Zobacz sobie Dependency Injection - komponent symfony.