Ответ 1
Я чувствую, что может быть немного путаницы в использовании отложенных, promises и труб здесь.
Я думаю, что вы хотите что-то вроде этого (http://jsfiddle.net/JohnMunsch/ghdgD/):
var testObject = {
loadOpportunities: function () {
// This is basically saying that we're getting something via AJAX, but we want to take a swipe at changing it
// before anyone else gets it, so by using pipe we're returning a different promise than the AJAX promise and
// the value returned from that is what we want others to get.
return $.ajax({
url: "/echo/json/",
data: opportunities,
type: "POST"
}).pipe(
function (result) {
console.log("[1]", result);
// Change the result in some way and then return the changed result. Otherwise there no point in using
// a pipe() here, a done() would work as well.
result.opportunities[0].name = "meat packer";
return result;
}
);
},
loadTypes: function () {
// This example is different from loadOpportunities in that it isn't using a pipe() so what is returned from
// this function is the same promise that $.ajax() gave us. Any call to .done() just echos back the same
// promise you gave it.
return $.ajax({
url : "/echo/json/",
data : opportunityTypes,
type : "POST"
}).done(
function (result) {
console.log("[1]", result);
// We can do something with the data received here, but outside of this function, anything that tied to
// the promise from the $.ajax() will see the original data, not any changes we might make here.
}
);
},
loadView: function () {
// The self = this, scope = this, etc. that you see at the start of functions is only necessary if there are
// closures below where this might be something else. This code doesn't have that so this line is unneeded.
//
// Be careful about cargo-cult programming and just doing something because you do it everywhere else or you
// see someone else do something. Try to understand why it there.
// var self = this;
if (!this.viewLoaded) {
console.log("loading");
var viewPromise = $.ajax({
url : "/echo/json/",
data : view,
type : "POST"
}).done(
function (result) {
console.log("[1]", result);
// We might take this and dump it in a <div> somewhere.
}
);
var typesPromise = this.loadTypes();
var opportunitiesPromise = this.loadOpportunities();
// The following line hands back a single promise (that what $.when() does) that wraps all three of the
// other promises and only fires when all three have completed OR when any one of them has failed.
return $.when(typesPromise, opportunitiesPromise, viewPromise).done(
function () {
// Here I've tied a function to the promise as soon as I've created it. But since I'm handing back the
// promise, I can also tie functions to it later as well, even after the promise has resolved or been
// rejected.
console.log("[2]", "All done loading types, views, and opportunities");
}
);
}
return true;
}
};
// Note: Unlike with all the steps labeled [1], I don't believe there is a formal ordering for the
// .done() attached to the same promise like both [2] and [3]. So it may be possible for those two steps to
// happen [2]/[3] or [3]/[2] although they always happened [2]/[3] in my testing.
$.when(testObject.loadView()).done(
function () {
console.log("[3]", "This will only trigger after all the steps in load view have completed.");
}
);
Примечание. Я изменил стиль ваших вызовов AJAX, но это было просто потому, что мне нужно было создать рабочий пример в jsFiddle. Нажмите на ссылку выше, чтобы увидеть это в действии. Он работает, и он заказывает вещи, как вы ожидаете. Вам, вероятно, не нужна труба столько, сколько вы думаете, если вы не планируете манипулировать результатами AJAX, прежде чем кто-либо, связанный с обещанием, увидит результаты. В противном случае .done(),.fail() и .when() будут обрабатывать большинство ситуаций.