Как можно организовать работу с асинхронным кодом?

Question

[bio]

Добрый день!
Есть код

var paramPamPam = {

    data : null,

    get : function () {
        var _this = this;

        return $.post('/path/to/file', function(data, textStatus, xhr) {
            _this.data = data;
        }); 
    },


    _enable: function () {
        console.log('enable');
    },
    enable: function () {

        var _this = this;
        
        if ( this.data ) {
            _this._enable();
        } else {
            this.get().done(function () {
                _this._enable();
            });
        };

    },


    _disable: function () {
        console.log('disable'); 
    },
    disable: function () {
        var _this = this;
        
        if ( this.data ) {
            _this._disable();
        } else {
            this.get().done(function () {
                _this._disable();
            });
        };
    }

}

paramPamPam.enable();

Как его можно упросить ? Писать в каждом методе на подобии enable и disable, не радует.

Answer 1

[Евгений Петров]

Что-то вы напридумывали, причём даже не избыточное, а ещё и неработающее.
Для начала вам нужно разобраться с Promise, о чём уже сказал Алексей Уколов, а потом почитать про Deferred Object, раз уж используете jQuery. Для справки — этот объект создаётся при вызове ajax() автоматически.

Вы используете в Deferred Object один и тот же метод done(), который вызывается только в случае успешного ответа. Вам следует открыть для себя методы fail(), который вызывается в случае провала, и always(), который вызывается всегда, независимо от результата.

$.ajax({
	url: '...'
}).done(function (data) {
	console.log('Успех');
}).fail(function (data) {
	console.log('Провал');
}).always(function (data) {
	console.log('А мне пофигу');
});

Update: Не ручаюсь за безукоризненность кода. Возможно, использование двух Deferred не есть гуд.
На скорую руку, примерно вот так (при нескольких обращениях подряд к методу запрос будет всё равно один):

var paramPamPam = {
	data: null,
	defer: $.Deferred(),

	get: function () {
		if (this.data) {
			this.defer.resolve(this.data);
		} else {
			if (!this.transport) {
				this.transport = $.ajax({
					url: '...',
					type: 'post'
				});
			}

			this.transport
				.done(this.enable.bind(this))
				.fail(this.disable.bind(this))
				.always(function () {
					delete this.transport;
				}.bind(this));
		};

		return this.defer.promise();
	},

	enable: function (data) {
		this.data = data;
		this.defer.resolve(data);
	},

	disable: function () {
		this.defer.resolve();
	}
}

paramPamPam.get().then(function (data) {
	console.log(data);
});

paramPamPam.get().then(function (data) {
	console.log(data);
});

Comments

[bio]

Вы не поняли, это всё я знаю.
Есть данные которые я получаю с сервера через api.
Есть методы которые используют эти данные.

Проблема в том, что в каждом методе нужно проверять, получал ли я эти данные, если не получал, то отправлять запрос. Вот это мне и не нравится, хочется найти более лучшее решение.

[Евгений Петров]

bio: Обновил ответ.. Можете после первого запроса выставлять флаг и не делать последующие, если после первого не вернулось ничего.

[bio]

Евгений Петров: Тоже пришёл к похожему решению, но к сожалению, при увеличении кол-во методов, которые работают с этими данными, всё превращается в кашу.

[Евгений Петров]

bio: В какую кашу? Я организовал 10 обращений — всё норм. Каждое обращение получает данные, как только они получены с сервера. Запрос выполняется один раз.

[bio]

Евгений Петров:
Ну вот смотрите, например есть много других методов которые работают с данными полученным с сервера. Каждый раз вызывать их через
paramPamPam.get().then(obj.method);
меня не много напрягает, некрасивый это :)
Хочется избавится от однотипных вызовов.
Может кончено я хочу слишком много.

[Евгений Петров]

bio: Вам нужна шашечки или ехать? Не хотите писать then(...) — передавайте callback.

[bio]

Евгений Петров: Вот и хочется того и того, но по всей видимости так не получится. :)
Спасибо!