7

i want to check var check_val in between two time var open_time and var close_time

var open_time  = "23:30";
var close_time = "06:30";
var check_val  ="02:30";
if(Date.parse ( check_val ) > Date.parse ( open_time ) && Date.parse ( check_val ) < Date.parse ( close_time )){
    var flag=1;
} else { 
    var flag=2
}

the result is always else part

gprathour
  • 14,813
  • 5
  • 66
  • 90
user3805285
  • 73
  • 1
  • 4

2 Answers2

5

Date.parse() accepts dates in RFC2822 or ISO8601 formats.

In your case, it always returns NaN.

Date.parse("23:30"); // NaN

Using the appropriate Date format works as expected:

var open_time = Date.parse("2011-10-09T23:30");
var close_time = Date.parse("2011-10-10T06:30");
var check_val = Date.parse("2011-10-10T02:30");

if( check_val > open_time && check_val < close_time ) {
    var flag=1;
} else { 
    var flag=2
}
Community
  • 1
  • 1
Jay
  • 3,445
  • 2
  • 24
  • 29
5

You could create your own object to hold time and then write a function which uses it:

var Time = function(timeString) {
    var t = timeString.split(":");
    this.hour = parseInt(t[0]);
    this.minutes = parseInt(t[1]);
    this.isBiggerThan = function(other) { 
        return (this.hour > other.hour) || (this.hour === other.hour) && (this.minutes > other.minutes);
    };
}

var timeIsBetween = function(start, end, check) {
    return (start.hour <= end.hour) ? check.isBiggerThan(start) && !check.isBiggerThan(end)
    : (check.isBiggerThan(start) && check.isBiggerThan(end)) || (!check.isBiggerThan(start) && !check.isBiggerThan(end));    
}

var openTime = new Time("23:30");
var closeTime = new Time("06:30");
var checkTime = new Time("02:30");

var isBetween  = timeIsBetween(openTime, closeTime, checkTime);
jaho
  • 4,852
  • 6
  • 40
  • 66