Левшин Михаил by LevshinMisha · Pull Request #18 · urfu-2016/javascript-task-3 · GitHub

No description provided.

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

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

это константа? посмотри, как в гайде рекомендуют называть константы.

Да Это по сути константа. Исправлю

зачем тебе этот массив? можно просто взять ключи словаря)

Для читаемости кода. daysDict.keys()[i] хуже чем daysArray[i]

по-моему, это не повод у всех словарей выделять ключи в массив) в конце концов, можешь уж список-то заново не создавать

числа нужно вынести в константы

в названии функции надо указывать, что она делает (то бишь глагол)

аналогично с timeInInterval

а зачем тут отступ перед return?)

str.split(' ') наверное стоит вынести в переменную, раз часто используется

константы

по-моему, это не самое подходящее название переменной. мало ли чего там m может быть?) да и h тоже

кажется, тут есть дублирование... и константы, конечно.

ой, как много констант

goodTimeToHack = !timeInIntervals(newSchedule[gangster], i) ?

В случае если время неподходит первому, но подходит второму и третьему, то значение переменной будет true. Что не совсем то что нужно.

опять имя функции не по канону

просто интересно: а почему не >=? а именно отрицание от <?

Потому что я могу)

кажется, что это можно переписать в один if, ну или как-то более сокращенно

везде есть константы, которые надо вынести/переименовать.

🍅

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

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

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

лучше isTimeInIntervals

и аналогично во всех остальных местах)

меня смущает, что у тебя 4 раза встречается строка
strToIntDate(something) + bankTimezone * MINUTES_IN_HOUR
это никак не сделать поизящнее?
имею в виду DRY

иногда отступы всплывают в самых неожиданных местах. не рекомендую их ставить после фигурных скобочек (не только здесь, но и во всём коде). тут же нечего отделять логически?

а ещё название этой переменной предполагает, что в ней хранится время.

название функции следует начинать с глагола

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

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

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

можно сделать isGoodTimeToHack = !isTimeInIntervals(schedule[gangster], i)

Сделать, то можно. Только это не совсем правильно(я уже писал почему). Даже базовые тесты перестают проходить.

В данном while у тебя i не изменяется, кроме как увеличивается при каждой итерации? Почему бы тогда не сделать это циклом?

функция isGangsterNotBusy использует внешние переменные, одной из которой является переменная i. А так как функции объявлять внутри цикла нельзя(lint), то следовательно нужно объявить эту переменную и функцию ее использующую заранее. Я вижу выход из этой ситуации созданием функции, которая возвращает другую функцию. Попробую это сделать.

исправил даже изящнее чем сам предполагал)

А зачем ты это вынес в отдельную функцию? У тебя она вынесена далеко от этого места, да и содержит в себе 1 строчку...

В lint'е стоит ограничение, что нельзя создавать функции внутри цикла

Очень сложная строка. И давай у тебя весь парсинг будет вынесен в одно место?

Эту функцию можно разбить на наполнение нового расписания гангстерами и банком. И объединить результат как раз в этой функции.

Лучше создай новую переменную с соответствующим названием. А то не ясно, чем результат этой функции отличается от предыдущей. А так -- мне нравится организация кода 👍

Кажется, тут лишний перенос строки

Попробуй написать это как-то по-другому. Попроще :)

Ну если честно я не знаю как. https://learn.javascript.ru/object-for-in смотрел тут. hasOwnProperty для lint'a

https://urfu-2016.github.io/javascript-slides/03-types/#/19

Почему удалил доки?

мешает

Но они же несут полезную информацию!

Если честно, на первый взгляд тут адище происходит. Слайсы не подойдут?

🍅

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

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

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

В этой функции слишком часто используется arrayIntervals.length, лучше вынеси в отдельную переменную. Да и название покороче будет)

Можно написать здесь return false;, а в конце функции -- return true;. Тогда без этой переменной обойдемся и меньше бегать по циклу будем)

