By using this site, you agree to our updated Privacy Policy and our Terms of Use. Manage your Cookies Settings.
424,688 Members | 1,888 Online
Bytes IT Community
+ Ask a Question
Need help? Post your question and get tips & solutions from a community of 424,688 IT Pros & Developers. It's quick & easy.

Problem with Focus... Help needed.

P: n/a
Hi

I have created function to check date and time. at the time of
execution, if date is left empty the function returns the error
message but then the focus goes to next field. Next filed is for time
and there is also a check on that. If time is empty return error
message. Again the explorer returns error message and then shifts the
focus to date field, again error message and focus goes to time and
this story goes on and on, untill browser is killed. can somebody help
me in debuggin this script.

Thanks a lot

Nitin

<head>
</HEAD>
<body>
<script language=javascript>
function CheckDate(sDate) {
var dateArray = sDate.value.split("/");
if (!/^\d{2}\/\d{2}\/\d{2}$/.test(sDate.value)) {
alert("Please enter a date in this format: dd/mm/yy.");
sDate.focus();
return false;
}
if (sDate.value.length < 8 )
{
alert("Date in not valid format");
sDate.focus();
return false;
}
var now= new Date();
var longYear= now.getYear();
longYear= (longYear+"").substring(2,4);
if (dateArray[2] < longYear) {
alert("Year "+dateArray[2]+" is wrong.");
sDate.focus();
return false;
}
if (dateArray[1] > 12 || dateArray[1] < 1) {
alert("Month "+dateArray[1]+" is wrong.");
sDate.focus();
return false;
}
if (dateArray[1] == 1 || dateArray[1] == 3 || dateArray[1] == 5 ||
dateArray[1] == 9 || dateArray[1] == 11){
if (dateArray[0] > 30 || dateArray[0] < 1){
alert("Date "+dateArray[0]+" in Month of "+dateArray[1]+" is
wrong");
sDate.focus();
return (false);
}
}
if (dateArray[1] == 4 || dateArray[1] == 6 || dateArray[1] == 7 ||
dateArray[1] == 8||dateArray[1] == 10||dateArray[1] == 12){
if (dateArray[0] > 31 || dateArray[0] < 1) {
alert("Date "+dateArray[0]+" in Month of "+dateArray[1]+" is
wrong");
sDate.focus();
return (false);
}
}
if (dateArray[1] == 2 && dateArray[0] > 29 || dateArray[0] < 1) {
alert("Date "+dateArray[0]+" in Month of "+dateArray[1]+" is
wrong.");
sDate.focus();
return (false);
}
return (true);
}
function CheckTime(sTime){
if (!/^([01]?[0-9]|[2][0-3])([:][0-5][0-9])?$/.test(sTime.value))
{
alert("Please enter the Time in this format:\n\n
hh:mm.");
sTime.focus();
return (false);
}
var timeArray = sTime.value.split(":");
if (sTime.value.length < 5 )
{
alert("Time in not valid format");
sTime.focus();
return (false);
}
if (sTime[0] > 23) {
alert(" hrs is not possible in a day. Invalid time.");
sTime.focus();
return (false);
}
if (sTime[1] > 60) {
alert ("mins is not possible in an hour. Invalid time.");
sTime.focus();
return (false);
}
return (true);
}
</script><HR><FORM METHOD="POST"
ACTION="/sso/cgi-bin/g100/g100_input_form.cgi/confirm"
ENCTYPE="application/x-www-form-urlencoded" NAME="input">
<TABLE CELLSPACING="0" BORDER="0" CELLPADDING="5" WIDTH="100%"><TR
ALIGN="left"><TD>Requestor</TD> <TD>Nitin Thakur</TD></TR>
<TR ALIGN="left"><TD>Email</TD> <TD>ni**********@ubs.com</TD></TR>
<TR ALIGN="left"><TD>Date (GMT) (dd/mm/yy)</TD> <TD><INPUT TYPE="text"
NAME="startdate" VALUE="" SIZE=8 MAXLENGTH=8 ONBLUR="return
CheckDate(this)"></TD></TR>
<TR ALIGN="left"><TD>Time (GMT) 24 hrs format (hh:mm)</TD> <TD><INPUT
TYPE="text" NAME="starttime" VALUE="" SIZE=5 MAXLENGHT="5"
ONBLUR="return CheckTime(this)"></TD></TR>
<BR>
<TABLE CELLSPACING="0" BORDER="0" CELLPADDING="5" WIDTH="50%"><TR
ALIGN="center"><TD>
<INPUT TYPE="submit" NAME="confirm" VALUE="Submit"
</FORM></BODY></HTML>
Jul 23 '05 #1
Share this Question
Share on Google+
4 Replies


P: n/a
On 16 Aug 2004 08:51:56 -0700, Nitin <ni**********@ubs.com> wrote:
I have created function to check date and time. at the time of
execution, if date is left empty the function returns the error
message but then the focus goes to next field. Next filed is for time
and there is also a check on that. If time is empty return error
message. Again the explorer returns error message and then shifts the
focus to date field, again error message and focus goes to time and
this story goes on and on, untill browser is killed. can somebody help
me in debuggin this script.
I don't believe that there's a problem with your script (I haven't taken a
close look), I think it's your HTML.

[snip]
<INPUT TYPE="text" NAME="startdate" VALUE="" SIZE=8 MAXLENGTH=8
ONBLUR="return CheckDate(this)"> [...]


The blur event is fired *every* time the control loses focus. So:

1) Click or tab to next control (B)
2) Current control (A) loses focus (event fires) and control B gains focus.
3) Validation fails, alert shows error message and user dismisses it.
4) Control A requests focus.
5) Control B loses focus and control A gains focus.
6) Validation for B fails, alert shows, and user dismisses it.
7) B requests focus.
8) Repeat from 3.

