PHP 5.3 > 5.4 / Probleme mit (Parameter n to function() expected to be a reference, v

[thotti]

Hi,

ich hatte unter PHP 5.3 ein Script laufen, welches ohne Probleme lief. Nach Umstellung auf Debian 6 und damit auf PHP 5.4 erhalte ich eine Fehlermeldung. Kann mir bitte einer sagen, warum das nicht mehr geht, und/oder was ich besser machen kann?

Das Script hier mal auf das Wesentliche reduziert:

<?php

class MyClass
{
    public function __construct()
    {
        $this->my_external_handler = 'my_external_user_function';
        declare(ticks = 1);
        pcntl_signal(SIGINT, array($this, "signal_handler"));
    }
    
    public function signal_handler($signal_number)
    {
        $this->info = "signal $signal_number caught";
        call_user_func($this->my_external_handler, $this);
    }
}

ini_set("display_errors", 1); 
ini_set("error_reporting", E_ALL);

$TestClass = new MyClass;

$i = 10;
while($i--) {
    echo("Waiting $i seconds for SIGINT (CONTROL+C) ...\n");
    sleep(1);
}

function my_external_user_function(&$data)
{
    echo("info: '".$data->info."'\n");
    exit;
}
  
?>

Wenn ich es in der Shell wie folgt ausführe

php test.php

dann erhalte ich die Meldung:

Parameter 1 to my_external_user_function() expected to be a reference, value given in /var/lib/asterisk/agi-bin/test.php on line 15

Wenn ich Zeile 15 wie folgt ändere, dann klappt es:

call_user_func($this->my_external_handler, &$this); 

Aber: Ist das so richtig?

[h3ll]

Warum verwendest du hier eine Referenz? Lass sie doch einfach weg.

[thotti]

Ja hier in der reduzierten Form wäre das kein Problem. In meinem Script spielt das schon eine Rolle ... Unabhängig davon wüsste ich schon gerne, warum das nicht mehr geht.

[h3ll]

Ja hier in der reduzierten Form wäre das kein Problem. In meinem Script spielt das schon eine Rolle ... Unabhängig davon wüsste ich schon gerne, warum das nicht mehr geht.

Dann zeig bitte ein Beispiel, wo das so notwendig ist. Ich seh keine Notwendigkeit für eine Referenz. Mit Fantasiecode, der nichts mit dem Problem zu tun hat, kann man schlecht arbeiten.

[combie]

call_user_func($this->my_external_handler, &$this); 

Aber: Ist das so richtig?

Nein!
Das ist Unsinn!
Denn $this ist IMMER eine Referenz.

[AmicaNoctis]

Nein!
Das ist Unsinn!
Denn $this ist IMMER eine Referenz.

$this ist immer ein Objekt, aber keine Referenz, jedenfalls nicht in PHP 5. Ich würde den Parameter $data einfach als MyClass typen und fertig. Es gibt keinen Grund, ihn als Referenz zu deklarieren.

[thotti]

Ich würde den Parameter $data einfach als MyClass typen und fertig.

Was heisst das bitte?

[AmicaNoctis]

Was heisst das bitte?

function my_external_user_function (MyClass $data) {
    // ...
} 

Alles was du in dieser Funktion mit $data anstellst, wird sich auf das übergebene Objekt auswirken. Es gibt also keinen Grund für eine Referenz. Du darfst $data nur keinen völlig neuen Wert zuweisen.

[thotti]

function my_external_user_function (MyClass $data) {
    // ...
} 

Danke - wo kann ich darüber bitte mehr lesen? Das habe ich noch nie gesehen. "Casten" ansich kenne ich - aber nicht so.

Und noch eine Frage bitte: Damit wäre es also eigentlich egal, ob ich eine Referenz übergebe und dies als Refernz in der Funktion deklariere ODER ob ich als value übergebe und dies ebenso in der Funktion als value angebe? Oder anders: die folgenden Varianten sind praktisch(!) identisch?

1)
call_user_func($this->my_external_handler, $this);
[...]
function my_external_user_function($data)

2)
call_user_func($this->my_external_handler, &$this);
[...]
function my_external_user_function(&$data)

[AmicaNoctis]

Danke - wo kann ich darüber bitte mehr lesen?

PHP: Type Hinting - Manual

Und noch eine Frage bitte: Damit wäre es also eigentlich egal, ob ich eine Referenz übergebe und dies als Refernz in der Funktion deklariere ODER ob ich als value übergebe und dies ebenso in der Funktion als value angebe?

Wenn ein Parameter als Referenz deklariert ist, wird immer eine Referenz übergeben. Wenn du dann myfunc(&$var) aufrufst (statt myfunc($var)), kommt sogar eine Fehlermeldung (ab PHP 5.4 sogar als Fatal Error).

Egal ist es allerdings nicht ganz. Referenzen sollten nach Möglichkeit gar nicht benutzt werden und wenn doch, dann aus gutem Grund. Wenn man sauber objektorientiert arbeitet, braucht man sie in der Regel erst recht nicht.

Oder anders: die folgenden Varianten sind praktisch(!) identisch?

1)
call_user_func($this->my_external_handler, $this);
[...]
function my_external_user_function($data)

2)
call_user_func($this->my_external_handler, &$this);
[...]
function my_external_user_function(&$data)

Das kommt darauf an, was du mit $data machst. Abgesehen davon, dass die zweite Variante wie gesagt ungültig ist, wirst du vermutlich dasselbe Resultat sehen, also ist es praktisch fast gleich, allerdings würde ich nicht behaupten „identisch“.

[thotti]

Gelöst

Danke an alle!

[h3ll]

die folgenden Varianten sind praktisch(!) identisch?

Nein. Ein $this als Referenz zu übergeben, macht generell keinen Sinn, weil $this nicht überschrieben werden kann.

Bei anderen Objekten wirkt es sich so aus:

function doSomething($object) {
    $object->bar = 2;
    $object = null;
}

$foo = new foo();
$foo->bar = 1;
doSomething($foo);
var_dump($foo);

// object(foo)#1 (1) {
//   ["bar"]=>
//   int(2)
// } 

function doSomething(&$object) {
    $object->bar = 2;
    $object = null;
}

$foo = new foo();
$foo->bar = 1;
doSomething($foo);
var_dump($foo);

// NULL 

Wie du hier siehst, kann es ziemlich gefährlich werden, wenn du ein Objekt als Referenz übergibst, weil du so Objekte löschen/überschreiben kannst, die noch an anderer Stelle gebraucht werden. Deswegen ist ganz klar darauf zu verzichten.

[AmicaNoctis]

@h3ll: Genau, schönes Beispiel!