Замена пользовательской части адреса электронной почты в Java

Question

У вас есть

user.nick@domain.com

и результат должен быть:

******@domain.com

В настоящее время я делаю это так:

public static String removeUserFromEmail(String email) {
    StringBuffer sbEmail = new StringBuffer(email);
    int start = sbEmail.indexOf("@");
    sbEmail.delete(0, start);
    return "******" + sbEmail.toString();
}

Есть что-то более простое или более элегантное?

Answer 1

Я бы хотел запустить indexOf для строки электронной почты, прежде чем помещать ее в буфер строк ...

int start = email.indexOf( '@' );

if( start == -1 )
{
   // handle invalid e-mail
}
else
{
   return "*****" + email.substring( start );
}

Answer 2

Ничего плохого в этом решении, хотя у меня есть два предложения:

1) Используйте StringBuilder вместо StringBuffer, если вам не нужно синхронизировать доступ между несколькими потоками. С StringBuffer связано снижение производительности, которое для этого приложения, скорее всего, не требуется.

2) Одним из преимуществ StringBuilder / Buffer является исключение чрезмерной конкатенации строк.

Ваша обратная строка преобразует буфер в строку, а затем объединяет. Я, вероятно, сделал бы это вместо этого:

int start = email.indexOf("@");

if (start < 0) {
    return "";  // pick your poison for the error condition
}

StringBuilder sbEmail = new StringBuilder(email);
sbEmail.replace(0, start, "******");
return sbEmail.toString();

К вашему сведению, мое решение - это всего лишь некоторые мысли о вашем текущем использовании StringBuffer (которые, надеюсь, будут полезны) Я бы порекомендовал решение Константина для этого простого струнного упражнения. Простой, читаемый, и это дает вам возможность справиться с ошибкой.

Answer 3

"some.user@domain.com".replaceAll("^[^@]+", "******");

Answer 4

выглядит хорошо. Лучше проверить, если indexOf возвращает -1.

Answer 5

public static String removeUserFromEmail(String email) {
    String[] pieces = email.split("@");
    return (pieces.length > 1 ? "******" + pieces[1] : email);
}

Answer 6

Вы можете использовать регулярное выражение, но ваше решение мне подходит.Вероятно, быстрее, чем регулярное выражение тоже.