C++ Pointer auf Klasseninterne Funktionen

[Markus F. (Firma: Turck Beierfeld GmbH) (markup)]

Schönen guten Abend,

in versuche recht neu in c++ einzusteigen und stehe derzeit vor einem 
Problem. Allgemein versuche ich die wieder Verwendbarkeit des Codes im 
Auge zu behalten.

Ziel: Es soll ein Modul geschrieben werden, welches nur an einer Stelle 
angepasst werden muss und dann dementsprechend für andere Projekte 
verwendet werden kann. Diese Prüfungen sind nicht statisch und können 
sich selbst bei einer Variante, je nach der Konfiguration unterscheiden. 
Bei mir geht es um eine typische Busverbindung mit Index basierenden 
Zugriff.

Ich habe dazu einmal eine stark vereinfachte Klasse geschrieben. Diese 
hat derzeit 2 Check Funktionen (biggerCheck und smallerCheck). Es soll 
jetzt in der Hauptfunktion eine Idee und 2 Zahlen übergeben werden und 
die Funktion schaut nach ob der Eintrag in der Parameter Liste vorhanden 
ist und führt die hinterlegte Check Funktion aus.

Wenn ich dies versuche zu Kompilieren erhalte ich die Meldung:

error C2064: Ausdruck ergibt keine Funktion, die 2 Argumente übernimmt

Diese Meldung kommt von der Zeile

ret = ((myParameterSet[i].Checker)(num1, num2));

Also werde ich etwas bei der Definition oder beim Aufrufen verkehrt 
gemacht werden, ich komme derzeit jedoch nicht weiter. Ich hatte ebenso 
die Meldung bei der Schreibweise:

ret = myParameterSet[i].Checker(num1, num2);

Kann mir dort vielleicht einer kurz auf die Sprünge helfen?

class ParameterTest
{
  #define PARAM_LIST_SIZE 4

  typedef  bool(ParameterTest::*functor_testFnc) (uint8_t, uint8_t);
  typedef struct
  {
    uint8_t      refIndex;
    functor_testFnc  Checker;    /* check if the written value in range */
  }ParameterSet_Type;

  bool biggerCheck(uint8_t num1, uint8_t num2)
  {
    bool ret;
    if (num1 > num2)
    {
      ret = true;
    }
    else
    {
      ret = false;
    }
    return(ret);
  }

  bool smallerCheck(uint8_t num1, uint8_t num2)
  {
    bool ret;
    if (num1 < num2)
    {
      ret = true;
    }
    else
    {
      ret = false;
    }
    return(ret);
  }


  const ParameterSet_Type myParameterSet[PARAM_LIST_SIZE] =
  {
    {  0, &ParameterTest::biggerCheck },
    {  1, &ParameterTest::smallerCheck },
    {  21, &ParameterTest::biggerCheck },
    {  42, &ParameterTest::smallerCheck }
  };

public:
  bool doCheck(uint8_t id, uint8_t num1, uint8_t num2)
  {
    bool ret = false;
    for (uint8_t i = 0; i < PARAM_LIST_SIZE; i++)
    {
      if (myParameterSet[i].refIndex == id)
      {
        ret = ((myParameterSet[i].Checker)(num1, num2));
      }
    }
    return(ret);
  }
};

[Niklas G. (erlkoenig)]

Auf welcher Instanz von "ParameterTest" soll denn "Checker" aufgerufen 
werden? Du musst eine Instanz angeben, wenn du einen 
Member-Funktions-Zeiger aufrufen willst.
Wenn der Aufruf ohne Instanz erfolgen soll, mach die Funktionen "static" 
und verwende normale Funktions-Zeiger statt Member-Funktions-Zeiger.

[Jemand (Gast)]

Niklas G. schrieb:
> Du musst eine Instanz angeben, wenn du einen
> Member-Funktions-Zeiger aufrufen willst.

Syntaxbeispiel der Vollständigkeit halber:

ret = (this->*myParameterSet[i].Checker)(num1, num2);

[Markus F. (Firma: Turck Beierfeld GmbH) (markup)]

Der Checker soll voraussichtlich in einer Klasse aufgerufen werden, 
welche alle Information von ParameterTest erbt.

