Zmienna data is undefined Opcje

[finito]

W kodzie ze strony oraz w moim występuje ten sam błąd. Chodzi o sytuację, w której w index.html moją publiczną metodę wywołuję w ten sposób: changeColor('swapColors'); Parser krzyczy:"data is undefined" w 18 linii. i nie wiem, czy to jest kwestia tego, że gdzieś ta data musi być zadeklarowana? a może chodzi o to, że data nie jest zainicjowana tzn. nie ma wartości. Tak czy siak, stanąłem przez to w miejscu.

[HTML] pobierz, plaintext 
(function($){
	$.fn.changeColor = function(options)
	{	
 
		function onClick(event){
			//cialo metody prywatnej
			$this = $(this);
			data = $this.data('changeColor');
			data.first = !data.first;
			$this.css("backgroundColor", data.first ? data.colorSecond : data.colorFirst);
			return true;
		}
		//metody publiczne
		var methods = {
			swapColors: function(){
			$this=$(this);
			data = $this.data('changeColor');
			tmp = data.colorSecond;
			data.colorSecond = data.colorFirst;
			data.colorFirst = tmp;
		},
		destroy: function(){
		//destruktor
			$this = $(this);
			$this.unbind("click");
			$this.css("background-color","");
			$this.removeData("changeColor");
		}
	};
	return this.each(function(){
			if(methods[options])
				{
					return methods[options].apply(this,arguments);
				}
				else if (typeof options === 'object' || ! options){
				//parametry pluginu
					var settings = $.extend({
						colorFirst:'Red',
						colorSecond:'Green'
						},options);
 
				$this = $(this);
				data = $this.data('changeColor');
				if (!data)
				{
					data = $this.data('changeColor',settings);
					$this.bind("click",onClick);
				}
				$this.css("background-color",settings.colorFirst);
				data.first=true;
				return;
			}
			else{
                //bład
                $.error('changeColor: no method: '+ options);
            } 
 
 
 
		});
 
	}
})(jQuery);
[HTML] pobierz, plaintext 

Ten post edytował finito 11.05.2014, 13:40:34

[PrinceOfPersia]

nie wiem, czy to jest kwestia tego, że gdzieś ta data musi być zadeklarowana?

Tak, w twoim wypadku najlepiej zadeklarować ją na początku samowywołującej się funkcji:

[JAVASCRIPT] pobierz, plaintext 
(function($){
// WSTAW
var data = // TUTAJ WSTAW WARTOSC POCZATKOWOA ZMIENNEJ, ewentualnie zrob var data; bez znaku rownosci
[JAVASCRIPT] pobierz, plaintext 

generalnie w JavaScript zmienne trzeba gdzieś zadeklarować. Jeśli ich nie zadeklarujesz, to zmienne robią się globalne, albo idzie błąd(jeśli próbujesz odczytac zmienną, której nie ma).

Możliwe też, że mimo deklaracji coś ci nie będzie działać, bo gdzieś coś źle inicjalizujesz.

[finito]

Niestety próbowałem już tego. Próbowałem deklarować data oraz tmp w różnych miejscach i nawet gdy deklaruje na samym początku samowywołującej się funkcji, to i tak wyskakuje ten błąd. To się dzieje też w oryginale, który hulał aż dopisałem w index.html: changeColors('swapColors');. Cała reszta kodu jest okej, więc nie wiem już o co chodzi.

Tak, w twoim wypadku najlepiej zadeklarować ją na początku samowywołującej się funkcji:

[JAVASCRIPT] pobierz, plaintext 
(function($){
// WSTAW
var data = // TUTAJ WSTAW WARTOSC POCZATKOWOA ZMIENNEJ, ewentualnie zrob var data; bez znaku rownosci
[JAVASCRIPT] pobierz, plaintext 

generalnie w JavaScript zmienne trzeba gdzieś zadeklarować. Jeśli ich nie zadeklarujesz, to zmienne robią się globalne, albo idzie błąd(jeśli próbujesz odczytac zmienną, której nie ma).

Możliwe też, że mimo deklaracji coś ci nie będzie działać, bo gdzieś coś źle inicjalizujesz.

[Damonsson]

[JAVASCRIPT] pobierz, plaintext 
tmp = data.colorSecond;
data.colorSecond = data.colorFirst;
data.colorFirst = tmp;
[JAVASCRIPT] pobierz, plaintext 

A możesz powiedzieć, co te 3 linijki mają niby robić?

[finito]

[JAVASCRIPT] pobierz, plaintext 
tmp = data.colorSecond;
data.colorSecond = data.colorFirst;
data.colorFirst = tmp;
[JAVASCRIPT] pobierz, plaintext 

A możesz powiedzieć, co te 3 linijki mają niby robić?

Zamiana kolejnością koloru tła.