hwsoft Posted March 16, 2016 at 11:52 PM Posted March 16, 2016 at 11:52 PM 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 Quote
photron Posted March 17, 2016 at 11:13 AM Posted March 17, 2016 at 11:13 AM 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. Quote
hwsoft Posted March 17, 2016 at 11:46 AM Author Posted March 17, 2016 at 11:46 AM 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. Quote
photron Posted March 23, 2016 at 02:23 PM Posted March 23, 2016 at 02:23 PM 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 Quote
hwsoft Posted March 23, 2016 at 04:08 PM Author Posted March 23, 2016 at 04:08 PM Klasse Sache, probiere es nachher gleich mal aus! Quote
hwsoft Posted March 24, 2016 at 08:12 AM Author Posted March 24, 2016 at 08:12 AM 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); Quote
photron Posted March 24, 2016 at 12:12 PM Posted March 24, 2016 at 12:12 PM 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. Quote
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.