Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Ошибка function is not defined (https://javascript.ru/forum/misc/11706-oshibka-function-not-defined.html)

KamaZz 26.09.2010 17:47

if (page) {
	var buttons = document.getElementById("buttonz");
	for (i = 0; i < page.length; i++) {
		try {
			buildings = buttons.getElementsByClassName(buildId + page[i][0]);
			buildings[0].firstChild.nextSibling.innerHTML += "<a href=\"javascript: setQueue('" + pagename + "', " + page[i][0] + ");\" title=\"|Поставить в очередь на постройку\" class=\"fastBuild tipsStandard feldwebel\"><img width=\"22\" height=\"14\" src=\"img/layout/pixel.gif\"><img width=\"22\" height=\"14\" src=\"img/layout/pixel.gif\"></a>";
		} catch (e) {
			GM_log(e.message);
		}
	}
}

function setQueue(pagename, id) {
	GM_setValue('build', new Array(id, pagename));
	alert(GM_getValue('build'));
}


В консоли выдает:
Error: setQueue is not defined
javascript: setQueue('station', 14);

Судя по всему проблема опять с замыканием... Подскажите пожалуйста как ее решить, т.к. по примерам статьи ничего не понятно...

exec 26.09.2010 17:52

Функция setQueue объявлена внутри замыкания? Если да, то надо добавить

window.setQueue = setQueue;

KamaZz 26.09.2010 19:50

Замыкание возникает внутри цикла?
Если да, то она вне замыкания... Просто при вызове этой функции (на ссылку жму), в консоли пишется что функция не объявлена...

KamaZz 27.09.2010 12:25

Функция и ее вызов находятся в одной области видимости...
Я имею ввиду что ни цикл, ни функция не находятся внутри другой функции...

KamaZz 27.09.2010 15:49

Помогло unsafeWindow.setQueue = setQueue;
Цитата:

Error: Greasemonkey access violation: unsafeWindow cannot call GM_setValue.
Но при использовании unsafeWindow, GreaseMonkey не даёт мне пользоваться функциями GM'a... Такими как GM_setValue, etc...
Существуют-ли другие способы решения этой проблемы?

KamaZz 28.09.2010 13:36

Error: missing formal parameter
Line: 1, Column: 10
Source Code:
(function('station', 21) { return function() { setQueue('station', 21); } })(1);
"<a href=\"#\" onclick=\"(function('" + pagename + "', " + page[i][0] + ") { return function() { setQueue('" + pagename + "', " + page[i][0] + "); } })(" + i + ");\" class=\"fastBuild\">";

exec 28.09.2010 13:38

Цитата:

(function('station', 21) {
В аргументах функции должны быть только их названия, а не строки и числа.

KamaZz 28.09.2010 14:07

Error: missing ) after formal parameters (указатель показывает на квадратную скобку)
(function(pagename, page[i][0]) { return function() { setQueue('station', 14); } })(0);

Error: missing formal parameter (указатель показывает на одинарную кавычку)
(function('pagename', page[i][0]) { return function() { setQueue('station', 14); } })(0);

Kolyaj 28.09.2010 14:09

KamaZz,
за вас тут ещё и синтаксические ошибки разбирать? Учебник почитайте, чтобы бред не писать.

KamaZz 28.09.2010 14:38

Прошу прощение, действительно глупость написал :)
Но замыкание не срабатывает... Зато ошибок уже не выдаёт...
Укажите пожалуйста, где я еще обшибся?

if (page) {
	var buttons = document.getElementById("buttonz");
	for (i = 0; i < page.length; i++) {
		try {
			buildings = buttons
					.getElementsByClassName(buildId + page[i][0]);
			buildings[0].firstChild.nextSibling.innerHTML += "<a href=\"#\" onclick=\"(function(pagename, page) { return function() { setQueue('" + pagename + "', " + page[i][0] + "); } })(" + i + ");\" class=\"fastBuild\"><img width=\"22\" height=\"14\" src=\"img/layout/pixel.gif\"></a>";
		} catch (e) {
			GM_log(e.message + "\nOccurred in function \"turnButtons\"");
		}
	}
}

//setQueue('sadasd', 1212);

function setQueue(pagename, id) {
	/*
	 * var array = new Array(id, pagename); GM_setValue('build', array);
	 * alert(GM_getValue('build'));
	 */
	alert('test');
}


Часовой пояс GMT +3, время: 02:43.