Question

我有以下代码，它有一个问题，在第一次加载时它不显示任何东西，但是当我在我的旋转木马上按next按钮时，一切都会完美运行，所以问题就在第一次调用的某个地方，因为其他电话完全相同但是正在工作：S

我不会把显示代码放在哪里，因为它只是jquery将数据添加到div和东西，但正如我所说，第一次，数据是空的。

var dificildecision = function() {
}

dificildecision.prototype = {
    is_animating : false,
    base_url : window.base_url,
    current_decision : null,
    decisiones_seen : 0,
    historial_decisiones : {
        "decisiones" : []
    },
    is_previous : false,
    previous_id : 1,

    next_decision : function(replace) {
        if (this.is_animating != true) {
            this.is_animating = true;
            if (this.is_previous) {
                decision = this.get_decision(this.previous_id);
            } else {
                if (this.get_current_decision() === false)
                    decision_id = 1;
                else
                    decision_id = this.current_decision.id;
                this.get_new_decision(decision_id);
                decision = this.get_current_decision();
                $('#decision-carousel').html("This is: " + decision.key);
                this.is_animating = false;
                this.is_previous = false;
            }
        } else {
            // console.log("Slow down");
        }
    },

    get_new_decision : function(decision_id) {
        if (typeof (decision_id) === "undefined" || decision_id === null) {
            decision_id = 1;
        }
        $.getJSON('http://echo.jsontest.com/key/value', this.set_current_decision);
    },
    get_current_decision : function() {
        return (this.current_decision) ? this.current_decision : false;
    },
    set_current_decision : function(decision) {

        // THIS DOESN'T WORK
        this.current_decision = decision;

        // THIS WORK SECOND+ TIME EXECUTED
        //dificildecision.prototype.current_decision = decision;
    }
};

    var dificil = new dificildecision();
    dificil.next_decision(true);
$('#testlink').on('click', function(){
    dificil.next_decision(true);
});

使用此代码，控制台上不显示任何内容，但如果我更改

set_current_decision : function(decision) { this.current_decision = decision; }

到

set_current_decision : function(decision) { dificildecision.prototype.current_decision = decision; }

仅在处理轮播功能时才输出对象，但在页面加载时不输出...

我也试图改变

get_new_decision : function(decision_id) { if (typeof (decision_id) === "undefined" || decision_id === null) { decision_id = 1; } $.getJSON('/~robhunter/dificildecision/web/app_dev.php/new/' + decision_id, this.set_current_decision); },

到

get_new_decision : function(decision_id) { if (typeof (decision_id) === "undefined" || decision_id === null) { decision_id = 1; } $.getJSON('/~robhunter/dificildecision/web/app_dev.php/new/' + decision_id, dificildecision.prototype.set_current_decision); },

但与原始代码完全一样，没有任何内容显示出来：S

你可以在这里试试http://jsfiddle.net/TUX5a/

Answer 1

尝试：

$(document).ready(function() {

  //  window.dificildecision = new dificildecision();
      var dificildecision = new dificildecision();
}

在this

中使用dificildecision.prototype.next_decision

<强>解释

当您在全局范围内使用以下行声明函数：function dificildecision()时，将会有一个名为dificildecision的新属性分配给您的窗口对象。当您致电window.dificildecision = new dificildecision();时，此属性（对函数的引用） 将被替换为 并带有实例

=＆GT;这会导致应用程序出现不可预测的行为，应该避免使用。

OP创建小提琴后更新：

我检查了你的小提琴，你有ajax异步性质的问题。当您调用decision = this.get_new_decision();时，您向服务器发出ajax请求以获得决定。在您调用decision = this.get_current_decision();的下一行，该决定尚未设置（回调set_current_decision尚未运行）。

这个问题有两个解决方案：

将$.ajax函数与async: false一起使用，以使用同步ajax。但是，建议不要使用此解决方案，因为它会阻止浏览器并降低用户体验。
使用回调或承诺API（推荐）。以下是演示如何执行此操作：

DEMO

从getJSON返回一个承诺

get_new_decision : function(decision_id) {
    if (typeof (decision_id) === "undefined" || decision_id === null) {
                decision_id = 1;
    }
  return $.getJSON('http://echo.jsontest.com/key/value', this.set_current_decision); 
}

然后在next_decision内再次返回：

next_decision : function(replace) {
        if (this.is_animating != true) {
            this.is_animating = true;
            if (this.is_previous) {
                decision = this.get_decision(this.previous_id);
            } else {
                if (this.get_current_decision() === false)
                    decision_id = 1;
                else
                    decision_id = this.current_decision.id;
                decision = this.get_new_decision(decision_id);

                this.is_animating = false;
                this.is_previous = false;
            }
        } else {
            // console.log("Slow down");
        }
        return decision;//this could be a promise or a value.
    },

使用$.when在数据就绪时执行回调，如果参数不是promise，则立即调用回调（将其视为已解决的承诺）：

$.when(dificil.next_decision(true)).then(function(decision){
        $('#decision-carousel').html("This is: " + decision.key);
});

此代码只是如何使用promise API的演示，您可能需要重新构建代码以使其适合此编程模型。

Answer 2

实际上，我发现我需要更改get_new_decision方法以使用$.ajax代替$.getJSON并设置async: false以此方式存储变量

get_new_decision : function(decision_id) {
        if (typeof (decision_id) === "undefined" || decision_id === null) {
            decision_id = 1;
        }
        $.ajax({
            async : false,
            type : 'GET',
            url : '/~robhunter/dificildecision/web/app_dev.php/new/'
                    + decision_id,
            dataType : 'json',
            // data: myData,
            success : function(data) {

                dificildecision.prototype.set_current_decision(data);
            }
        });
    },

并转换为set_current_decision方法以使用this.current_decision = decision; :) 有了这个改变，一切都很完美！

Javascript数据未在第一时间加载

2 个答案: