if (Foo) and if (!Foo) seem to imply Foo is a boolean variable, at least for me. That being said, as long as your identifier is sufficiently descriptive it really shouldn't matter. I would use !Foo for new code, but follow existing conventions if there are any. Consistency trumps all.