No, certamente non la migliore pratica a tutti. Tutte le lingue che ho mai usato hanno un operatore "non", quindi usalo. È molto chiaro, molto facile da leggere e salva i metodi di scrittura essenzialmente duplicati.
es. il significato e l'intenzione del codice qui sotto mi sembra abbastanza chiaro:
if (!In(...))
{...}
Anche se ho visto un po 'di codice come questo:
if (NotIn(...))
{...}
Io penserei, "questo è probabilmente l'opposto di In() , ma se fosse perché non hanno appena scritto !In() ". Quindi, finirò per avere per controllare i documenti o il codice: (
Ovviamente non è sintatticamente sbagliato scrivere un metodo del genere, non è solo idiomatico (in qualsiasi lingua langua che abbia mai usato).
Modifica Come Amir menziona sui commenti, questo è il tipo di cosa che potrebbe essere trattata negli standard di codifica, insieme a come denominare un metodo (o una proprietà) che restituisce un valore booleano.