Публичный менторинг. Игорь Шульга. Модуль 3.1. Код ревью № 2.

Рет қаралды 2,004

Күн бұрын

Пікірлер: 12

@maksimus.ssirotkin1124 Жыл бұрын

Добрый вечер. Огромное спасибо за видео, есть несколько небольших вопросов, первый - Какой смысл каждые 5 минут вычитывать список компаний, Они что IPO, каждые 5 минут проводят? (Может данную таску достаточно раз в день или кэширование). - и второй вопрос, ссылки в очень и если я правильно из базы, происходит их вычитывание, но так ссылка одинаковая, меняется только символ компании, не лучше ли избежать базы и очереди в этом. случае?

@EugeneSuleimanov Жыл бұрын

Спасибо за вопросы! 1. Проект учебный, поэтому бизнес-логика может быть не самой прямолинейной. 2. Очередь можно убрать, а вот данные в БД нужны, конечно же.

@Марк-м3х5п Жыл бұрын

Добрый день! Я решил попробовать сделать проект вместе с Вами , пишу его параллельно и после проверяю как получилось у Игоря. Но у меня почему-то перестал работать API c которого мы получаем response. Я к ним даже не могу попасть на сайт , не говоря о получении ответа. Подскажите в чем может быть проблема ? Пробовал и через VPN , результат прежний. Буду рад ответу!

@EugeneSuleimanov Жыл бұрын

Спасибо за вопрос! Вероятнее всего, истёк срок действия токена или завершился триальный период. Вы можете проверить на сайте.

@Марк-м3х5п Жыл бұрын

Сам сайт не работает на который ссылка в документации и с ним же API для запросов, даже в браузере не возможно открыть его уже 2-й день@@EugeneSuleimanov

@EugeneSuleimanov Жыл бұрын

@@Марк-м3х5п проверил - все отработало. Возможно, стоит попробовать под другим VPN.

@Марк-м3х5п Жыл бұрын

Да , действительно! VPN другой страны отработал , спасибо@@EugeneSuleimanov

@gennadiypetrov3799 Жыл бұрын

Я, честно говоря, ждал большего от менторинга. Думал по сути будут комментарии, а не: тут лишняя инициализация, тут private final, добавь логи, тут NPE проверь... После такого ревью суть кода вообще не поменяется... Сори за хэйт. Люблю твой канал, просто ожидал большего.