未捕获的SyntaxError：意外的令牌if

Question

我创建WordPress ShortCode选项卡并编写此代码以收集短代码

jQuery('body').on('click', '#tapSubmit',function(){
    var shortcode = '[tapWrap]';
    jQuery('.tapForm').each(function(){
        var title = jQuery('.Title').val(),
            content = jQuery('.Content').val(),
            shortcode += '[tap ';
        if(title){shortcode += 'title="'+title+'"';}
        shortcode += ']';
        if(content){shortcode += ''+content+'';}
        shortcode += '[/tap]';
    });
    shortcode += '[/tapWrap]';

    tinyMCE.activeEditor.execCommand('mceInsertContent', false, shortcode);
});

我得到这个错误

Uncaught SyntaxError: Unexpected token if 

我尝试了http://jsfiddle.net/中的代码，我在这行代码中得到了这个错误

shortcode += '[tap ';
Expected an assignment or function call and instead saw an expression.

怎么解决？

Answer 1

当你有

var title = jQuery('.Title').val(),
        content = jQuery('.Content').val(),
        shortcode += '[tap ';

您正在该链中定义新变量，但已定义了shortcode ，因此您要在此范围内创建新变量。 作为一个新的变量你不能使用+= 。 无论如何我认为你只想用这个：

var title = jQuery('.Title').val(),
    content = jQuery('.Content').val(); // changed the last comma with semicolon
shortcode += '[tap ';

阅读：
关于范围
关于var

Answer 2

问题就在这里

var title     = jQuery('.Title').val(),
    content   = jQuery('.Content').val(),
    shortcode += '[tap ';

shortcode已经是上面定义的var。 你不能在var表达式中使用+=

只需将其更改为

var title     = jQuery('.Title').val(),
    content   = jQuery('.Content').val(); // note the semicolon here

shortcode += '[tap ';

---

我想你也会遇到一些嵌套问题。 而不是为循环的每次迭代调用jQuery('.Content').val() ，我认为你正在寻找更像$(this).find('.Content').val()或$('.Content', this) 。 这将在给定的.tapForm范围内找到相关的.Content输入。

我在想这样的事情，但这只是一个想法

jQuery('body').on('click', '#tapSubmit', function(){

  function title(context) {
    var value = jQuery(".Title", context).val();
    return value ? 'title="' + value + '"' : '';
  }

  function content(context) {
    var value = jQuery(".Content", context).val();
    return value || '';
  }

  var taps = jQuery('.tapForm').map(function(){
    return '[tap ' + title(this) + ']' + content(this) + '[/tap]';
  }).join();

  tinyMCE.activeEditor.execCommand('mceInsertContent', false, '[tapWrap]' + taps + '[/tapWrap]');  
});