0

我将整个函数粘贴在这里，顺便说一下，我在这个脚本中使用了 mustache 模板库，但这在问题中不是必需的：

tmplReplaceContent : function(json, tmpl, target){
    var regex = new RegExp("\{{[a-zA-Z\.\_]*\}}");
    var template = '';
    var view = '';
    /* json -> check if object */
    if (typeof json == 'object') {
        view = json;
        if(!regex.test(tmpl)){
            /* get mustache tmpl from the path */
            $.get(msi.vars.tmpl_url + tmpl + '.mustache', function(tmplOut){
                template = tmplOut;
                var content = Mustache.render(template, view);
                $(target).html(content).hide().fadeIn();
            });
        } else {
            template = tmpl;
            var content = Mustache.render(template, view);
            $(target).html(content).hide().fadeIn();
        }
    } else {
        /* getJSON from the path */
        $.getJSON(msi.vars.base_url + json, function(jsonOut){
            view = jsonOut;
            if(!regex.test(tmpl)){
                /* get mustache tmpl from the path */
                $.get(msi.vars.tmpl_url + tmpl + '.mustache', function(tmplOut){
                    template = tmplOut;
                    var content = Mustache.render(template, view);
                    $(target).html(content).hide().fadeIn();
                });
            } else {
                template = tmpl;
                var content = Mustache.render(template, view);
                $(target).html(content).hide().fadeIn();
            }
        });
    }

我无法缩短它并删除重复的代码，因为我无法在 Ajax 成功中分配局部变量，因为它是异步的。我在互联网上徘徊了大约 15 个小时。但仍然没有运气。

我怎样才能删除重复的代码并使这个东西更短？

4

2 回答 2

1

好吧，如果您有重复的代码，功能就在那里 :)

只是为了好玩，这里是你如何包括一个：

tmplReplaceContent : function(json, tmpl, target){

    function render(tmplOut) {
        template = tmplOut;
        var content = Mustache.render(template, view);
        $(target).html(content).hide().fadeIn();
    }

    var regex = new RegExp("\{{[a-zA-Z\.\_]*\}}");
    var template = '';
    var view = '';
    /* json -> check if object */
    if (typeof json == 'object') {
        view = json;
        if(!regex.test(tmpl)){
            /* get mustache tmpl from the path */
            $.get(msi.vars.tmpl_url + tmpl + '.mustache', render);
        } else {
            render();
        }
// Etc...

缩短了很多东西，对吧？

于 2012-12-22T14:42:28.137 回答
1 
tmplReplaceContent : function(json, tmpl, target) {

  var regex = new RegExp("\{{[a-zA-Z\.\_]*\}}"),
      view = '';

  function setOutput(template) {
      var content = Mustache.render(template, view);
      $(target).html(content).hide().fadeIn();
  }

  function doJSON(json) {
      view = json;
      if(!regex.test(tmpl)){
          /* get mustache tmpl from the path */
          $.get(msi.vars.tmpl_url + tmpl + '.mustache', setOutput);
      } else {
          setOutput(tmpl);
      }
  }

  /* json -> check if object */
  if (typeof json === 'object') {
    doJSON(json);
  } else {
      /* getJSON from the path */
      $.getJSON(msi.vars.base_url + json, doJSON);
}
于 2012-12-22T14:46:38.627 回答