1

以下函数的目的是访问雅虎服务器上的脚本并查找实时货币兑换率,该兑换率稍后将用于处理客户的购买。

我可以访问 JavaScript 警报中的费率,但似乎无法将它们返回到最初调用该getRate()函数的 Jquery 方法。

return rate;在函数末尾尝试了一个标准parseExchangeRate(),但它不起作用。我也尝试过在其中设置rate为全局变量,parseExchangeRate()但这也不起作用。

function getRate(from, to) {

    var script = document.createElement('script');

    script.setAttribute('src', "http://query.yahooapis.com/v1/public/yql?q=select%20rate%2Cname%20from%20csv%20where%20url%3D'http%3A%2F%2Fdownload.finance.yahoo.com%2Fd%2Fquotes%3Fs%3D"+from+to+"%253DX%26f%3Dl1n'%20and%20columns%3D'rate%2Cname'&format=json&callback=parseExchangeRate");
    document.body.appendChild(script);

}

function parseExchangeRate(data) {

    var name = data.query.results.row.name;
    rate = parseFloat(data.query.results.row.rate, 10);

    alert("Exchange rate " + name + " is " + rate);

}

$(function() {

    getRate('USD', 'PHP');
    xRatePHP = rate;

    /* Do stuff with rate */

});

rate当我尝试在 Jquery 函数中访问它时,Firebug 通知我未定义。

我尝试的另一件事是设置 http 请求的最后一个参数callback=rate=parseExchangeRate(不足为奇)也不起作用。

更新

@Relfor 解决了最初的问题,即rate没有在全局范围内正确声明为全局变量。我解决了这个问题,然后发现了另一个问题,一些人也在下面确定了这个问题,即在getRate()调用之后(可能需要大约 2000 毫秒来更新变量rate),脚本立即继续运行,无需等待rate更新并使用速率,无论是否准备好。

我试图window.setInterval用来创建一个延迟来解决这个问题,当我注意到他们仍然在线程中活动时,即使我已经接受了@Relfor 的回答,所以我更愿意把它带回这里所以当我们进行这项工作时,其他人可以从解决方案中受益。

有一个最终(我希望 - 最终!)问题,那就是为了简化发布的原始问题,我没有透露我实际上是在尝试从 Yahoo! 获得两个费率。(可能还有更多计划),因此,我getRate()在循环中调用如下:

function getRate(from, to) {

    var script = document.createElement('script');

    script.setAttribute('src', "http://query.yahooapis.com/v1/public/yql?q=select%20rate%2Cname%20from%20csv%20where%20url%3D'http%3A%2F%2Fdownload.finance.yahoo.com%2Fd%2Fquotes%3Fs%3D"+from+to+"%253DX%26f%3Dl1n'%20and%20columns%3D'rate%2Cname'&format=json&callback=parseExchangeRate");
    document.body.appendChild(script);

}

function parseExchangeRate(data) {

    var name = data.query.results.row.name;
    rate = parseFloat(data.query.results.row.rate, 10);

}

var rate = 1.00;
var timer;
var q;
var xRatePHP, xRateGBP;
$(function() {

    function getTheRates() {

        var rateArr = new Array('PHP','GBP');

        for (var x=0; x < rateArr.length; x++) {
            getRate('USD', rateArr[x]);

            q = 0;
            timer = window.setInterval(function(){manageTimer(rateArr[x])},100);

        }
    }

    function manageTimer(c) {

        if (rate != 1) {

            window.clearInterval(timer);

            /* Note that 'c' is undefined according to 'alert' below,
             * so this next line is not working correctly.
             */
            eval("xRate"+c+" = rate;"); 


            alert(c + " = " + rate); // displays 'undefined 43.543'

            rate = 1.00;

        }

        q++;

        if (q > 30 ) {

            window.clearInterval(timer);

            // added below because above isn't working (but neither does this!)
            timer = '';
            alert(c+' timeout'); // 'c' is undefined according to alert ???
            q = 0;

        }

    }

    getTheRates();

    /* Do stuff with the rates */

});

有人建议我移到/* Do stuff with the rates */函数内部,parseExchangeRate()但不确定考虑到我对getRate()循环调用的启示,该建议是否仍然有效?

更新 3(替换更新 2)

我在这里创建了一个 JSbin:http://jsbin.com/udikas/3/edit上面除了这两个问题之外似乎正在工作:

1) 超时机制似乎不起作用。

2)在第 33 行这条线alert('start timer (' + x +')');没有它,计时器似乎没有启动!我不知道为什么,但我不能把那条线留在里面。

4

3 回答 3

3

变量rate尚未定义

$(function() {

    getRate('USD', 'PHP');
    xRatePHP = rate;

    /* Do stuff with rate */

});

在研究了您的代码之后,似乎parseExchange(data)函数中将速率定义为

function parseExchangeRate(data) {

    var name = data.query.results.row.name;
    rate = parseFloat(data.query.results.row.rate, 10);

    alert("Exchange rate " + name + " is " + rate);

}

如果您希望从函数命名空间访问速率而不从内部声明它们,则必须在全局命名空间中指定速率,该命名空间位于任何函数或循环之外。

编辑:命名空间问题已解决,我给出的答案已被接受,但是我想在此处添加有关您正在处理的代码的详细信息。它取自这里的要点:https ://gist.github.com/henrik/265014

我们在这里处理 JSONP

parseExchangeRate乍一看,它存在的原因和它的上下文似乎很神秘,尽管它的存在是您的 JSONP 请求和响应返回的数据之间的主要连接。

