Michael3468 / RangeSlider

Range slider jquery plugin
https://whitedevilman.github.io/RangeSlider/
1 stars 0 forks source link

Валидация настроек модели #5

Closed romansilivestrov closed 2 years ago

romansilivestrov commented 2 years ago

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

Когда объект, скопирован по ссылке, при мутации будут изменяться оба объекта: image Чтобы создать новый объект проще всего использовать spread operator: const obj2 = {...obj1};

Настройки дублируются в this.settings и полях класса. Следует оставить их в одном месте.

Данный метод можно заменить обычным присвоением: image