我想知道是否有更优雅的方法来修改onclick事件的参数。我有一个表,我动态添加/删除元素,我重新索引行。每行都有一个delete
链接,该链接具有行的索引(和duplicate
链接),需要更新其参数以匹配修改后的行ID。
目前我的代码看起来像(简化)
<a onclick="delRow(1)">delete</a>
和javascript: ...
html = element.innerHTML;
html = html.replace(/dupRow(\\d+)/g, "dupRow(" + newIndex + ")");
html = html.replace(/delRow(\\d+)/g, "delRow(" + newIndex + ")");
element.innerHTML = html
我希望它成为
的内容if (element.onclick != null) {
element.onclick.params[0] = newIndex;
}
任何这样的方式来完成这个?如果这有帮助我也有jQuery。谢谢!
更新
感谢@ rich.okelly的光荣帮助,我已经解决了我的问题
<script>
...
var newRow = '\
<tr>\
<td class="index" col="0">0</td>\
<td>this is content...</td>\
<td><a href="#" row-delete="true">Del</a></td>\
</tr>';
// re-index table indices in a non-efficient manner
function reIndexTable() {
$("#rpc-builder-table").find('.index').each(function (i) {
$(this).html(i)
})
}
// add row
function addRow() {
for (i = 0; i < $('#addRowCount').attr("value"); i++) {
$("#rpc-builder-table").append(newRow);
}
reIndexTable();
}
$(document).ready(function () {
// add row button
$('#addRowsButton').on('click', function () {
addRow();
});
// delete row
$('#rpc-builder-table').on('click', 'td a[row-delete="true"]', function () {
$(this).closest('tr').remove();
reIndexTable();
});
...
}
</script>
...
<div>
<label>Rows to add: </label>
<input id="addRowCount" value="1" size="2" />
<button id="addRowsButton">Add Row(s)</button>
</div>
<div><table id="rpc-builder-table"><tbody>
<tr>
<th>Idx </th>
<th>Some content (1)</td>
</tr>
</tbody></table></div>
...
我使用了.on()
函数而不是建议的.delegate()
函数,因为它已被弃用。解决方案效果很好 - 希望它可以帮助某人:)
答案 0 :(得分:2)
如果您将html更改为类似于:
<tr>
<td>
<a href="#" data-delete="true">delete</a>
</td>
</tr>
然后你的javascript就像:
$('td a[data-delete="true"]').on('click', function() {
$(this).closest('tr').remove();
});
<强>更新强>
如果将行动态添加到预先存在的表(表可以与任何父元素互换),则可以使用委托方法,如下所示:
$('table').delegate('td a[data-delete="true"]', 'click', function() {
$(this).closest('tr').remove();
});
答案 1 :(得分:1)
使用事件委派来附加事件处理程序
,而不是内联处理程序$("#tableID").delegate("a", "click", delRow);
$("#tableID").on("click", "a", delRow); //jQuery 1.7
处理程序内部
var row = $(this).closest("tr").index(); //Get the index of the parent row
内联处理程序被解析为函数:
function onclick() {
delRow(1);
}
所以改变它们很困难。您的示例使用新参数重写整行,这是不好的做法。
答案 2 :(得分:0)
最脑死亡的解决方案是摆脱参数并设置变量isntead。
var row_to_dup = 42;
$("#a_row_dupper").bind('click', function (){
dupItem(row_to_dup);
});
//changing the row to dup
row_to_dup = 17;