Chcialbym sie dowiedziec czmu wszyscu na forum tak bardzo nie lubia global w klasach i funkcjach.
Jak oladalem sobie kod typo3 to tam jest pelno globalow zreszta tak jest w b. duzej ilosci skryptow ...

Globale są "be" z kilku powodów. Jednym z ważniejszych jest to, że wiążą ze sobą w sposób bardzo ścisły elementy aplikacji, które powinny byc maksymalnie niezależne. potem masz problemy np. z przeniesieniem kodu do innego projektu.
Z takich bardziej przyziemnych powodów, to globale mają tę wadę, że mogą zostac nadpisane w miejscu najmniej spodziewanym i może być kłopot.

no ale jak robie sobie program i nie mam zamiary zeby jego kod byl framewokiem albo uzywany gdzies indziej, to chyba nie jest "be" ?

No dobrze. Możesz tak zrobić i będzie działało Po co jednak pisać klasy, używać obiektów? Jedna z podstawowych zasad OOP jest zasada hermetyzacji - wlicza się w to również stosowanie zmienych globalnych.
Żeby nie było, że zmienne globalne są zupełnie nieużywane, to podam przykład singletona, który też jest pewnego rodzaju daną globalną. Ma tylko trochę mniej wad

Odpowiedz sobie na takie pytania:
- Jak się upewnisz, czy zawartość zmiennej globalnej jest prawidłowa?
- Czy będziesz potrafił zachować spójność danych w tej zmiennej?
- Gdzie będziesz szukał błędów związanych z tą zmienną, skoro wszędzie będziesz miał do niej całkowity dostęp?

Myślę, że nie bez podstaw nikt nie toleruje używania zmiennych globalnych. W przypadku zmian związanych z tą zmienną, cały skrypt Ci się sypie...


To czy coś jest "be", czy nie, nie określa zastosowanie kodu, a jego jakość.


Oj tak, w bardzo dużej ilości skryptów to znajdziesz. Teraz popatrz jak ludzie się rzucili na php, dzięki jego dostępności i prostocie. Popatrz jakie posty znajdują się czasem w przedszkolu... To, że dużo ludzi tego używa, nie znaczy, że jest to prawidłowe zachowanie.

tak samo jak sie upewniam z narmalnymi zmiennymi, a pozatym w globalach to zazwyczaj jest link to $_DB, itd.

Ale chyba TYPO3 nie jest zle, i nie jest robione przez amatorow ?

ja nie probuje udowodnic ze global sa dobre, ale ja ich uzywam np http://crystalpage.sf.net/code , tam zobaczysz o co mi chodzi z globalami jak zobaczysz plik init.php i pliki z katalogu cpLib, i chce sie dowiedziec czy oz tak zle sa czy poprostu nie lubiane bo nie lubiane i koniec .

Tak sądzisz? Zmienne globalne nie mają żadnej kontroli typów. Przekazując zmienną do metody możesz wpłynąć na jej typ i robisz to w jednym miejscu. Potem zapisujesz wartość i wiesz, że zawsze będzie ona poprawna... A co mi z tego, że zmienna nazywa się $_DB, skoro za chwilę może wyglądać identycznie jak $_CP (zupełnie nieintuicyjna nazwa...) albo $string?

$_CP => CrystalPage ale na localu od dawna zmienilem na $_SYS

a co mi z kontroli typow jak ja tam chowam tylko zmienne comfig-u i db?
np. gdzie jest folder z cachem albo user do bazy danych...
tutaj niema co sie bawic w typy w takim przypadku

Moze zdanie o slowie global jest takie ze powinni dac w php.ini mozliwosc wylaczenia tego slowa wogole ze skladni

Typo3 - hmm - zajrzyj sobie do eZ publisha i ew stamtad bierz przyklady.

OT: TYPO3 jest co najmniej 1000 razy lepsze od eZ publish

LOL
Ładniej wygląda czy jest lepiej i funkcjonalniej napisane?
To pierwsze to może i tak
To drugie? ROTFL

Rozumiem, że było fajną rozrywką zmieniać wszelkie odwołania do zmiennej $_CP na $_SYS.

