323
True crime (europe.pub)
submitted 11 months ago by to c/programmer_humor@programming.dev
74 comments fedilink hide all child comments
 
you are viewing a single comment's thread
view the rest of the comments
[–] 34 points 11 months ago (1 child)

I would certainly rather see this than {isAdmin: bool; isLoggedIn: bool}. With boolean | null, at least illegal states are unrepresentable... even if the legal states are represented in an... interesting way.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 3 points 11 months ago (1 child)

    Admin false LoggedIn false doesn't feel illegal to me, more redundant if anything

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 9 points 11 months ago* (2 children)

    I was thinking of the three legal states as:

    • not logged in (null or {isAdmin: false, isLoggedIn: false})
    • logged in as non-admin (false or {isAdmin: false, isLoggedIn: true})
    • logged in as admin (true or {isAdmin: true, isLoggedIn: true})

    which leaves {isAdmin: true, isLoggedIn: false} as an invalid, nonsensical state. (How would you know the user's an admin if they're not logged in?) Of course, in a different context, all four states could potentially be distinctly meaningful.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 4 child comments
    •