Case-Anweisung - Zwingen alle Elemente des Typen auszuprogrammieren

[DeddyH]

Wie wäre es mit einem ganz anderen Ansatz?

Delphi-Quellcode:

type
  TEnum = (t1, t2, t3);

const 
  EnumCheck: array[TEnum] of Boolean = (true, true, false);

Wenn jetzt TEnum um weitere Elemente erweitert wird, muss zwangsläufig auch die Konstante erweitert werden. Eine andere Möglichkeit wäre so etwas:

Delphi-Quellcode:

function EnumCheck(Value: TEnum): Boolean;
begin
  case Value of
    t1: Result := true;
    t2: Result := true;
    else
      raise EForgottenValue.Create('Hamwa nich was vergessen?');
  end;
end;

[lxo]

Zitat von DeddyH:

Wie wäre es mit einem ganz anderen Ansatz?

Delphi-Quellcode:

type
  TEnum = (t1, t2, t3);

const 
  EnumCheck: array[TEnum] of Boolean = (true, true, false);

Wenn jetzt TEnum um weitere Elemente erweitert wird, muss zwangsläufig auch die Konstante erweitert werden. Eine andere Möglichkeit wäre so etwas:

Delphi-Quellcode:

function EnumCheck(Value: TEnum): Boolean;
begin
  case Value of
    t1: Result := true;
    t2: Result := true;
    else
      raise EForgottenValue.Create('Hamwa nich was vergessen?');
  end;
end;

Das mit der Konstante mache ich auch schon.
Ich möchte aber gerne direkt beim case hängen bleiben.
Nur weil ich dann die Konstante erweitert habe, habe ich mein case ja nicht angepasst.
Wenn ich mehrere case-anweisungen habe die ich beachten muss, müsste ich ja so eine Zuweisung über jede case-Anweisung von dieser Enumeration machen.

Den zweiten Fall mache ich auch schon.
Das bekomme ich aber erst zur Laufzeit mit.
Hätte es halt gerne direkt beim kompilieren.
Unit-Test wäre das einzige was mir noch einfällt.
Hätte es aber wie gesagt gerne schon direkt beim kompilieren als Fehler, damit man direkt dahin geführt wird und das ausprogrammieren kann.

[Rollo62]

Zitat von lxo:

Hätte es halt gerne direkt beim kompilieren.

Zitat von lxo:

Hallo,
ich möchte bei einer Case-Anweisung einen Compiler-Fehler bekommen ....

Genau, beim Compilieren kommt der Fehler mit so einer Lösung
CEnum : array[ TEnum ] of String = ( 't1', 't2', 't3' ); //Knallt schon hier, wenn es nicht übereinstimmt Dass man das jetzt erweitern muss, ist ja genau Sinn des Crashes.
Nebenbei hätte so eine noch weitere Vorteile, also zum Beispiel Textausgabe, Rückgabe beliebiger Werte, etc.

Eventuell sucht Du ja sowas, etwas "Verboseres":

Delphi-Quellcode:

type
  TMyEnum = (Value0, Value1, Value2);

const
  ExpectedEnumSize = 3; // Erwartete Anzahl der Elemente in TMyEnum

{$IF     Ord(High(TMyEnum)) > (ExpectedEnumSize-1) }
  {$MESSAGE ERROR 'TMyEnum wurde erweitert!'       }
{$ELSEIF Ord(High(TMyEnum)) < (ExpectedEnumSize-1) }
  {$MESSAGE ERROR 'TMyEnum wurde verkleinert!'     }
{$IFEND}

[dummzeuch]

Zitat von Rollo62:

Genau, beim Compilieren kommt der Fehler mit so einer Lösung
CEnum : array[ TEnum ] of String = ( 't1', 't2', 't3' ); //Knallt schon hier, wenn es nicht übereinstimmt

Das Problem ist, dass man dann aber die betroffenen case-Statements alle manuell raussuchen muss, was nicht ganz so einfach ist, da dort ja nicht der Typ sondern eine Variable des Typs verwendet wird, man kann also nicht einfach ein grep auf den Namen des Typs machen. Wenn der Compiler bei jedem davon einen Fehler schmeißt, ist das einfacher. Kann ja auch gerne abschaltbar sein (wobei man das ja auch mit einem else erreichen könnte).

[himitsu]

Er muß ja keinen Fehler werfen, aber z.B. eine Warnung oder Info wäre eigentlich problemlos möglich, wenn bei einem Enum nicht alle Werte benutzt wurden.

Ja, natürlich auch abschaltbar, bzw. vielleicht über ein Attribut oder Keyword, explizit anforderbar,
denn es kommt vor, dass ich in manchen Case absichtlich garnicht alles verwende,
z.B. wenn an einer Stelle nur ein Teil benötigt wird und der Rest einfach nichts machen soll.

[dummzeuch]

Zitat von himitsu:

Ja, natürlich auch abschaltbar, bzw. vielleicht über ein Attribut oder Keyword, explizit anforderbar, denn es kommt vor, dass ich in manchen Case absichtlich garnicht alles verwende,
z.B. wenn an einer Stelle nur ein Teil benötigt wird und der Rest einfach nichts machen soll.

Dafür gibt's ja schon else. Dann muss man allerdings evtl. anderen Tools beibringen, dass ein leeres else durchaus OK ist.

Delphi-Quellcode:

case bla of
  b1: HandleB1;
  b2: HandleB2;
else
  // nix tun
end;

[Uwe Raabe]

Zitat von dummzeuch:

da dort ja nicht der Typ sondern eine Variable des Typs verwendet wird, man kann also nicht einfach ein grep auf den Namen des Typs machen.

Wenn man SCOPEDENUMS verwendet, schon...

[bernau]

Zitat von Uwe Raabe:

Wenn man SCOPEDENUMS verwendet, schon...

Wenn da nicht mein 25 Jahre alter Code wäre.

[Sinspin]

Zitat von bernau:

Zitat von Uwe Raabe:

Wenn man SCOPEDENUMS verwendet, schon...

Wenn da nicht mein 25 Jahre alter Code wäre.

... wie ein guter Wein.

[dummzeuch]

Zitat von Uwe Raabe:

Zitat von dummzeuch:

da dort ja nicht der Typ sondern eine Variable des Typs verwendet wird, man kann also nicht einfach ein grep auf den Namen des Typs machen.

Wenn man SCOPEDENUMS verwendet, schon...

Wenn ich so viel tippen wollte, kann ich ja gleich Java oder C# verwenden.

Das wäre mal eine interessante IDE-Funktion: Man tippt einen Enum-Wert und die IDE schlägt automatisch einen Scope vor. Also quasi ein umgekehrtes TEnumType.<ctrl+Space>
Oder gibt's das am Ende schon?