Callback Funktion mit static function pointer

[molleonair]

Hallo zusammen ich habe ein Problem bei der Umsetzung einer callback Funktion..

Ziel soll sein das aus einer Klasse(TrackControl) heraus eine Funktion (sendmidi) im Haupt Sketch aufgerufen wird
dazu wollte ich einem static function pointer der Klasse die Referenz auf die Funktion übergeben.
Die einzelnen Instanzen der Klasse sollen dann die Funktion im Hauptsketch aufrufen können.
Ich habe das ganze schon umgeschrieben gehabt auf "extern void sendMidi (byte value);" und das hat funktioniert aber ich würde gern wissen
wie das mit dem callback richtig gemacht wird.

Code Hauptsketch:

#include "TrackControl.h"
//void sendMidi (byte); //prototyp needed ?
TrackControl trackcontrol[8];
void setup() {
  TrackControl::setCallback(&sendMidi);
  Serial.begin(9600);
}
void loop() {
  trackcontrol[0].setVolume(1); 
  delay(1000);
}
void sendMidi (byte value){
  Serial.print(value);
}

TrackControl.h:

#ifndef TrackControl_h
  #define TrackControl_h
  #include <arduino.h>

  class TrackControl {
    static void (*sendmidi)(byte); //pointer to function
    public:
      void setVolume (byte value);
      static void setCallback (void (*pCallbackFunction)(byte)); //static method to setup callback function for all instances
  };

#endif

TrackControl.cpp

#include "TrackControl.h"
#include <arduino.h>

void TrackControl::setCallback (void (*pCallbackFunction)(byte)){
  sendmidi=pCallbackFunction;
}

void TrackControl::setVolume (byte value){
  sendmidi(value);
}

Danke für eure Hilfe ..

[shedepe]

Um mal kurz in blaue zu Fragen: Was geht denn nicht? Macht das beantworten einfacher, dann muss man eventuell den Code nicht mal anschauen

Prinzipiell hast du schon alles richtig gemacht. Du brauchst eine Statische Instanz des Callbacks. Da fehlt in der Cpp Datei noch

void (*TrackControl::sendmidi)(byte) = 0

eben weil es ja eine statische Variable sein soll.
Weil die Syntax bisschen nervig wird auf Dauer verwendet man für Functionpointers gerne typedefs.
Also z.B. in der Klasse drin:

typedef void (*sendmidi_fptr)(byte);
static sendmidi_ftpr callback;

und im cpp:
TrackControl::sendmidi_ftpr TrackControl::callback = 0;

Ansonsten ist die idee wie gesagt gut so:
Man hat eine setCallback Funktion und ruft den wie du es gemacht hast eben auf.
Ich würde vom Design her gerade bei Funktionspointern statische instanzen davon vermeiden, sondern lieber jeder Instanz den Pointer einzeln mitgeben. Das hat weniger Risiko für Nebenwirkungen wenn der Code mal komplexer wird.

Edit was ich noch vergessen habe: Das funktioniert so nur für Funktionen die außerhalb einer Klasse sind bzw. statisch in einer Klasse sind. Willst du einen Funktionspointer auf eine Memberfunction übergeben kannst du das nicht mit einem klassischen Funktionspointer so ohne Probleme machen. Mit C++ am PC geht das am besten mit std::function. Es es das auch für arduino gibt weiß ich nicht.

[molleonair]

Hallo shepede ..
Danke für deine Antwort .. als VB.Net Hobbyist kann ich das nur nicht umsetzten
meiner Meinung nach habe ich einen Funktionspointer in der Header Datei deklariert
und mit deiner "void (*TrackControl::sendmidi)(byte) = 0" initialisiert
die cpp sieht jetzt so aus:

#include "TrackControl.h"
#include <arduino.h>

void TrackControl::setCallback (void (*pCallbackFunction)(byte)){
  void (*TrackControl::sendmidi)(byte) = 0;
  TrackControl::sendmidi=pCallbackFunction;
}

void TrackControl::setVolume (byte value){
  sendmidi(value);
}

die Fehlermeldung lautet:
sketch/TrackControl.cpp:6: undefined reference to `TrackControl::sendmidi'

die Lösung mit der statischen Klassen Funktion habe ich gewählt da ich sonst in jeder Instanz noch einen Pointer bräuchte und schon bei 70Prozent dynamischer Speicherauslastung bin
und dann noch für 128 Instanzen Pointer bräuchte.

[shedepe]

Lies dir noch mal durch wie man in C++ Statische Variablen verwendet. https://de.wikibooks.org/wiki/C%2B%2...nte_in_Klassen
Nur im Headerfile anlegen reicht nicht, weil das dem Compiler nur sagt, es soll so eine Variable geben. Erst durch anlegen im C++ File wird diese auch wirklich erst im Speicher erstellt.

[molleonair]

und was ist mit der
void (*TrackControl::sendmidi)(byte) = 0;

in der cpp ist das keine Intitialisierung ?

[shedepe]

Schau dir mal an was du geposted hast. Da steht das in einer Funktion drin.
void TrackControl::setCallback (void (*pCallbackFunction)(byte)){
void (*TrackControl::sendmidi)(byte) = 0;

[molleonair]

Sorry das ich dir da nicht folgen kann...

du hast geschrieben das diese Zeile in der cpp fehlt ..die einzige cpp ist TrackControl.cpp bestehend aus 2 Funktionen .. also muss ich deine Zeile zwangsweise ja in eine Funktion legen.
Danach hab ichs versuchsweise mal in der setup im .ino file probiert (vieleicht hast du ja die gemeint ) und habe trotzdem noch dieselbe Fehlermeldung.

(dein Code snippet aus Post #2 kompiliert auch nicht fehlerfrei.)

Ich hoffe hier findet sich noch jemand im Forum der in der Lage ist die 2 Zeilen Code die ich falsch habe zu berichtigen so daß ich das Konzept verstehen kann.
Ich Suche jetzt schon mehrere Tage im Internet und finde keine Lösung für eine statische callback Methode.

An alle noch eine schönes Wochenende

[shedepe]

Dein Problem ist: Du willst eine statische Variable anlegen. Eine statische Variable im Headerfile anzulegen reicht nicht aus, weil aus einem Headerfile heraus der Compiler keine Speicherallokation durchführen kann. Deshalb musst du für eine statische Variable im allgemeinen Scope des C++ Files (Also nicht in einer funktion drin -> Da solltest du dir auch wirklich noch mal den Link den ich dir geposted habe durchlesen, ich poste das ja nicht umsonst).

D.h. diese Zeile solltest du außerhalb des Funktionsscopes haben:
void (*TrackControl::sendmidi)(byte) = 0;

[molleonair]

Hallo shepede ...
wie schon gesagt ich habe diese zeile in der setup im .ino file probiert die Fehlermeldung ist die gleiche.
Und in der CPP sind nur 2 Funktionen wo soll ich denn da außerhalb was hinschreiben.
kannst du mir nicht konkret sagen wo genau diese Zeile stehen muss damit es funktioniert ?
und den Link hab ich mehr als einmal gelesen.

[shedepe]

Dann solltest du vllt auch danach googlen was der globale Scope eines C++ Files ist.
Du gehst hin und schreibst direkt unter dem #include <dein headerfile> die Zeile für die Statische Variable hin. So steht das auch in dem Link und gehört leider so sehr zu den Grundlagen, dass ich dir noch mal ein gutes Tutorial anraten würde.

Also um es noch mal klar zustellen.

#include <TrackControl.h>

void (*TrackControl::sendmidi)(byte) = 0;

void TrackControl::setCallback (void (*pCallbackFunction)(byte)){
}