C#•7mo ago

Best way to set all applicable flags?

I have a string that represent possible flags. How can I set all the flags that are in the string?

Classifiers getClassifiers(string s) {
    Classifiers c = new();
    foreach (var enumVal in Enum.GetValues(typeof(Classifiers)))
    {
        if (s.Contains(enumVal.ToString())) c += (int)enumVal;
    }
    
    return c;
}

Classifiers c = getClassifiers("CTR HI LAG");
Console.WriteLine(c); // HI, CTR, LAG

[Flags]
enum Classifiers
{
    LO = 1<<0,
    HI = 1<<1,
    LFT = 1<<2,
    RIT = 1<<3,
    CTR = 1<<4,
    LAG = 1<<5
}

Classifiers getClassifiers(string s) {
    Classifiers c = new();
    foreach (var enumVal in Enum.GetValues(typeof(Classifiers)))
    {
        if (s.Contains(enumVal.ToString())) c += (int)enumVal;
    }
    
    return c;
}

Classifiers c = getClassifiers("CTR HI LAG");
Console.WriteLine(c); // HI, CTR, LAG

[Flags]
enum Classifiers
{
    LO = 1<<0,
    HI = 1<<1,
    LFT = 1<<2,
    RIT = 1<<3,
    CTR = 1<<4,
    LAG = 1<<5
}

Classifiers getClassifiers(string s) {
    Classifiers c = new();
    foreach (var enumVal in Enum.GetValues(typeof(Classifiers)))
    {
        if (s.Contains(enumVal.ToString())) c += (int)enumVal;
    }
    
    return c;
}

Classifiers c = getClassifiers("CTR HI LAG");
Console.WriteLine(c); // HI, CTR, LAG

[Flags]
enum Classifiers
{
    LO = 1<<0,
    HI = 1<<1,
    LFT = 1<<2,
    RIT = 1<<3,
    CTR = 1<<4,
    LAG = 1<<5
}

Classifiers getClassifiers(string s) {
    Classifiers c = new();
    foreach (var enumVal in Enum.GetValues(typeof(Classifiers)))
    {
        if (s.Contains(enumVal.ToString())) c += (int)enumVal;
    }
    
    return c;
}

Classifiers c = getClassifiers("CTR HI LAG");
Console.WriteLine(c); // HI, CTR, LAG

[Flags]
enum Classifiers
{
    LO = 1<<0,
    HI = 1<<1,
    LFT = 1<<2,
    RIT = 1<<3,
    CTR = 1<<4,
    LAG = 1<<5
}

This works but is there a better way?

reflectronic•6/1/25, 11:50 PM

can you change the format of the string

reflectronic•6/1/25, 11:50 PM

Enum.Parse<Classifiers>("CTR,HI,LAG")Enum.Parse<Classifiers>("CTR,HI,LAG")

Jason_BjornOP•6/1/25, 11:51 PM

I supposed I could change the format of the string, It might also include other information not needed by the classifier though

JJason_Bjorn I supposed I could change the format of the string, It might also include other ...

Anton•6/2/25, 3:55 AM

what's the input look like?

FusedQyou•6/2/25, 9:26 AM

This is horrible, why would you do this?

FusedQyou•6/2/25, 9:28 AM

If you want to set all the flags, just apply them normally.
Classifiers c = LO | HI | LFT | RIT | CTR | LAG;Classifiers c = LO | HI | LFT | RIT | CTR | LAG;
People also usually add a flag to the enum that specifies groups of flags, or all flags.

[Flags]
enum Classifiers
{
    LO = 1<<0,
    HI = 1<<1,
    LFT = 1<<2,
    RIT = 1<<3,
    CTR = 1<<4,
    LAG = 1<<5,
    ALL = LO | HI | LFT | RIT | CTR | LAG,
}

[Flags]
enum Classifiers
{
    LO = 1<<0,
    HI = 1<<1,
    LFT = 1<<2,
    RIT = 1<<3,
    CTR = 1<<4,
    LAG = 1<<5,
    ALL = LO | HI | LFT | RIT | CTR | LAG,
}

FusedQyou•6/2/25, 9:28 AM

Doing weird string parsing is unnecessary complication and just straight up confusing. This is also a very poor and slow solution.

FusedQyou•6/2/25, 9:31 AM

Also, this is your solution regardless if you remove this parsing system. You can then pass "ALL" to the string and parse all of them.

FFusedQyou This is horrible, why would you do this?

Anton•6/2/25, 2:00 PM

I suspect they are trying to parse it out in some larger piece of text

cned•6/3/25, 10:22 PM

string.Contains is, 90% of a code smell to me. It's almost never correct, since a lot of text "cotains" substrings unexpectedly. Like in this example, "LO" and "HI" are going to show up in a LOT of strings. Like is "CTRIT" supposed to be both "CTR" an "RIT"? I strongly suspect no.

And I've seen it used for things like ".Contains(googl.com)", or the like in URL's. But "my.site.evil/#google.com" contains it.

You should already know what piece of string you are looking at is supposed to contain, then it either "is" that or not. No contains.

FusedQyou•6/4/25, 7:51 AM

I assume that the whole point is to have a concatenated string of the flags, meaning that it doesn't contain bloat like other text

FusedQyou•6/4/25, 7:51 AM

Not that it matters, it's still horrible and inefficient. I doubt they are familiar with how to use flags properly

Ccned string.Contains is, 90% of a code smell to me. It's almost never correct, since ...

FusedQyou•6/4/25, 7:52 AM

You should already know what piece of string you are looking at is supposed to contain, then it either "is" that or not. No contains.

Also this, it doesn't really matter. They should not use a string, period.

Jason_BjornOP•6/4/25, 10:51 PM

This is how I get the json

Jason_BjornOP•6/4/25, 10:51 PM

I didn't chose to put it all in a string with a bunch of other random crap

FusedQyou•6/5/25, 7:05 AM

What is this JSON? Can you change how it returns the data? Then just return an integer that represents the flag

FusedQyou•6/5/25, 7:11 AM

This still is excessive overcomplication. If you get this JSON from somebody else then they really need to improve it.

Best way to set all applicable flags?

Similar Threads

Similar Threads

Similar Threads