好吧......凌晨2点,这是我划线的地方。帮助......在我的笔记本电脑最终走出窗外之前。 :)
我尝试过使用setTimer,回调以及我能想到的其他一切(当然还有其他一些Stackoverflow提示)。我已经删除了所有内容,所以我只留下基本代码。
我要做的是调用parseRow()并在最后保存记录之前,我需要获取相关的类别(通过AJAX);然而,它正好经过它,所以类别总是“未定义”。
function parseRow(row){
var rowArray = row.trim().split(",");
var date = rowArray[0];
var checknum = rowArray[1];
var payee = rowArray[2];
var memo = rowArray[3];
var amount = rowArray[4];
//ERROR: blows right past this one and sets the category variable BEFORE ajax returns
var category = autoSelectCategory(payee);
saveRecord(date, checkNum, payee, memo, category, payment, deposit);
}
function autoSelectCategory(payee) {
var data;
$.ajax({
async: false,
url: "autoselectcategory",
dataType: "json",
data: {
string: payee
},
success: function (returnedData) {
data = returnedData;
}
});
return data;
}
答案 0 :(得分:15)
AJAX代表异步。这意味着在原始代码中,saveRecord
将在之前执行,客户端将从服务器接收响应(并且,根据$.ajax
实现,它可能在之前客户端将请求发送到服务器。)
此外,您似乎误解了JS中函数的工作原理。 var category = autoSelectCategory(payee);
会将类别设置为autoSelectCategory
的返回值;但是代码中的autoSelectCategory
函数不会返回任何内容。
另一方面,匿名函数的data
返回值只能由$.ajax
函数使用(而$.ajax
可能会忽略success
参数返回值)
以下是应该有效的代码:
function parseRow(row){
var rowArray = row.trim().split(",");
var date = rowArray[0];
var checknum = rowArray[1];
var payee = rowArray[2];
var memo = rowArray[3];
var amount = rowArray[4];
autoSelectCategory(payee, function (category) {
saveRecord(date, checkNum, payee, memo, category, payment, deposit);
});
}
function autoSelectCategory(payee, callback) {
$.ajax({
async: false,
url: "autoselectcategory",
dataType: "json",
data: {
string: payee
},
success: callback
});
}
答案 1 :(得分:10)
请勿使用async: false
选项。这是一个纯粹的邪恶(在浏览器甚至其他标签中阻止所有脚本!)并且自jQuery 1.8以来它已被弃用。你应该像往常一样使用回调。
function parseRow(row) {
/* the other code */
autoSelectCategory(payee, function() {
saveRecord(date, checkNum, payee, memo, category, payment, deposit);
});
}
function autoSelectCategory(payee, callback) { // <---- note the additional arg
$.ajax({
url: "autoselectcategory",
dataType: "json",
data: {
string: payee
},
success: function(res) {
/* the other code */
callback();
}
});
}