Aが1、または2または3の場合、正常それ以外は異常という判定で
以下のようにコーディングしたら先輩からそんな糞コードかくなと怒られてしまった。
実際はもっと条件が複雑で、逆の条件を考えるのが億劫だったからこうしたんだけど
やっぱりダメですか？

if (a == 1 || a == 2 || a ==3)
{
/* 何もしない */
} else {
/* エラー */
return 1
}

正常の場合、処理が続く