Веб-версияОткрыть в Telegram

ВеткаВернулись ко мне с фидбеком из Яндекс, кому интересно (я не прошёл).

15 сообщений · –
M
Вернулись ко мне с фидбеком из Яндекс, кому интересно (я не прошёл). Я конечно лажал, не спорю и в принципе с выводом согласен. Времени было мало и поэтому часть вещей я переписывал на ходу, когда понял, что не учёл один момент из задачи и вышло, что вышло. Но вот, что меня удивило. 1. Надо буквально все мысли в слух произносить, я начинал использовать stream api, но понял, что лаконичней и проще (в тот момент) перейти на циклы. Он написал, что я не помню, как использовать стримы. 2. Они говорили, что можно спокойно использовать гугл, я постоянно путаю computeIfAbsent и putIfAbsent, поэтому прогуглил, в фидбек это тоже попало. 3. Я говорил, что не привык писать тесты без спринга, через время я сказал, что мне так же непривычно без Мокито и Ассертионс, я не имел в виду, что это не работает без спринга, просто подметил. Он написал, что я не знаю, к чему это относится. Хотя это и часть spring-test я понимаю, что это можно вытащить через JUnit. 4. Использовал Long, как идентификатор. Ну я мог использовать UUID, просто, как по мне это скорее докопаться. К тому же я не разрабатывал БД и данные должны были приходить от других компонентов. 5. Пытайтесь всё засунуть в кеш, мне показалось это излишнем в рамках моего кода, для некоторых пунктов, поэтому я использовал его там, где мне казалось логичным его использование, на деле это попало в фидбек. В общем главная мысль. Произносите каждую свою мысль в слух, у них очень детальный фидбек и многие проблемы возникли из-за недопонимания
  1. J
    Надеюсь в следующий раз повезет, у них есть бан на 6 месяцев в отдел куда подавал или нет? (Ни разу не подавал резюме в современный Яндекс)
  2. А
    Может, имелось в виду, что long надо было? Он же not null по своей природе
    1. M
      У меня лонг как раз и был, тут скорее имелось использование UUID
    2. M
      А ты примитив имеешь в виду, не. Там было написано, что возможна атака переборами при использовании лонги
          1. K
            В задачке есть условие про повторный запрос с тем же id. Наверно это имелось ввиду
            1. А
              Ну это бизнес-кейс, он должен быть просто корректно обработан, как и повторный запрос с тем же UUID
            2. M
              я создал Set и прописал equals&hashcode и использовал это в виде кеша для запросов батчей
      1. А
        Ну, скорее, тут не атака перебором, а возможность многопоточной записи кучи энтитей. Лонга должна быть засинкана чтобы коллизию не получить, а UUID’у пофиг. Там вероятность коллизии настолько мала, что можно на нее забить и генерить на ходу
  3. А
    Попало бы даже если бы ты произносил. У меня было, давно правда, что я сказал «вот тут в будущем могут быть такие-то проблемы, могу расписать валидацию, могу просто коммент сделать, что валидируем то и это», сошлись на комменте. В итоге в ревью мне тоже написали что не смог в валидацию

Открытая публичная лента из поискового индекса ChatCrawler — «Google по публичному Telegram»; обновляется по мере обхода площадки. Время — UTC.

Только публичный контент, официальный API Telegram. О проекте · Вопросы · Чего мы не делаем · Убрать страницу из выдачи · Каталог · Поиск · Как мы считаем