asd95
commented
3 years ago Добрый вечер Андрей в тикете Logic была указана строка "Рефакторинга как такого не было." Я не делал как такого кода ревью. Мне было важно чтобы все работало и отправить вам чтобы: 1) дать понять что для меня это важно и работа над тестом шла. 2) важен и feedback который сейчас получаю, которому я очень благодарен. Что касается того что вы мне скинули согласен. Можно было в функцию getArrayDates передать в качестве аргументов StardDate и endDate. Открою ветку для рефакторинга и учту ваши моменты, а так же и пройдусь сам. Спасибо
Добрый день, меня зовут Андрей. Мы совместно с @viktorovich87 делаем ревью Вашего тестового задания. По вопросам которые возникают, открываем тикеты. По ним сделаете соответствующие правки в коде, если считаете нужным, либо оставьте ответ в комментарии.
При передачи параметров в функцию
getArrayDatesпараметры оборачиваются в array. В функцию передается array и из него берется по индексам оба параметра:Подскажите, пожалуйста, в связи с чем был выбран данный подход ? Почему не передать даты отдельными параметрами ?
При таком подходе использования массивом, очень тяжело читается код. В целом это не совсем правильно