包装一系列闭包

Question

我继承了一大段代码，其中有一大组 DataTable 渲染器。我想在前面的数据对象中插入一些额外的数据而不重写所有渲染器，只重写那些实际受影响的渲染器。

columnDefs 字段生成为具有 render 函数的对象数组，通常还有其他字段（targets、type 等） .我想即时重写它，使 render 函数成为调用原始函数的包装器。我找不到正确的代码来关闭原始函数，因此我可以在处理过程中修改数据并仍然调用该原始函数。

例如，给定一个 columnDefs 为：

var columnDefs = [
{
  "render": function (data, type, row) {
    return stuff(row[3]);
  },
  "targets": 2
},
{
  "render": function (data, type, row) {
    return stuff(row[0]);
  },
  "targets": 1
},
// etc.
]

我正在尝试做这样的事情：

var mapper = function (def) {
    (function (d, old_render) {
        d.render = (data, type, row) => {
            let extra_data = undefined;
            if (row.length > 1 && typeof (row[0]) == 'object') {
                extra_data = row[0];
                row = row.slice(1);
            }
            return old_render(data, type, row, extra_data);
        }
    })(def, def.render);
};
columnDefs.forEach(mapper);

在调用 $("#my-datatable").DataTable({ ..., columnDefs: columnDefs, ... }).

之前

目前有效的是正在调用新的渲染函数。但是，当调用 render 时，old_render 始终未定义，这当然会死亡。

完整的代码示例 on JSFiddle 为了方便起见，也在这里：

function stuff(x) {
  return "<b>" + x + "</b>";
}

var columnDefs = [
{
  "render": function (data, type, row) {
    return stuff(row[3]);
  },
  "targets": 2
},
{
  "render": function (data, type, row) {
    return stuff(row[0]);
  },
  "targets": 1
},
]

var dataSource = [{"a", "b", "c", "d", "e"}, {"f","g","h","i","j"}];
columnDefs.forEach(d => {
  d.render = function(old_render) {
    return (data, type, row) => {
      let extra_data = undefined;
      return old_render(data, type, row.map(, extra_data);
    };
  }(d.render);
});
var table = $("#my-table").DataTable({
processing: true,
data: dataSource,
columnDefs: columnDefs
});

而关联的 HTML 只是：

<link href="//cdn.datatables.net/1.10.19/css/jquery.dataTables.min.css" rel="stylesheet" type="text/css" />
<script src="https://cdn.datatables.net/1.10.19/js/jquery.dataTables.min.js"></script>

<p>
hello
</p>
<table id="my-table">
<thead>
  <tr>
    <th>1</th><th>2</th><th>3</th><th>4</th>
  </tr>
</thead>
</table>

更新...JSLint 正在运行，我只是不确定为什么它不在我较大的代码库中。我想回到绘图板，找出原因。

Answer 1

像这样的东西应该有用...

function newRender(oldRender, data, type, row) {
    let extra_data = undefined;
    if (row.length > 1 && typeof (row[0]) == 'object') {
        extra_data = row[0];
        row = row.slice(1);
    }
    return oldRender(data, type, row, extra_data);
}

var mapper = function (def) {
    def.render = newRender.bind(null, def.render);
};

columnDefs.forEach(mapper);

Answer 2

事实证明，如果某些记录没有输入渲染器，那么当然无法调用旧渲染器。这里处理的列数足够多，我没有注意到其中一些没有渲染器。

var columnDefs = [
{
  "render": function (data, type, row) {
    return stuff(row[3]);
  },
  "targets": 2
},
{
  //"render": function (data, type, row) {
  //  return stuff(row[0]);
  //},
  "targets": 1
},
]

现在当代码到达此处的第二列时，没有渲染函数，因此当然是未定义的。通过用 if (d.render) {...} 包装映射器，问题就这样避免了。

包装一系列闭包

Wrapping a series of closures

javascript

monkeypatching

datatables