Javascript: want to create a function based on something inputted through html (need some help)

Question

I'm a newbie scripter and these are my first javascript attempts. What this code is supposed to do is: It's a calculator but first it requires registration. (It's kinda fiction though, as you can just press Continue without registering). What I wanted to do is: if the password I entered is less than 5 characters then it should say a message that an error occured and nothing to happen after that. If the password is more tahn 5 characters then it should continue on with the calculator.

So here is the code but no matter how many characters my password has it simply launches the calculator directly going to the else statement.

var pw = document.getElementByID("userInput2").value;

function triggerCalc(){
        if (pw<5){
                alert("An error occured. Your password must be more than 5 symbols!");
                }
        else { ... }
}

Include the code in this question.

johnnycardy
– johnnycardy

2014-01-03 12:22:20 +00:00
Commented Jan 3, 2014 at 12:22 — johnnycardy
– johnnycardy, Commented Jan 3, 2014 at 12:22
Please post your code here, not on some 3rd party site.

deceze
– deceze ♦

2014-01-03 12:22:54 +00:00
Commented Jan 3, 2014 at 12:22 — deceze
– deceze ♦, Commented Jan 3, 2014 at 12:22

johnnycardy · Accepted Answer · 2014-01-03 12:22:39Z

1

Try:

if(pw.length < 5) {
 ...
}

The length part checks the length of the password. Beforehand, you were checking if the password itself is less than 5 (and it isn't a number, so that wouldn't make sense).

answered Jan 3, 2014 at 12:22

johnnycardy

3,0451 gold badge21 silver badges27 bronze badges

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

2 Comments

user3157145 Over a year ago

Thanks for your fast response, but now nothing happens after I press Continue. Do you have an idea why this happens?

johnnycardy Over a year ago

Nope, but that's a separate problem so raise a new question :-)

jackotonye · Accepted Answer · 2016-01-28 16:29:28Z

1

This will be a better fit

if(pw.length <= 5)

{
 alert("An error occurred. Your password must be more than 5 symbols!");
}
else{}

answered Jan 28, 2016 at 16:29

jackotonye

3,84326 silver badges31 bronze badges

Comments

Mohammad Areeb Siddiqui · Accepted Answer · 2014-01-03 12:23:47Z

0

Just pw would return an object. You need to use the length property so that you could target it's length.

Try this:

if(pw.length <= 5) { //do something here }

answered Jan 3, 2014 at 12:23

Mohammad Areeb Siddiqui

10.2k16 gold badges74 silver badges118 bronze badges

Comments

Everton Lenger · Accepted Answer · 2014-01-03 13:14:35Z

0

I think you can do this:

function triggerCalc() {
    var pw = document.getElementByID("userInput2").value;
    if (pw.length < 5){
        alert("An error occured. Your password must be more than 5 symbols!");
    }
}

The variable pw should be inside the function, and not before it.

And a hint: some indentation in the code would be very good!

edited Jan 3, 2014 at 13:14

answered Jan 3, 2014 at 12:28

Everton Lenger

1,4572 gold badges18 silver badges34 bronze badges

Collectives™ on Stack Overflow

Javascript: want to create a function based on something inputted through html (need some help)

4 Answers 4

2 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related