Optymalizacja kodu jQuery sprawdzajacego poprawność hasła

pytanie zadane 2 sierpnia 2016 w JavaScript przez Śmieszek_;3 Użytkownik (780 p.)
edycja 2 sierpnia 2016 przez Śmieszek_;3

Hej.

Napisałem kod sprawdzający hasło i prosiłbym Was o wskazówki jak ten kod zoptymalizować, bo te if else if dziwnie wygląda, wiec pewnie można to zrobić jakoś łatwiej :D

<script>
$(document).ready(function() {
	
	$('#password').blur(function() {
        var passwordInputValue = $('#password').val();
        var errorSpan = $('.error');
        var minLength = /.{8}/
        var oneLetterRegExp = /[a-z]+/
        var oneBigLetterRegExp = /[A-Z]+/
        var oneDigitRegExp = /[0-9]+/
        var oneSpecialSign = /.[^0-9a-zA-Z]/
        if (passwordInputValue.search(minLength) == -1) {
            errorSpan.text('Hasło powinno mieć co najmniej 8 znaków')
        } else if (passwordInputValue.search(oneLetterRegExp) == -1) {
            errorSpan.text('Hasło powinno mieć co najmniej jedną małą literę')
        } else if (passwordInputValue.search(oneBigLetterRegExp) == -1) {
            errorSpan.text('Hasło powinno mieć co najmniej jedną dużą literę')
        } else if (passwordInputValue.search(oneDigitRegExp) == -1 ) {
            errorSpan.text('Hasło powinno mieć co najmniej jedną cyfrę')
        } else if (passwordInputValue.search(oneSpecialSign) == -1) {
            errorSpan.text('Hasło powinno mieć minimum jeden znak specialny.')
        } else {
            errorSpan.text('Hasło przeszło walidację.')
        }
    })
})
</script>
</head>
<body>
	<span class="error"></span><br>
Podaj hasło: <input type="text" id="password"><br>
</body>

Aha i czy jest sens trzymać te regexy w zmiennych, czy lepiej wrzucić po prostu do search() ?

1 odpowiedź

odpowiedź 2 sierpnia 2016 przez xmentor Nałogowiec (49,520 p.)

A czemu nie zrobisz jednego wyrażenia regularnego?

Poza tym, część walidacji można już przeprowadzić dodając atrybuty do inputów np.: required, pattern.

komentarz 2 sierpnia 2016 przez Śmieszek_;3 Użytkownik (780 p.)

Jednego nie mogę zrobić, bo chodzi mi oto, że jak ktoś wyśle hasło niepoprawne, to dostanie komunikat czego konkretnie brakuje. Np, brakuje znaku specjalnego, duzej litery. Co prawda mógłbym to określić w jednym komunikacie błędu wymieniając co powinno mieć hasło, jednak tak mi się bardziej podoba, ot taki bajer ;)

Chodzi mi bardziej o poćwiczenie regex'ów (jezeli macie pomysły na jakieś wyrazenia regularne zeby potrenowac to mozecie podac ;p), ale skoro już zrobiłem taki formularz z taką weryfikacją, to chciałbym wiedzieć, czy dałoby radę go zoptymalizować, żeby w przyszłości stworzyć lepszy kod.

1	komentarz 2 sierpnia 2016 przez kubaapk Nałogowiec (44,270 p.) Optymalizacją będzie jeden regexp. ;)

komentarz 2 sierpnia 2016 przez xmentor Nałogowiec (49,520 p.)

Nie wiem czasem czy metoda reg.test(string) nie byłaby szybsza od string.search(reg)

Wydaje mi się, że gdzieś o tym czytałem na stackoverflow.

Jednego nie mogę zrobić, bo chodzi mi oto, że jak ktoś wyśle hasło niepoprawne, to dostanie komunikat czego konkretnie brakuje. Np, brakuje znaku specjalnego, duzej litery. Co prawda mógłbym to określić w jednym komunikacie błędu wymieniając co powinno mieć hasło, jednak tak mi się bardziej podoba, ot taki bajer ;)

Tutaj np. możesz wyświetlać jeden komnunikat o błędnym haśle(tzn. nie pasującym do wyrażenia) a obok inputa mieć jakąś podpowiedź.