Czyli oczekujesz, że zawsze znajdzie się tam tablica? Czyli jednak potrzebujesz kontroli typów... Zrozum, że do tej zmiennej możesz przypisać co chcesz, gdzie chcesz i kiedy chcesz. Na dobrą sprawę przed każdym użyciem wypadało by sprawdzić, czy wciąż zawartość jest poprawna. Chcę zobaczyć, jak piszesz do tego testy... Bo chyba poważny CMS, powinien być przetestowany?

Takim sposobem otwierasz sobie furtkę do błędów, których nie wyłapią Ci żadne testy. Ja, kiedy przekazuję obiekt z referencją do DAO, wiem, że dostanę wewnątrz klasy zawsze to, czego potrzebuję. Natomiast po twojej zmiennej $_DB nie wiem czego się mam spodziewać? AdoDB? PEAR:B? Propel? PDO? Moje klasy zawsze będą poprawne, bez względu na to, co przekażesz im jako obiekt DAO, bo zawsze interfejs będzie prawidłowy. Twoje zależą od zmiennej globalnej, nad którą nie masz kontroli. W momencie testu nie jesteś w stanie zapewnić, że ta zmienna zawsze będzie poprawna.

Na dobrą sprawę, wywołując dwa zapytania, a pomiędzy nimi jakieś instrukcje, możesz nawet nie dowiedzieć się, że używasz innej bazy danych...

OT: Ty dddddddduuuuuuuuuuzzzzzzzzzzooooooooooo jeszcze musisz sie nauczyc o programowaniu - oj duzo

EOT

A ty jeszcze wiecej...
"Krowa, która dużo ryczy, mało mleka daje"
---------------------------------------------------------------------------

W tym $_DB zawsze jest tylko i wylacznie link to objektu ADO i nie bede tam wstawiac PDO czy innych perdul, a moge byc pewny ze tam jest to co chce bo sa w globalach zachowywane tylko dane z plikow ini ktore sa zbierane za pomoca PEAR::Config, i ja sie nie zbiorol to skrypt nie konczy tylko pokazuje errora ( to co jest uplodowane to jest stare i chcialem zebyc tylko zobaczyl o co mi chodzilo). i z $_DB tak samo jak sie nie poloczy to nie knoczy egzekucji.
Po drugie zarowno $_SYS jak i $_DB sa uzywane tylko przez cms i nikt niema do nich dostepu oprucz samego cms-u.

Rozumiem, że zabronisz ich używania? Czyli nie przewidujesz w swoim systemie rozszerzeń? To cóż to za CMS, którego nie można rozszerzać...

A teraz mam dla Ciebie zagadkę w postaci kodu php.

[PHP] pobierz, plaintext 
<?php
function chaos() {
	global $_DB;
	$db[1] = new PDO('...');
	$db[2] = new AdoDBConnection('...');
	$db[3] = new InnyObiektDAO('...');
	$random = rand(1, 3);
	$_DB = $db[$random];
}
 
$_DB = new PDO('..');
$_DB->query('...');
chaos();
$_DB->query('...');
chaos();
$_DB->query('...');
?>
[PHP] pobierz, plaintext 

Wiem, że przykład jest naciągany, ale dokładnie pokazuje jak "bezpieczne" są zmienne globalne. Uwierz mi, że zawsze, kiedy inna osoba dobiera się do kodu istnieje prawdopodobieństwo, że zrobić coś podobnego, lecz nie w tak dobitny sposób.

Czekam tylko, aż ustosunkujesz się do mojego pytania z testami

LOL

[PHP] pobierz, plaintext 
<?php
$GLOBALS['_SYS']=null;
$GLOBALS['_DB']=null;
?>
[PHP] pobierz, plaintext 

Dopisze sobie w jakims module ( o ile CP bedzie modulowy ) czy szablonie i caly Twoj CMS mozna sobie.... ( dopisac samemu wg wlasnej opinii )

@Ludvik - dobre Ten temat niedlugo bedzie mozna przerzucic do dzialu Humor

NuLL, nie krzycz tylko spokojnie wyjasniaj

Zmienne globalne sabotuja hermetyzacje i głownie o to chodzi. Nie jest to zgodne z aspektami programowania OOP, staja sie przyczyna smietniku w kodzie. Postronny programista nie wie skad nagle dana zmienna sie wziela. Zmienne globalne sa nie tylko omijane z daleka w php ale w kazdym innym jezyku programowania.

