I generally dislike that because hardcoded strings can change any time. May be we should have them being in a variable somewhere and check it against the contents of that variable?