Returning the other value of boolean enum

Question

I have a class with a "mode" state. There are currently two modes and we are not likely to add more in the future.

The class has a CurrentMode property, and a function to swap to the other mode. I have implemented this using another property called ModeNotInUse which returns the opposite value of CurrentMode.

However, I am in two minds about how to implement this ModeNotInUse

public enum PlayerMode
{
    Puzzle,
    Action
}

public class ModeController
{
    public PlayerMode CurrentMode { get; private set; }

   
    private PlayerMode ModeNotInUse => ...


    public void SwitchMode()
    {
        CurrentMode = ModeNotInUse
    }

}

My two implimentations of this are:

private PlayerMode ModeNotInUse 
   => CurrentMode switch
   {
       PlayerMode.Puzzle => PlayerMode.Action,
       _ => PlayerMode.Puzzle
   };
   

Or

private PlayerMode ModeNotInUse 
    => CurrentMode == PlayerMode.Puzzle 
        ? PlayerMode.Action 
        : PlayerMode.Puzzle;

Which better captures my intent here? The first is my preference but I worry it's difficult to read.

---

(edit) I could of course do this with a backing boolean field but that feels even less clean

private bool inPuzzleMode;
public PlayerMode CurrentMode
{
    get => inPuzzleMode switch
        {
            true => PlayerMode.Puzzle,
            false => PlayerMode.Action
        };
    private set => inPuzzleMode = value == PlayerMode.Puzzle;
}

private PlayerMode ModeNotInUse 
   =>  inPuzzleMode switch
   {
       true => PlayerMode.Action,
       false => PlayerMode.Puzzle
   };

Which do you think is best?

Answer 1

I think a method would be better than a property in this case. However, I would do it this way :

public class ModeController
{
    public PlayerMode CurrentMode { get; private set; }

    public void SwitchMode() 
          => CurrentMode == PlayerMode.Puzzle ? PlayerMode.Action : PlayerMode.Puzzle;
}

if the PlayerMode is used in different classes, and you want to get the opposite mode, then you can move SwitchMode() into an extension :

public static class Extension {
    public static PlayerMode  SwitchMode(this PlayerMode current) 
            => current == PlayerMode.Puzzle ? PlayerMode.Action : PlayerMode.Puzzle;    
}

public class ModeController
{
    public PlayerMode CurrentMode { get; private set; }

    public void SwitchMode() => CurrentMode = CurrentMode.SwitchMode();
}

if there is a requirement where you want to know the old mode, then you could store the old mode before switching it, and add another method to get the old mode.

Answer 2

First, kudos to you for using an enum even though it would be very tempting to just use a bool in its place. Not enough can be said for semantic programming.

Second, I would use your first method, but perhaps extract it to an extension method:

public static PlayerMode NotInUse(this PlayerMode currentMode) => currentMode switch
{
    PlayerMode.Puzzle => PlayerMode.Action,
    _ => PlayerMode.Puzzle
};

Answer 3

How about simply just this?

=> (PlayerMode)((CurrentMode + 1) % 2);

Here you are taking advantage of the underlying values: 0 and 1.

---

UPDATE #1

I think if you look at this problem for the domain perspective then the whole idea of ModeNotInUse is pointless. Since if you want to branch based on the mode then you

• either use CurrentMode == PlayerMode.Puzzle
• or CurrentMode != PlayerMode.Puzzle

and you have covered all cases without introducing a redundant property.

My solution is more fragile than others because if you set explicitly the value for the PlayerMode enums (different than the default ones) then it will not work anymore. For example:

public enum PlayerMode
{
    Puzzle = 1,
    Action = 10
}