Переназначение переменных
performanceArtist opened this issue · 1 comments
performanceArtist commented
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.
performanceArtist commented
if (min >= max) {
const temp = min;
min = max;
max = temp;
}
Тут можно сразу объект вернуть с перевёрнутыми значениям.