Jump to content

[PHP] IO16 Callback fehlt ein UserData Argument


Recommended Posts

Hallo,

 

ich bastel gerade an einem Stack mit mehreren IO16 Bricklets. Dabei ist aufgefallen, dass anderes als bei IPConnection

 

void IPConnection::registerCallback(int $id, callable $callback, mixed $userData=NULL)

 

kein 3. Argument "mixed $userData=NULL" zu Verfügung steht.

Stattdessen wird nur

 

void BrickletIO16::registerCallback(int $id, callable $callback)

 

angeboten. Dies führt nun zu einem ziemlich hässlichen Konstrukt um eine Lösung dafür zu schaffen.

 

function cb_io1($p,$i,$v) { cb_interrupt($p,$i,$v,'io1'); }
function cb_io2($p,$i,$v) { cb_interrupt($p,$i,$v,'io2'); }
function cb_io3($p,$i,$v) { cb_interrupt($p,$i,$v,'io3'); }
function cb_io4($p,$i,$v) { cb_interrupt($p,$i,$v,'io4'); }
...
$io1->registerCallback(BrickletIO16::CALLBACK_INTERRUPT, 'cb_io1');
$io2->registerCallback(BrickletIO16::CALLBACK_INTERRUPT, 'cb_io2');
$io3->registerCallback(BrickletIO16::CALLBACK_INTERRUPT, 'cb_io3');
...

 

könntet ihr bitte in den PHP Bindings (ggf. auch in anderen wenn notwendig) das 3. Argument $userData einfügen? Generell denke ich mach es Sinn, dies bei allen registerCallBack einzufügen.

 

Alternativ wäre es hilfreich wenn der CallBack Aufruf die UID des Auslösenden Bricklets mitliefert. Andernfalls weis man zwar das A1 einen Interrupt ausgelöst hat, aber nicht von welchem Bricklet das ganze kommt.

 

Vielen Dank

 

Grüße

Martin

 

Link to comment
Share on other sites

Die dokumentierte und implementierte Signatur ist schon diese:

 

void BrickletIO16::registerCallback(int $id, callable $callback, mixed $userData=NULL)

 

Allerdings fehlte beim Aufruf des eigentlichen Callbacks die Übergabe des $userData Parameters, so dass bei deiner cb_io1 Funktion dann kein $userData Parameter ankommt. Das ist jetzt korrigiert.

 

Angehängt die korrigierte Version zum Testen.

tinkerforge_php_bindings_2_0_13_rc1.zip

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...