Skip to content

Conversation

@makhalin
Copy link

No description provided.

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Пройдено тестов 14 из 19

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройдено тестов 19 из 19

@honest-hrundel
Copy link

@Victoria-Vladimirova обрати внимание решено доп. задание

@mokhov
Copy link

mokhov commented Oct 27, 2016

Николай, отправь PR ещё раз

@honest-hrundel
Copy link

🍏 Пройдено тестов 19 из 19

@honest-hrundel
Copy link

@FrodoTheTrue обрати внимание решено доп. задание

@FrodoTheTrue
Copy link

почему билд сломан?

}

function convertTime(time, timezone) {
time = /(\d{2}):(\d{2})\+(\d+)/.exec(time);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DRY

function convertSchedule(schedule, timezone) {
var newSchedule = [];
for (var name in schedule) {
if (schedule.hasOwnProperty(name)) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем эта проверка?

var newSchedule = [];
for (var name in schedule) {
if (schedule.hasOwnProperty(name)) {
var persSchedule = schedule[name]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

persSchedule не очень понятное название

function getRobberyTimes(fullSchedule, duration) {
var robberyTimes = [];
var iter = [];
var iterMax = [];

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

хитро сделано с хранением числа в массиве :)
iter, iterMax здесь не подходят, не понятно что итерируем и как

getIters(iter, iterMax, fullSchedule);

while (iter[0] < iterMax[0]) {
var minMax = getMinMax(fullSchedule, iter);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

странное название для функции

*/
exports.getAppropriateMoment = function (schedule, duration, workingHours) {
console.info(schedule, duration, workingHours);
exports.getAppropriateMoment = function sad(schedule, duration, workingHours) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sad ? 😢 😞 😿

@FrodoTheTrue
Copy link

🍅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants