Uncaught ReferenceError: функція не визначена за допомогою onclick

Question 1

Я намагаюся створити скрипт користувача для веб-сайту, щоб додати власні емоції. Однак я отримую багато помилок.

Ось функція:

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атрибутом, з’являється така помилка:

Uncaught ReferenceError: функція не визначена.

Будь-яка допомога буде вдячна.

Дякую!

Оновлення

Я спробував використовувати:

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помилку.

Ось сценарій .

Я спробував зробити це, щоб перевірити, чи працюють слухачі, а вони не для мене:

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);

Question 2

Ніколи не використовуйте .onclick()або подібні атрибути з скрипту користувача! (Це також погана практика на звичайній веб-сторінці ).

Причиною є те, що скрипти користувачів працюють у пісочниці ("ізольований світ"), і onclickпрацюють у межах цільової сторінки і не бачать жодних функцій, створених вашим сценарієм.

Завжди використовуйте addEventListener()^Doc (або еквівалентну бібліотечну функцію, наприклад jQuery .on () ).

Отже, замість коду типу:

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

Ви б використали:

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

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

Що стосується циклу, ви не можете передавати дані слухачеві подій, як це. Перегляньте документ . Плюс кожен раз, коли ви змінюєтесь 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;
}

ПОПЕРЕДЖЕННЯ:

Ваш код повторно використовує один і той же ідентифікатор для кількох елементів. Не робіть цього, це недійсно. Даний ідентифікатор повинен зустрічатися лише один раз на сторінці.
Кожного разу, коли ви використовуєте .outerHTMLабо .innerHTML, ви видаляєте будь-які обробники подій на уражені вузли. Якщо ви використовуєте цей метод, остерігайтеся цього.

Question 3

Я отримав помилку (я використовую Vue), і я переключився onclick="someFunction()"на, @click="someFunction"і тепер вони працюють.

Question 4

Якщо функція не визначена при використанні цієї функції в html, наприклад onclick = 'function ()', це означає, що функція знаходиться у зворотному виклику, у моєму випадку це 'DOMContentLoaded'.