AdoDB + __construct(), można tak? zadziała? Opcje

[Aztech]

Pytanko: mam klasę a w niej wiadomo konstruktor. Czy tworzenie w nim połączenia z AdoDB jest dobrym pomysłem (by nie musieć się łączyć za każdym razem w wywołaniu metody update, create, alter, delete z bazą danych?

[PHP] pobierz, plaintext 
<?php
 
include("adodb.inc.php");
 
class TUser {
 
 
/*
pola prywatne
*/
 
public function __construct() {
  $db = NewADOConnection('mysql');
  $db->Connect("localhost", "root", "password", "mydb");
}
 
public function updateUser(){
/*
kod
*/
$db->Execute(/* formuła na update */)
}
 
public function createUser(){
/*
kod
*/
$db->Execute(/* formuła na tworzenie usera */)
}
 
public function deleteUser(){
/*
kod
*/
 
$db->Execute(/* formuła na usuwanie */)
}
 
}
 
?>
[PHP] pobierz, plaintext 

[mike]

Przechowywanie w polu prywatnym połączenia z bazą danych to bardzo dobry pomysł.

BTW:
Chyba zresztą każdy tak robi.

[Aztech]

raczej chodziło mi o to żeby nie robić czegoś takiego:

[PHP] pobierz, plaintext 
<?php
 
include("adodb.inc.php");
 
class TUser {
 
 
/*
pola prywatne
*/
 
public function __construct() {
/*
kod
*/
}
 
public function updateUser(){
/*
kod
*/
$db = NewADOConnection('mysql');
$db->Connect("localhost", "root", "password", "mydb");
$db->Execute(/* formuła na update */)
}
 
public function createUser(){
/*
kod
*/
$db = NewADOConnection('mysql');
$db->Connect("localhost", "root", "password", "mydb");
$db->Execute(/* formuła na tworzenie usera */)
}
 
public function deleteUser(){
/*
kod
*/
$db = NewADOConnection('mysql');
$db->Connect("localhost", "root", "password", "mydb");
$db->Execute(/* formuła na usuwanie */)
}
 
}
 
 
?>
[PHP] pobierz, plaintext 

zawsze można też:

[PHP] pobierz, plaintext 
<?php
 
public function connect($db) {
  $db = NewADOConnection('mysql');
  $db->Connect("localhost", "root", "password", "mydb");
}
 
?>
[PHP] pobierz, plaintext 

i potem w miejsce

[PHP] pobierz, plaintext 
<?php
 
  $db = NewADOConnection('mysql');
  $db->Connect("localhost", "root", "password", "mydb");
 
?>
[PHP] pobierz, plaintext 

dajemy

[PHP] pobierz, plaintext 
<?php
 
 $this->connect($db);
 
?>
[PHP] pobierz, plaintext 

więc pytanie, czy nie lepiej w konstruktorze a jeśli nie to dlaczego?

[mike]

W skrócie:

[PHP] pobierz, plaintext 
<?php
 
include("adodb.inc.php");
 
class TUser {
 
 
/*
pola prywatne
*/
private $db = null;
 
public function __construct() {
	$db = NewADOConnection('mysql');
	$this->db = $db->Connect("localhost", "root", "password", "mydb");
 
}
 
public function updateUser(){
/*
kod
*/
$this->db->Execute(/* formuła na update */)
}
 
public function createUser(){
/*
kod
*/
$this->db->Execute(/* formuła na tworzenie usera */)
}
 
public function deleteUser(){
/*
kod
*/
$this->db->Execute(/* formuła na usuwanie */)
}
 
}
 
 
?>
[PHP] pobierz, plaintext 

[Aztech]

Dzięki bardzo, właśnie o to mi chodziło, czy tak wolno.
EOT

[ActivePlayer]

a na moje oko tak najpoprawniej:

[PHP] pobierz, plaintext 
<?php
 
include("adodb.inc.php");
 
class TUser {
 
 
/*
pola prywatne
*/
private $db = null;
 
public function __construct(&$db) {
	$this->db = $db;
 
}
//...
}
$db = NewADOConnection('mysql');
$db->Connect("localhost", "root", "password", "mydb");
$user = new TUser(&$db);
 
?>
[PHP] pobierz, plaintext 

moglem cos z referencjami popaprac bo na oczka nie widze...

[dr_bonzo]

Dokładnie - w php5 obiekty przekazywane sa jako referencje -- wiec wszystkie '&' sa niepotrzebne. A "TUser(&$db);" to nawet dostaniesz warninga/notice'a.

[krzysztof f.]

Można też tak:

[PHP] pobierz, plaintext 
<?php
class TUser {
   /**
	* Połączenie z bazą danych
	* 
	* @var Connection
	* @access protected
	* @static
	**/
   protected static $oDbConn;
 
   /**
	* Konstruktor 
	*
	* Ustwienie obiektu połączenia z bazą danych
	*
	* @return TUser
	* @access public
	* @throws RegistryException
	* @throws SQLException 
	**/
   public function __construct()
   {
	  try
	  {
		 $this->oDbConn = ServiceLocator::getInstance()->getDbConn();
	  }
	  catch ( RegistryException $oE )
	  {
		 throw $oE;
	  }
	  catch ( DbException $oE )
	  {
		 throw $oE;
	  }
   }
}
?>
[PHP] pobierz, plaintext 

Gdzie ServiceLocator http://java.sun.com/blueprints/corej2eepat...iceLocator.html to klasa odpowiedzialna za wyszukiwanie usług powszechnie używanych w obrębie całej palikacji. Możemy ją zaimplementować w najprostrzy sposób jako rejestr http://www.phppatterns.com/docs/design/the_registry

Ten post edytował krzysztof f. 7.12.2005, 10:01:56

[NuLL]

Nie lepiej singleton :?:

[PHP] pobierz, plaintext 
<?
class costam
{
public function jakasMetoda()
{
$result=db::instance()->query("jakies zapytanie do sql-a");
}
}
?>
[PHP] pobierz, plaintext