Forum PHP.pl

Witam Was… Ostatnio zainstalowałem sobie nowy serwer (zrobiłem update) z php 5.1.1 na 5.2.0. W rezultacie połowa tablic przestała mi działać… pięknie co? Powiem szczeże że się zdenerwowałem jak nic… już kilka dni badam o co chodzi. Przykładowy kod z widoku mojego frameworka:

[PHP] pobierz, plaintext 
<?php
$oView->aError = array(); //declare an array
var_dump($oView->aError); // it will output: Array( )
 
$oView->aError[] = '1';   // add element to array...
$oView->aError[] = 2;	 // ... some int? ...
$oView->aError[] = '3';   // ... and string ...
 
var_dump($oView->aError); // it output: Array( )
						  // hmmm, it should output: Array( [0] => string(1), "1" [1] => int(1), "2" => string(1) "3" )
?>
[PHP] pobierz, plaintext 

Obiekt $oView oczywiście został wcześniej zainicjowany. Przejdźmy do rzeczy… Zaraz po zadeklarowaniu tablicy sprawdzamy ją tak dla pewności, że jest. „No dziwne by było żeby jej nie było” (masło maślane ale prawda sama w sobie )

Ok. jedziemy dalej… dodajemy do tablicy kolejne elementy poprzez użycie [] – co to oznacza każdy wie. Po dodaniu elementów lumpujemy tablice po raz kolejny. O dziwo jest pusta .

Przypominam, że zrobiłem update z php 5.1.1 na 5.2.0. Czyżby wlepili tam jakiegoś buga? Zaraz po updacie ta część kodu przestała działać. Podobnie warstwa ACL, gdzie klasa User w podobny sposób sama sobie dodaje grupy użytkownika – dziwiłem się, że do żadnego kontrolera nie mam dostępu

Teraz wszystko jasne. Zainteresowało mnie to, przeprowadziłem testy. Napisałem dwa przykłady, które działają…

Test 01:

[PHP] pobierz, plaintext 
<?php
$aArray = array();
 
$aArray[1] = 'first element';
$aArray[2] = 'second element';
$aArray[3] = 'third element';
 
var_dump($aArray);
?>
[PHP] pobierz, plaintext 

Test 02:

[PHP] pobierz, plaintext 
<?php
class ArrayTest
{
	private $aAttributes = array();
 
	public function __construct()
	{
		$this->aAttributes['TestCase'] = array();
	}
 
	public function __destruct()
	{
		var_dump($this->aAttributes['TestCase']);
	}
 
	public function __get($sName)
	{
		return $this->aAttributes['TestCase'][$sName];
	}
 
	public function __set($sName, $mValue)
	{
		return $this->aAttributes['TestCase'][$sName] = $mValue;
	}
}
 
$oArrayTest = ArrayTest();
 
$oArrayTest->FirstElement = 1;
$oArrayTest->SecondElement = 2;
$oArrayTest->ThirdElement = 3;
?>
[PHP] pobierz, plaintext 

Co jest nie tak? Może załącze jeszcze klase mojego widoku I usera, aby pokazać, że dodaję elementy w taki sam sposób:

User Class (patrz metody setGroup(), hasGroup(), getGroup() )
http://phpfi.com/179458
View Class (patrz metody __get(), __set() )
http://phpfi.com/179456

Ten post edytował Athlan 28.11.2006, 16:37:22

Witam
Nie instalowałem jeszcze 5.2 ale tutaj : http://www.php.net/UPDATE_5_2.txt znalazłem coś takiego :

New functions
=============

In the php core
===============
array array_fill_keys(array keys, mixed val)
- Create an array using the elements of the first parameter as keys, each initialized to val

i tak sobie pomyślałem czy nie o to chodzi ?
W działających przykładach podałeś i klucze i wartości a w pierwszym nie działającym nie podajesz kluczy (tak się do teraz robi) może w 5.2 nie da się już $oView->aError[] ? tyko trzeba podać klucz ?

Witam
Rozwiazaniem tego problemu jest użycie funkcji array_push.

Pytanie dlaczego nie można przypisywać elementu w zwykły sposób.. :/

Potestowałem troszkę i nie zauważyłem różnic w obsłudze tablic ;| Może wina leży po stronie metod magicznych, albo gdzieś coś Twój framework gubi?

Musi Ci wywalać gdzieś jakiś errro, daj PEŁNE raportowanie i się dowiesz. Na pewno nic w php nie zmienili, gdyż mi działa wszystko ok.

$oArrayTest = ArrayTest();

a nie powinno byc

$oArrayTest = new ArrayTest();

?

Dziś odkryłem swój błąd, który popełniłem przy tworzeniu tablic. Pokażę to na poniższym przykładzie:

[PHP] pobierz, plaintext 
<?php
 
class ArrayTest
{
	private $_aAttributes = array();
 
