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 zu diesem Kommentar
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 zu diesem Kommentar
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.

Gast
Reply to this topic...

×   Du hast formatierten Text eingefügt.   Formatierung jetzt entfernen

  Only 75 emoji are allowed.

×   Dein Link wurde automatisch eingebettet.   Einbetten rückgängig machen und als Link darstellen

×   Dein vorheriger Inhalt wurde wiederhergestellt.   Clear editor

×   Du kannst Bilder nicht direkt einfügen. Lade Bilder hoch oder lade sie von einer URL.

×
×
  • Neu erstellen...