ArbakovMaxim
commented
1 year ago Недавно, от вашей компаний, опять появилась вакансия на front-end разработчика. Могу ли я повторно, попробовать пройти собеседование, учитывае, что тестовое, я уже делал? За время что прошло, я сделал проект, выучил новые технологий. в прошлый раз, в самом конце, выбор, пал не на меня))
Решение с фильтрацией массива переменных по вхождению
ID.includes(v.ID), гдеtypeof ID == 'string', не слишком удачное, потому что обнаруживает совпадения с лишними переменными: например, в "125" входят переменные 1, 2, 5, 12, 25, и собственно 125, что мы и видим на странице https://arbakovmaxim.github.io/APB-Test/variables/125.https://github.com/ArbakovMaxim/APB-Test/blob/5225ae5d87a216684a48f67a680174ed99b79fcb/src/pages/VariablesId/VariablesId.jsx#L21-L47
Правильнее было бы привести параметр ID из адресной строки к числу и сравнить
+ID === v.ID.Кроме того, могут быть ситуации когда параметр ID не совпадает ни с одной из переменных — это нормально, потому что пользователь имеет доступ к адресной строке и ничто не мешает ему зайти на адрес https://arbakovmaxim.github.io/APB-Test/variables/0, например. Хорошо что в нашем случае это не приводит к ошибке (пустой массив просто не рендерится), но ещё лучше чтобы такая ситуация была обработана либо перенаправлением на 404, либо сообщением "Такой переменной не существует".
P.S. Просим прощения за очень долгий ответ по вашей кандидатуре, мы столкнулись с потоком откликов, который многократно превышал наши ожидания. Последние собеседования проходят на этой неделе, после чего мы обязательно оповещаем всех кандидатов о результатах. Спасибо за работу и ожидание!