command liste

[Philipp Karbach (Gast)]

baue mir gerade eine kleine shell über die UART. Ich hab schon oft sowas
gemacht aber sonst immer die commandliste von hand gebaut mit switch und
so, geht, find ich aber irgendwie hässlich! Das ganze kann man ja auch
universeller machen. ich sehe das oft mal da wird dann eine liste
gebaut wie:
(pseudocode)
cmdlist = { "setup", setup_func, .... } usw.

wenn ich jetzt durch diese liste durchloopen würde, dann immer den
commandkey ("setup") mit der eingabe vergleiche wie würde ich dann
klarmachen, dass setup_func eine funktion ist die dann aufgerufen
werden soll? Hat jemand ne idee? danke :)

[Rolf Magnus (Gast)]

Du definierst das Element als Funktionszeiger. Vielleicht hilft
folgendes:

struct cmd
{
    const char* name,
    void (*func)(void);
};

struct cmd cmdlist[] =
{
    { "setup", setup_func},
    { 0, 0}
};

...

int execute(const char* command)
{
    for (cmd* c = cmdlist; cmd->name != 0; ++cmd)
    {
        if (strcmp(cmd->name, command))
        {
            cmd->func();
            return 1;
        }
    }
    printf("Unknown command\n");
    return 0;
}

Wenn du noch Parameter übergeben willst, mußt du dir halt überlegen,
wie du das einbaust.

[Philipp Karbach (Gast)]

danke das ist genau was ich haben wollte!

[Rolf Magnus (Gast)]

Dann ist ja gut. Das Finden der (wie ich gerade sehe) zahlreichen Fehler
bleibt als Übung ;-)

[Philipp Karbach (Gast)]

aber warum krieg ich denn storage size of cmdlist isnt known?

[Philipp Karbach (Gast)]

und strcmp returned 0 wenn der vergleich positiv ist ;)

[Philipp Karbach (Gast)]

so hab glaub ich alle gefunden :p

struct cmd
{
  const char* name;
  void (*func)(void);
};

void poll()
{
  float val = ((float)ADC/1024.0f)*5.0f;
  char string[10];
  dtostrf(val,5,1,string);
  printf("Polling.....\nADC: %sV\n",string);
}


struct cmd cmdlist[] =
{
  { "poll", poll },
  { 0, 0 }
};


int execute(const char* command)
{
  struct cmd *c = 0;
  for (c = cmdlist; c->name != 0; ++c) {
    if (!strcmp(c->name, command)) {
      c->func();
      return 1;
    }
  }
  printf("Unknown command\n");
  return 0;
}