Mit statischen Funktionen funktioniert das Konstrukt in c recht einfach. 
Jetzt muss ich jedoch auf gewisse Daten in andere Klassen bei mir in den 
Checks zugreifen und das hätte ich versucht über friends zu lösen.

Stehe anderen Lösungen aber auch offen gegenüber.

[Markus F. (Firma: Turck Beierfeld GmbH) (markup)]

> Syntaxbeispiel der Vollständigkeit halber:
>

ret = (this->*myParameterSet[i].Checker)(num1, num2);

Super das läuft ohne Kompilierfehler durch. Dann geht es morgen ums 
testen.

Vielen Dank!

[Niklas G. (erlkoenig)]

Markus F. schrieb:
> Jetzt muss ich jedoch auf gewisse Daten in andere Klassen bei mir in den
> Checks zugreifen und das hätte ich versucht über friends zu lösen.

Das, und public/private, hat überhaupt nichts mit dem Problem zu tun. Es 
geht nur darum, auf welcher Instanz die Funktion aufgerufen werden soll. 
Wahrscheinlich ist Jemand's Lösung das was du suchst.

[Arc N. (arc)]

Etwas moderner darf die Lösung schon sein oder darf/kann nur C++ vor 
C++11 verwendet werden?

using CheckFun = std::function<bool(uint8_t, uint8_t)>;

struct ParameterSet {
    uint8_t RefIndex;
    CheckFun Function;
};

class ParameterTest {
    CheckFun larger = [](uint8_t n1, uint8_t n2) { return n1 > n2; };
    CheckFun smaller = [](uint8_t n1, uint8_t n2) { return n1 < n2; };
    std::vector<ParameterSet> parameters = { 
        { 0, larger }, 
        { 1, smaller }, 
        { 21, larger }, 
        { 42, smaller } 
    };
public:
    bool DoCheck(uint8_t id, uint8_t n1, uint8_t n2) {
        for (auto p : parameters) {
            if (p.RefIndex == id) return p.Function(n1, n2);
        }
        return false;
    }
};

[cppbert3 (Gast)]

Markus F. schrieb:
> bool biggerCheck(uint8_t num1, uint8_t num2)
>   {
>     bool ret;
>     if (num1 > num2)
>     {
>       ret = true;
>     }
>     else
>     {
>       ret = false;
>     }
>     return(ret);
>   }

Auch wenn das vielleicht nur ein Beispiel ist, aber

return num1 > num2;

ist mehr als ausreichend und verständlich

[Markus F. (Firma: Turck Beierfeld GmbH) (markup)]

Arc N. schrieb:
> Etwas moderner darf die Lösung schon sein oder darf/kann nur C++ vor
> C++11 verwendet werden?

Vielen Dank, die Lösung gefällt mir auf anhieb besser. Ich hatte mich 
derzeit noch daran gescheut Funktionen aus der STL zu nehmen, da ich 
gerade erst angefangen habe, mich in dieser einzuarbeiten.

Ich bin derzeit am schauen, scheinbar unterstützt der GCC (V 6.3.1) aus 
dem Atmel Studio C++11, also werde ich das gleich umbauen.

[cppbert3 (Gast)]

cppbert3 schrieb:
>> bool biggerCheck(uint8_t num1, uint8_t num2)
>>   {
>>     bool ret;
>>     if (num1 > num2)
>>     {
>>       ret = true;
>>     }
>>     else
>>     {
>>       ret = false;
>>     }
>>     return(ret);
>>   }

als Reminder: das würde dir bei einem Review von jedem guten Entwickler 
angekreidet werden

kannst auch

bool ret = num1 > num2;
return ret;

schreiben aber alles andere ist wirklich totaler murks - zeigt das dir 
vielleicht nicht klar ist das num1 > num2 direkt auf bool evaluiert d.h. 
dein Code sachen macht die der Kompiler genau so auch macht

[Mark B. (markbrandis)]

cppbert3 schrieb:
> Auch wenn das vielleicht nur ein Beispiel ist, aber
>
> return num1 > num2;
>
> ist mehr als ausreichend und verständlich

Woher kommt dieser Aberglaube, dass Code automatisch besser wird wenn 
man ihn in weniger Zeilen quetscht? 🤔

[mh (Gast)]

