1

正在使用fourquare api获取场地，之前有一个从api写出的可点击列表，但将其缩减为仅写入屏幕的一个场地名称。然后决定最好将其发送到php。所以当我做了我认为是逻辑切割代码的事情时，它完全停止了工作。

我的程序有这个，工作：

$(document).ready(function doEverything(element) {
    $.getJSON("https://api.foursquare.com/v2/venues/search?ll=" + lat + "," + lng + "&client_id=L2VWBKPOW45D5X3FJ3P4MJB5TGVJ4ST2J005RIVAFIWG44ND%20&client_secret=ZKDAOLHASCA31VUOGMBTAS3RFYUOMXL4IFFYPRURIDQA3QMA%20&v=20111107", function(data) {
        one = data.response.venues[0].name;
        var list = [];
        list[0] = [one];

        function Make() {
            for (var i = 0; i < 1; i++) {
                var div = document.createElement("div");
                div.style.margin = "-435px 100px 0px 110px";
                div.innerHTML = list[i];
                !
                function() {
                    var index = 0;
                    div.onclick = function() {
                        doSomething(this);
                    };
                }();
                document.body.appendChild(div);
            }
        }

        function doSomething(element) {
            var value = element.innerHTML;
            switch (value) {
            case one:
                break;
            }
        }
        Make();
    });
});

然后我决定我想使用这个将一个变量传递给 php：

theVar = 10; //just to make things simple.
urlString = "cookiestesttwo.php?var=" +theVar;
window.location = urlString;

因此，我尝试将我的 api 代码简化为此，但它停止了工作：

$(document).ready() {
    $.getJSON("https://api.foursquare.com/v2/venues/search?ll=" + lat + "," + lng + "&client_id=L2VWBKPOW45D5X3FJ3P4MJB5TGVJ4ST2J005RIVAFIWG44ND%20&client_secret=ZKDAOLHASCA31VUOGMBTAS3RFYUOMXL4IFFYPRURIDQA3QMA%20&v=20111107", function(data) {
        one = data.response.venues[0].name;
        document.write(one)
        theVar = one
        urlString = "cookiestesttwo.php?var=" + theVar;
        window.location = urlString;)
    };
};​
4

2 回答 2

2

您的问题可能是您document.write()在 DOM 已经完成时使用。你不应该那样做。创建一个元素document.createElement( "div" )并设置innerText()，然后将其附加到 dom。就像你在重构之前所做的一样。

编辑

我知道这不是，document.write()但只是澄清我在说什么我写了一个小重构。我也扔掉了，theVar = one因为那是多余的。还要确保在正确的范围内声明您的变量。所以我var在前面加了一个one。

$(document).ready(function() {
    $.getJSON("https://api.foursquare.com/v2/venues/search?ll="+lat+","+lng+"&client_id=L2VWBKPOW45D5X3FJ3P4MJB5TGVJ4ST2J005RIVAFIWG44ND%20&client_secret=ZKDAOLHASCA31VUOGMBTAS3RFYUOMXL4IFFYPRURIDQA3QMA%20&v=20111107",

    function(data){
       var one = data.response.venues[0].name;

       var div = document.createElement( "div" );
       div.innerText( one );
       document.appendChild( div );

       window.location = "cookiestesttwo.php?var=" + one;
    });
  });

但是如果你改变窗口的位置。document.write()因为无论如何你离开了网站，所以没有必要添加新的 div 或添加新的 div。

于 2012-12-20T20:50:14.703 回答
2

$(document).ready() {不是正确的语法并且会引发错误。

此外，函数末尾还有另一个语法错误。你颠倒了}和）

$(document).ready(function() {
    $.getJSON("https://api.foursquare.com/v2/venues/search?ll=" + lat + "," + lng + "&client_id=L2VWBKPOW45D5X3FJ3P4MJB5TGVJ4ST2J005RIVAFIWG44ND%20&client_secret=ZKDAOLHASCA31VUOGMBTAS3RFYUOMXL4IFFYPRURIDQA3QMA%20&v=20111107", function(data) {
        one = data.response.venues[0].name; // if one is local to this function then use VAR otherwise you'll leak it globally.
        document.write(one);
        theVar = one; // same here for the VAR keyword.
        urlString = "cookiestesttwo.php?var=" + theVar; // and here as well. NO GLOBAL LEAKS!
        window.location = urlString;
    });
});

我在评论中提出了更多提示。

于 2012-12-20T21:16:45.030 回答