if(定数 == x)という書き方もありとは思うがな...
理由が先に来てた方がコードが読みやすい場合もあるし

まあ、NULLを先頭に持っていく意味はわかんないが...
if(ALLOC_ERROR == x)
とかだと、読みやすくて良いと思う