Better way to determine date

Question

I am looking for best practice on how I may write this function more efficiently.

  var d = new Date(),
      time = d.getHours(),
      day  = d.getDay();

  if (day > 0 || day < 7){
    if (time > 18 || time < 8){
      // execute code

    }    
  }                     

Answer 1

if ( (day > 0 || day < 7) && (t > 18 || t < 8))

Is this what you want?

Second condition will only be evaluated only if the first one is true. If the first one is true, then only the second one is evaluated.

Answer 2

Change the first conditional to

if(day>0)

The Date.getDay() function always returns an integer between 0 - 6 so it will never be greater than or equal to seven.