JavaScript - RegEx for time or a specific string

Question

In my form I have a field which asks the user what time they would like me to call them.

The code for the input is as follows (excluding the <form> tags):

<p>
   <label>When can we call you?:</label>
   <br>
   <input type="text" name="contactconvenience" placeholder="HH:MM or &#34;anytime&#34;" onchange="checkCallTime()">
</p>
<div id="callbackwarn"></div>

I want to use a RegEx to check whether the user has entered a time in the format HH:MM or the specific string "anytime", but the code I'm using doesn't work.

var callback = /^\d{2}([:]\d{2})$/;

var callbackanytime = str.match(/anytime/g);

function checkCallTime() {
    var valid = true;

    if ((!callback.test(document.bookingsform.contactconvenience.value)) || (!callbackanytime.test(document.bookingsform.contactconvenience.value))) {
        document.bookingsform.contactconvenience.style.border = "1px solid red";
        document.getElementById("callbackwarn").innerHTML = "Enter a time in the format HH:MM or type \"anytime\".";
        document.bookingsform.contactconvenience.title = "Please enter a time in the format HH:MM or type \"anytime\".";
        document.getElementById("callbackwarn").style.display = "block";
        valid = false;
    } else {
        document.bookingsform.contactconvenience.style.border = "1px inset #EBE9ED";
        document.bookingsform.contactconvenience.style.borderRadius = "2px";
        document.getElementById("callbackwarn").style.display = "none";
    }
}

As you can see I have use two separate RegEx expressions, one for matching a time in the format HH:MM; and the other for matching the specific string "anytime".

In the function itself, I'm using the || to see if either of the RegExs have been matched or not.

NB: I am aware that I'm using the onchange rather than onsubmit, the <form> tag consists of the onsubmit calling on a master function.

Alex K. · Accepted Answer · 2014-01-31 15:32:04Z

1

Assumming 00:00 for midnight;

var callback = /^(([01]\d|2[0-3]):([0-5]\d)|anytime)$/;

if (!callback.test(document.bookingsform.contactconvenience.value))
{
   // error code
}

edited Jan 31, 2014 at 15:32

answered Jan 31, 2014 at 15:21

Alex K.

177k32 gold badges276 silver badges299 bronze badges

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

4 Comments

RoyalSwish Over a year ago

This isn't working, if I type in arbitrary characters it won't show the error. I removed the || and now just using your one RegEx. Any ideas?

RoyalSwish Over a year ago

Yep, that isn't working for me. I even copied and pasted your code, still nothing.

Alex K. Over a year ago

Works ok for me? jsfiddle.net/alexk/4AZqs (must click out of the input to raise the event)

RoyalSwish Over a year ago

How peculiar, I copied and pasted your jsfiddle code and now it works perfectly. There must've been some mistyped code which I failed to pick up in the error code area of the function. Works perfectly now. Thanks!

Toto · Accepted Answer · 2014-01-31 15:15:33Z

0

You have to use an AND && not an OR || between the two test:

if ((!callback.test(document.bookingsform.contactconvenience.value)) 
    && 
    (!callbackanytime.test(document.bookingsform.contactconvenience.value))) {

NOT (A OR B) <=> NOT A AND NOT B

You could also use an unique regex:

/^\d{2}:\d{2}|\s*anytime\s*$/;

answered Jan 31, 2014 at 15:15

Toto

91.7k63 gold badges97 silver badges135 bronze badges

4 Comments

RoyalSwish Over a year ago

I changed it and this didn't work. I then tried your unique RegEx and still didn't work. Any ideas?

Toto Over a year ago

What is the value of contactconvenience.

RoyalSwish Over a year ago

I think None? I haven't given the tag a value attribute.

Toto Over a year ago

@RoyalSwish: I meant the value when you enter the function.

Collectives™ on Stack Overflow

JavaScript - RegEx for time or a specific string

2 Answers 2

4 Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related