Extension: Enum kann mit char*

Gast #4541319 
Hallo,
ich wollte von euch mal ein Urteil zu meinem "Codekonstrukt"..
Die Problemstellung war ich bekomme von einem Endgerät einen char* 
übermittelt dieses sollte von einem int zugewiesen werden und der index 
bzw die zahl wird zu einem Enum...

Am Programm Anfang wird der Mode gesetzt und im laufe wollte ich mit dem 
Enum typ vergleichen  if( log == Mode ) then.....

1
 

2


3
//   --------Set mode-------

4
uint8_t Mode = 0 ;

5
enum MODE{ NO_OPTION = 0, LOG = 1, DIAGNOSTICS = 2 , GETFILE = 3 , SETFILE = 4 };

6


7
const char* getEnumMODEString[] = {

8
     "NO_OPTION",

9
     "LOG",

10
     "DIAGNOSTICS",

11
     "GETFILE",

12
     "SETFILE",

13
};

14


15
uint8_t cmpEnum( char*c){

16
   

17
    for (uint8_t i = 0 ; i >= sizeof(c)/sizeof(char); i++){ // "sizeof(c)/sizeof(char)" get elements of char* Array

18
        if (strcmp(c , getEnumMODEString[i]) !=0 ) return i ;

19
    }

20
}

21


22
bool setMode(char* c ){

23
    switch (cmpEnum(c)) {

24
        case LOG:

25
            Mode = LOG;

26
            return true;

27
        case DIAGNOSTICS:

28
            Mode = DIAGNOSTICS;

29
            return true;

30
        case GETFILE:

31
            Mode = GETFILE;

32
            return true;

33
        case SETFILE:

34
            Mode = SETFILE;

35
            return true;

36
        default:

37
            Mode = NO_OPTION;

38
//         set error handler

39
            return false;

40
    }

41
}

42


43
uint8_t getMode(){

44
    return Mode;

45
}

46


47


48
.......

49
char mode[10] = "LOG";     

50
                if (!setMode(mode)) { // set mode 

51
//                  no mode, error???

52
                }

53
......

Was sagt ihr dazu ?
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Langsamer Modus Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Melden Danke
Gast 
Olaf schrieb:
> uint8_t cmpEnum( char*c){
>
>     for (uint8_t i = 0 ; i >= sizeof(c)/sizeof(char); i++){ //
> "sizeof(c)/sizeof(char)" get elements of char* Array
>         if (strcmp(c , getEnumMODEString[i]) !=0 ) return i ;
>     }
> }

Das sieht irgendwie ziemlich kaputt aus. c ist hier ein char pointer, 
sizeof(c) sollte also auf 32bit Systemen immer 4, auf 64bit Systemen 
immer 8 liefern. Was genau soll in der Schleife passieren? Ein 
Durchlaufen aller Möglichkeiten? Die sind ja nur in getEnumMODEString 
gelistet. "sizeof(getEnumMODEString)/sizeof(char*)" würde schon viel 
mehr Sinn machen, aber auch dann muss < für die Schleife verwendet 
werden. Und strcmp muss auf 0 Vergleichen. 0 heißt "0 Unterschied".

Olaf schrieb:
> case LOG:
>             Mode = LOG;

Diese ganzen Konstrukte kannst du dir sparen, gib einfach außerhalb der 
Schleife der cmpEnum Funktion als Return Wert "NO_OPTION" zurück. Dann 
kannst du nur für die Fälle wo du reagieren willst switch cases machen 
(in dem Beispiel is das nur case NO_OPTION:) und für den Rest (default:) 
immer MODE = <was immer cmpEnum returnt>
Antwort Zitieren Bearbeiten Löschen Melden Danke 
Wie Bernhard Kerbl bereis erwähnte, wird mit sizeof(c)
 aka sizeof(char*) immer die grösse eines Pointers zurückgegeben. Du 
brauchst aber die Grösse von getEnumMODEString. Ausserdem kann man 
setMode massiv vereinfachen. Ich würde es so versuchen: (alles 
ungetestet)

1
#include <assert.h>

2
#include <stddef.h>

3


4
// Define modes (must be valid identifiers)

5
#define MOD_MODE_LIST(X) \

6
  X(NO_OPTION), \

7
  X(LOG), \

8
  X(DIAGNOSTICS), \

9
  X(GETFILE), \

10
  X(SETFILE)

11


12
// Use a modul specific prefix for the enum values to prevent colisions with other macro names. (MOD_ in this example)

13


14
#define MOD_MODE_LIST_ENUM(X) \

15
  MOD_ ## X,

16


17
#define MOD_MODE_LIST_STRING(X) \

18
  #X,

19


20
enum MOD_Mode {

21
  MOD_MODE_LIST(MOD_MODE_LIST_ENUM)

22
  MOD_MODE_COUNT,

23
  MOD_MODE_INVALID = MOD_MODE_COUNT

24
};

25


26
static enum MOD_Mode mode = MOD_NO_OPTION;

27


28
const char* modeNameList[] = {

29
  MOD_MODE_LIST(MOD_MODE_LIST_ENUM)

30
};

31


32
static_assert(

33
  sizeof(modeNameList)/sizeof(*modeNameList) == MOD_MODE_COUNT,

34
  "Number of mode enums and number of mode name list must match"

35
);

36


37
enum MOD_Mode cmpEnum( char* c ){

38
  enum MOD_Mode i;

39
  for( i=0; i < MOD_MODE_COUNT; i++ ){

40
    if( strcmp( c, modeNameList[i] ) != 0 )

41
      break;

42
  }

43
  return i;

44
}

45


46
bool setMode( char* c ){

47


48
  enum MOD_Mode newMode = cmpEnum(c);

49


50
  if( newMode == MOD_MODE_INVALID )

51
    return false;

52


53
  mode = newMode;

54


55
  return true;

56
}

57


58
enum MOD_Mode getMode(){

59
  return mode;

60
}
Antwort Zitieren Bearbeiten Löschen Melden Danke
Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 10 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.