Наверное, здесь функция должна называться getFreeTimes? Или можешь придумать еще понятнее?

• Очень большая вложенность.
• Идет привязка с именам бандитов, что не есть хорошо. Вдруг они изменят состав?
• Переменные begin и end инициализированы далеко от места использования, и неясно, начало и конец чего они обозначают.
• i, j, k тоже никак себя не объясняют.

Привязка к конкретной дате

Здесь больше подойдет глагол get. Ты же его возвращаешь?)

Это название функции тоже не годится)

Однобуквенных переменных вообще быть не должно, только если в крайних случаях это не цикл по индексам.

Можно записать в 1 строку тернарными операторами)

Есть метод toString на крайний случай

Ай-ай-ай, константы!

Это уже не просто hours

можно же так: hours += bankTimeZone - timeZone

Тут не нужна пустая строка

Почему нельзя воспользоваться функцией Math.max?

у меня не работает Math.max для дат

А если один из этих способов?

О, давай через reduce. Типа пройти по всем датам и найти такую, которая больше других. Очевидно, что sort имеет сложность больше линейной - O(log(n) * n), а reduce будет O(n)

Функция должна занимать не больше 25 строк

Кажется, что можно сначала просчитать все leadToDataType(schedule[key][i].to, bankTimeZone) для каждого i, а потом заполнить ими словарь.

И попробуй использовать reduce

Привязка к именам бандитов. И слишком большая вложенность. Такого быть не должно. Нельзя ли просто по ключам пробежаться?

+1

так делать совсем нельзя - а если будет сколько угодно бандитов или имена могу быть другими?

Не проще ли сделать так: var day = WEEK[date.getDate()]?
Почему словарь называется WEEK? Почему метод getDate возвращает день?

Что значит это название?)

Ты уверена, что без этих волшебных дат тут не обойтись?

Лучше названия не сокращать

Ты много где пушишь объект с этими полями. Можно вынести в отдельную функцию

судя по смыслу, аргумент лучше назвать freeIntervals

а почему такое название функции? formatE ? Судя по коду, мы тут интервалы пересекаем с [ПН, СР] и интервалы разбиваем по дням. Это скорее некая нормализация - normalizeIntervals

а что за даты такие? Чтобы стало понятнее, нужно сделать переменные с говорящими именами

мы тут пытаемся разложить interval на отрезки по дням? и в итоге будут теже интервалы разложенные по дням - так? forEach, который собирает некий результат - лучше писать через reduce

хм, возможно, лучше назвать fromDay и toDay.

типа не дальше среды?

О, давай через reduce. Типа пройти по всем датам и найти такую, которая больше других. Очевидно, что sort имеет сложность больше линейной - O(log(n) * n), а reduce будет O(n)

опять по смыслу reduce = проход по массиву и получение одного результата

ты часто используешь конструкцию: new Date(0, 0, x, y, z) - кмк для нее лучше завести хелпер-функцию с говорящим именем, тогда такие вызовы будут не такими таинственными

а зачем нам объект? можно было бы сделать две переменные - bankStart и bankEnd

опять reduce нужен, смотри как коротко получается:

var intervals = timeToRobbery.reduce(function(res, interval) {
    if (interval.to - interval.from >= msInDuration) {
        res.push(interval);
    }
    return res;
}, []);

и в конце можно написать тернарник: return intervals.length === 0 ? nul : intervals;

можно без скобок