	public function __get($sName)
	{
		return $this->_aAttributes[$sName];
	}
 
	public function __set($sName, $mValue)
	{
		return $this->_aAttributes[$sName] = $mValue;
	}
}
 
$oArray = new ArrayTest();
 
$oArray->test = array();
$oArray->test[] = 'test1';
$oArray->test[] = 'test2';
$oArray->test[] = 'test3';
 
var_dump($oArray->test);
 
?>
[PHP] pobierz, plaintext 

Dump tablicy wykaże pustą – ma wykazać. Dlaczego? Do metody __get() dostaje się atrybut test – ok. Następnie deklarujemy atrybut test[]. Co pomyśli sobie __set() i __get() ? Dla zmiennej $sName oznacza po prostu $this->_aAttributes[test[]], a powinno $this->_aAttributes[test][]. Po części prawda, po części paradoks. Moim zdaniem php zamo sobie powinno wrzucać elementy w tablice przy __set() i __get(), no ale skoro nie… Więc jak można rozwiązać powyższy przykład? Ano bardzo podobnie, budowa klasy nie zmieni się:

[PHP] pobierz, plaintext 
<?php
 
// …
 
$oArray = new ArrayTest();
 
$aArray = array();
$aArray[] = 'a';
$aArray[] = 'b';
$aArray[] = 'c';
 
$oArray->test = $aArray;
 
var_dump($oArray->test);
?>
[PHP] pobierz, plaintext 

Teraz działa?