Oczywiscie sa tacy ktorzy korzystaja z nich... na wlasna odpowiedzialnosc, ja z takim kodem nie chcialbym pracowac.

Jak dla mnie eZ to: "Krowa, która dużo mleka daje".

A globale z reguły stwarzają dużo kłopotów...

Gdyby globale były super fajne i bezpieczne, to twórcy php nie ustawialiby ich domyślnie na OFF

@nasty_psycho: No to fajne CMSy piszesz, bez możliwości rozbudowy... i jeszcze uważasz, że TYPO3 jest lepsze od eZ Publish :| Poprostu gratuluję

Sa mozliwosci rozbudowy nawet b. duzo jest ok 10 klas do pracowania z rozszezeniami.
ALE:
Jak kots robi extension to nie bedzie chyba wysadzal sobie cms-u albo hackowac. Bedzie caly dokument o tym jak pisac dodatki do cms-u i tam bedzie jasno napisane zeby tych zmiennych nie uzywac.

@Ludvik: tak to jest naciagane i to bardzo , to samo mozna zrobic z kazda zmienna. a $_DB nie jest zmienna typu array, ja w cp uzywam tylko ADO i nic innego
@Strzalek: to nie bylo o eZ, bo eZ jest bardzo dobry, tylko o NuLL.
@tiraeth: nie chce sie powtarzac ale moj cms ma rozbudowane mozliwosci podczepiania extension-ow i ustalania ich rodzaju jak np. Startup, filter czy content i wiele innych.
@NuLL: z tego co widze jak myslisz o budowie modulow to masz sklonnosci samobujcze
-----------
no i wedlug was jak mozna w alternatywny sposob dla kazdej klasy / funkcji dac mozliwosc korzystania z $_DB ? zeby nie bylo balaganu i w kazdej klasie miec public $dblink.

pozdrawiam

No kurcze, przecież to podstawy programowania obiektowego.

• Singleton Pattern
• Singleton

to to znam ale mi chodzi o to w moim kodzie pozmieniac, bo nie chce mi sie zmieniac w kazdej prawie funkcji $_DB na jakies tam linki do singletona, za duzo czasu mi to zabierze.

a tak pozatym to testowalem wszystko u mnie na globalach i jest dobrze w niczym nie przerzkadza.

ps. Joomla (mambo) uzywa global - wordpress tez (wiem ze to nie jest 100% OOP ale ...)

gluwnie to typo3 mnie troche uspokoil bo najpierw sie naprawde matrwilem.

Wciąż nie dotarło do Ciebie moje przesłanie. Dla mnie możesz używać dowolnego typu/interfejsu/klasy bazowej. Ja i tak sobie zrobię z tą zmienną co tylko będę chciał. Ba, nawet nie będę musiał się namęczyć.
Chyba nie powiesz, że ta klasa nie będzie bezpieczna...

[PHP] pobierz, plaintext 
<?php
class Example {
	public function __construct(PDO $db) {
		$this->db = $db;
	}
 
	public function pobierzDane() {
		$result = $pdo->query('...');
		return $result->fetchAll();
	}
 
	protected $db;
}
?>
[PHP] pobierz, plaintext 

ZAWSZE będę miał interfejs, o który mi chodzi... Singleton też zapewni bezpieczeństwo.


Rozumiem, że wierzysz w bezbłędnych programistów. Dobre API poza tym da radę bez żadnej dodatkowej dokumentacji. Opis metod powinien starczyć, a nie rozpisywanie się na temat tego, co można zrobić, a czego nie... Jeżeli czegoś nie można zrobić, to ty masz zapewnić, że to się nie stanie. Kusisz los zostawiając furtkę...

Kieruj się tym, co napisał Vomit, bo ujął to krótko i ładnie. Taka jest właśnie prawda o zmiennych globalnych...


Nieprzemyślany projekt rodzi problemy przy rozwoju. Miało być pro, a jakoś nie wyszło chyba. Nigdy nie wierzyłem, że porzucając jedną wersję i zaczynając pisać drugą, można zmienić styl pisania w tak znaczny sposób.

OT: Popraw bloga. Chwalisz się, że waliduje się, a tak nie jest.