К сожалению из forEach нельзя ничего возвращать(

В целом стало намного лучше, молодец. Исправь мелкие замечания, и я передаю тебя ментору.

🚀

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

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

В данном решении очень много избыточного кода. Из-за этого его очень сложно читать. Хочу в ответном комментарии увидеть ответ на вопрос: как работает твой алгоритм?

Это ключи предыдущего объекта

Название читаю, а понять что она делает не могу

Зачем это нужно?

https://urfu-2016.github.io/javascript-slides/03-types/#/19

start и end известны на этапе создания объекта, почему length нужно вычислять, а нельзя сделать полем?

Это нечитаемый однострочник, нужно вынести аргументы TimeIntreval в читаемые переменные

Название функции переводится как «Упростить временные интервалы». Я честно не понимаю что тут подразумевается

Для фильтрации массивов придумали специальную функцию https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Global_Objects/Array/filter

Из названия функции абсолютно не понятно что она делает. И по коду я тоже ничего не понял

🍅

Первый коммит

4aab65b

honest-hrundel assigned chipolinka Oct 23, 2016

chipolinka reviewed Oct 24, 2016

View reviewed changes

+Константы, +глаголы-функции, -костыли

82eda4e

К слову о костылях

c5e2d5a

chipolinka reviewed Oct 24, 2016

View reviewed changes

Систему счисления всем parseInt'ам за мой счет!

ad30429

chipolinka reviewed Oct 28, 2016

View reviewed changes

все что смог сделал

73a3ba5

chipolinka reviewed Oct 28, 2016

View reviewed changes

honest-hrundel assigned mokhov and unassigned chipolinka Oct 28, 2016

Вернул комментарии, переименовал некоторые переменные

652f8e8

mokhov reviewed Oct 30, 2016

View reviewed changes

-               * @param {String} workingHours.to – Время закрытия, например, "18:00+5"
-               * @returns {Object}
-               */
+              var daysDict = { 'ПН': 0, 'ВТ': 1, 'СР': 2 };

-               * @returns {Object}
-               */
+              var daysDict = { 'ПН': 0, 'ВТ': 1, 'СР': 2 };
+              var daysArray = ['ПН', 'ВТ', 'СР'];

+                  var day = 0;
+                  if (daysDict[str.slice(0, 2)] !== undefined) {
+                      day = daysDict[str.split(' ')[0]];
+                      str = str.split(' ')[1];

+                  var timezone = parseInt(str.split('+')[1]);
+                  str = str.split('+')[0];
+                  var h = parseInt(str.split(':')[0]);
+                  var m = parseInt(str.split(':')[1]);

+                      });
+                  });
+                  newSchedule.Bank.push(new TimeInterval(strToIntDate(workingHours.from) +
+                      bankTimezone * 60, strToIntDate(workingHours.to) + bankTimezone * 60));

