Получить дни недели и определенное время дня

Question

Я использую javaScript для отображения элемента в будние дни и в рабочие часы (с понедельника по пятницу с 08:00 до 16:50).

Я использую этот код:

var date = new Date(); 
var thisMin = date.getMinutes(); 
var thisHour = date.getHours(); 
var thisWeekday = date.getDay(); 
// True if Mon - Fri between 08:00 - 16:50 
if (thisWeekday > 0 && thisWeekday < 6 && thisHour > 7 && (thisHour < 17 && thisMin <= 50)) { 
    alert("True"); 
} else { 
    alert("False"); 
} 

- но это лучший способ его код? Это похоже на множество условий, и он работает все время ... Может ли это быть сделано лучше с помощью javaScript?

Fiddle here.

Спасибо.

Answer 1

Обратите внимание, что у вас есть условие с двумя отдельными частями «должны быть в пределах недельного интервала» и «должны быть в пределах временного интервала». Код может отражать это более четко, будучи явным.

Для сравнения времени я бы перекроить текущее время, так что вы можете использовать Date «s <= и >= операторы вместо Переопределение времени сравнения себе:

var startTime = new Date(0, 0, 0, 8, 0); 
var endTime = new Date(0, 0, 0, 16, 50); 
var startWeekday = 1; 
var endWeekday = 5; 

function shouldDisplay() { 
    var d = new Date(); 

    function withinWeekInterval() { 
     return (d.getDay() >= startWeekday && 
       d.getDay() <= endWeekday); 
    } 

    function withinTimeInterval() { 
     var now = new Date(0, 0, 0, d.getHours(), d.getMinutes()); 
     return (now >= startTime && 
       now <= endTime); 
    } 

    return (withinWeekInterval() && withinTimeInterval()); 
} 

Ручной тест:

alert(shouldDisplay()); 

Answer 2

Try ниже Код:

var date = new Date(); 
var thisMin = date.getMinutes(); 
var thisHour = date.getHours(); 
var thisWeekday = date.getDay(); 
var isSuccess = false; 
// True if Mon - Fri between 08:00 - 16:50 
if (thisWeekday > 0 && thisWeekday < 6 && thisHour > 7 && (thisHour < 17)) { 
    if (thisHour == 16 && thisMin > 50) { 
     isSuccess = false; 
    } else { 
     isSuccess = true; 
    } 
} else { 
    isSuccess = false; 
} 

alert(isSuccess);