Как вернуть значение из одного метода и получить в другом?

Question

[Unknown]

Привет!
Только что понял истину, что задавая переменные глобально я расходую память понапрасну. Поэтому ищу способ не задавая переменную глобально получать ее значение в любом из методов.
Таким образом:
Как мне получить значение из метода checkMinMax() в методе iterator() не задавая переменные глобально?
Код на codepan
Всем спасибо!

Comments

[Егор Живагин]

Если интересно, маленький фидбек по коду ) Мне кажется, вам может быть полезно.

Если вы вызываете метод только для проверки условия - вы можете спокойно его вызвать прям в самом условии, как тут кто-то отвечал уже. Это нормальная практика.

Вы можете смело убрать === true из условия, не изменится вообще ничего.

Очень странно видеть переменную sliderInit внутри класса вместо this
Вы привязываете класс к определённому экземпляру класса... зачем? По идее, классы должны быть изолированы от остального кода. А так вы переименуете переменную - придётся лезть в код класса, захотите ещё слайдер - придётся лезть в код класса, ...

А разве функцию checkMinMax нельзя привести к одной строчке, вместо цикла и сравнений:
return [this._slider.length - 1, 0];
Номер последнего слайда всегда будет равен slider.length - 1 , номер первого слайда всегда будет 0

Опять же, ради масштабируемости есть смысл передавать переменную aria при вызове слайдера

[Unknown]

Егор Живагин, вы имели ввиду заменить sliderInit.iterator() вот здесь?

class Slider {
    constructor(slider, startIndex = 0) {
        this._slider = slider;
        this._currentSlideIndex = startIndex;
        this._events = area.addEventListener('wheel', function(event) {
            if (event.deltaY > 0) {

                sliderInit.iterator();    // поменять на this.iterator()?

            } else if (event.deltaY < 0) {

            }
        }, false);
    }

А вот это return [this._slider.length - 1, 0]; супер альтернатива

спасибо!

[Егор Живагин]

Richard, да, именно здесь

[Unknown]

Егор Живагин, >>> посмотрите куда указывает this в моем случае.

[Егор Живагин]

Richard, точно, внутри функции же. Это обрабатывается так:

constructor(slider, startIndex = 0) {
let self = this;
// остальной ваш код

Соответственно, внутри функции вызываете self.iterator()

[Unknown]

Егор Живагин, да, все так) Забыл совсем про практику с self, видел же что так делают)
Огромное спасибо!)

Answer 1

[Stalker_RED]

class foo {
  method1() {
     return 'hello' // return вы и сами написали
  }
  method2() {
     let x = this.method1() // а вот результат никуда не присвоили
     console.log(x)
  }
}

Comments

[Иван Корюков]

На самом деле присвоили, но не глобальной переменной, которая останется висеть в памяти, а локальной, которая будет удалена при выходе из метода (если не замыкание, но это уже совсем другая история...)

[Unknown]

Сделал так

checkMinMax() {
        const arr = [];
        let allCountMax, allCountMin;
        for (let i = 0; i < this._slider.length; i++) {
            arr.push(i);
            allCountMax = Math.max.apply(null, arr);
            allCountMin = Math.min.apply(null, arr);
        }
        return [allCountMax, allCountMin];
    }
    iterator() {
        let allCountMax = this.checkMinMax();      // Вернуть можно только значение, а я думал можно переменные возвращать, но на самом деле они удаляются так как заданы локально
        let allCountMin = this.checkMinMax();
        console.log(allCountMax[0]);                 // Получаю 3
        console.log(allCountMin[1]);                  // Получаю 0
        
        if (sliderInit.checkDelta() === true) {
            if (!allCountMax) {
                console.log('asdasd');
                curItemIndex = this._currentSlideIndex++;
            }
        } else {
            curItemIndex = this._currentSlideIndex--;
        }
        
    }

[Stalker_RED]

Richard, не нужно два раза вызывать, и в первый раз возвращаются обе переменные.

class Foo {
  method1() {
     return [1, 3]
  }
  method2() {
     let [x, y] = this.method1()
     console.log(x)  // -> 1
     console.log(y)  // -> 3
  }
}

[Unknown]

Stalker_RED, я понимаю что возвращаются, просто не знал как по другому написать, напишу также как вы)
Спасибо!

[Unknown]

Stalker_RED, хмм, тогда перестает выполняться условие (!allCountMax). Интересно почему в консоль я могу вывести значение, а обычным способом получить не могу?

iterator() {
        let [allCountMax, allCountMin] = this.checkMinMax();
        if (sliderInit.checkDelta() === true) {
            console.log(allCountMax); // 3
            if (!allCountMax) {     // Ни ответа, ни привета, ни ошибки
                console.log('asdasd');
                curItemIndex = this._currentSlideIndex++;
            }
        } else {
            curItemIndex = this._currentSlideIndex--;
        }
    }

[Stalker_RED]

Richard, дык !3 === false
А что вы ожидаете от if (false) ... ?

[Unknown]

Stalker_RED, :D
точно