Telnet Server - Menü Auswahl effizienter lösen

[Ardu (Gast)]

Moin Moin,
ich arbeite zur Zeit an einem Telnet Server, dieser läuft auch ziemlich 
gut, jedoch in er Menü Auswahl nicht wirklich effizient.

Siehe Code:

void parseCommand() {

  if(cmd.equals("quit")) {
      client.stop();
      connected = false;
  
      else if(cmd.equals("1 on")) 
      {
        digitalWrite(2, HIGH);
      }
      
      else if(cmd.equals("1 off")) 
      {
        digitalWrite(2, LOW);
      }
      else if(cmd.equals("2 on")) 
      {
        digitalWrite(3, HIGH);
      }
      else if(cmd.equals("2 off")) 
      {
        digitalWrite(3, LOW);
      }
      else if(cmd.equals("3 on")) 
      {
        digitalWrite(4, HIGH);
      }
      else if(cmd.equals("3 off")) 
      {
        digitalWrite(4, LOW);
      }
      else if(cmd.equals("4 on")) 
      {
        digitalWrite(5, HIGH);
      }
      else if(cmd.equals("4 off")) 
      {
        digitalWrite(5, LOW);
      }
      
      else if(cmd.equals("all on"))
      {
        digitalWrite(2, HIGH);
        digitalWrite(3, HIGH);
        digitalWrite(4, HIGH);
        digitalWrite(5, HIGH);
      }
      else if(cmd.equals("all off"))
      {
         digitalWrite(2, LOW);
         digitalWrite(3, LOW);
         digitalWrite(4, LOW);
         digitalWrite(5, LOW);
      }
      
      
      else 
      {
        server.println("Invalid command, type help");
      }
  cmd = "";
}

ich habe es mit switch case versucht aber da Zeigt mir mein Compiler den 
Mittelfinger. Es läuft auf einem Arduino DUE ( ARM Cortex M3)

[Ardu (Gast)]

hm :(

[Daniel S. (daniel_s49)]

Ist die Frage jetzt, wie du diesen Code effizienter gestalten kannst 
oder ist die Frage warum dieser Code nicht compiliert?
Letzteres läge wohl an der fehlenden } vor dem ersten else if.

Für ersteres würde ich vermutlich den String am Leerzeichen trennen 
(split oder so - kenn die Arduino-Libs da nicht).
Wenn vorne zwischen '1' und '5', dann umwandeln in Zahl ('5'-'1'=5 - 
alternativ atoi()). Wenn hinten "off" dann LOW merken, sonst HIGH und 
dann mit den beiden Werten die digitalWrite() aufrufen.

[Stephan (Gast)]

Ich würde den String auch decodieren macht das Programm universeller und 
den Code besser wartbar.

[Daniel A. (daniel-a)]

Ich weiss nicht, ob es Effizienter ist, aber hier hatte ich einen 
iterativen ansatz vorgestellt:
Beitrag "Re: Arduino Serial ganze wörter abfragen"

Möglicherweise kann man hier etwas ähnliches machen.

[Ardu (Gast)]

Daniel S. schrieb:
> Ist die Frage jetzt, wie du diesen Code effizienter gestalten
> kannst
> oder ist die Frage warum dieser Code nicht compiliert?
> Letzteres läge wohl an der fehlenden } vor dem ersten else if.
>
> Für ersteres würde ich vermutlich den String am Leerzeichen trennen
> (split oder so - kenn die Arduino-Libs da nicht).
> Wenn vorne zwischen '1' und '5', dann umwandeln in Zahl ('5'-'1'=5 -
> alternativ atoi()). Wenn hinten "off" dann LOW merken, sonst HIGH und
> dann mit den beiden Werten die digitalWrite() aufrufen.

ne es funktioniert ja, ich suche nach alternativen, bzw weniger schreib 
arbeit bzw weniger if Anweisungen.

Daniel A. schrieb:
> Ich weiss nicht, ob es Effizienter ist, aber hier hatte ich einen
> iterativen ansatz vorgestellt:
> Beitrag "Re: Arduino Serial ganze wörter abfragen"
>
> Möglicherweise kann man hier etwas ähnliches machen.

werde es mir mal angucken :)

[nicht“Gast“ (Gast)]

Moin,

Du kannst die Kommandos im einem struckt ablegen und dazu einen 
Funktionzeiger.

typedef struck {
   char *command,
   void (*commandFunction)(int arg1, bool arg2 )
} command;

Dann machst du dir ein Array aus Commands

command commandList[] = { { “1 on“, funktionZumAusführen},
                          { “1 off“, NeudefInition}};

Dann kannst du einfach über das Array iterieren und bei einem Treffer 
mit Equal die funktion ausführen.

Ich hoffe, ich konnte den Ansatz verständlich machen. Da ist leider 
ziemlich umständlich mit den Tablet.

Grüße

[Bernd K. (prof7bit)]

Du könntest den String so parsen dass Du einerseits ein Kommando "on" 
oder "off" (oder noch andere Kommandos) erkennst und andererseits deren 
Argument(e) (in diesem Falle eine Zahl oder "all") (oder kein Argument 
bei manchen anderen Kommandos).

Dann musst Du Dir eine sinnvolle simple und konsistente und einfach zu 
parsende Grammatik für alle Deine jetzigen und zukünftigen Kommandos 
ausdenken (zum Beispiel wäre es evtl leichter zu parsen wenn das 
Kommando zuerst und dann erst die (optionalen) Argumente kommen.

[Andi (Gast)]

setz den command an erste stelle und schreib einen generischen handler.
Sprich

void onHandler(char * argp, int argLeng) {

//zahl zu beginn von argp auswerten und  pin setzen
}
  
struct cmdDef {
  char * cmd;
  void (fun*)(char * argp, int argLen);
};

struct cmdDef handlers[] = {
  {"on", &onHandler},
  {"off", &ofHandler}
};

[Andi (Gast)]

Du prüfst dann in einem for-loop, ob der befehlsstring mit einem der 
commands beginnt. Ist das der fall, wird der entsprechende handler 
aufgerufen.
Dabei ist argp ein pointer auf das erste zeichen nach dem befehl, argLen 
die anzahl der noch bleibenden zeichen.

[Klaus (Gast)]

Ardu schrieb:
> else if(cmd.equals("1 on"))

Aber in die Doku schreiben, daß nur exakt 1 Leerzeichen zwischen der 
Zahl und dem Kommando zulässig ist.

MfG Klaus