如果您仔细查看请求:

http://query.yahooapis.com/v1/public/yql?
q=select%20rate%2Cname%20from%20csv%20where%20url%3D'http%3A%2F%2F
download.finance.yahoo.com%2Fd%2F
quotes%3Fs%3DUSDPHP%253DX%26f%3Dl1n'%20and%20
columns%3D'rate%2Cname'&format=json&callback=parseExchangeRate

(我将链接分成多行以使其更易于阅读)
仔细查看 url 的最后一段:callback=parseExchangeRate
这是连接,当 JSONP 请求完成时将调用 parseExchangeRate。

那么为什么它不起作用呢?

让我再次显示代码:

$(function() {

    getRate('USD', 'PHP');
    xRatePHP = rate;

    /* Do stuff with rate */

});

我们应该打破这个:

  • getRate('USD', 'PHP')使用相应的货币类型“USD”和“PHP”加载 JSONP 的工作
  • xRatePHP = rate将右侧分配ratexRatePHP. 但是这条线给了我们一个问题!我们的控制台朋友告诉我们这rate是未定义的!

真相: mr.console 没有说谎,rate实际上是未定义的,但是如果在几分钟后再次询问mr.console没有您给出的任何进一步命令,rate则会回复实际已定义。这是魔法吗?

实际发生的是在你从这条线上走的时间之间

getRate('USD', 'PHP');

xRatePHP = rate;

mr.yahoo 的 JSONP 响应尚未返回,这就是为什么xRatePHP = rate发出rate的时间似乎未定义。

硬编码解决方案
让我们硬编码一段代码在使用前等待的时间,rate这样我们就知道 mr.yahoo 响应了,setTimeout这将帮助我们:

getRate('USD', 'PHP');
setTimeout(function(){alert(rate)}, 2000);

现在一切正常!查看演示:http: //jsbin.com/udikas/1/edit

Soft Code
有没有考虑过 mr.yahoo 响应时间超过 2000 毫秒的情况?或者甚至比这还少?(雅虎非常快!)让我们采取不同的方法,这将让我们使用我们计算它rate确切时刻parseExchangeRate

为此,我们必须从以下位置添加回调parseExchangeRate

function parseExchangeRate(data) {

    var name = data.query.results.row.name;
    rate = parseFloat(data.query.results.row.rate, 10);

    alert("Exchange rate " + name + " is " + rate);

}

function parseExchangeRate(data) {

    var name = data.query.results.row.name;
    rate = parseFloat(data.query.results.row.rate, 10);

    alert("Exchange rate " + name + " is " + rate);
    gotTheRate(rate)

}

然后改变

$(function() {

    getRate('USD', 'PHP');
    alert(rate)

});

function gotTheRate(rate){
    alert(rate);
  }

$(function() {

    getRate('USD', 'PHP');
      
});

可以在http://jsbin.com/udikas/2/edit找到一个演示

多个回调(响应问题更新)

请记住,硬编码setTimeouts并不好玩,所以让我们从您的代码中删除 、manageTimerq和其他此类元素,取而代之的是:

function getRate(from, to) {

    var script = document.createElement('script');

    script.setAttribute('src', "http://query.yahooapis.com/v1/public/yql?q=select%20rate%2Cname%20from%20csv%20where%20url%3D'http%3A%2F%2Fdownload.finance.yahoo.com%2Fd%2Fquotes%3Fs%3D"+from+to+"%253DX%26f%3Dl1n'%20and%20columns%3D'rate%2Cname'&format=json&callback=parseExchangeRate");
    document.body.appendChild(script);

}

function parseExchangeRate(data) {

    var name = data.query.results.row.name;
  
    rateDict[name.match(/USD to ([\w]*)/)[1]] = parseFloat(data.query.results.row.rate, 10);
    total_responses++;
  if (total_responses === rateArr.length){
    for (var k in rateDict){
        alert("USD to " + k + " is " + rateDict[k]);
    }
  }
}

var rate = 1.00;
var timer;
var q;
var rateArr = new Array('PHP','GBP')
var total_responses = 0;
var rateDict = {};

$(function() {

    function getTheRates() {


        for (var x=0; x < rateArr.length; x++) {

            getRate('USD', rateArr[x]);

        }
    }

    getTheRates();
  

}); 

http://jsbin.com/udikas/4/edit

-Relfor

于 2013-07-04T08:13:35.763 回答
1

您应该重新订购您的代码流:

function getRate(from, to) {
    var script = document.createElement('script');
    script.setAttribute('src', "http://query.yahooapis.com/v1/public/yql?q=select%20rate%2Cname%20from%20csv%20where%20url%3D'http%3A%2F%2Fdownload.finance.yahoo.com%2Fd%2Fquotes%3Fs%3D"+from+to+"%253DX%26f%3Dl1n'%20and%20columns%3D'rate%2Cname'&format=json&callback=parseExchangeRate");
    document.body.appendChild(script);
}
function parseExchangeRate(data) {
    var name = data.query.results.row.name;
    var rate = parseFloat(data.query.results.row.rate, 10);
    alert("Exchange rate " + name + " is " + rate);
    xRatePHP = rate;
    /* Do stuff with rate */
}
$(function() {
    getRate('USD', 'PHP');
    // rate is not yet available here, so don't do anything with it
});
于 2013-07-04T08:27:14.083 回答
0

试着说:

var rate = 0;

在您的代码之上。这将修复错误。然后你必须考虑如果你得到 0 可能你在错误的时间读取变量,在它被填充之前。

于 2013-07-04T08:10:18.937 回答