Sam jestem zaskoczony, można z tego wyciągnąć jakiś wniosek, dla mnie jest jednoznaczny: twórcy php engine nie popisali się pod tym względem, nie możemy operować na tablicach obiektu poprzez metodę __set(). Teoretycznie nawet nie powinniśmy, ba co przyjmie pierwszy argument tej funkcji? Przyjmie Stinga jak ma się nazywać zmienna (kolejny element tablicy private $_aAttributes. Teoretycznie nie powinno działać, ale ja i tak wnoszę sprzeciw. Takie jest moje zdanie

Tak więc prostuje moje poprzednie pytanie . Fajnie jest dojść do takiego wniosku, więc zachęcam innych do tworzenia mnóstwa przykładów odwzorowujących kawałek naszego kodu, który rzekomo nie działa. To nauczka dla mnie, jak i przykład dla młodych programistów. Szczerze po tej wypowiedzi w ogóle nie wstydzę się pytania którego zadałem, gdyż nikt nie był w stanie mi na nie odpowiedzieć – nie ma pola do wstydu

Dziwną rzeczą jednak jest to, że kolizja wystąpiła dopiero w php 5.2.0, a w poprzedniej (mojej) wersji na serwerze 5.1.1 rzekomy błąd nowego pokolenia php 5 gładko działał. Zapraszam do dyskusji na ten temat w tym topicku – może ktos wie coś na owy przypadek traktowania zmiennych __set() w pierwszym parametrze.

Pozdrawiam, Athlan


----- [EDIT] -----

P.S.

Jak wspomniałem w pytaniu, przez to padł mi ACL a precyzując: klasa User, która nadawała grupy użytkownikowi do sesji. Kod wyglądał następująco:

[PHP] pobierz, plaintext 
<?php
public function setGroup()
{
	$aGroups = func_get_args();
 
	foreach($aGroups as $iKey => $sValue)
	{
		$this->_oUserSessionInfo->groups[] = $sValue;
	}
}
?>
[PHP] pobierz, plaintext 

Jak widać, sesja $this->_oUserSessionInfo jest obiektem (po prefiksie wiedeńskim). Zmienna groups[] przyjmowana jest do klasy sesji poprzez metodę __set() w pierwszym jej parametrze. Wczesniej udowodniłem, że tak nie może być… jak rozwiązać problem? Ano namotamy trochę w kodzie – ale będzie działało.

[PHP] pobierz, plaintext 
<?php
public function setGroup()
{
	$aGroups = func_get_args();
 
	foreach($aGroups as $iKey => $sValue)
	{
		$aArray = $this->_oUserSessionInfo->groups;
		$aArray[] = $sValue;
		$this->_oUserSessionInfo->groups = $aArray;
	}
}
?>
[PHP] pobierz, plaintext 

Najpierw pobieramy tablicę $aArray z sesji jako tablica grup. Potem wykonujemy operację dodania grupy już poza klasą nie używając metody __set(), po czym tablice ponownie zapisujemy do __set()’a. Innego sposobu nie ma… ale jakiegoś haka udało mi się wymyślić.

Ten post edytował Athlan 30.11.2006, 17:07:18

Proponuje zamiast tablic w tym przypadku użyć ArrayObject

[PHP] pobierz, plaintext 
<?php
class ArrayTest
{
	private $_aAttributes = array();
 
	public function __get($sName)
	{
		return $this->_aAttributes[$sName];
	}
 
	public function __set($sName, $mValue)
	{
		return $this->_aAttributes[$sName] = $mValue;
	}
}
 
$oArray = new ArrayTest();
 
$oArray->test = new ArrayObject();
$oArray->test[] = 'test1';
$oArray->test[] = 'test2';
$oArray->test[] = 'test3';
 
var_dump($oArray->test);
?>
[PHP] pobierz, plaintext 

Problem pojawia się na przykład przy użyciu array_push lub implode itp
Dla implode można stosować rzutowanie, np. implode( ',', (array) $oArray->test)

Przecież obiekt ArrayTest nie posiada atrybutu test .. Powinno wywalić błąd a nie wywala..

Do tego właśnie służą magiczne metody __get() __set() __call() - jak czegoś nie ma to one się tym zajmują
Manual

Tylko jaki jest sens przypisywać nieistniejącej zmiennej wartość? Lepiej ją najpierw zadeklarować.. Przez to kod dla kogoś innego staje się mniej zrozumiały.

maly przyklad
przed uzyciem skontaktuj sie z baza dany i przeczytaj SQLowa ulotke nizej

[PHP] pobierz, plaintext 
<?php
class Cars {
	protected $items;
 
	public function __get( $model) {
		if ($this->items === null) {
			$query = 'SELECT * FROM cars';
			$result = mysql_query( $query);
			while ($row = mysql_fetch_assoc( $result)) {
				$this->items[$row['model']] = $row['description'];
			}
		}
		if (false == isset( $this->items[$model])) {
			throw new Exception( 'nie ma takiej pozycji');
		}
		return( $this->items[$model]);
	}
 
	public function __set( $model, $description) {
		try {
			$this->__get( $model);
			$query = 'UPDATE cars SET description=''.$description.'' WHERE model=''.$model.''';
			mysql_query( $query);
		} catch( Exception $e) {
			$query = 'INSERT INTO cars (model, description) VALUES (''.$model.'', ''.$description.'')';
			mysql_query( $query);
		}
		$this->items[$model] = $description;
	}
}
 
$cars = new Cars();
$cars->skarpeta = 'fajne auto';
 
try {
	echo $cars->skarpeta;
} catch (Exception $e) {
	echo $e;
}
?>
[PHP] pobierz, plaintext 

[SQL] pobierz, plaintext 
CREATE TABLE `cars` (
  `model` varchar(255) NOT NULL,
  `description` text NOT NULL,
  PRIMARY KEY  (`model`)
)
[SQL] pobierz, plaintext 

Oczywiscie mozna to usprawniac na wiele sposobow. Zastosowania __get() __set() sa bardzo przydatne.
Zauwaz ze tutaj moznaby zamiast rzucac wyjatek w przypadku braku rekordu zwrocic jakas domyslna wartosc. Wedlug mnie dzieki tego typu rozwiazaniom kod staje sie abrdziej przejrzysty i elegancki. Nieco ciezej sie wykrywa bledy ale mozna zrobic rzeczy bardziej uniwersalne, elastyczne i wielokrotnie wykorzystywac . Moznaby do tej klasy dodac licznik zapytan i mase innych gadzetow...
Mam nadzieje ze przekonalem nieco...
Pisane na szybko wiec wybaczcie jakies bledy...

Panowie, panowie

Chyba zupełnie odbiegliście od tematu. @cassis już kompletnie nie czai blusa. Chodzi o tworzenie elementów tablicowych z __set() i wkładanie ich do tablicy jako atrybut klasy prawda? Bo co przyjmie pierwszy parametr metody magicznej __set() przy podaniu $oObject->element[] = 'value'; ?

Takie coś: $this->_aAttributes[$sName] - przekształci się w : $this->_aAttributes[element[]] a nie $this->_aAttributes[element][] :!:

Nie czujecie fazy czy udajecie? Temat jest imho EOT. Problem wyjaśniony, możnaby podyskutować o tym, dlaczego przechodziło to w wersjach starszych niż php 5.2.0 ? hę?

Pozdrawiam, Athlan.

Ten post edytował Athlan 5.12.2006, 14:38:47

no tak - troche poplynalem. Chcialem wytlumaczyc jedynie o co chodzi z __get i __set...

Wracajac do glownego tematu na liscie bugow php jest taki temat 39449 ze statusem Open. Ktos tam tlumaczy ze to jest prawidlowe zachowanie...

Nie wiem czy to bug czy nie ale nie ladnie ze to zmieniono. Ja poradzilem sobie z tym zmieniajac wszedzie gdzie to potrzebne zwykle tablice na ArrayObject. Do tego zrobilem sobie klase rozszerzajaca i dodalem kilka usprawnien...

Na tym przykladzie widac ze warto sobie wszystko obudowac ladnie tak ze jak sie pojawi nowy "feature" to latwo bedzie to mozna dostosowac...