Mark B. schrieb:
> cppbert3 schrieb:
>> Auch wenn das vielleicht nur ein Beispiel ist, aber
>>
>> return num1 > num2;
>>
>> ist mehr als ausreichend und verständlich
>
> Woher kommt dieser Aberglaube, dass Code automatisch besser wird wenn
> man ihn in weniger Zeilen quetscht? 🤔

Man könnte genauso gut fragen:
Woher kommt dieser Aberglaube, dass Code automatisch besser wird wenn
man ihn in über mehr Zeilen verteilt? 🤔

Für mich ist

bool biggerCheck(uint8_t num1, uint8_t num2)
{
    return num1 > num2;
}

deutlich besser lesbar als

bool biggerCheck(uint8_t num1, uint8_t num2)
{
    bool ret;
    if (num1 > num2)
    {
        ret = true;
    }
    else
    {
        ret = false;
    }
    return(ret);
}

Vor allem wenn man bedenkt, dass sich biggerCheck und smallerCheck in 
nur einem Zeichen unterscheiden.

[Yalu X. (yalu) (Moderator)]

Mark B. schrieb:
> cppbert3 schrieb:
>> Auch wenn das vielleicht nur ein Beispiel ist, aber
>>
>> return num1 > num2;
>>
>> ist mehr als ausreichend und verständlich
>
> Woher kommt dieser Aberglaube, dass Code automatisch besser wird wenn
> man ihn in weniger Zeilen quetscht? 🤔

Nicht automatisch, aber in diesem Fall schon. Da wird auch nichts
gequetscht, sondern nur sinnloses Zeug weggelassen.

Die meisten Leute schreiben ja auch nicht

  if (x + y == 0)
    sum = 0;
  else if (x + y == 1)
    sum = 1;
  else if (x + y == 2)
    sum = 2;
  else if (x + y == 3)
    sum = 3;
  ...

sondern "quetschen" das zu einer Zeile zusammen:

  sum = x + y;

[rµ (Gast)]

Markus F. schrieb:
> Arc N. schrieb:
>> Etwas moderner darf die Lösung schon sein oder darf/kann nur C++ vor
>> C++11 verwendet werden?
>
> Vielen Dank, die Lösung gefällt mir auf anhieb besser. Ich hatte mich
> derzeit noch daran gescheut Funktionen aus der STL zu nehmen, da ich
> gerade erst angefangen habe, mich in dieser einzuarbeiten.
>
> Ich bin derzeit am schauen, scheinbar unterstützt der GCC (V 6.3.1) aus
> dem Atmel Studio C++11, also werde ich das gleich umbauen.

Auf dem kleinen µC ist Vorsicht geboten bei der Verwendung von 
std::function, das benötigt unter Umständen dynamisch allozierten 
Speicher.

So wie es aussieht greifen smallerCheck und biggerCheck auf keine 
Instanzvariablen oder -methoden zu, dann könnte man die beiden Methoden 
also schon "static" machen, man benötigt dann auch keinen Zeiger auf die 
Instanz mehr.

[cppbert3 (Gast)]

Mark B. schrieb:
> cppbert3 schrieb:
>> Auch wenn das vielleicht nur ein Beispiel ist, aber
>>
>> return num1 > num2;
>>
>> ist mehr als ausreichend und verständlich
>
> Woher kommt dieser Aberglaube, dass Code automatisch besser wird wenn
> man ihn in weniger Zeilen quetscht? 🤔

Vielleicht weil diese 10 Zeilen exakt 0 Mehrwert bringen?

wenn die Menge des Code hier was "bringt" - warum dann nicht gleich die 
Zeile und Zeichen-Maximierung?

bool biggerCheck(uint8_t num1, uint8_t num2)
{
    bool ret;
    bool vergleich;
    vergleich = num1 > num2;
    if (vergleich == true)
    {
        ret = true;
    }
    else if(vergleich == false)
    {
        ret = false;
    }
    return(ret);
}

weils noch sinnfreier ist

[mh (Gast)]

cppbert3 schrieb:
> wenn die Menge des Code hier was "bringt" - warum dann nicht gleich die
> Zeile und Zeichen-Maximierung?

Wenn schon, dann mit

bool vergleich1 = false;
bool vergleich2 = false;
vergleich1 = num1 > num2;
vergleich2 = num1 <= num2;