Question

我的代码中有一个错误，但看不出我做错了什么。

我所拥有的是facebook用户ID的隐藏输入，它是通过jQuery UI自动完成填充的：

<input id="fbid" type="hidden" value="12345, 567890, ">

然后我有一个jQuery函数，当点击链接发布到朋友的墙上时，它会运行。

<div id="fb-root"></div>
<script>
window.fbAsyncInit = function() {
    FB.init({
        appId      : '12345678', // App ID
        channelUrl : '/channel.html', // Channel File
        status     : true, // check login status
        cookie     : true, // enable cookies to allow the server to access the session
        oauth      : true, // enable OAuth 2.0
        xfbml      : true  // parse XFBML
    });

    // Additional initialization code here

    FB.login(function(response)
    {
        if (response.authResponse)
        {                                               
            $("#send-voucher").click(function() {

                // Post message to friend's wall

                var opts = {
                    message : 'This is the message',
                    name : 'This is the name',
                    link : 'http://www.opticalexpress.co.uk',
                    description : 'This is the description',
                    picture : '/voucher_valid.png'
                };

                var referees = $("#fbid").val();

                // remove last comma and space from the end of the string
                referees = $.trim(referees);
                referees = referees.substring(0, referees.length - 1);

                var referee = referees.split(',');
                referee.each(function() {

                    FB.api('/'+ referee +'/feed', 'post', opts, function(response)
                    {
                        if (!response || response.error)
                        {
                            alert('Posting error occured');
                        }
                        else
                        {
                            alert('Success - Post ID: ' + response.id);
                            $("#send-voucher").hide();
                            $("#voucher-sent").fadeIn('slow');
                        }
                    });
                });
            });
        }
        else
        {
            alert('Not logged in');
        }
    }, { scope : 'publish_stream' });
};

当我点击发送优惠券时，错误来自我上面的else声明中的提醒对话框：

Posting error occured

我在另一个警报中添加了一个警告，看看为什么在循环开始后发生这种情况：

$.each(referee, function() {
    referee = $.trim(referee);
    alert(referee); // new alert for debugging
    ....

此警报输出让我感到惊讶的是referee的值为12345, 567890（与隐藏输入相同但删除了尾随空格和最后一个逗号）。

因此，jQuery似乎没有正确地拆分它，但它运行循环正确的次数，因为警报框弹出两次告诉我裁判然后错误。（两个警报框只显示两次以澄清）。如果这是巧合我尝试在隐藏输入中添加更多id，并且可以确认警告框显示与id的数量相同的次数。

所以我想知道我做错了什么因为循环运行了正确的次数但是逗号分隔的id似乎根本没有分开。

Answer 1

在名为referee.each(function()的数组上调用

referee，而不是jQuery对象。

请尝试$.each(referee, function()。

http://api.jquery.com/jQuery.each/

更新

您也可以使用正则表达式查看.split，看看这是否有所不同：

var referee = referees.split(/\s*,\s*/);

https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/String/split

Answer 2

程序中的

referee是分割字符串列表，在“每个循环”中，您使用列表而不是实例。尝试更换：

FB.api('/'+ referee +'/feed', 'post', opts, function(response)

与

FB.api('/'+ $(this) +'/feed', 'post', opts, function(response)

Answer 3

您的代码存在一些问题。正如其他人所提到的，您使用不存在的.each()方法循环数组。您可以使用标准for()循环。此外，使用正则表达式可以更有效地剥离空格和多余的逗号。

以下是一个简化示例，也可在http://jsfiddle.net/hJQ6W/

处找到

<input id="fbid" type="hidden" value="12345, 567890, ">
<script>
  var referees = $("#fbid").val();
  // trim string and remove all commas from the end of it
  referees = referees.replace(/\s/g,'').replace(/,*$/,'');
  var referee = referees.split(',');
  for(i=0; i<referee.length; i++) {
    alert('"'+referee[i]+'"');
  }
</script>

jQuery split（）不....分裂？

3 个答案: