Notice: Only variables should be assigned by reference

[noviKorisnik]

Ovo je nešto specifično za PHP 4.4.0. Nisam uspeo nešto više da nađem o temi jer je google indeksirao ispise greške na gomili sajtova.

Interesuje me da li može da ubije neki deo koda da se ne izvrši kako treba?

Ispis greške mogu da izbegnem ako umesto...
$varijabla =& funkcija ();... koristim...
$varijabla = funkcija ();... (znači, samo maknem &). Da li je to OK? Ima li suštinske razlike, ili je samo stvar u tome što ono prvo parče koda generiše Notice u PHP 4.4.0 a drugo ne?

Da li je dovoljno da isključim E_NOTICE iz error_reportinga (da ne moram da trčim kroz hiljade linija koda)?

[dinke]

http://www.php.net/manual/en/language.references.return.php

Note: Unlike parameter passing, here you have to use & in both places - to indicate that you return by-reference, not a copy as usual, and to indicate that reference binding, rather than usual assignment, should be done for $foo.
Ukratko moras da imas & i u definiciji f-je kao u primeru na linku gore.

Isto tako:
Note: If you try to return a reference from a function with the syntax: return ($found_var); this will not work as you are attempting to return the result of an expression, and not a variable, by reference. You can only return variables by reference from a function - nothing else. E_NOTICE error is issued since PHP 4.4.0 and PHP 5.1.0 if the code tries to return a dynamic expression or a result of the new operator.
Dakle, moguce je da ti ovo pravi problem. Ako imas iskaz tipa: return ($found_var); u f-ji onda ti vracas expression a ne variablu. Skloni zagrade i bice ok.

[noviKorisnik]

Konkretno sam menjao na jednom mestu gde je originalno bio sledeći kod
$parser =& xml_parser_create('');
Dakle, funkcija koja vraća resource (a to je valjda objekat).

Kao što sam pomenuo, izgleda da je u pitanju nešto što se odnosi isključivo na verziju 4.4.0, a to je sada masovna poruka, evo google: http://www.google.com/search?q=%22Only+variables+should+be+assigned+by+r eference%22
Jedva sam uspeo da iskopam nešto o tome, ovde: http://www.pmachine.com/forums/viewthread/25062/
The error you’re quoting is one that came up for people when PHP 4.4.0 was released. That release changes a core feature of PHP in a non-backwards-compatible way.

[Ilija Studen]

Imao sam isti problem, a rešio sam ga tako što sam instalirao PHP 4.3.11 i PHP 5.0.4 umesto stabilnih 4.4 i 5.1. Sad više nemam tih problema jer sam prešao na PHP5 gde & jednostavno više ne koristiš (osim u slučajevima foreacha gde želiš reference, a ne koristiš iteratore).

Koliko je meni poznato, postoji config opcija kojom se ovaj notice isključuje, ali se ne sećam imena. Ono što sigurno možeš da uradiš (ali nije neko rešenje) je:

error_reporting(0);

// tvoj kod

error_reporting(E_ALL);

Isključiš pa uključiš prikazivanje grešaka. Odradiće posao ako ti ni jedan drugi metod nije dostupan...

[noviKorisnik]

error_reporting (E_ALL ^ E_NOTICE);

Interesuje me samo da li je to dovoljno, da li se možda neki deo koda neće dobro izvršiti.

Evo kako ja to vidim, ali nisam siguran da je to baš ovako:
- to je Notice, primedba, koja kaže da bi nešto trebalo da se radi na drugi način, koliko razumem radi se o stvari koju PHP rešava i ispravlja usput tako da ne dolazi do brljotine, za razliku od Warnig poruke koja ukazuje na verovatnu grešku (npr. referenciranje nepostojećeg indeksa niza).

[dinke]

Ja sam uvek za to da se prilikom razvoja drzi ukljucen i E_NOTICE level. Dakle error_reporting = E_ALL.

E, sad sto se tvog problema tice, molim te pastuje mi definiciju funkcije xml_parser_create i return k-du u istoj.

[noviKorisnik]

To je PHP funkcija: www.php.net/xml_parser_create

Gledam sada ostale slučajeve dodele referencom...

Naravno tu su varijable
$nextNode =& $currNode->nextSibling;

Pozivi funkcija, npr
$newxmldoc =& unserialize($fileContents);

Pozivi konstruktora, npr
$this->connection =& new php_http_connection($host, $path, $port, $timeout);

Potom pozive statičkih metoda, npr
$fileContents =& php_file_utilities::getDataFromFile($name, 'r');

I metoda
$childTextNode =& $this->ownerDocument->createTextNode($data);

...

Uglavnom, toga ima dosta, a dok ovako gledam ne deluje mi pametno da redom krenem s uklanjanjem referentne dodele jer, npr, metod može da vrati neki svoj objekat i tada nije svejedno da li se dodeljuje sam objekat ili njegova kopija.

I da ne bude zabune, sav ovaj kod nije moj niti sam ga nasledio, već sam samo krenuo da konfigurišem gotov CMS i pritom naleteo na ovakve poruke.

Konačno, ostaju mi 3 puta: isključivanje Notice obaveštenja, instalacija starije verzije PHP i izmena koda. Ovo treće mi deluje riskantno jer još nisam dovoljno upoznat sa sistemom da bih znao šta se sve menja brisanjem jednog karaktera.

Interesuje me kako se PHP ponaša kada uoči takvo nešto, da li ipak izvršava kako je navedeno u kodu uz malo gunđanja ili radi kako smatra da bi trebalo. Izuzev samih poruka zasad nisam primetio neke probleme u funkcionalnosti aplikacije.

Ako ipak dodeli kako je navedeno, čini mi se da je dovoljno isključivanje obaveštavanja.

[dinke]

Verovatno se sam php buni jer se svejedno vraca referenca pa nema potrebe koristiti & fun_name vec je dovoljno fun_name prilikom poziva. To je uvedeno koliko vidim u php 4.4.0 i php 5.1.0

Umesto da prepravljas tudj kod, ja ti u ovom slucaju koji si naveo savetujem da iskljucis E_NOTICE nivo. Ne brini, to ce samo iskljuciti "gundjanje" kako ti kazes, nista se lose nece desiti.

[noviKorisnik]

Tačno. Skoknuo sam do foruma (to je Mambo, da ne budem misteriozan) i našao neke teme u kojima se spominje ovaj problem. Ovo dalje može da vodi do sledeće poruke...
Warning: Cannot modify header information - headers already sent by (output started at...
... a tada već nije svejedno.

I tamo sam našao preporuke da se briše &, ali toga ima dosta, a kako mi predstoji još instalacija komponenti i modula... puno muke.

Tako da ću za početak isključiti E_NOTICE, a ako se pojave dodatni problemi, idem na stariju verziju jezika.

[Ilija Studen]

Ukloni & ako znaš da će se skripta vrteti na PHP5 ili na PHP4.4 i novijim
U suprotnom samo isključi notice na problematičnom mestu...

Ništa gore nego kad razbiju kompatibilnost sa starim verzijama na takav način :(

[zekica]

PHP 4.4.0 Release Announcement (http://www.php.net/release_4_4_0.php):
...
As part of the solution for the reference bug, you are very likely to find that your own or third-party PHP scripts, considered 'clean' code under previous versions of PHP, will now throw an E_NOTICE when references are incorrectly used in the script. This is intended to alert developers to minor errors in their approach, and does not affect the script's performance in any other way.
...

Što znači da će tvoja skripta skoro sigurno raditi ako samo ubaciš error_reporting(E_ALL ^ E_NOTICE);.

[noviKorisnik]

To je pravi link, zekice :-)

@Ilija - ne radi se o jednom problematičnom mestu, već o gomili, kao i o komponentama koje tek treba da se instaliraju a pate od iste boljke. Na sreću, pregledao sam malo kako se rukuje error_reportingom i jedino se pri samoj instalaciji postavlja na E_ALL, a za sve ostalo se prati opcija iz globalne konfiguracije (ne diraj ništa, NONE, bez E_NOTICE, E_ALL).

[noviKorisnik]

... epilog...

php 4.3.11

Našao sam da ponešto ipak ne radi očekivano, mogao bih doduše trčati po kodu i uklanjati '&', negde da, negde ne, ali kad se sve rešava jednostavnim prelaskom na valjanu verziju koja ima probleme s memorijom, hajde, to je to.

ps. - e sad, ako uletim u fazu eksperimenta pa krenem i na php 5... ko zna gde bi bio kraj?

[Ilija Studen]

Na PHP 5.0.x nećeš imati nikakvih problem, ali ti ne garantujem za PHP 5.1.

U PHP5 definitivno možeš da pobrišeš & jer se uvek prenose reference za razliku od PHP4. Ima par izuzetaka (nrp. foreach uvek prenosi kopiju ako nije naglašeno drugačije npr), ali oni ne utiču na problem koji tebe muči.

[noviKorisnik]

Upecao sam nešto :-)
<?php
class parent
{
var $child;
function parent ()
{
$this -> child = new child ();
}
function getChild ()
{
return $this -> child;
}
}
class child
{
var $property;
function setValue ($value)
{
$this -> property = $value;
}
function getValue ()
{
return (string) (int) (bool) $this -> property;
}
}
$parent = new parent ();
@ $child = & $parent -> getChild ();
$parent -> child -> setValue (1);
die (PHP_VERSION . ' : ' . $child -> getValue ());
?>
Outputi su
4.4.0 : 0
i
4.3.11 : 1
:-)))
probajte u peticama, živo me interesuje...

[Ilija Studen]

Fatal error: Cannot use 'parent' as class name as it is reserved in D:\webroot\testbre.php on line 3

Znači ne radi (parent je rezervisana reč u PHP5), ali uz blage modifikacije (parent => ParentClass i child => ChildClass) uspeo je da vrati ovo:

5.0.4 : 1

Varijacije:

...
$child = $parent -> getChild ();
...

...
$child =& $parent -> getChild ();
...

Ovo vraća 1 sa ili bez "forsirane" reference. Dakle, nemaš šta da se misliš ili da se cimaš.

PS: Da li sam spomenuo da je jedan od najjačih razloga koji me je naterao da se u potpunosti prebacim na PHP5 upravo rešen sistem za rad sa referencama. Dosadila mi glavobolja i jurcanje za fantomskim bugovima koji nastaju zbog njih.