code optimieren stringarray, enum

[optimierer (Gast)]

char *event_id_string[] = {
              "Tor auf",
              "Tor zu",
              "Fenster auf",
              "Fenster zu",
              "Türe auf",
              "Türe zu" }

enum event_id_code {
              TORAUF,
              TORZU,
              FENSTERAUF,
              FENSTERZU,
              TUEREAUF,
              TUEREZU }
              
  
switch (event_id_code) {

  case TORAUF    : printf(event_id_string[TORAUF]); break:
  case TORZU     : printf(event_id_string[TORZU]); break;
  case FENSTERAUF: printf(event_id_string[FENSTERAUF]); break;
  case FENSTERZU : printf(event_id_string[FENSTERZU); break;
  case TUEREAUF  : printf(event_id_string[TUEREAUF]); break;
  case TUEREZU   : printf(event_id_string[TUEREZU]); break;
  }

Wie würdet ihr das angehen?
So muss man immer aufpassen, bei der Erstellung der enumeration den 
richtigen Stringfortlauf einzuhalten, und wenn da was vertauscht wird, 
sieht es nicht mehr gut aus.

Fängt ihr mit der enumeration an, und dann die Stringliste
oder ganz anders?

[Yalu X. (yalu) (Moderator)]

Falls C: Vielleicht helfen dir die "designated array initializers"
weiter. In diesem Beispiel liegen die Strings im Array in der gleichen
Reihenfolge wie die Werte im Enum, auch wenn es optisch anders aussieht:

enum event_id_code {
  TORAUF,
  TORZU,
  FENSTERAUF,
  FENSTERZU,
  TUEREAUF,
  TUEREZU
};

char *event_id_string[] = {
  [TORZU     ] = "Tor zu",
  [TUEREZU   ] = "Türe zu",
  [TORAUF    ] = "Tor auf",
  [FENSTERZU ] = "Fenster zu",
  [TUEREAUF  ] = "Türe auf",
  [FENSTERAUF] = "Fenster auf"
};

Der Ausdruck

event_id_string[FENSTERAUF]

liefert also immer "Fenster auf", egal wie sehr du die Zeilen in
event_id_code oder event_id_code durcheinanderwürfelst.

Natürlich müssen bei der Array-Initialisierung die Indizes in den
eckigen Klammern mit dem Text jeweils rechts daneben korrespondieren.
Das aber beim Einfügen neuer Elemente die einzelnen Zeilen i.Allg. nicht
geändert werden, besteht hier kaum die Gefahr von Fehlern. Falls doch
ein Fehler gemacht wurde, kann dieser schnell gefunden werden, da die
korrespondierenden Dinge ja direkt nebeneinander stehen.

[Rufus Τ. F. (rufus)]

Yalu X. schrieb:
> "designated array initializers"

Es sei darauf hingewiesen, daß auch 2016 noch C-Compiler existieren, die 
nichts neueres als C90 kennen, und da gibt es so etwas leider nicht.

Ein "berühmtes" Beispiel ist der Compiler, der zu Microsofts "Visual 
C++" gehört.

Der unterstützt C++11 (oder neuer), aber nichts neueres als C90:


https://herbsutter.com/2012/05/03/reader-qa-what-about-vc-and-c99/

[Yalu X. (yalu) (Moderator)]

Rufus Τ. F. schrieb:
> Der unterstützt C++11 (oder neuer), aber nichts neueres als C90:
>
> https://herbsutter.com/2012/05/03/reader-qa-what-about-vc-and-c99/

Der Artikel ist ja auch schon ein paar Tage alt :)

Seit der 2013er-Version soll es die designated Initializers wohl geben
und mittlerweile auch ein paar diesbezügliche Bugs gefixt worden sein.
Ich habe allerdings keinen MS-Compiler hier, wo ich das auf die Schnelle
mal ausprobieren könnte.

Außerdem: Wer programmiert auf einem PC unter Windows in C? ;-)

Microsoft hat die Priorität für den C-Modus ihres Compilers aus gutem
Grund herabgesetzt (scheint es sich jetzt aber doch wieder anders zu
überlegen).

[Mark B. (markbrandis)]

Yalu X. schrieb:
> Außerdem: Wer programmiert auf einem PC unter Windows in C? ;-)

Viele Leute, dank MinGW und Cygwin. :-)

[Rufus Τ. F. (rufus)]

Yalu X. schrieb:
> Außerdem: Wer programmiert auf einem PC unter Windows in C? ;-)

Wer beispielsweise Konsolenanwendungen schreiben möchte (wie z.B. 
Kommandozeilencompiler) oder Systemdienste etc.

Windows-GUI-Anwendungen in C zu schreiben à la Petzoldt ist definitiv 
außerhalb der Genfer Konvention.

[Bert3 (Gast)]

Auf jeden Fall mal das case

switch (event_id_code) {

  case TORAUF    : printf(event_id_string[TORAUF]); break:
  case TORZU     : printf(event_id_string[TORZU]); break;
  case FENSTERAUF: printf(event_id_string[FENSTERAUF]); break;
  case FENSTERZU : printf(event_id_string[FENSTERZU); break;
  case TUEREAUF  : printf(event_id_string[TUEREAUF]); break;
  case TUEREZU   : printf(event_id_string[TUEREZU]); break;
  }

durch

printf(event_id_string[event_id_code]);

ersetzen

[Rufus Τ. F. (rufus)]

Vorher aber überprüfen, ob "event_id_code" im gültigen wertebereich 
liegt. Das wird einem nicht abgenommen, auch wenn man einen enum 
verwendet.

[Kollege von oben (Gast)]

Du könntest dir ein Array anlegen mit einer eindeutigen Zuordnung, so 
kannst du sicher stellen das diese nicht verloren geht:

typedef struct _GateStates
{
  event_id_code id;
  char* text;
} GateStates

Deklaration:

static GateStates STATES[];

Definition:

GateStates STATES[] =
{ TORAUF, "Torauf" },
...

[Rufus Τ. F. (rufus)]

Kollege von oben schrieb:
> Definition:GateStates STATES[] =
> { TORAUF, "Torauf" },
> ...

Da hast Du ein Paar Klammern vergessen.

GateStates STATES[] =
{
  { TORAUF, "Torauf" },
  ...
};

[Daniel A. (daniel-a)]

Ich mache sowas manchmal mit Macros:

#define EVENT_ID_LIST \
  X( TORAUF, "Tor auf" ), \
  X( TORZU, "Tor zu" ), \
  X( FENSTERAUF, "Fenster auf" ), \
  X( FENSTERZU, "Fenster zu" ), \
  X( TUEREAUF, "Türe auf" ), \
  X( TUEREZU, "Türe zu" )

#define X(ID,STRING) STRING
char *event_id_string[] = { EVENT_ID_LIST };
#undef X

#define X(ID,STRING) ID
enum event_id_code { EVENT_ID_LIST };
#undef X

[Kollege von oben (Gast)]

Rufus Τ. F. schrieb:
> Da hast Du ein Paar Klammern vergessen.

Mea culpa. Was Rufus schreibt ist richtig.