+                  };
+                  this.getNextDay = function () {
+                      return new TimeInterval(this.start + 24 * 60, this.end + 24 * 60);

+                  };
+              }
+              function timeInIntervals(timeIntervalsArray, time) {

+                  for (var i = 0; i < timeIntervalsArray.length; i++) {
+                      if (timeIntervalsArray[i].timeInInterval(time)) {
+                          return true;

+                              }
+                          }
+                          for (; i < 3 * 24 * 60; i++) {

+                          var i = 0;
+                          function gangsterNotBusy(gangster) {
+                              if (timeInIntervals(newSchedule[gangster], i)) {
+                                  goodTimeToHack = false;

+                  function simplifyTimesIntervals() {
+                      var newTimeIntervals = [];
+                      goodTimesIntervals.forEach(function (timeInterval) {
+                          if (!(timeInterval.getLength() < duration)) {

                       tryLater: function () {
+                          if (this.exists()) {
+                              var startInterval = goodTimesIntervals[0];
+                              if (goodTimesIntervals.length > 1 || startInterval.getLength() >= duration + 30) {

+                  };
+              }
+              function timeIsInIntervals(timeIntervalsArray, time) {

+                  });
+                  newSchedule.Bank.push(new TimeInterval(
+                      strToIntDate(workingHours.from) + bankTimezone * MINUTES_IN_HOUR,
+                      strToIntDate(workingHours.to) + bankTimezone * MINUTES_IN_HOUR));


		function createGoodTimesIntArray() {

		var goodTimeToHack = true;

+                          var goodTimeToHack = true;
+                          var i = 0;
+                          function gangsterIsNotBusy(gangster) {

+                  var i = -1;
+                  function isGangsterNotBusy(gangster) {
+                      if (isTimeInIntervals(schedule[gangster], i)) {
+                          isGoodTimeToHack = false;

+                      }
+                  }
+                  while (i < DAYS_TO_HACK * MINUTES_IN_DAY) {

+                      if (!isTimeInIntervals(schedule.Bank, i)) {
+                          continue;
+                      }
+                      gangsterNames.forEach(isGangsterNotBusy);

+              function createScheduleWithTimeIntervals(bankWorkingHours, schedule, gangsterNames) {
+                  var newSchedule = {};
+                  var bankShift = parseInt(bankWorkingHours.from.split('+')[1], 10) * MINUTES_IN_HOUR;

+                  var gangsterNames = getGangsterNames(schedule);
+                  var newSchedule = createScheduleWithTimeIntervals(workingHours, schedule, gangsterNames);
+                  var goodTimesIntervals = getGoodTimeIntervals(newSchedule, gangsterNames);
+                  goodTimesIntervals = simplifyTimesIntervals(goodTimesIntervals, duration);

-                          return template;
+                          if (!this.exists()) {
+                              return '';

+              function getGangsterNames(schedule) {
+                  var gangsterNames = [];
+                  for (var gangsterName in schedule) {
+                      if (schedule.hasOwnProperty(gangsterName)) {

               exports.isStar = true;
-              /**
-               * @param {Object} schedule – Расписание Банды

+                  return false;
+              }
+              function strDateToInt(strDate) {

Левшин Михаил by LevshinMisha · Pull Request #18 · urfu-2016/javascript-task-3 · GitHub

Navigation Menu

Uh oh!

Левшин Михаил - #18

Левшин Михаил#18LevshinMisha wants to merge 8 commits intourfu-2016:masterurfu-2016/javascript-task-3:masterfrom LevshinMisha:masterCopy head branch name to clipboard

Conversation

LevshinMisha commented Oct 23, 2016

Uh oh!

honest-hrundel commented Oct 23, 2016

Uh oh!

honest-hrundel commented Oct 23, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chipolinka Oct 24, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chipolinka commented Oct 24, 2016

Uh oh!

chipolinka commented Oct 24, 2016

Uh oh!

honest-hrundel commented Oct 24, 2016

Uh oh!

honest-hrundel commented Oct 24, 2016

Uh oh!

honest-hrundel commented Oct 24, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chipolinka Oct 24, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

honest-hrundel commented Oct 27, 2016

Uh oh!

Левшин Михаил#18
LevshinMisha wants to merge 8 commits into
urfu-2016:masterfrom
LevshinMisha:master

chipolinka Oct 24, 2016 •
edited
Loading

chipolinka Oct 24, 2016 •
edited
Loading

chipolinka Oct 28, 2016 •
edited
Loading

+                  if (DAYS_INDEXES[strDateCopy.slice(0, 2)] === undefined) {
+                      strDateCopy = 'ПН ' + strDateCopy;
+                  }
+                  var o = {

+              function addOrChangeLastTimeInterval(arrayIntervals, timeToStartNewInterval) {
+                  if (arrayIntervals.length === 0 ||
+                      arrayIntervals[arrayIntervals.length - 1].end !== timeToStartNewInterval - 1) {

+                  var gangstersIsNotBusy = true;
+                  gangsterNames.forEach(function (gangster) {
+                      if (isTimeInIntervals(schedule[gangster], time)) {
+                          gangstersIsNotBusy = false;

               exports.isStar = true;
+              var DAYS_INDEXES = { 'ПН': 0, 'ВТ': 1, 'СР': 2, 'ЧТ': 3, 'ПТ': 4, 'СБ': 5, 'ВС': 6 };
+              var DAYS_NAMES = ['ПН', 'ВТ', 'СР', 'ЧТ', 'ПТ', 'СБ', 'ВС'];

+                  };
+              }
+              function isTimeInIntervals(timeIntervalsArray, time) {

+              }
+              function strDateToDateObj(strDate) {
+                  var strDateCopy = strDate.slice();

+                  this.start = start;
+                  this.end = end;
+                  this.getLength = function () {

+              }
+              function createTimeIntervalWithShift(time, shift) {
+                  return new TimeInterval(strDateToDateObj(time.from).intValue + shift,

+                      strDateToDateObj(time.to).intValue + shift);
+              }
+              function simplifyTimesIntervals(timeIntervals, duration) {