Zmienne globalne Opcje

[tmk]

Witam wszystkich

robie serwis skladający się z wieleu pod stron, w jednym pliku definiuję sobie zmienne przechowujące dane takie jak: nazwa bazy danych, login, hasło, użytkownik bazy itp.
Żeby te dane były dostepnę w funkcjach, potrzebuję zmiennych globalnych. I teraz pytanie, czy byłoby bezpieczne zrobienie tych wszystlich zmiennych jako global? Chodzi mi o

[PHP] pobierz, plaintext 
<?php
global $baza, $username, $user_pass ...;
?>
[PHP] pobierz, plaintext 

jakie z tego moga wynikać zagrożenia?

[tomekp]

Rozumiem, że chcesz wykorzystać zmienne globalne tylko do przerzucenia danych logowania do bazy, wydaje mi się, że nie ma w tym nic niebezpiecznego.
Ja mam takie dwie funkcje do łączenia z bazą i wykonywania zapytania:

[PHP] pobierz, plaintext 
<?php
function connection() {
	global $conn;
	global $cfg;
	if ($conn) {
		return $conn;
	} else {
		$conn = mysql_connect($cfg['mysql']['host'],$cfg['mysql']['login'],$cfg['mysql']['pass']);
 
		if (!$conn || !mysql_select_db($cfg['mysql']['db'],$conn)) {
			echo("Połączenie z bazą jest niemożliwe, skontaktuj się z Administratorem.");
		} else {
			return $conn;
		}
	}
}
 
function sql($query) {
	if(!($conn = connection())) {
		echo("Połączenie z bazą jest niemożliwe, skontaktuj się z Administratorem.");
	} else {
		$result = mysql_query($query,$conn);
		if(!$result) {
			echo("<b>Zapytanie :</b><br>['$query']<br>");
			echo("<hr color='black'><b>Błąd :</b><br>[".mysql_error()."]");
			echo("<hr color='black'>");
		} else {
			return $result;
		}
	}
}
?>
[PHP] pobierz, plaintext 

[Cysiaczek]

Unikaj zmiennych globalnych jak ognia !
Jeśli już potrzebujesz takich informacji, to najlepiej zainteresuj się obiektami i czymś, co nazywa się wzorcem 'singleton'
To trochę bezpieczniejsza forma zmiennych globalnych (nawet, jeśli nie chcesz projektowa i programowa obiektowo)

[tomekp]

Dlaczego unikać zmiennych globalnych jak ognia?

[Prph]

A dlaczego zmienne a nie stale?
Rzeczy takie jak nazwa bazy, haslo itp to raczej stale:

[PHP] pobierz, plaintext 
<?php
define('_DB_HOST', 'localhost');
...
 
mysql_connect(_DB_HOST, .., .., ..);
?>
[PHP] pobierz, plaintext 

W ogole jak na to spojrzec bardziej profesjonalnie, to sa to dane konfiguracyjne. Warto wiec napisac Klase np. Registry i uzywac:

[PHP] pobierz, plaintext 
<?php
mysql_connect(Registry::getEntry('db_host'));
?>
[PHP] pobierz, plaintext 

Pozdrawiam.

[Cysiaczek]

Należy ich unikac, bo są niebezpieczne dla zdrowia twoich aplikacji (możesz je np. przypadkowo nadpisac i będziesz miał problem).
Prph ma rację - dużo lepszym wyjściem jest użycie stałych lub ja mówiłem singletonów

[tomekp]

Użycie stałych wygląda rzeczywiście dobrze, a co to są te singletony?

[Prph]

Singleton to sposob na stworzenie klasy takiej, aby w aplikacji mozna bylo utowrzyc jej tylko jeden obiekt.

Co to znaczy? Ano nie zrobisz $obiekt = new MojaKlasa();

A dlaczego?

1. Kontruktor klasy powinien byc prywatny (powinien, bo jak uzywasz php4, to nie mozesz tego okreslic).
2. Obiekty tworzysz za pomoca innej metody, najczesniej nazwanej getInstance.

Przyklad:

[PHP] pobierz, plaintext 
<?php
class Klasa
{
  private static $_oInstance;
 
  private function __construct()
  {
	// tu robisz to co w normalnym konstruktorze chcialbys zrobic.
  }
 
  public function getInstance()
  {
	if(!isset(self::$_oInstance))
	{
	  $sClassName = __CLASS__;
	  self::$_oInstance = new $sClassName;
	}
 
	return self::$_oInstance;
  }
}
?>
[PHP] pobierz, plaintext 

Jak widzisz getInstace sprawdza czy obiekt juz istnieje. Jezeli nie: tworzy go i zwraca. Jezeli instnieje: zwraca go.

A jak tego uzywac?

[PHP] pobierz, plaintext 
<?php
$obiekt = Klasa(); // zwroci blad!
 
$obiekt = Klasa::getInstance(); // tak jest dobrze
?>
[PHP] pobierz, plaintext 

Pamietajac, o tym ,ze klasy sa dostepne w kazdym pliku, poniewaz ich nazwy sa globalne, mozesz utworzyc tzw instancje (egzemplarz klasy - obiekt) w dowolnym miejscu.... Oczywiscie jezeli plik z klasa zostal wczesniej dolaczony.

O singletonie przeczytasz takze na php.net.

PS. Dodam jeszcze, ze ten rodzaj klas stosujesz tak, gdzie nie chcesz aby w aplikacji pojawil sie drugi obiekt tej klasy. Przyklad? Baza danych - jedna baza danych wystarczy w aplikacji. Inny? - cos co steruje aplikacja - jądro - dwa jądra? Hyh tylko u istot zywych

Pozdrawiam.

Ten post edytował Prph 24.05.2006, 17:35:35