Android email validation with regular expression not working

Question

I have a regular expression to validate an email and when I run it, it constantly tells me that its an invalid email address but the email is correct

email = (EditText) findViewById(R.id.email);   

final String Email = email.getText().toString();

if (!Email.matches("^([a-zA-Z0-9_\\-\\.]+)@([a-zA-Z0-9_\\-\\.]+)\\.([a-zA-Z]{2,5})$"))
     {
         email.requestFocus();
         email.setError("INVALID EMAIL ADDRESS");
     }

Does anyone know it would give me the error even though it is correct?

But when I run it on my android it just prints the error Invalid email address — Michelle, Apr 27 '16 at 16:10
Do you have a space at the end? That would return false. What is the content of `Email`? — Clark Kent, Apr 27 '16 at 16:10
How do you get `Email`? Please post that snippet too. And log the value of `Email`. Likely the error is there. — BackSlash, Apr 27 '16 at 16:10
Email is just converting it to a string so that .matches can work I have edited the code — Michelle, Apr 27 '16 at 16:13
I would try `email.getText().toString().trim()` to see if there's any whitespace you're not accounting for, but I would still like to see a log statement with the contents of the Email String to see what you're testing against. — Clark Kent, Apr 27 '16 at 16:14
.trim() didn't work but I can't seem to find the log statement regarding Email String — Michelle, Apr 27 '16 at 16:22
What is that "else"? It seems that you set email var on one "if" branch and check its value in another... — Daniel, Apr 27 '16 at 16:34
It was an else if because I had an if above it that checks that email wasn't empty — Michelle, Apr 27 '16 at 16:37

Angel Koh · Answer 1 · 2016-04-27T16:28:46.583

0

try this instead. this should remove any hidden spaces/characters after the email.

Email.matches
    ("([a-zA-Z0-9_\\-\\.]+)@([a-zA-Z0-9_\\-\\.]+)\\.([a-zA-Z]{2,5}).*"))

according to http://developer.android.com/reference/java/lang/String.html

public boolean matches (String regularExpression)

Added in API level 1 Tests whether this string matches the given regularExpression. This method returns true only if the regular expression matches the entire input string. A common mistake is to assume that this method behaves like contains(CharSequence); if you want to match anywhere within the input string, you need to add .* to the beginning and end of your regular expression. See matches(String, CharSequence).

edited Apr 27 '16 at 16:28

answered Apr 27 '16 at 16:24

Angel Koh

12,479
7
64
91

1

@Michelle, can you do a Log.d("TAG", "-->" + Email +"<--" ); just before the matches call? then filter by "TAG". – Angel Koh Apr 27 '16 at 16:29
How do I filter by "TAG"? – Michelle Apr 27 '16 at 18:35
@Michelle you can do filtering like the following http://stackoverflow.com/questions/19931987/how-to-filter-logcat-in-android-studio – Angel Koh Apr 28 '16 at 00:05

Android email validation with regular expression not working

1 Answers1