Как побороть ReferenceError в JS?

Question

[Александр]

Собственно прошу помощи в поиске ошибки, не могу понять, какая переменная не объявлена? о_О
Простой код:

<div class="form-group" onclick="delMaster(\''.$value['id'].'\');">
                   блабла
                </div>

И функция:

//Удаление мастеров
        function delMaster(id) {
            $.get("page/retail/order/ajax/addMaster.php", {act: 3, masterid: id, orderId: <?=$orderID?>})
            .done(function (data) {
                if(data == '') {
                    $('#alertAddMaster').fadeIn();
                }
                $('#loadResultAddMaster').html(data);
            });
        };

Что не так? :)

Консоль:

Uncaught ReferenceError: delMaster is not defined
at HTMLDivElement.onclick (retail.php?p=order&id=15:1)

Answer 1

[Александр]

Смотря где объявлена функция. Попробуй
window.delMaster = function...
И, соответственно,

<div class="form-group" onclick="window.delMaster(...);">

Comments

[Александр]

Спасибо, помогло...

[Александр]

Александр: Значит функция была объявлена в другой области видимости. Вообще так не стоит делать. Лучше добавить функцию внутрь тега скрипт, вне других блоков. И обойтись тогда без window

[Александр]

Александр: если можно, поподробнее, почему так не стоит делать? В JS не профи :) Ошибку кстати нашел и поправил...

[Александр]

Александр: Во-первых - дурной тон, во-вторых можно по невнимательности затереть какое-нибудь свойство объекта windows. Так что лучше этого избегать, хотя в редких случаях и бывает полезно.

Answer 2

[GreatRash]

Очевидно же: delMaster is not defined
Вероятно вы скрипт на страницу добавляете позже чем вызываете клик.

Comments

[Александр]

Да нет, сначала подгружается функция, а потом инклюдиться файл...

Answer 3

[emp1re]

Интерпритатор не зарезервировал имя delMaster, значит файл с функцией не подключен. Или функции в не зоны видимости для евента onclick