Question

For some reason this does not look correct. Is there a better way to write/refractor this?

if (!(auth.DoesAuthMatch && auth.DoesEmailMatch))
{
    statusText = "Access Denied";
}

I think I have a ‘case of the Mondays!..’

Answer 1

Refactor to a method:

private bool IsAccessDenied(Auth auth)
{
    return !(auth.DoesAuthMatch && auth.DoesEmailMatch);
}

if (IsAccessDenied(auth))
{
    statusText = "Access Denied";
}

As for the Boolean logic:

!(auth.DoesAuthMatch && auth.DoesEmailMatch) == 
(!auth.DoesAuthMatch || !auth.DoesEmailMatch)

A simple application of De Morgan’s Laws.