Conversation
|
🍅 Пройдено тестов 6 из 19 |
|
🍅 Пройдено тестов 6 из 19 |
|
🍅 Пройдено тестов 12 из 19 |
|
🍅 Пройдено тестов 10 из 16 |
savichev-igor
left a comment
savichev-igor
left a comment
There was a problem hiding this comment.
Читать код сложно, постарайся внести побольше семантики и избавиться от этих индексов, посмотри в сторону использования объектов и свойств
Добавь где-то JSDoc или лаконичные комментарии, чтобы объяснить что происходит
🍅
| var MINUTES_IN_HOUR = 60; | ||
| var HOUR_IN_DAY = 24; | ||
| var TOTAL_TIME_LINE = WEEK_DAYS.length * HOUR_IN_DAY * MINUTES_IN_HOUR; | ||
| // var TRY_LATER_MINUTES = 30; |
There was a problem hiding this comment.
Такого рода комментарии лучше удалять
| .replace('%MM', (minutes < 10 ? '0' : '') + minutes); | ||
| } | ||
|
|
||
| // /** |
|
|
||
| function calcGangPartyFreeTime(scheduleArr) { | ||
| var result = []; | ||
| for (let personName in scheduleArr) { |
| continue; | ||
| } | ||
| var personResult = []; | ||
| for (let i = 0; i < scheduleArr[personName].length; i++) { |
There was a problem hiding this comment.
Используешь где-то let, но при этом не используешь const для констант
| return hours * MINUTES_IN_HOUR + Number(format[3]); | ||
| } | ||
|
|
||
| function busyTimeToFreeTime(timeArr) { |
There was a problem hiding this comment.
Как-то много индексов и каких-то проверок в этом методе, постарайся хотя бы сделать, чтобы было обращение к свойствам, а не индексам, ибо вообще сложно разобраться что к чему здесь
There was a problem hiding this comment.
За 2 минуты так и не смог понять, что делает этот метод
| function busyTimeToFreeTime(timeArr) { | ||
| var result = []; | ||
| for (let i = 0; i <= timeArr.length; i++) { | ||
| if (i === 0) { |
There was a problem hiding this comment.
Почему такое исключение для первой итерации?
| function getBankWorkingTime(timeArr) { | ||
| var result = []; | ||
| for (let i = 0; i < WEEK_DAYS.length; i++) { | ||
| result.push([stringToInt(WEEK_DAYS[i] + ' ' + timeArr.from), |
There was a problem hiding this comment.
Почему бы здесь не использовать интерполяцию строк?
|
|
||
| function getBankWorkingTime(timeArr) { | ||
| var result = []; | ||
| for (let i = 0; i < WEEK_DAYS.length; i++) { |
| var arr2 = compare ? arrays[0] : arrays[1]; | ||
| for (let i = 0; i < arr1.length; i++) { | ||
| for (let k = 0; k < arr2.length; k++) { | ||
| // if (arr1[i][0] >= arr2[k][1] || arr1[i][1] <= arr2[k][0]) { |
| } | ||
|
|
||
| function checkTimeInterval(arr1, arr2, newArr) { | ||
| if (arr1[0] >= arr2[1] || arr1[1] <= arr2[0]) { |
|
Почему-то в почту стали падать уведомления про твой PR, думал что уже можно проверять Поздно увидел, что тесты ещё не все пройдены Можешь уже опираться на моё частичное ревью) |
3 participants
No description provided.