Efficient way to replace char from each index of a String array if condition match

Question

I'm looking for efficient way to replace specific char from a list of email addresses if the char matches in any index of the array. But it should return the full array list.

my current code:

List<String> emailList = new ArrayList<>();
    emailList.addAll(Arrays.asList("[email protected]","[email protected]","[email protected]"));
    List<String>  updatedEmail = emailList.stream().filter(a->a.contains("+")).
            map(s -> s.replaceAll("\\+", ""))
            .collect(Collectors.toList());
    emailList.stream().filter(a->!a.contains("+")).forEach(b->updatedEmail.add(b));

    updatedEmail.stream().forEach(b->System.out.println("email: "+b));

Console:

email: [email protected]
email: [email protected]
email: [email protected]

Note that s.replace("+", "") is better than replaceAll as it avoids regex each call so it is just emailList.stream().map(s -> s.replace("+", "")).forEach(System.out::println) or emailList.stream().map(s -> s.replace("+", "")).toList() — DuncG
– DuncG, Commented Jul 27, 2022 at 8:56
Note that for Gmail addresses, emailadress [email protected] refers to just [email protected], so the suffix behind the + can be used as label. However, they're not the same as [email protected], which would be the result of your string replacement. — MC Emperor
– MC Emperor, Commented Jul 27, 2022 at 9:08

Ryuzaki L · Accepted Answer · 2022-07-27 06:16:38Z

1

You can use replaceAll on all strings and string not having + symbol will remains unchanged

List<String>  updatedEmail = emailList.stream()
        .map(s -> s.replaceAll("\\+", ""))
        .collect(Collectors.toList());

answered Jul 27, 2022 at 6:16

Ryuzaki L

40.4k14 gold badges81 silver badges111 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Eritrean · Accepted Answer · 2022-07-27 09:02:11Z

1

In case you want to modify the original list without using streams and creating a new list, you can do something like:

emailList.replaceAll(s -> s.replace("+", ""));

answered Jul 27, 2022 at 9:02

Eritrean

16.6k3 gold badges25 silver badges28 bronze badges

Comments

Peter Stoilkov · Accepted Answer · 2022-07-27 07:03:05Z

0

You already use replaceAll method and you can directly modify every element in current list and print it with foreach.

emailList.stream()
                .map(s -> s.replaceAll("\\+", ""))
                .forEach(b -> System.out.println("email: " + b));

answered Jul 27, 2022 at 7:03

Peter Stoilkov

113 bronze badges

Collectives™ on Stack Overflow

Efficient way to replace char from each index of a String array if condition match

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related