Dziedziczenie PHP class Opcje

[metalog]

Witam.
Mam plik z klasą baza są tam fukcje(polacz,zamknij itd) do obsługi bazy danych.
Robię panel też na klasach.
W każdej klasie muszę dziedziczyć klasę baza. ( class cmsArtykuly extends baza{ })
Chodzi oto ze za każdym razem kiedy wykonywana jest jakaś klasa. konstruktor i destruktor łaczy się i rozłancza z bazą co nie jest za wydajne.

[PHP] pobierz, plaintext 
<?php
function __construct($db){						
						$this->polacz($db);
 
				}
function __destruct(){						
						$this->zamknij();
 
				}
?>
[PHP] pobierz, plaintext 

Moje pytanie: czy da się to zrobić zeby połaczenie się trzymało i nie trzeba było za każdym razem ładzyć się i rozłanczać

[mike]

Ogromny błąd, który popełniasz to dziedziczenie za każdym razem po tej klasie.
Nie rób tego. To postępowanie jest pozbawione sensu.

[l0ud]

Zła metoda, zdecydowanie. Powinieneś na początku utworzyć jeden obiekt łączący się z bazą, później w określony sposób przekazywać jego instancję do każdego innego obiektu wymagającego połączenia z bazą.

[metalog]

Może ktoś podać przykład takiego rozwiązania?
Albo jakieś wskazówki. ?

[dr_bonzo]

np. w najprostszej postaci

[PHP] pobierz, plaintext 
<?php
$module = new cmsArtykuly( $polaczenie_z_baza )
?>
[PHP] pobierz, plaintext 

[metalog]

Czy to nie jest ten sam sposób co wyżej? i co ma być(obiekt czy co) w $polaczenie_z_baza

[Gecco]

skorzystaj z singletona, wtedy bedziesz mial jedna instancje klasy

[mike]

Czy to nie jest ten sam sposób co wyżej? i co ma być(obiekt czy co) w $polaczenie_z_baza

$polaczenie_z_baza to obiekt klasy (w Twoim przypadku) baza. Rozwiązanie jest na tyle inne że tworzysz obiekt (który łączy się z bazą) tylko raz, a później przekazujesz go każdej potrzebującej klasie.
Klasa kiedy dostała obiekt, już nie musi łączyć się z bazą tylko takie połącznie pobrać przekazanego obiektu.
Na przykład:

[PHP] pobierz, plaintext 
<?php
 
$polaczenie_z_baza->getConnection();
 
?>
[PHP] pobierz, plaintext 

i na połącznieu działać dalej.

[Babcia@Stefa]

Spróbuj w sesji zapisywać stan połączenia z bazą danych, jeśli połączenie jest aktywne to nie zostanie ponownie ustanowione oraz zamknięte.
A na końcu kodu będziesz musiał niestety sam zamykać połączenie, chyba że piszesz w OOP to nie będziesz miał problemu.

