Javascript var没有变化

时间:2015-03-13 16:51:10

标签: javascript jquery scope

我想修改这个在提交前进行一些检查的vtiger函数。

我添加了globalvar我想用来检查表单是否可以提交。我可以在这里更改什么来获得所需的结果?

globalvar在函数结束时应该是2或3,但是它保持为1,我还添加了警报,其中设置它以确保它到达那些点。

registerRecordPreSaveEvent : function(form) {

    var thisInstance = this;
    if(typeof form == 'undefined') {
        form = this.getForm();
    }

    form.on(Vtiger_Edit_Js.recordPreSave, function(e, data) {
        var accountName = thisInstance.getAccountName(form);
        var recordId = thisInstance.getRecordId(form);
        globalvar = 1;

        var params = {};
        if(!(accountName in thisInstance.duplicateCheckCache)) {
            Vtiger_Helper_Js.checkDuplicateName({
                'accountName' : accountName, 
                'recordId' : recordId,
                'moduleName' : 'Accounts'
            }).then(
                function(data){
                   thisInstance.duplicateCheckCache[accountName+'_accounts'] = data['success'];
                   globalvar =2;

                },
                function(data, err){

                    thisInstance.duplicateCheckCache[accountName+'_accounts'] = data['success'];
                    thisInstance.duplicateCheckCache['message'] = data['message'];
                    var message = 'Company already exists';
                    Vtiger_Helper_Js.showMessage({'text' : message,'type': 'error'})
                }
            );
        }

        else {
            if(thisInstance.duplicateCheckCache[accountName+'_accounts'] == true){
                var message = 'Company already exists';
                    Vtiger_Helper_Js.showMessage({'text' : message,'type': 'error'})
            } else {
                delete thisInstance.duplicateCheckCache[accountName+'_accounts'];
                return true;
            }
        }
         if(!(accountName in thisInstance.duplicateCheckCache)) {
            Vtiger_Helper_Js.checkDuplicateName({
                'accountName' : accountName, 
                'recordId' : recordId,
                'moduleName' : 'Leads'
            }).then(
                function(data){
                    globalvar =3;
                    console.log(globalvar);
                    thisInstance.duplicateCheckCache[accountName+'_leads'] = data['success'];

                },
                function(data, err){

                    thisInstance.duplicateCheckCache[accountName+'_leads'] = data['success'];
                    thisInstance.duplicateCheckCache['message'] = data['message'];
                    var message = 'Lead already exists';
                    Vtiger_Helper_Js.showMessage({'text' : message,'type': 'error'})
                }
            );
        }

        else {
            if(thisInstance.duplicateCheckCache[accountName+'_leads'] == true){
                var message = 'Lead already exists';
                    Vtiger_Helper_Js.showMessage({'text' : message,'type': 'error'})
            } else {
                delete thisInstance.duplicateCheckCache[accountName+'_leads'];
                return true;
            }
        }
    console.log(globalvar);
        e.preventDefault();
    })
}

1 个答案:

答案 0 :(得分:1)

此代码中包含许多异步操作。在包含函数实际返回后,异步操作稍后完成。因此,在包含函数完成后,您不能立即使用异步操作的结果,因为异步操作本身仍然存在且尚未完成。

因此,在async操作(您正在执行的操作)中设置globalVar将不会在包含的函数完成时显示它的值。

使用异步操作结果的唯一方法是在回调中使用它来表示异步操作已完成,或者从该回调中调用函数并将所需数据传递给它。这些是你的选择。你根本无法按照你尝试的方式去做。

与此处的所有问题一样,如果您描述了您尝试解决的实际问题而不是解释您尝试的解决方案的问题,我们可能会在更高的层次上提供帮助。

在StackOverflow上,未能描述您真正想要完成的内容被称为XY problem


您无法在异步响应中运行e.preventDefault(),因为在异步响应完成时,表单已经提交。

您可能需要翻转逻辑,以便在所有异步操作完成且没有错误之前不提交表单。然后,您将手动提交表单。