Javascript Validation database

Question

I have a Javascript code (below) which validates mobile numbers. It's working fine. But the problem I am facing is both valid and invalid mobile numbers are being inserted into the database. I want only valid mobile numbers to be inserted into the database.

The code is as follows. Can anyone suggest any ideas to change the code?

<script type="text/javascript">
  function CheckINumber(INumber) {
    var INum = /^[+]{1}[0-9]{2}-\d{10}$/;
    if(INum.test(INumber)) {
      alert("Your Mobile Number Is Valid.");
    } else {
      alert("Your Mobile Number Is Not Valid.");
    }
  }
  <input name="Upload" type="submit" height="25" class="uploadbutton" onClick="CheckINumber(mobile.value);"

return False after "Invalid alert". i.e. before closing brace of else in this case. — Sujit Singh
– Sujit Singh, Commented Jan 5, 2013 at 5:36
otherwise, by default it executes further code, considering that every entered number is valid. — Sujit Singh
– Sujit Singh, Commented Jan 5, 2013 at 5:43
You need to validate in the server-side too, because the user can disable javascript and bypass your client-side validation. — Gabriel Gartz
– Gabriel Gartz, Commented Jan 5, 2013 at 6:01
Hi sujith thanks for your suggestion I tried but then also it is being inserted into database... — sudarshan
– sudarshan, Commented Jan 5, 2013 at 8:15

msivri · Accepted Answer · 2013-01-05 05:43:34Z

2

Are you using a form? If so you should do:

<form method="post" name="form_name" action="your_submit_url" onsubmit="return(CheckINumber(mobile.value));">

Also:

function CheckINumber(INumber)
{
  var INum = /^[+]{1}[0-9]{2}-\d{10}$/; 

  if(INum.test(INumber)) { return true; }
  return false;
}

answered Jan 5, 2013 at 5:43

msivri

3221 gold badge2 silver badges12 bronze badges

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

Comments

Kirk · Accepted Answer · 2013-01-05 05:57:16Z

0

Insert false in the invalid case to signal it failed to prevent the script from continuing.

if(INum.test(INumber))
{
 alert("Your Mobile Number Is Valid.");  
} 
else
{
  alert("Your Mobile Number Is Not Valid.");  
  return false;
}

edited Jan 5, 2013 at 5:57

Kirk

16.3k20 gold badges85 silver badges117 bronze badges

answered Jan 5, 2013 at 5:37

Prashant16

1,5264 gold badges18 silver badges40 bronze badges

1 Comment

techfoobar Over a year ago

+1 for correct answer. But an explanation to go with the answer will make it useful to everyone viewing the post and not just the OP.

Collectives™ on Stack Overflow

Javascript Validation database

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related