Javascript validation error [closed]

Question

Closed. This question is not reproducible or was caused by typos. It is not currently accepting answers.

This question was caused by a typo or a problem that can no longer be reproduced. While similar questions may be on-topic here, this one was resolved in a way less likely to help future readers.

Closed 3 months ago.

Improve this question

I have written the following code in html and the javascript file is also embedded below

<!DOCTYPE html>
<meta charset="utf-8">
<HTML>
<head>
<link rel="stylesheet" href="admin.css"/>
<script type="text/javascript" src="common.js"></script>
</head>
<BODY>
<div class="adminnav" id="adminnav">
<ul>
<li><a href="#">Insert</a>
<ul>
<li><a href="addcourse.html">Insert Course</a></li>
<li><a href="addstudent.html">Insert Student</a></li>
<li><a href="addteacher.html">Insert Teacher</a></li>
<li><a href="addsubject.html">Insert Subject</a></li>
</ul>
</li>
</ul>
</div>
<div class="addcourse" >
<form name="course" onSubmit='return validcourse()' method="POST">
Course Name:
  <input type="text" name="coursename" id="cname" /><br>
Duration:<input type="text" name="cd"/>
<br>
Course Id:<input name="cid" type="text" /><br>
<input  type="submit" value="submit" name="submit"><input type="reset" name="coursereset">
</form>
</div>
</BODY>
</HTML>

common.js

function validcourse()
{
    var course_name=document.course.coursename;
    var course_duration=document.course.cd;
    var course_id=document.course.cid;

    if(course_name_valid(course_name))
    {
        {
            if(course_duration_valid(course_duration))
            {
                if(course_id_valid(cid))
                {
                }
            }
        }
    }


}

function course_name_valid(course_name)
{
    var letters=/^[A-Za-z]+$/;
    if(course_name.value.match(letters))
    {
        return true;
    }
    else
    {
        alert("Course name must have alphabets only");
        course_name.focus();
        return false;

    }
}

function course_duration_valid(course_duration)
{
    var letters=/^[1-9]+$/;
    if(cd.value.match(letters))
    {
        return true;
    }
    else
    {   
        alert("Course Duration can have numbers only");
        course_duration.focus();    
        retun false;
    }   


}

function course_id_valid(course_id)
{
    var letters=/^[0-9a-zA-z]+$/;
    if(course_id.value.match(letters))
    {
        return true;
    }
    else
    {
        alert('Course ID must have character and numeric values only');
        course_id.focus();
        return false;           
    }
}

My problem is that nothing is happening no error messages are given. Error messages are given only for course name but not for course duration and course id?

Here is a link to jsfiddle http://jsfiddle.net/amolkarale/aTfq6/1/

Put an alert at the top of your js and see if that executes. I suspect you are calling the wrong file. — Sudipta Chatterjee
– Sudipta Chatterjee, Commented Mar 13, 2013 at 23:02
I am calling common.js file which contains the code for validation.You can check the script tag. — Amol
– Amol, Commented Mar 13, 2013 at 23:05
Can you create a jsfiddle and update to post the link - much easier to debug that way — Sudipta Chatterjee
– Sudipta Chatterjee, Commented Mar 13, 2013 at 23:09

Steve · Accepted Answer · 2013-03-13 23:04:57Z

1

You are not returning anything from validcourse. Add a return true; and return false; in there, to either let the submission continue or stop the submission from happening:

function validcourse()
{
    var course_name=document.course.coursename;
    var course_duration=document.course.cd;
    var course_id=document.course.cid;

    if(course_name_valid(course_name))
    {
        {
            if(course_duration_valid(course_duration))
            {
                if(course_id_valid(cid))
                {
                    return true;
                }
            }
        }
    }
    return false;

}

answered Mar 13, 2013 at 23:04

Steve

8,6487 gold badges42 silver badges54 bronze badges

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

3 Comments

Amol Over a year ago

I have tried man but it is not working still validations are not taking place

Steve Over a year ago

Have you set a breakpoint on the first line inside validcourse? Just to see if your function ever gets executed?

Steve Over a year ago

You don't "write" breakpoints, you "set" them in your browser's debugger. In Chrome for example: developers.google.com/chrome-developer-tools/docs/…

blamonet · Accepted Answer · 2013-03-13 23:19:58Z

0

Looks like you have a syntax error in your third functioncourse_duration_valid(course_duration)

retrun false;

should be:

return false;

answered Mar 13, 2013 at 23:19

blamonet

6901 gold badge6 silver badges13 bronze badges

1 Comment

Amol Over a year ago

ya spelling mistake i have made it return

Sudipta Chatterjee · Accepted Answer · 2013-03-14 00:40:03Z

0

Okay, so I got your stuff to work. Here are the changes needed:

First, object references need to be cleaner in your DOM

var course_name=document.getElementById("cname");
var course_duration=document.getElementById("cd");
var course_id=document.getElementById("cid");

As you can see, the elements need to have their individual ids.

Course Name:
<input type="text" name="coursename" id="cname" /><br>
Duration:
<input type="text" name="cd" id="cd"/><br>
Course Id:
<input name="cid" type="text" id="cid" /><br>

Finally, the retrun needed to be corrected, as already pointed out by @Steve

answered Mar 14, 2013 at 0:40

Sudipta Chatterjee

4,6702 gold badges29 silver badges30 bronze badges

Collectives™ on Stack Overflow

Javascript validation error [closed]

3 Answers 3

3 Comments

1 Comment

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

1 Comment

Comments

Related