if (!value)
is easier/faster to follow. Subjective as you said. As long as you are consistent, this is the main thing.
EDIT
One other point to add – omitting the true/false keywords should also (hopefully) force the coder to use better named variables. Bool variables should always indicate meaning or state purpose, such as:
if (MyWallet.IsEmpty)
There is no reason with the above to use == false
or == true
as it’s redundant. The above is human readable immediately.
Far better than having to decipher:
if (MyWallet.EmptyStatus == true)
or something ridiculous like this.