0

Всем привет!

Есть необходимость валидировать пароли на наличие букв идущих в алфавитном порядке.

Возможно кто-нибудь сталкивался с подобными задачами ранее и знает какой-нибудь оптимальный вариант.

Может есть какие-нибудь готовые библиотеки, которые могут делать что-то подобное. Не хочется просто свой велосипед придумывать.

Любые подсказки/советы приветствуются.

9
  • Только букв? А с цифрами что? 123456?
    – aepot
    20 янв 2022 в 15:15
  • @aepot, только букв.
    – Viktor
    20 янв 2022 в 15:29
  • 4
    Убивал бы всех авторов валидаторов паролей. Если я устанавливаю пароль 123 значит мне не важно то, что этот пароль защищает и мой выбор пароля - это осознанное действие 20 янв 2022 в 15:36
  • @AntonShchyrov, оффтоп конечно с вашей стороны, но прокомментирую. Сегодня вы ставите пароль 123 на проде, а завтра база с прода уже лежит в открытом доступе в сети. Последствия думаю всем понятны.
    – Viktor
    20 янв 2022 в 15:47
  • @AntonShchyrov к счастью этот валидатор 123 пропустит. :)
    – aepot
    20 янв 2022 в 16:04

1 ответ 1

0

Кажется, вы слегка перемудрили. Не нужно хранить алфавит, вам же только последовательность следующий-предыдущий обрабатывать, так?

private static bool IsValidPassword(string password, int maxCharsInOrderCount)
{
    if (password == null)
        throw new NullReferenceException($"{nameof(password)} был null");
    if (maxCharsInOrderCount < 2)
        throw new ArgumentException($"{nameof(maxCharsInOrderCount)} должно быть больше, чем 1");
    if (password.Length < maxCharsInOrderCount)
        return true;

    int current = 1;
    int max = 1;

    var cleanPassword = password.Select(c => (c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z') ? c : '\0');

    char lastChar = '\0';
    int direction = 0;
    foreach (char c in cleanPassword)
    {
        int diff = c - lastChar;
        if (Math.Abs(diff) == 1)
        {
            if (current == 1)
                direction = diff;
            if (diff == direction)
            {
                current++;
                if (current > max)
                    max = current;
            }
            else
                current = 1;
        }
        else
            current = 1;
        lastChar = c;
    }
    return max < maxCharsInOrderCount;
}
6
  • Смысл в алфавите как раз был, чтобы исключить цифры и спецсимволы, которые так же могут быть в пароле. В вашем варианте, как понимаю их порядок тоже будет учитываться.
    – Viktor
    20 янв 2022 в 16:01
  • @DmitryRuchin нет, не будет, с чего вы взяли? Вы ж написали, что надо только большие и маленькие буквы, я так и сделал.
    – aepot
    20 янв 2022 в 16:02
  • @DmitryRuchin комментарии нужны, или код понятен?
    – aepot
    20 янв 2022 в 16:07
  • Да, увидел вашу чистку от лишних символов. Но тут тогда другая проблема будет. Если изначальная строка будет: a1b2c, то он будет искать в строке abc , что неправильно.
    – Viktor
    20 янв 2022 в 16:08
  • 1
    Да, всё верно. С телефона читаю, не всё сразу заметил.
    – Viktor
    20 янв 2022 в 16:12

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.