在javascript中将函数写为字符串

时间:2012-06-26 02:45:39

标签: javascript jquery

我想创建一个上下文驱动的菜单。当用户单击树中的任何节点时,会出现此菜单。每个节点都有一个“treedropdownmenu”类。在单击节点上,应打开上下文驱动的菜单。我正在使用“删除”菜单选项传递方法“_deleteClick”。但它给我一个错误:找不到“_deleteClick”菜单。

我的小部件中有以下代码:

$(".treedropdownmenu").live("click", function (event) {
    var pos;
    if(($(window).height() - event.pageY) < 80) {
        pos = {
            left: event.pageX + 20,
            top: event.pageY - 60
        };
    } else {
        pos = {
            left: event.pageX + 20,
            top: event.pageY + 20
        };
    }
    if(ko.dataFor(this).nodeId() && ko.dataFor(this).nodeId() !== 0) {
        var item = ko.dataFor(this);
        var strHtml = "<a href='#' onclick='_deleteClick(item)'>Delete:</a> " + "<br/>" + "<b>Create Date:</b>" + "<br/>" + "<b>Exposed Party Name:</b>" + "<br/>" + "<b>Portfolio Type:</b>" + "<br/>" + "<b>Owner:</b>";
        $("#dataManagerMenuItem1234").show().offset(pos).html(strHtml);
    }
});

我的删除菜单是:

function _deleteClick(item) {
    alert("delete clicked");
}

任何人都可以让我知道我哪里错了吗?

4 个答案:

答案 0 :(得分:1)

是的,我认为这不会奏效。试试这个:

var strHtml = "<a href='#'>Delete:</a> " + "<br/>" + "<b>Create Date:</b>" + "<br/>" + "<b>Exposed Party Name:</b>" + "<br/>" + "<b>Portfolio Type:</b>" + "<br/>" + "<b>Owner:</b>";
 $("#dataManagerMenuItem1234").show().offset(pos).html(strHtml).find('a').click(function() { _deleteClick(item); });

答案 1 :(得分:1)

这是一种向链接添加事件处理程序的笨拙方式,尤其如此 因为你无论如何都是以编程方式创建的。为什么不添加点击 动态处理程序,如此?

var item = ko.dataFor(this);
var deleteLink = $('<a>', {
    href: '#',
    click: function() {
        _deleteClick(item);
    },
    text: 'Delete:'
});
$("#...").show().offset(pos).append(deleteLink);

答案 2 :(得分:0)

不要在字符串中添加事件处理程序,这是不好的做法。

var strHtml = "<a href='#'>foo</a>;
var lnk = jQuery(strHtml);
lnk.on("click", function(){ _deleteClick(item); });
$("#dataManagerMenuItem1234").show().offset(pos).append(lnk);

答案 3 :(得分:0)

您尚未在此处发布所有代码,但我假设您已在本地范围内定义了函数_deleteClick,并且您的onclick处理程序无法从全局范围访问它。这是您不应使用onclick属性将事件附加到DOM元素的原因之一!您的onclick属性(以字符串形式写出)不会在正确的上下文中执行。使用jQuery来订阅。

看起来你正在使用KnockoutJS。更好的方法是使用Knockout template来编写菜单。