skinnynpale/rangeSlider.js

Переназначение переменных

performanceArtist opened this issue · 1 comments

    let step = this.getAvailableValue(state, "step");
    const max = this.getAvailableValue(state, "max") as number;
    const min = this.getAvailableValue(state, "min") as number;

    if (step > max) {
      step = max;
    }

    const diff = max - min;
    if (step > diff) {
      step = diff;
    }

Переназначений лучше избегать. Что если у тебя два условия пройдут проверку? Лучше чётко обозначить условия и при выполнении сразу возращать значение через return.

    if (min >= max) {
      const temp = min;
      min = max;
      max = temp;
    }

Тут можно сразу объект вернуть с перевёрнутыми значениям.