PHP Connection mit stream_socket_*

[hwsoft]

Hallo, vorweg ich mag den PHP Code Stile in den Class nicht. Es gibt da verschiedene Dinge, nur ein Beispiel:

}

?>

 

wie man sieht wird nach dem "?>" noch ein Zeilenumbruch gemacht. In PHP 5 kann man das schließen des PHP-Tags eigentlich auch ganz weg lassen, damit eben keine Zeichen mit ausgeben werden.

 

Soweit, weiter in der Geschichte, mein Basis PHP Server lässt aus Sicherheitsgründen die Sockets nicht standardmäßig zu. Ich hab mich entschieden den IPConnection umzuschreiben auf "stream_socket_", da wenig Zeit war, ist das natürlich Quick and Dirty.

 

Siehe: https://www.christophh.net/2012/07/24/php-socket-programming/

Neue Klasse im Anhang funktioniert soweit mit dem

 

$t = new BrickletTemperature("XYZ", $connector);
$temperature = $t->getTemperature();

 

sehr gut.

 

Ich kann erstmal mein Projekt weiter machen, bei Fragen stehe ich euch gerne zur Verfügung.

IPConnection.php

[photron]

Stream Sockets waren mir noch nicht bekannt. Ich denke es spricht nichts dagegen, dass für die nächste Version der PHP Bindings entsprechend umzustellen. Ich kümmer mich drum. Danke für die Überarbeitung.

[hwsoft]

Ich könnte mir einen SocketProvider Class vorstellen, die dann von stream und sockets abgeleitet wird, wenn IPConnection instance erzeugt wird, kann diese dann prüfen auf PHP Version > 5.* und auf die extension sockets (vorhanden ist.) Und lädt dann die entsprechende Klasse.

 

Nur mal so eine Idee  .

 

Es ist nicht verkehrt mit der "extension sockets" nur wie in meinen Link erklärt (oben) ist diese extension standardmäßig aus bei einer Serverinstallation mit PHP.

 

 

[photron]

Hier eine Version der PHP Bindings die die Sockets Extension verwendet, wenn vorhanden und ansonsten Stream Sockets verwendet.

 

Stream Sockets haben einen Nachteil, die TCP_NODELAY Option wird nicht unterstütz.

tinkerforge_php_bindings_2_1_8_07807acf8c78b.zip

[hwsoft]

Klasse Sache, probiere es nachher gleich mal aus! 

[hwsoft]

Ich hab mich mal bissel umgesehen wegen "TCP_NODELAY", laut einer Anfrage wurde dies (schon länger her) gepacht (siehe hier: https://bugs.php.net/patch-display.php?bug=51879&patch=add-tcp-nodelay&revision=1297975503).

 

Hab create stream mir angeschaut:

 

...
function __construct($address, $port)
    {
        // FIXME: stream sockets don't support TCP_NODELAY, see https://bugs.php.net/bug.php?id=51879
        $this->handle = stream_socket_client("tcp://$address:$port", $errno, $message);
....

 

Versuche mal folgendes Beispiel, was gehen sollte:

$opts = array(
    'socket' => array(
        'tcp_nodelay' => false,
    ),
);
$ctx = stream_context_create($opts);
$stream = stream_socket_client($uri, $errno, $errstr, $timeout, $flags, $ctx);

[photron]

In https://bugs.php.net/bug.php?id=51879 wird das Problem diskutiert. Der vorgeschlagene Patch wurde aber nicht gemerged und der Bug steht immer noch auf Assigned. Das Problem besteht also weiterhin.