0

Есть такая функция, которая принимает в себя две даты, заводит пустой счётчик и увеличивает его на один если в массиве дат производственного календаря есть такая дата.

function countNonWorkingDays(fromDate, toDate) {
  let counter = 0;
  const formatProductionCalendar = productionCalendar.map(element => {
    const d = new Date(element);
    return `${('0' + d.getDate()).slice(-2)}-${('0' + (d.getMonth() + 1)).slice(
      -2,
    )}-${d.getFullYear()}`;
  });
  let currentDate = moment(fromDate, 'DD-MM-YYYY');
  const finishDate = moment(toDate, 'DD-MM-YYYY');
  while (currentDate <= finishDate) {
    if (formatProductionCalendar.includes(currentDate.format('DD-MM-YYYY'))) {
      counter++;
    }
    currentDate = moment(currentDate).add(1, 'days');
  }
  return counter;
}

Массив дат нерабочих дней в производственном календаре (в самой функции я форматирую productionCalendar под формат 'DD-MM-YYYY' и в итоге получается правильный массив formatProductionCalenar):

[
  '2022-01-01',
  '2022-01-02',
  '2022-01-03',
  '2022-01-04',
  '2022-01-05',
  '2022-01-06',
  '2022-01-07',
  '2022-01-08',
  '2022-01-09',
  '2022-01-15',
  '2022-01-16',
  '2022-01-22',
  '2022-01-23',
  '2022-01-29',
  '2022-01-30',
]

При запуске подобного теста я ожидаю получить 15 нерабочих дней

const fromDate = new Date(2022, 1, 1);
const toDate = new Date(2022, 2, 1);

describe('countNonWorkingDays working correct', () => {
  it('countNonWorkingDays for January', () => {
    expect(countNonWorkingDays(fromDate, toDate)).to.equal(15);
  });
});

Но вместо этого счётчик возвращает 10. Что я делаю неправильно и есть ли варианты упростить такую функцию? А также рассмотреть вариант при котором fromDate > toDate и получать корректную ошибку

1
  • new Date(2022, 0, 1)
    – Konst
    28 дек 2021 в 14:35

1 ответ 1

0

В комментарии @Konst правильно отметил мою ошибку в самом тесте.

const fromDate = new Date(2022, 1, 1);
const toDate = new Date(2022, 2, 1);

describe('countNonWorkingDays working correct', () => {
  it('countNonWorkingDays for January', () => {
    expect(countNonWorkingDays(fromDate, toDate)).to.equal(15);
  });
});

Правильно будет вот так:

const fromDate = new Date(2022, 0, 1);
const toDate = new Date(2022, 1, 1);

describe('countNonWorkingDays working correct', () => {
  it('countNonWorkingDays for January', () => {
    expect(countNonWorkingDays(fromDate, toDate)).to.equal(15);
  });
});

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.