This is fine, but just FYI you could make the regex case insensitive instead of listing all the upper and lower case letters in the character sets.