JavaScript / TypeScript

[romansilivestrov]

async и await используются для работы с промисами. В слайдере, где весь код синхронный они не к чему.

Тип void используется только для функций, которые не возвращают значение:

[romansilivestrov]

Много где встечаются type assertions, что нежелательно.

в подобных местах их можно избежать, добавив проверку или сделав операцию опциональной.

[romansilivestrov]

Насколько я понял, this.rangeSliderName - это константа, которая присваивается единожды. Соответственно, её лучше импортировать из файла констант, вместо того, чтобы передавать как параметр в компоненты.

[osyka-yuri]

Очень часто используется type assertion при поиске по dom дереву, неспроста же ведь там тип Element | null, так как элемент может быть как найден так и нет, и typescript соответственно подталкивает нас чтобы мы делали проверку на то что элемент был действительно найден это сделает код более надежным и менее подверженным ошибкам.
То же относится и к событиям, везде где используется event.target используется type assertion, чтобы действительно убиться что там находится нужный нам тип элемента можно использовать event.target instanceof HTMLDivElement.

---

MetaLamp_four_task/src/plugins/rangeSliderFox/mvc/model/Model.ts

Line 423 in 1faf42a

min = Number(checkProperty(this, min, 'min' as keyof Model));

Здесь, по-моему, вообще бессмысленный type assertion