@edit
Jak widzisz lepsze rozwiązanie, to je przedstaw (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Sesja jako global też może być

@edit
Nie serializować!
Tylko zapisywać że jest połączenie...
Gdy łączysz się poprzez mysql_connect() to połączenie jest GLOBALNE.

Dziękuję, Babcia@Stefa

Ten post edytował Babcia@Stefa 25.06.2008, 20:23:36

[dr_bonzo]

LOL, w sesji?
- albo nie wiesz ze nie da sie polaczenia z baza zaserializowac
- albo chcesz $_SESSION jako globala uzywac

[metalog]

Zrobiłem tak jak mówiliście:
na początku index.php:

[PHP] pobierz, plaintext 
<?php
$baza=new baza($baza);
$baza->polacz($db);
 
a na końcu:
 
$baza->zamknij();
?>
[PHP] pobierz, plaintext 

w pliku cmsDokumenty.php

[PHP] pobierz, plaintext 
<?php
class cmsDokument {
function pokazDokumenty($znacznik){
$zap="zapytanie....";
$baza->zapytanie($zap);
													  }
							  }
?>
[PHP] pobierz, plaintext 

w pliku pokazDokumenty.php:

[PHP] pobierz, plaintext 
<?php
.... 
$cmsDokument=new cmsDokument($baza);
$cmsDokument->pokazDokumenty($_GET['znacznik']);
......
?>
[PHP] pobierz, plaintext 

i error Call to a member function zapytanie() on a non-object in [....]/cmsDokumenty.php

Ten post edytował metalog 26.06.2008, 07:42:38

[l0ud]

A gdzie konstruktor tej klasy, co przechwytuje obiekt bazy?

PHP4:

[PHP] pobierz, plaintext 
<?php
class cmsDokument {
 
var $baza;
 
function cmsDokument($baza) {
$this->baza = $baza;
}
 
function pokazDokumenty($znacznik){
$zap="zapytanie....";
$this->baza->zapytanie($zap);
//jakaś dalsza akcja
}
 
}
?>
[PHP] pobierz, plaintext 

PHP5:

[PHP] pobierz, plaintext 
<?php
class cmsDokument {
 
private $baza;
 
public function __construct($baza) {
$this->baza = $baza;
}
 
public function pokazDokumenty($znacznik){
$zap="zapytanie....";
$this->baza->zapytanie($zap);
//jakaś dalsza akcja
}
 
}
?>
[PHP] pobierz, plaintext 

i wywal to : $baza->zamknij(); na końcu. Nie musisz ręcznie zamykać połączenia.

Ten post edytował l0ud 26.06.2008, 11:15:42

[metalog]

Jest ok już nie wyświetla się błąd ze niema funkcji zapytanie lecz nie zwraca mi ta funkcja wyników :| nie wiecie czemu tak może być? pewno to związane jest z obiektem baza.

[Cysiaczek]

Widzisz tam gdzieś return w tej funkcji? Ja nie, więc pewnie dlatego nic nie zwraca.

Pozdrawiam.

[jezoo]

a czy mozna w ten sposob, tzn czy to jest bezpieczna metoda polaczenia z baza

[PHP] pobierz, plaintext 
<?php
class DatabaseConnector {
 
		private static $instance;
 
		public function __construct(){}
 
		public static function getInstance(){
 
				if(!is_object($instance))
						$this->instance = new DatabaseConnector();
				return $this->instance;
		}
 
		public function connect(){
				pg_connect("host='".Constants::$DB_HOST."'
						user='".Constants::$DB_USER."' password='".Constants::$DB_PASSWORD."'
						port='".Constants::$DB_PORT."' dbname='".Constants::$DB_NAME."'")
				or die("Błąd połączenia z baza danych!");
		}
 
		public function complete($arg) {
				pg_query($arg);
				pg_close();
		}
}
?>
[PHP] pobierz, plaintext 

[wlamywacz]

Wydaje mi się że Twoja aplikacja jest źle zaprojektowana i tym samym źle napisana.

[bim2]

Hmm, trzymaj takie rzeczy (jak baza, obsługa błędów, języki itd.) "globalnie". Czyli tak, żebyś miał do tego dostęp wszędzie. Ja przekazuje je do głównego kontrolera, który dodaje ich instancje przy tworzeniu klasy akcji np.

[PHP] pobierz, plaintext 
<?php
$oUser = new Action_User;
$oUser->oDb = $obiekt_bazy;
?>
[PHP] pobierz, plaintext 

To tylko taki przykład, ale powinieneś mieć dostep do bazy tam gdzie potrzebujesz. Nie rób nic na globalu, bo nie ma sensu dawać całej aplikacji dostęp do bazy. :]

[metalog]

Mam takie cos:

[PHP] pobierz, plaintext 
<?php
....
  class cmsLinki {
 
				var $prefix='';
				var $linkarray=array();
				private $baza;
 
				public function __construct($baza) 
						{
							$this->baza = $baza;
 
							}
 
				function pokazLinki($znacznik){
 
						$zap="select * from linki";
 
						$this->$baza->zapytanie($zap);
...
?>
[PHP] pobierz, plaintext 

Wywołanie:

[PHP] pobierz, plaintext 
<?php
$baza=new baza();
$baza->polacz($db);
$cmsLinki = new cmsLinki();
$cmsLinki->baza = $baza;
$cmsLinki->pokazlinka($_GET['ida']);
?>
[PHP] pobierz, plaintext 

I wyskakuje mi Fatal error: Cannot access empty propert

Ten post edytował batman 2.07.2008, 10:55:45

Powód edycji: poprawiłem bbcode

[mike]

Jak cytujesz błed to cytuj go w całości, bo przez niewiedzę możesz obciąć najbardziej istotne informacje.

A co do kodu. Odwołujesz się do pola klasy $baza w sposób publiczny a zmienna ta została zadeklarowana jako prywatna.
To błąd.

Poza tym dlaczego nie przekazałeś bazy w konstruktorze? Przecież po to chyba go tak skonstruowałeś.

Poza tym piszesz niechlujnie. Raz dobrze: $this->baza->...
raz źle $this->$baza->....