hwsoft Posted March 16, 2016 at 11:52 PM Share 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 Link to comment Share on other sites More sharing options...
photron Posted March 17, 2016 at 11:13 AM Share 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 Link to comment Share on other sites More sharing options...
hwsoft Posted March 17, 2016 at 11:46 AM Author Share 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 Link to comment Share on other sites More sharing options...
photron Posted March 23, 2016 at 02:23 PM Share 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 Link to comment Share on other sites More sharing options...
hwsoft Posted March 23, 2016 at 04:08 PM Author Share Posted March 23, 2016 at 04:08 PM Klasse Sache, probiere es nachher gleich mal aus! Quote Link to comment Share on other sites More sharing options...
hwsoft Posted March 24, 2016 at 08:12 AM Author Share 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 Link to comment Share on other sites More sharing options...
photron Posted March 24, 2016 at 12:12 PM Share 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 Link to comment Share on other sites More sharing options...
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.