The solution is to use the change event; the event will only fire if the
value has actually changed. The user can skip a field without changing it,
and you won't end up in a loop. Of course, this means that you'll need to
validate the form at submission in case the user never enters a value.

Hope that helps,
Mike

--
Michael Winter
Replace ".invalid" with ".uk" to reply by e-mail
Jul 23 '05 #2

P: n/a
Michael Winter wrote:
Nitin wrote:

<snip>
<INPUT TYPE="text" NAME="startdate" VALUE="" SIZE=8
MAXLENGTH=8 ONBLUR="return CheckDate(this)"> [...]


The blur event is fired *every* time the control loses focus. So:

1) Click or tab to next control (B)
2) Current control (A) loses focus (event fires) and control B
gains focus. 3) Validation fails, alert shows error message
and user dismisses it.
4) Control A requests focus.
5) Control B loses focus and control A gains focus.
6) Validation for B fails, alert shows, and user dismisses it.
7) B requests focus.
8) Repeat from 3.

<snip>

The same never ending loop is the consequence but in practice when an
alert is put up the focus goes to the OK button on the alert, so control
B loses focus at that point. Assuming that it has actually been passed
focus at that point. I have a recollection of a browser where the onblur
code is executed prior to the focusing of the next control, so when the
onblur code re-focuses the first control the browser then goes on to do
what it was planning anyway and focuses the next control, re-blurring
the first.

I too would recommend not attempting validation with onblur events.

Richard.
Jul 23 '05 #3

P: n/a
On Mon, 16 Aug 2004 18:28:31 +0100, Richard Cornford
<Ri*****@litotes.demon.co.uk> wrote:

[snip]
The same never ending loop is the consequence but in practice when an
alert is put up the focus goes to the OK button on the alert, so control
B loses focus at that point. [...]
I assumed that it would, but it doesn't appear to fire an event. At least
not in my tests with Opera when an event is being executed. I didn't test
with other browsers as Opera was sufficient for building the sequence I
wrote.
I too would recommend not attempting validation with onblur events.


That's more important, anyway.

Mike

--
Michael Winter
Replace ".invalid" with ".uk" to reply by e-mail
Jul 23 '05 #4

P: n/a
JRS: In article <ff**************************@posting.google.com >,
dated Mon, 16 Aug 2004 08:51:56, seen in news:comp.lang.javascript,
Nitin <ni**********@ubs.com> posted :

I have created function to check date and time.


One presumes that you are paid per yard of code; a much shorter date
(and time) validator can be found via the FAQ : see below.

Also see the recent thread here "Multiple Datepicker".

You can I think solve your perceived problem by validating onChange
rather than onBlur; but it would be better IMHO to validate when the
user signals that data entry is completed.

When posting code to News, it is your duty not to allow the system to
wrap your code lines - anyway, you indent in units which are too large
for convenience.

HTML should also be indented to show structure, if people are to be
expected to read it.

You seem to ask for the time in GMT; the date also should be in GMT;
then, for consistency, var longYear= now.getYear();
should probably use getUTCyear()
--
John Stockton, Surrey, UK. ?@merlyn.demon.co.uk Turnpike v4.00 IE 4
<URL:http://www.jibbering.com/faq/> JL/RC: FAQ of news:comp.lang.javascript
<URL:http://www.merlyn.demon.co.uk/js-index.htm> jscr maths, dates, sources.
<URL:http://www.merlyn.demon.co.uk/> TP/BP/Delphi/jscr/&c, FAQ items, links.
Jul 23 '05 #5

This discussion thread is closed

Replies have been disabled for this discussion.