Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Махалин Николай #6

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

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.

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

* @returns {Object}
*/
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