Где недочет в моей функции :\?

Question

[Extramezz]

Написал функцию, код ниже. Она задает CSS стили элементу через заданное время и убирает их через второе заданное время, а точнее возвращает к старому значению. Работает она в содружестве с Jquerry....
Суть проблемы - как видите, использую setTimeout и если вызывать функцию часто, она работает некорректно - стили задаются, но не убираются =\ Почему?

Синтаксис:

animate(                                                  
{
	свойство:значение,
	свойство:значение,                                         
	....                                                      
	time: [время установки(мс), время существования(мс)] 
},                                      
{                                     
	свойство:значение,              
	свойство:значение,                   
	....                                          
	time: время установки(мс)                             
},                                                        
........                                        
);

//elem - элемент которому задаются стили...объявлен он в другом месте
for(var e in arguments){
	e = arguments[e];
	
	if(e.time){
		var r = (type(e.time) == "array") ? e.time[1] : false,
		t = (type(e.time) == "array") ? e.time[0] : e.time;
		delete e.time;
	}else{
		var t = 0, r = false;
	}

	var o = {};
	for(var i in e) o[i] = $(elem).css(i);
	
	(function(a,b,c,d){
	     setTimeout(function(){
	          $(elem).css(a);
	          if(d) setTimeout(function(){$(elem).css(c);}, d);
	     },b);
	})(e,t,o,r);
}

Пример использования.....

var elem = document.getElementById("test");
animate({"opacity":0, time:[0,1000]});
//Прозрачность мгновенно сменится на 0 и спустя 1с вернется к исходному значению.

Answer 1

[Владлен Грачев]

Прекрасное односимвольное именование переменных, отсутствие рабочего примера — это все очень хорошо.

Не вникал в код и не искал логические ошибки, но рекомендую почитать про асинхронность в JS и коллбэки. Если просто взять и написать два setTimeout, первое и второе заданное время будет отсчитываться от одного и того же момента времени. То есть, если первое время будет больше второго, то сначала выполнится содержимое второго setTimeout.

Если вкратце, то второй setTimeout нужно устанавливать изнутри первого, и никак иначе.

Comments

[Extramezz]

Влад Грачев
1. Обновил пост, сделал запуск таймера "удаления" из таймера "создания", но нужно же при переборе arguments запускать каждый таймер из предыдущего. Что-то не пойму как это сделать :/

2. А что плохого в односимвольном именовании? Это гораздо легче, на мой взгляд: t - время установки 1 таймера (от time), r - время второго (reload), o - объявлена раньше как пустой Object. Память хорошая, вот и сокращаю :)

3. Добавил пример.

[Владлен Грачев]

Extramezz, сначала в цикле сформировать все, что необходимо, а потом уже однократно расставить таймеры.

Про пример я говорил рабочий, например, на jsfiddle. Да не суть уже.