Match two dates and return true if match

Question

function isInDate(MatchDate, entries) {
    for (c = 0; c < entries.length; c++) {
        var entry = entries[c];
        var DateFrom = new Date(entry.DateFrom);
        var DateTo = new Date(entry.DateTo);

        if (
            DateFrom.getMonth() <= MatchDate.getMonth() &&
            DateFrom.getDay() <= MatchDate.getDay() &&
            DateTo.getDay() >= MatchDate.getDay()
        ) {
            return true;
        }
    }

    return false;
}

I am passing in a "MatchDate" that I want to compare with every looped entries "DateFrom" and "DateTo", if the matchdate is in the timespan I want to return true, otherwise false.

It should also return true if its the same day as DateFrom & same day as DateTo... I can't get it to work whatever I try.. Could anyone please assist?

Style note: use lower camel case (`matchDate`, `dateFrom`, etc.) for variable names. — John Montgomery, Sep 21 '17 at 23:50
You should also declare variables in loops: `for (var c = 0; ...)`. — RobG, Sep 22 '17 at 00:35
You want any of the dates to match or return false when they don't? — StackSlave, Sep 22 '17 at 00:42

score 3 · Answer 1 · answered Sep 21 '17 at 23:47

3

getDay returns the day of the week, not the day of the month.

You want to use getDate()

And you can simply just do

return MatchDate >= DateFrom && MatchDate <= DateTo;

answered Sep 21 '17 at 23:47

epascarello

204,599
20
195
236

score 0 · Answer 2 · answered Sep 22 '17 at 00:39

Should be like:

var pre = onload, isInDate; //for use on other loads
onload = function(){
if(pre)pre();
isInDate = function(MatchDate, entries){
  for(var c=0,l=entries.length; c<l; c++) {
    var entry = entries[c], DateFrom = new Date(entry.DateFrom), DateTo = new Date(entry.DateTo);
    var md = MatchDate.getTime();
    if(!(DateFrom.getTime() <= md && DateTo.getTime() >= md)){
      return false;
    }
  }
  return true;
}
}

Match two dates and return true if match

2 Answers2