Problem mit enum in C++

[Marcell (Gast)]

Hallo,
ich habe ein verständisproblem zu enums und hoffe, dass ihr mir dabei 
helfen könnt.

Ich definiere in einer Klasse mit dem Namen 'day' z.B. zwei Enums:

enum days{ monday=0,
           tuesday,
           wednesday
           thursday,
           friday,
           saturday,
           sunday}

enum time{ morning=0,
           midday,
           afternoon,
           night,
           midnight}

In meiner main-function definiere ich ein Objekt, welches somit diese 
beiden enums kennt.

aber wie kann ich jetzt die einzelnen inhalte abfragen?


ich würde gerne folgendes machen (dazu habe ich mir zwei variablen 
erstellt (tag, zeit)

if(tag == day::monday)
{ // do sth.
}
else if(tag== day::tuesday)
{
  if(zeit==time::morning)
    {//do sth.
    }
  else if(zeit==time::midday)
    {// do sth. different
    }
}

komme da aber absolut nicht weiter.
Marcell

[Slippin J. (gustavo_f)]

Dein Enum heißt doch days. Also muss du doch darauf abfragen. Wenn deine 
Klasse day (ungünstiger Name!), dann in etwa so day::days::monday

[Rufus Τ. F. (rufus)]

Enums sind keine Klassen, daher ist der ::-Oberator nicht korrekt.

Es sei denn, Deine enums gehören zu einer Klassendeklaration:

class bla
{
public:
  enum days{ monday=0,
           tuesday,
           wednesday
           thursday,
           friday,
           saturday,
           sunday};

  enum time{ morning=0,
           midday,
           afternoon,
           night,
           midnight};

etc. ...

}

Dann wird der ::-Operator verwendet, aber nicht mit dem Namen des 
jeweiligen enums, sondern mit dem der Klasse, in deren Namensraum der 
enum deklariert wurde.

Im Beispiel also

if (time == bla::midday)
...

Das ist wegen potentieller Namenskonflikte nicht besonders elegant.

In C11 gibt es eine Erweiterung namens "enum class", damit ist die von 
Dir verwendete Syntax verwendbar:

http://stackoverflow.com/questions/12183008/how-to-use-enums-in-c

[Rolf M. (rmagnus)]

Marcell schrieb:
> ich würde gerne folgendes machen
 [...]
> komme da aber absolut nicht weiter.

Was meinst du damit? Wenn du das gerne so machen würdest, dann mach es 
doch einfach so. Sieht alles korrekt aus.

[Klaus K. Klops (Gast)]

War doch schon fast richtig.

class Foo
{
public:

  enum days {
    monday = 0,
    tuesday,
    wednesday,
    thursday,
    friday,
    saturday,
    sunday
  };

  enum time {
    morning = 0,
    midday,
    afternoon,
    night,
    midnight
  };

  // Verwendung innerhalb der Klasse.
  void bla()
  {
    days tag = days::monday;

    // days tag2 = monday;

    time zeit = time::morning;

    if (tag == days::monday)
    {
    }
    else if (tag == days::tuesday)
    {
      if (zeit == time::morning)
      {
      }
      else if (zeit == time::midday)
      {
      }
    }
  }
};

// Verwendung außerhalb der Klasse.
int main()
{
  Foo::days tag = Foo::days::monday;
  
  // Foo::days tag2 = Foo::monday;
  
  Foo::time zeit = Foo::time::morning;

  if (tag == Foo::days::monday)
  {
  }
  else if (tag == Foo::days::tuesday)
  {
    if (zeit == Foo::time::morning)
    {
    }
    else if (zeit == Foo::time::midday)
    {
    }
  }
};

Dass

days tag2 = monday;
bzw.
Foo::days tag2 = Foo::monday;

auch funktionieren würde, ist eher ein Nachteil von enums, besonders von 
globalen. Darum setzt man sie (also globale enums) oft in eigene 
Sub-Namespaces. Mit "enum class" ist das allerdings nicht mehr nötig.

> In meiner main-function definiere ich ein Objekt, welches somit diese
> beiden enums kennt.

In diesem Sinn kennt die Klasse die Enums. Also 
DeineKlasse::time::morning oder DeineKlasse::morning, nicht 
DeinObjekt.time::morning oder wie auch immer du es probiert hast.

[Manno (Gast)]

> Marcell schrieb:
>  [...]
> Sieht alles korrekt aus.

Die bedingte Verzweigung mag korrekt ausschauen, wäre aber 
handwerklicher Pfusch.
Erstmal würde sich hier bei einem enum eine switch-case Abfrage 
anbieten.
Die würde es schon mal etwas leserlicher machen. Das eigentliche Problem 
bestünde aber weiterhin:
7 * 5 = 35 mögliche Ergebnisse = sehr viel, sehr unschöner Code.

Und Morgen kommt jemand und möchte genaue Urzeiten für 'time' definieren 
oder weitere hinzufügen. Dann hätte man schnell 400+x Zeilen mit 
Abfragen und inkludierten Code; und mit Sicherheit ebenso doppelten.

Soetwas möchte man nicht der Nachwelt hinterlassen.

ber lösbar z.B. mit einem Zustandsautomaten nach Fowler et al.