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

$(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;
    });
});

我在 cmets 中提供了更多提示。

Answer 2

您的问题可能是在 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 没有意义，因为你还是离开了网站。