Callback Funktion mit static function pointer

[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;