Flags enum in .NET

Ich versuche, eine Reihe von bedingten Anweisungen zu verwenden, die eine mit [Flags] zugewiesene Aufzählung einrichten. Der Compiler beklagt sich jedoch, dass 'm' nicht zugewiesen ist. Wie kann ich Folgendes umschreiben, um meine beabsichtigte Funktionalität zu erreichen?Flags enum in .NET

Media m; 
if (filterOptions.ShowAudioFiles) 
    m = m | Media.Audio; 
if (filterOptions.ShowDocumentFiles) 
    m = m | Media.Document; 
if (filterOptions.ShowImageFiles) 
    m = m | Media.Image; 
if (filterOptions.ShowVideoFiles) 
    m = m | Media.Video;

Quelle

2009-04-10 deckerdev

Sie müssen m initialisieren. Erstellen Sie ein "Kein" -Flag, das den Wert 0 hat, dann:

Media m = Media.None;

Dann den Rest Ihres Codes.

Quelle

2009-04-10 19:54:08

Wenn keine der Bedingungen zutrifft, ist m undefiniert. Setze es auf einen Anfangswert.

Quelle

2009-04-10 19:54:00 EricSchaefer

Haben Sie einen 'Standard' wie filterOptions.ShowNone? Wenn ja, beginnen Sie mit m auf diesen Wert. Der Compiler beklagt sich, weil am Ende aller if's m nichts gesetzt ist.

Quelle

2009-04-10 19:54:49 n8wrl

-3

Sie müssen nicht wirklich eine Media.None erstellen. Sie können einen beliebigen Wert in die Flag-Enumerierung umwandeln, auch wenn sie nicht mit einem Wert der Flags übereinstimmt.

Media m = (Media)0; 

if (filterOptions.ShowAudioFiles)  m |= Media.Audio; 

if (filterOptions.ShowDocumentFiles) m |= Media.Document; 

if (filterOptions.ShowImageFiles)  m |= Media.Image; 

if (filterOptions.ShowVideoFiles)  m |= Media.Video;

Quelle

2009-04-10 20:04:09

Was passiert, wenn alle Bedingungen fehlschlagen? Was ist, wenn 0 z.B. Medien.Audio? Sie benötigen einen * gültigen * Anfangswert. – EricSchaefer

Das ist ein schlechter Ratschlag. Es ist immer besser, explizite Werte zu verwenden, als sich auf Integer-Umwandlungen zu verlassen. – Randolpho

Nicht wirklich. Die Enum funktioniert auch dann einwandfrei, wenn kein "gültiges" Flag gesetzt ist. Es wird immer noch einen Wert geben. Alle Enums werden standardmäßig von "Int32" übernommen. Ja, später könnten andere Probleme auftreten, wenn Sie keine Standardwerte verwenden. Aber dafür gibt es Ausnahmen. –

Zusätzlich zu den oben genannten Antworten, neben der Tatsache, dass dieser Code ziemlich überflüssig scheint, würde Ich mag, dass Sie eine Select Case statt all diesen hässlichen Wenn die Nutzung vorzuschlagen.

Quelle

2009-04-10 20:06:44 hmcclungiii

Angesichts der Tatsache, dass eine beliebige Kombination der Flags gesetzt werden konnte, sehe ich nicht, wie ein Select Case geeignet wäre. –

Einverstanden, -1, weil Sie diese Wenns nicht mit einem Schalter ersetzen konnten. – Samuel

Ok, also hatte ich eine gute gesunde Gehirnhälfte dort. – hmcclungiii

Man könnte auch schreiben:

Media m = default(Media)

Nützlich in Fällen, in denen Sie nicht wissen, die Enum, Klasse, oder ob es ein Wert/Referenztyp ist.

Quelle

2009-04-10 20:15:03

Das ist wirklich eine schlechte Idee, es verbirgt die ganze Einfachheit der Aufzählungen. Sie wurden erstellt, um magische Zahlen zu entfernen, und Sie fügen sie wieder hinzu, wenn eine None für eine Flag-Enumeration vorhanden sein sollte, die Sie stattdessen verwenden sollten. – Samuel

Guter Punkt. Ich habe über den Fall nachgedacht, in dem Sie nicht der Autor sind, aber dann wäre die explizite Wahl der Vorgabe sowieso besser. –

Flags enum in .NET

Antwort

Verwandte Themen