Trying to validate JavaScript input using OnClick and a function. No matter what is entered, it says the format is correct.

Question

I have an HTML file and an external JavaScript file. I have input from the user in the HTML and on the click on the validate button, the courseValidation() function is called. The problem is with the if statement. When the input does not match myRegExp, it should write that the format is incorrect to the page, but it always writes Correct Format, regardless if it is correct or not. Not sure what I am doing wrong. I'm just learning so no negative comments please.

HTML File:

<html lang="en">
<head>
<title>Validation</title>
</head>
<body>
    <script src="course.js"></script>
    <p>Enter Course Information:</p>

    <input id="courseCode" type="text">
    <button onclick="courseValidation()">Validate</button>

        <p id="course"></p>
   </body>
</html>

External JavaScript File:

function courseValidation() {
var courseCode = document.getElementById("course").innerHTML;
var myRegExp = /\D{3}.\d{3}#\d{4}_\D{2}-\d{4}/;

if (courseCode.match(myRegExp)) {
    document.write("Incorrect Format");
} else {
    document.write("Correct Format");
}
}

Possible duplicate of How to get an input text value in JavaScript — Oghli
– Oghli, Commented Jun 18, 2017 at 23:34

Wayne Allen · Accepted Answer · 2017-06-18 23:33:22Z

2

You are not retrieving the value of the input correctly. var courseCode = document.getElementById("course").innerHTML; should be var courseCode = document.getElementById("courseCode").value;.

Also, your if statement is the wrong way around. It should read:

if (courseCode.match(myRegExp)) {
    document.write("Correct Format");
} else {
    document.write("Incorrect Format");
}

edited Jun 18, 2017 at 23:33

answered Jun 18, 2017 at 23:25

Wayne Allen

1,7451 gold badge11 silver badges18 bronze badges

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

Comments

dealwap · Accepted Answer · 2017-06-18 23:32:53Z

-1

The ID of your input is courseCode not course and you are using .innerhtml, it should be .value

var courseCode = document. getElementById('courseCode).value

answered Jun 18, 2017 at 23:32

dealwap

7313 gold badges17 silver badges38 bronze badges

Collectives™ on Stack Overflow

Trying to validate JavaScript input using OnClick and a function. No matter what is entered, it says the format is correct.

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related