using condition in jquery onclick button

Question

Im trying to confirm if the password strength is strong or weak and is the password is strong and when I submit it should have alert message like "You Have Strong Password" and when its weak "Invalid Password"

This is what I am now.

function checkPasswordStrength() {
        var passwordStrength = false;
        var number = /([0-9])/;
        var alphabets = /([a-zA-Z])/;
        var special_characters = /([~,!,@,#,$,%,^,&,*,-,_,+,=,?,>,<])/;
        if ($('#password').val().length < 8) {
            $('#password-strength-status').removeClass();
            $('#password-strength-status').addClass('weak-password');
            $('#password-strength-status').html("Weak (should be atleast 8 characters.)");
        } else {
            if ($('#password').val().match(number) && $('#password').val().match(alphabets) && $('#password').val().match(special_characters)) {
                $('#password-strength-status').removeClass();
                $('#password-strength-status').addClass('strong-password');
                $('#password-strength-status').html("Strong");
                return passwordStrength = true;
            } else {
                $('#password-strength-status').removeClass();
                $('#password-strength-status').addClass('medium-password');
                $('#password-strength-status').html("Medium (should include alphabets, numbers and special characters.)");
            }
        }
    }

    $('#btn-submit').click(function () {
        if (passwordStrength == false) {
        alert("INVALID PASSWORD");
        } else {
        alert("You have Strong PASSWORD");
        }

</script>

its for Educational Purpose only im just starting jquery.. thank you in advance..

What is the problem?

Don't Panic
– Don't Panic

2022-01-08 01:17:44 +00:00
Commented Jan 8, 2022 at 1:17 — Don't Panic
– Don't Panic, Commented Jan 8, 2022 at 1:17

Joel Rosenberg · Accepted Answer · 2022-01-08 03:04:22Z

1

You need to call the function instead of just checking your variable. So rather do

$('#btn-submit').click(function () {
    if (checkPasswordStrength() === false) {

instead of

$('#btn-submit').click(function () {
    if (passwordStrength == false) {

Then, instead of return passwordStrength = true; you should do just passwordStrength = true and add a return passwordStrength to the very end of your function so it will return either false or true.

answered Jan 8, 2022 at 3:04

Joel Rosenberg

664 bronze badges

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

Comments

arshovon · Accepted Answer · 2022-01-08 03:06:07Z

1

It looks like the variable scope is incorrect. var passwordStrength should be put outside of the checkPasswordStrength function.

var passwordStrength
function checkPasswordStrength() {

    ....

edited Jan 8, 2022 at 3:06

arshovon

13.7k9 gold badges54 silver badges72 bronze badges

answered Jan 8, 2022 at 1:28

Tuvshinjargal Byambajav

574 bronze badges

3 Comments

Tuvshinjargal Byambajav Over a year ago

You're welcome.

arshovon Over a year ago

@EyeConic, if the answer solves the problem, mark it as accepted.

Don't Panic Over a year ago

This will not help. checkPasswordStrength() is never called, so passwordStrength will never change from whatever you definie it as outside that function (null in this answer). @joel-rosenberg's answer is correct.

Collectives™ on Stack Overflow

using condition in jquery onclick button

2 Answers 2

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related