Check if String contains words that are set in an array android studio

Question

I am trying to check if a string is valid by checking if it contains special words that are set in an array

String email = "this@webmaster";
    tv.setText(Boolean.toString(checkEmailValidity(email)));

}

public boolean checkEmailValidity(String email) {

    String[] specialWords = {"webmaster", "government"};
    if( email.contains(specialWords.toString())||email.contains(" ")){
        return false;
    }
    return true;
}

The result is always true so the statement that checks the email with the specialWords is not good. What can I do?

"this@webmaster" has "webmaster" in itself, so it returns true, what did you expect? — Mehran Zamani, May 20 '17 at 09:29
I wanted it to return false if the email contained one of the special words. I found the logical error thank you as I needed to insert ! in front of email.contains(specialWords.toString()). Thank you for the explanation — ladyBug, May 20 '17 at 09:40
from [this](http://stackoverflow.com/questions/8992100/test-if-a-string-contains-any-of-the-strings-from-an-array) : `public static boolean stringContainsItemFromList(String inputStr, String[] items) { return Arrays.stream(items).parallel().anyMatch(inputStr::contains); }` — Mehran Zamani, May 20 '17 at 09:46

score 2 · Answer 1 · answered May 20 '17 at 09:31

2

If you are using java 8 you can do this way

return Arrays.stream(specialwords).parallel().anyMatch(email::contains);

answered May 20 '17 at 09:31

Denis Ismailovski

311
1
7
15

`parallel()` is overkill for something like this. – Andreas May 20 '17 at 09:32
Also note that OP returns `false` when `contains()` return true, so you need to use `noneMatch()` instead. – Andreas May 20 '17 at 09:33
Yes. Parallel is an overkill I agree. – Denis Ismailovski May 20 '17 at 09:35
@Andreas Don't tell _him_ that, he just copied the answer from here: [Test if a string contains any of the strings from an array](//stackoverflow.com/a/8995988) – Tom May 20 '17 at 11:46

score 1 · Answer 2 · answered May 20 '17 at 09:48

1

Pattern regex = Pattern.compile("\\b(?:webmaster|government)\\b");
Matcher regexMatcher = regex.matcher(email);
boolean contains = regexMatcher.find();

answered May 20 '17 at 09:48

Cătălin Florescu

5,012
1
25
36

Jay Prakash · Answer 3 · 2017-05-20T09:48:48.173

Try This:

public static boolean checkEmailValidity(String email) {
        String[] specialWords = {"webmaster", "government"};
        Set<String> set = new HashSet<String>(Arrays.asList(specialWords));
    return set.contains(email);
    }

This is for your code:

public static boolean checkEmailValidity(String email) {
        String[] specialWords = {"webmaster", "government"};
        boolean flag = false;
        if (!email.contains(" ")) {
            for (int index = 0; index < specialWords.length; index++) {
                if (email.contains(specialWords[index].toString())) {
                    flag = false;
                    break;
                } else {
                    flag = true;
                }
            }
        }
        return flag;
    }

score -1 · Answer 4 · answered May 20 '17 at 09:53

-1

String email = "this@webmaster";
    tv.setText(Boolean.toString(checkEmailValidity(email)));


public boolean checkEmailValidity(String email) {

    String[] specialWords = {"webmaster", "government"};
    if( email.contains(specialWords.get(0))||email.contains(specialWords.get(1))){
        return false;
    }
    return true;
}

answered May 20 '17 at 09:53

Mehran Zamani

831
9
31

What if there was more than 2 special words? U should use `forech` in this case. – Mohammad Zarei May 20 '17 at 10:35
It is to handle the task at hand, as i said before in the comment, there is a general solution. – Mehran Zamani May 20 '17 at 10:47

Check if String contains words that are set in an array android studio

4 Answers4