Pretty good. There's no need to create a
String object from a String literal though. Just use the literal. In other words.
Also, it's good when doing a String comparison to make the call on the String you know isn't null. So:
Finally, if you're trying to get a case insensitive comparison there, you're missing a lot of cases: "Bad", "BAd", "bAd", etc. Instead do: