javascript api简化版

Question

使用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;)
    };
};​

Answer 1

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

编辑

我知道这不是document.write()的问题，但请澄清一下我在说什么，我写了一点重构。我还去掉了theVar = one，因为它是多余的。还要确保在正确的作用域中声明变量。因此，我在one前面添加了一个var。

$(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就没有意义了。

Answer 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;
    });
});

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