Ale się postów namnożyło
Nasty: Obrona globali nie ma sensu. Świat poszedł do przodu. Programiści dostając do reki nową klasę chcą jej używać, a nie zastanawiać się, czy klasa nie ma gdzieś jakiegoś globala. Sam, kiedy chcę użyć nowej klasy, patrzę najpierw na interfejs. Od razu wiem, co musze przekazać do konstruktora, jakie ewentualnie metody potrzebują informacji z zewnątrz. Bardzo jestem zadowolony, kiedy mogę przekazać wartości do konstruktora i cieszyć się z używania getCostam() i setCostam(). Gdybym musiał przejmować się tym, że jakaś metoda w klasie używa globala i muszę zadbać o dostarczenie jego wartosci, to przestaje się robic wesoło :/

Piszesz, że nie chcesz robić frameworka. W porządku. Pisz dla siebie. Zauważ, że sam zwróciłes uwagę na to, że nie chce Ci się zmieniać X wystąpień twojej zmiennej na Singletona. To jest właśnie jedna z nieporządanych cech klasy/aplikacji - olbrzymia ilość miejsc, w których należy dokonac poprawek, jeśli coś zmieniasz. Nieużywanie zmiennych globalnych redukuje ten proces do niezbędnego minimum (czuję się jakbym książkę cytował ).
Rozumiem, że twój post był próbą zwrócenia uwagii na istnienie tego typu zmiennych w wielu publikacjach, które są przecież używane. Coż - one kiedyś muszą zostać przepisane lub napisane od nowa.

Pozdrawiam

problemy przy rozwoju? u mnie global nie sa problemem i nie mam aktualnie zadnych problemow z rozwojem (cms-u oczywiscie ).
Jutro postaram sie uplodowac to co mam napisane do tej pory.

ogulnie to narazie nie bede zmienial niczego bo jest za duzo pracy przy tym, ale dziekuje za opinje.

pokaz ten Twój CMS ciekaw jestem co to za cudo. Nasto ja robie byki ortograficzne ale Ty bijesz mnie o stokroć.

~acztery: bedzie gotowy za jakis 2-3 misiace ale screeny mozesz zobaczyc na stronie,
a co do ortorafji to **** ***** ***** *****

(poprostu nie chce sie za bardzo powtarzac)

http://wortal.php.pl/wortal/artykuly/php/p...asieg_zmiennych

no strasznie mi pomogles, jestem dozgonnie wdzieczny za tak bardzo solidnie napisany i rozbudowany artukul, myslalem ze nigdzie sie nie dowiem jak pacowac ze zmiennymi

Haha, nie wiem co powiedzieć. Pomijam to że jest bardzo krótki i g**no daje to jeszcze ma błędy w kodzie php



Dziwne, u mnie na localhost wyświetla:



Pozdrawiam

Pokaże Ci ciekawy sposób


I jedizemy tak,
strona.php?_SYS=lol

[PHP] pobierz, plaintext 
<?php
function foo() {
   global $_SYS;
   echo $_SYS;
}
 
foo();
?>
[PHP] pobierz, plaintext 

Wynik


Pomijajac juz inne aspetky bezpieczenstwa

---

No tak, powinno byc On nie Off ale to tylko dla tego że kopiowałem z mojego php.ini i zapomniałem zamienic. Przykład był wymyslany jako najprostrzy jaki wpadł mi w tym momecie do głowy, ale obrazuje co sie moze stac... jego zadaniem nie bylo pokazania jakiegos "super" zagrozenia tylko ujawnienie ze ono jest mozliwe(!) i dlaczego nie powinno sie stosowac global. Taka sytuacja nie miala by miejsca gdyby obiekt byl tworzony wewnatrz jakiegos obiektu.

Także przykład ma działać "na wyobraznie" a nie do gdybania co by było gdyby...

~hwao: Przekonales mnie (naprawde!), i dzieki, zaczolem zmieniac powoli $_DB;

jak skoncze to dam ci znac i dasz mi kolejne uwagi, ok ?

fipaj i hwao zwrocili ostatnio uwage na wzorzec Registry. W jednej z ksiazek przeczytalem, ze jest to zwykly Singleton.

Chciałbym jednak zapytac czemuby nie zrobic takiego API? Powinno zdialac tak samo a nie korzystamy z Singletona.

