В чем ошибка проверки на уровень сложности пароля?

Question

[neitoo]

Требования для пароля
1 уровень
Количество символов меньше 8.
Состоит из строчных букв.
2 уровень
Количество символов >= 8
Используются строчные и прописные буквы.
3 уровень
Предыдущие условия + используются цифры.
4 уровень
Предыдущие условия + используются спец символы (пример: !»№;%:?*()_+<>?|)

Мой код

var password = ValuePass.Text;
            var regex = new Regex(@"([a - z])");
            if (password.Length < 8 && regex.IsMatch(password))
            {
                resultValue.Text = "1 уровень";
                return;
            }

            var regex2 = new Regex(@"([a-zA-Z])");
            if (password.Length >= 8 && regex2.IsMatch(password))
            {
                resultValue.Text = "2 уровень";
                return;
            }

            var regex1 = new Regex(@"([0 - 9])");
            if (password.Length >= 8 &&  regex1.IsMatch(password) && regex2.IsMatch(password))
            {
                resultValue.Text = "3 уровень";
                return;
            }

            var regex3 = new Regex(@"([!,@,#,$,%,^,&,*,?,_,~])");
            if (password.Length >= 8 && regex1.IsMatch(password) && regex2.IsMatch(password) && regex3.IsMatch(password))
            {
                resultValue.Text = "4 уровень";
                return;
            }

Но он работает только до 2 уровня, а дальше что не пиши, уровня выше не будет.

Comments

[Владимир Коротенко]

Если же хотите на регулярках возьмите вот это
https://social.msdn.microsoft.com/Forums/vstudio/e...

[Василий Банников]

В регехах символы в [] не нужно разделять запятыми.
А вообще есть ContainsAny

[Lynn «Кофеман»]

Да и пробелы вокруг дефиса тоже не нужны

Answer 1

[Planet_93]

У вас некорректная логика проверок. В данном случае, когда пароль соответствует 4 уровню, то он всё равно покажет 2 уровень, так как у вас стоит return. То есть return, при любом раскладе закончит выполнение метода после 2 уровня, хотя пароль и соответствует 3 или 4 уровню.
У вас есть два варианта, либо поменять уровни местами, то есть сначала проверять на соответствие 4 уровню, потом 3 уровню и так до первого, с return при проверки каждого уровня.
Либо вынести return из под каждого if и оставить один единственный в конце метода.

Второй вариант:

var password = ValuePass.Text;
            var regex = new Regex(@"([a - z])");
            var regex2 = new Regex(@"([a-zA-Z])");
            var regex1 = new Regex(@"([0 - 9])");
            var regex3 = new Regex(@"([!,@,#,$,%,^,&,*,?,_,~])");
            if (password.Length >= 8 && regex1.IsMatch(password) && regex2.IsMatch(password) && regex3.IsMatch(password))
            {
                resultValue.Text = "4 уровень";
                return;
            }
            if (password.Length >= 8 &&  regex1.IsMatch(password) && regex2.IsMatch(password))
            {
                resultValue.Text = "3 уровень";
                return;
            }
            if (password.Length >= 8 && regex2.IsMatch(password))
            {
                resultValue.Text = "2 уровень";
                return;
            }
            if (password.Length < 8 && regex.IsMatch(password))
            {
                resultValue.Text = "1 уровень";
                return;
            }

Первый вариант:

var password = ValuePass.Text;
            var regex = new Regex(@"([a - z])");
            if (password.Length < 8 && regex.IsMatch(password))
            {
                resultValue.Text = "1 уровень";
            }

            var regex2 = new Regex(@"([a-zA-Z])");
            if (password.Length >= 8 && regex2.IsMatch(password))
            {
                resultValue.Text = "2 уровень";
            }

            var regex1 = new Regex(@"([0 - 9])");
            if (password.Length >= 8 &&  regex1.IsMatch(password) && regex2.IsMatch(password))
            {
                resultValue.Text = "3 уровень";
            }

            var regex3 = new Regex(@"([!,@,#,$,%,^,&,*,?,_,~])");
            if (password.Length >= 8 && regex1.IsMatch(password) && regex2.IsMatch(password) && regex3.IsMatch(password))
            {
                resultValue.Text = "4 уровень";
            }
            return;

Answer 2

[Владимир Коротенко]

Не люблю я регекспы, может проще сделать?

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            var pwd = "Test1236";
            var r = new PasswordChecker(pwd);
            ;

        }
    }

    public class PasswordChecker
    {
        private readonly char[] _upper = new char[] { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z', };
        private readonly char[] _digit = new char[] { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' };
        private readonly char[] _special = new char[] { '!', '@', '#', '$', '%', '^', '&', '*', '?', '_', '~' };
        private readonly int _strongPasswordLength;
        private readonly string _password;
        public bool IsLengthOk => _password.Length > _strongPasswordLength;
        public bool HasUpper { get; private set; }
        public bool HasDigit { get; private set; }
        public bool HasSpecial { get; private set; }

        public PasswordChecker(string password, int passwordLength=8)
        {
            _password = password;
            _strongPasswordLength = passwordLength;
            foreach (var ch in _password)
            {
                HasUpper = _upper.Any(x => x == ch);
                HasDigit = _digit.Any(x => x == ch);
                HasSpecial = _special.Any(x => x == ch);
            }
        }
    }
}

Comments

[freeExec]

Можно ещё списки букв и цифр задать диапазонами, только первую и последнюю, т.к. их коды идут по порядку.