未捕获的引用错误:未使用onclick定义函数

nx7onnlm  于 2022-10-30  发布在  Java
关注(0)|答案(8)|浏览(155)

我试图为一个网站制作一个用户脚本来添加自定义表情。但是,我一直收到很多错误。
函数如下:

function saveEmotes() {
    removeLineBreaks();
    EmoteNameLines = EmoteName.value.split("\n");
    EmoteURLLines = EmoteURL.value.split("\n");
    EmoteUsageLines = EmoteUsage.value.split("\n");

    if (EmoteNameLines.length == EmoteURLLines.length && EmoteURLLines.length == EmoteUsageLines.length) {
        for (i = 0; i < EmoteURLLines.length; i++) {
            if (checkIMG(EmoteURLLines[i])) {
                localStorage.setItem("nameEmotes", JSON.stringify(EmoteNameLines));
                localStorage.setItem("urlEmotes", JSON.stringify(EmoteURLLines));
                localStorage.setItem("usageEmotes", JSON.stringify(EmoteUsageLines));
                if (i == 0) {
                    console.log(resetSlot());
                }
                emoteTab[2].innerHTML += '<span style="cursor:pointer;" onclick="appendEmote(\'' + EmoteUsageLines[i] + '\')"><img src="' + EmoteURLLines[i] + '" /></span>';
            } else {
                alert("The maximum emote(" + EmoteNameLines[i] + ") size is (36x36)");
            }
        }
    } else {
        alert("You have an unbalanced amount of emote parameters.");
    }
}

span标记的onclick调用此函数:

function appendEmote(em) {
    shoutdata.value += em;
}

每次单击具有onclick属性的按钮时,都会出现以下错误:
未捕获的引用错误:未定义函数

更新

我尝试使用:

emoteTab[2].innerHTML += '<span style="cursor:pointer;" id="'+ EmoteNameLines[i] +'"><img src="' + EmoteURLLines[i] + '" /></span>';
document.getElementById(EmoteNameLines[i]).addEventListener("click", appendEmote(EmoteUsageLines[i]), false);

但我得到了一个undefined错误。
这里是script
我试着这样做来测试听众是否有效,而他们对我无效:

emoteTab[2].innerHTML = '<td class="trow1" width="12%" align="center"><a id="togglemenu" style="cursor: pointer;">Custom Icons</a></br><a style="cursor: pointer;" id="smilies" onclick=\'window.open("misc.php?action=smilies&amp;popup=true&amp;editor=clickableEditor","Smilies","scrollbars=yes, menubar=no,width=460,height=360,toolbar=no");\' original-title="">Smilies</a><br><a style="cursor: pointer;" onclick=\'window.open("shoutbox.php","Shoutbox","scrollbars=yes, menubar=no,width=825,height=449,toolbar=no");\' original-title="">Popup</a></td></br>';
document.getElementById("togglemenu").addEventListener("click", changedisplay,false);
uz75evzq

uz75evzq1#

切勿使用.onclick()或用户脚本中的类似属性!(它也是poor practice in a regular web page)。

原因是用户脚本在沙盒(“隔离世界”)中运行,而onclick在目标页范围内运行,看不到脚本创建的任何函数。
请始终使用addEventListener() Doc(或等效的库函数,如jQuery .on())。
所以不要像这样编写代码:

something.outerHTML += '<input onclick="resetEmotes()" id="btnsave" ...>'

您可以用途:

something.outerHTML += '<input id="btnsave" ...>'

document.getElementById ("btnsave").addEventListener ("click", resetEmotes, false);

对于循环,你不能像这样把数据传递给一个事件监听器,请参阅the doc。另外,每次你像这样改变innerHTML,你就破坏了以前的事件监听器!
无需过多重构代码,就可以传递带有数据属性的数据。因此,请使用如下代码:

for (i = 0; i < EmoteURLLines.length; i++) {
    if (checkIMG (EmoteURLLines[i])) {
        localStorage.setItem ("nameEmotes", JSON.stringify (EmoteNameLines));
        localStorage.setItem ("urlEmotes", JSON.stringify (EmoteURLLines));
        localStorage.setItem ("usageEmotes", JSON.stringify (EmoteUsageLines));
        if (i == 0) {
            console.log (resetSlot ());
        }
        emoteTab[2].innerHTML  += '<span style="cursor:pointer;" id="' 
                                + EmoteNameLines[i] 
                                + '" data-usage="' + EmoteUsageLines[i] + '">'
                                + '<img src="' + EmoteURLLines[i] + '" /></span>'
                                ;
    } else {
        alert ("The maximum emote (" + EmoteNameLines[i] + ") size is (36x36)");
    }
}
//-- Only add events when innerHTML overwrites are done.
var targetSpans = emoteTab[2].querySelectorAll ("span[data-usage]");
for (var J in targetSpans) {
    targetSpans[J].addEventListener ("click", appendEmote, false);
}

其中appendEmote类似于:

function appendEmote (zEvent) {
    //-- this and the parameter are special in event handlers.  see the linked doc.
    var emoteUsage  = this.getAttribute ("data-usage");
    shoutdata.value += emoteUsage;
}

警告:

  • 您的代码对多个元素重复使用相同的ID。请不要这样做,这是无效的。给定的ID在每个页面中只能出现一次。
  • 每次使用.outerHTML.innerHTML时,都会丢弃受影响节点上的所有事件处理程序。
yduiuuwa

yduiuuwa2#

请确定您是否正在使用Javascript模块?!如果使用js 6模块,您的html事件属性将不起作用。在这种情况下,您必须将函数从全局作用域转移到模块作用域。只需将以下内容添加到您的Javascript文件中:window.functionName= functionName;
例如:

<h1 onClick="functionName">some thing</h1>
cdmah0mi

cdmah0mi3#

我认为你把函数放在了$(document).ready.......函数总是由$(document).ready......提供的。

n3h0vuf2

n3h0vuf24#

我在特定组件的.html文件中用(click) = "someFuncionName()"在Angular 上解决了这个问题。

irtuqstp

irtuqstp5#

请检查函数的大小写。

onclick="sillyLongFunctionName"


函数名称(){0}
不完全一样。有时候很难辨认!

niwlg2el

niwlg2el6#

如果在html中使用该函数时没有定义该函数,例如onclick = 'function()',这意味着函数在回调中,在我的例子中是'DOMContentLoaded'。

jslywgbw

jslywgbw7#

如果您使用的是外部js文件,请确保您的函数不在回调函数中。

(function() {  //comment this out

//your code

})(); //comment this out
ztyzrc3y

ztyzrc3y8#

我在Vue中通过将我的onclick="someFunction()"切换到@click="someFunction"解决了这个问题。

相关问题