2

i'm working on a registration page. im stuck in this javascript form validation. what i want to do is first validate the entered data then pass the value to action page. but the validation never works. i tried removing action and also removing (form) in function and tried passing form but none works.

here is my code

<html>

<head>
    <title>Registeration</title>
    <script language="javascript">
    function regval(form) {
        var a = document.forms["regform"]["username"].value;
        var b = document.forms["regform"]["password"].value;

        if (a == null || a == "") {
            alert("Enter username");
            return false;
        }
        if (b == null || b == "") {
            alert("enter password");
            return false;
        }

    }
    </script>
</head>

<body>
    <form name="regform" id="regform" action="registration.php" method="post" onSubmit="return regval();">
        <table width="274" border="0" align="center" cellpadding="2" cellspacing="0">
            <tr>
                <td width="95">
                    <div align="right">Username:</div>
                </td>
                <td width="171">
                    <input type="text" name="username" />
                </td>
            </tr>
            <tr>
                <td>
                    <div align="right">Password:</div>
                </td>
                <td>
                    <input type="password" name="password" />
                </td>
            </tr>
            <td>
                <div align="right"></div>
            </td>
            <td>
                <input name="submit" type="submit" value="register" onClick="validateForm()" />
            </td>
            </tr>
        </table>
    </form>
</body>

</html>
3
  • if(b==nul||b="") should be if(b==null || b="") I think. Commented Jun 25, 2015 at 5:32
  • using if(!a) {...logic...} if(!b) {...logic...} will be best option, and I guess you should return true also when control doesn't go into if statement Commented Jun 25, 2015 at 5:39
  • i added alert(); line in script code and it is working . Commented Jun 25, 2015 at 5:58

3 Answers 3

2
if(b==nul||b="")

Should be:

if(b==null||b=="")

Or shorter:

if(!a) { alert("Enter username"); return false; }
if(!b) { alert("Enter password"); return false; }
Sign up to request clarification or add additional context in comments.

Comments

1
<html>
<head>
<title>Registeration</title>
<script language="javascript">
function validateForm(form)
{
var a=document.forms["regform"]["username"].value;
var b=document.forms["regform"]["password"].value;
alert(a);

if(a==null || a=="")
{
alert("Enter username");
return false;
}
if(b==nul||b=="")
{
alert("enter password");
return false;
}

}
</script>
</head>
<body>
<form name="regform" id="regform" action="#"  method="post" onSubmit="return regval();">
<table width="274" border="0" align="center" cellpadding="2" cellspacing="0">
<tr>
<td width="95"><div align="right">Username:</div></td>
<td width="171"><input type="text" name="username" /></td>
</tr>
<tr>
<td><div align="right">Password:</div></td>
<td><input type="password" name="password" /></td>
</tr>
<td><div align="right"></div></td>
<td><input name="submit" type="submit" value="register" onClick="validateForm()"/></td>
</tr>
</table>
</form>
</body>
</html>

3 Comments

it prints the username
i just added a alert() without printing value and its working .
if(a=="") { document.getElementById("user-err").innerHTML = "Enter username"; return false; } <td width="171"><input type="text" name="username" /> <div id="user-err"></div> </td> Check this one
0

try this code

function regval(form)
    {
        var a=document.forms["regform"]["username"].value;
        var b=document.forms["regform"]["password"].value;

        if(a=="")
        {
            alert("Enter username");
            return false;
        }
        if(b == "")
        {
            alert("enter password");
            return false;
        }

    }

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.