[PHP] pobierz, plaintext 
<?php
class MyRegistry
{
	static private $aVars = array();
 
	static public function setVar( $var, $value )
	{
		   MyRegistry::$aVars[$var] = $value;
	}
 
	static public function getVar( $var )
	{
		   return MyRegistry::$aVars[$var];
	}
}
?>
[PHP] pobierz, plaintext 

~nasty_psycho, zastanawiam się tylko gdzie zostawiłeś narząd, który nazywa się M-Ó-Z-G-I-E-M. Jak byś chociaż przeczytał zawartość tego artykułu to być zrozumiał istotę keyword zwanego jako global" title="Zobacz w manualu php" target="_manual (ostatni kod w tym artykule). A tak to mogę tylko Ci pogratulować umiejętności czytania ze zrozumieniem

~tiraeth: ja zobaczylem tylko jak "wyglada" ten artykul to az mi sie slabo zrobilo, i mam takie pytanie: czy ty rozumiesz co tu sie pisze w tym topicu? bo chyba nie za bardzo ... :

Przeciez caly topic jest o globalach, a ty mnie odsylasz do tekstu gdzie jest napisane co to jest global !

Edit: Wracając do posta hwao, który jest błędny.

URL: blabla.php?_SYS=lol

[PHP] pobierz, plaintext 
<?php
function foo()
{
   global $_SYS;
   echo $_SYS;
}
 
foo();
?>
[PHP] pobierz, plaintext 

Returns:

[PHP] pobierz, plaintext 
<?php
function foo()
{
   global $_SYS;
   echo $_SYS;
}
 
foo();
?>
[PHP] pobierz, plaintext 

Returns:




URL: blabla.php

[PHP] pobierz, plaintext 
<?php
$_SYS = 'rotfl';
function foo()
{
   global $_SYS;
   echo $_SYS;
}
 
foo();
?>
[PHP] pobierz, plaintext 

Returns:


Tak to wygląda (przed chwilą sprawdzałem...)

ok, koniec klutni !
Postanowielm ze poprawie, i mozecie zobaczy cesciowe efekty poprawien ( bo nie bardzo mialem czasu) na http://crystalpage.sf.net/code/ , tak lepjej ?

Taa.....

Ja Ci moge pokazac jak wyglada moj stosunek do zmiennych superglobalnych i globalnych w moim frameworku ktory wykorzystuje na codzien w pracy ? O tak

[PHP] pobierz, plaintext 
<?php
class webRequest
{
//kod
 
//singleton
private function __construct(){
//kod
unset($_POST,$_GET,$_SERVER,$_FILES,$_COOKIE,$_ENV,$_SESSION,$GLOBALS,$_REQUEST);
}
}
?>
[PHP] pobierz, plaintext 

I moge Ci zagwaratowac ze zaden programista - nawet Ty nie jest w stanie niczego z nim popsuc piszac modul.

ja umie ;] daj mi tylko zrodla ;]

Swoja droga, to jak Twoj system potrafily polozyc zmienen globalne, to nie masz filtrowania zadnych danych...

Tzn Wszystkie superglobale przed zniszczeniem sa umieszczane w odpowiednich klasach i dostep do kazdej z nich odbywa sie przez filtry

no, panowie, moze wreszcie mi powiecie czy teraz jest zmiana na lepsze w kodzie ?
http://crystalpage.sf.net/code/
czy daley isc takim tropem ?

Jeżeli to poprafiles, to dobrze...

Co do kodu, to ja napewno bym tak nie pisal duzej aplikacji

Zebym nie był goło słowny

[PHP] pobierz, plaintext 
<?php
public function GetExtPath($extShotName) {
		global $_CP;
		if ($dir_handle = opendir('extensions/')) {
 
		$files = scandir('extensions/');
		while ((list($no, $file) = each($files)) ) {
?>
[PHP] pobierz, plaintext 

tak, no wiem jeszcze nie wszystkie global sa usuniete ale wszystkie $_DB i troche configu, ale to jest tylko kwestja czasu

nasty_psycho wiem ze to nie ma związku z tematem ale polecam Ci wykorzystać DB_DataObject w tym cms szybki o obsługuje różne bazy danych.