Pull to refresh

Comments 7

а где ссылка на «Красивое приложение для просмотра погоды с Angular 8»?

Но я бы не стал рекомендовать это приложение новичкам, которые могут решить, что так и надо писать на Angular.
А не-новички там тоже вряд ли что-то ценное найдут.

Можете сказать, почему? Из всех проектов этот пока заинтересовал больше всего, + был небольшой опыт на angular

Там в комментах все расписано, в общем-то.


Вкратце — автор совсем не разобрался с rxjs и асинхронным кодом в angular. WeatherService чудовищен — создаются Subject зачем-то, и вручную туда закидываются данные изнутри .subscribe. А еще один и тот же http запрос делается несколько раз, просто чтобы разные слайсы данных вернуть в компонент.
DetailsComponent — накопипастили свойства для 5 дней вместо массива. делать subscribe там тоже излишне — AsyncPipe более идиоматичен и лаконичен (и не нужно будет сохранять ссылки на подписки и вручную отписываться в onDestroy). А еще и subscribe() внутри другого subscribe() — что в итоговой реализации может привести к race condition и показу данных от другого города.


Если совсем придираться, то Injectable-сервисы рекомендуется определять с providedIn: root

Боже. До какого маразма довели JS. Все эти ангулары и прочие мегафреймворки. Просто уму не растяжимо.

Sign up to leave a comment.