Не отрабатывает проверка.

[Krisha]

Есть кнопка у которой есть следующий код:

if ( myFunction() )
{
trace ("ok");
} else {
trace ("oops");
}

код функции:


function getFailedLessons (ChapterID)
{
trace("getFailedLessons:");

query = new LoadVars();
query.action = "getFailedLessons";
query.ChapterID = ChapterID;

query.sendAndLoad(_root.scriptName, query, 'POST');

query.onLoad = function()
{
if (this.FailedLessons != undefined)
{
return false;
} else {
return true;
}
};
}

Как сделать так, чтобы пока функция не отработала проверка в кнопке не срабатывала, а то получается так, что данные еще не получены, а проверка уже срабатывает и выводит oops

[Nox Noctis]

будь проще :)

убрать проверку и всё что там происходит с кнопки, и поместить это всё прямо в онЛоад.

а на кнопке написать только вызов getFailedLessons (ChapterID);

[Krisha]

Можно конечно и так сделать, но тогда теряется весь смысл, функция не должна знать ничего о других, её задача получить данные, обработать их вернуть результат. А приложение уже должно в зависимости от результата реагировать...

У меня таких функций несколько предполагается, не могут же все они быть друг в друге ?

[Nox Noctis]

Оригинал написал(а) Krisha
её задача получить данные, обработать их вернуть результат. А приложение уже должно в зависимости от результата реагировать...

смысл тогда не теряется - а как раз таки появляется :)

ты уже нарушил принцип инкапсуляции, как только стал пользоваться в функции внешней переменной: query = new LoadVars(); - так какая разница ? :)

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

если хочешь чтобы всё было чисто - делай так.
но надо ли оно тебе ? :)

loadData = function (source, parser) {
	var loader = new LoadVars();
	loader.parser = parser;
	loader.onData = function(result) {
		this.parser(result);		
	};
	loader.load(source);
};

если ты грузишь переменные в виде строки запроса, и хочешь воспользоваться автоматическим раскодированием этого дела в переменные, как это делает объект LoadVars - то ничто не мешает вызвать метод раскодирования явно :)

LoadVars.decode(query_string)