OOP i dziedziczenie, Iterator -> InfiniteIterator Opcje

[mrok]

Mam sobie taki oto kodzik:

[HTML] pobierz, plaintext 
var xsApp = window.xsApp || {};
 
/**
 * @namespace xsApp
 * @class util
 */
xsApp.util = {};
 
/**
 * @namespace xsApp.util
 * @class iterator
 */
xsApp.util.iterator = {};
 
/**
 * Ensure prototypal inheritance
 * @namespace xsApp
 * @class util
 * @method createObject
 * @param baseObj Object which properties and method should be inherited
 * @return Object
 */
xsApp.util.createObject = function (baseObj) {
	function F(){}
	F.prototype = baseObj;
	return new F();
};
 
/**
 * @namespace xsApp.util
 * @class iterator
 * @method getIterator
 * @param arr Array used to build iterator
 * @return Iterator
 */
xsApp.util.iterator.getIterator = function (arr) {
	var list = arr,
	index = 0,
	length = list.length;
	if ($.isArray(list)){
		return {
			first : function () {
				return list[0];
			},
			next : function () {
				return list[index++] || null;
			},
			prev : function () {
				return list[--index] || null;
			},
			current : function () {
				return list[index] || null;
			},
			reset : function () {
				return this.seek(0);
			},
			seek : function (newIndex) {
				index = newIndex;
				return this;
			},
			hasNext : function () {
				return index < length;
			},
			hasPrev : function () {
				return index > 0;
			},
			getList : function () {
				return list;
			},
			getCurrentIndex : function () {
				return index;
			},
			getListLength : function () {
				return length;
			}
		};
	}
	return Error ('xsApp.util.iterator.getIterator - parameter have to be an array');
};
 
[HTML] pobierz, plaintext 

potrzebuję też dopisać infiniteIterator, tylko zamiast kopiowac mnustwo kodu chciałbym jakos go wydziedziczyc z poprzedniego iteratora.
Cos jak

[HTML] pobierz, plaintext 
/**
 * @namespace xsApp.util
 * @class iterator
 * @method getInfiniteIterator
 * @param arr Array used to build iterator
 * @return InfiniteIterator
 */
xsApp.util.iterator.getInfiniteIterator = function (arr) {
	var iter = xsApp.util.iterator.getIterator(arr);
	iter = xsApp.util.createObject(iter);
 
	iter.hasNext = function () {
		return true;
	};
	iter.hasPrev = function () {
		return true;
	};
	//next i prev oczywiscie podobnie
	return inter;
};
 
[HTML] pobierz, plaintext 

Ma ktoś pomysł jak to zrobić?
Oczywiście chciałbym uniemożliwość dostęp do zmiennych w iteratorze (cos jak private i byc moze z tym bedzie problem).

[wookieb]

[JAVASCRIPT] pobierz, plaintext 
xsApp.util.iterator.getInfiniteIterator = function (arr) {
	var iter = xsApp.util.iterator.getIterator(arr);
	iter = xsApp.util.createObject(iter);
 
	var length = iter.getListLength();
	var parentNext = iter.next;
	var parentHasNext = iter.hasNext;
	var parentPrev = iter.prev;
	var parentHasPrev = iter.hasPrev;
	iter.hasNext = function () {
		return true;
	};
	iter.hasPrev = function () {
		return true;
	};
 
	iter.next = function() {
		if (!parentHasNext()) {
			this.reset();
		}
		return parentNext();
	};
 
	iter.prev = function() {
		if (!parentHasPrev()) {
			this.seek(length - 1);
		} 
		return parentPrev();
	};
	//next i prev oczywiscie podobnie
	return iter;
};
[JAVASCRIPT] pobierz, plaintext 

Btw, mam wrażenie, iż twoje next() nie powinno zwracać wartości "następnego" elementu (choć i nawet tego nie robi do końca poprawnie)

[zegarek84]

Ma ktoś pomysł jak to zrobić?
Oczywiście chciałbym uniemożliwość dostęp do zmiennych w iteratorze (cos jak private i byc moze z tym bedzie problem).

Pomysłu nie trzeba mieć a jeśli chcesz to zrobić w miarę optymalnie jak na JS i na to jak jest skonstruowany to przy dziedziczeniu z ograniczeniem do zmiennych może być delikatny problem przynajmniej przy prototypowym dziedziczeniu gdyż jeśli dziedziczysz po jakimś "zwykłym" obiekcie do prototypu posiadającym referencję do zmiennych prywatnych deklarowanych przez var to w kontekście obiektów potomnych będą to zmienne statyczne... co do zmiennych lokalnych to głównie zachować "notację" z deklaracją zmiennnych - przynajmniej edytor będzie pokazywał zmienne prywatne mimo zmiennych publicznych - ale jeśli będziesz chciał to wszystko domknąć to przecież całą aplikację w js możesz zamknąć w anonimowej funkcji ;]

w między czasie temat dałem sobie ofline coby go przeczytać a i jak odpisuję nie jestem świeży jednak uważam, że najlepiej zrobię jak zamieszczę przykładowy kod po części skopiowany od Ciebie z próba przedstawienia dziedziczenia w js gdyż raczej mało osób zna logikę js [fakt nie programuję zawodowo nawet też w node.js ale raczej akurat js mam dopracowane...]

kodzik który w wcześniej naskrobałem bez neta dla ogólnej idei jak nie przepisywać kodów - btw. moje IDE pokazuje, że w JS jest dostępny iterator także na ie od wersji 7 - czyżby wynajdować koło na nowo?? - fakt nie sprawdzałem jakie metody gdzyż to zrobiłem offline...

[JAVASCRIPT] pobierz, plaintext 
// ZMIENNE PRYWATNE PRZY DZIEDZICZENIU PROTOTYPOWYM DLA OBIEKTÓW POTOMNYCH
// GDZIE MA DZIAŁAĆ instanceof BĘDĄ SIĘ ZACHOWYWAŁY JAK ZMIENNE
// STATYCZNE - SKOŻYSTAJ Z UMOWNEJ NOTACJI DLA ZMIENNYCH PRYWATNYCH
// Z PODKREŚLENIEM - DODATKOWO JESZCZE MOŻNA "ZAMKNĄĆ" TE OBIEKTY W JAKIEJŚ
// FUNKCJI BY NIE BYŁY WIDOCZNE NA ZEWNĄTRZ
 
// by mieć obiekt bez konstruktora ;] - Twoja funkcja
xsApp.util.createObject = function (baseObj) {
  function F(){}
  F.prototype = baseObj;
  return new F();
};
 
/**
 *
 * @constructor
 */
function Iterator(aArray){
  if(aArray instanceof Array){
    this._aList = aArray;
  } else {
    this._aList = Array.prototype.slice.call(aArray);
  //throw new Error('Argument nie jest tablicą!!!'); // lub jakiś trace na konsole błędów
  }
  this._iLength = this._aList.length;
  aArray = null;
}
 
Iterator.prototype = {
  _aList: null, // array jest przekazywane przez referencję więc trzeba przypisywać w konstruktorze
  _iIndex: 0,
  _iLength:null,
  first : function () {
    return this._aList[0] || null;
  },
  next : function () {
    ++this._iIndex;
    return this.current();
  },
  prev : function () {
    --this._iIndex;
    return this.current();
  },
  current : function () {
    return this._aList[this._iIndex] || null;
  },
  reset : function () {
    return this.seek(0);
  },
  seek : function (iNewIndex) {
    this._iIndex = iNewIndex;
    iNewIndex = null;
    return this;
  },
  hasNext : function () {
    return this._iIndex<this._iLength;
  },
  hasPrev : function () {
    return this._iIndex>0;
  },
  getList : function () {
    return this._aList;
  },
  getCurrentIndex : function () {
    return this._iIndex;
  },
  getListLength : function () {
    return this._iLength;
  }
};
/**
 *
 * @constructor
 * @extends {Iterator}
 */
function InfiniteIterator(aArray){
  Iterator.call(this, aArray); // NICZYM parent::__construktor - POCZYTAJ JESZCZE O apply
  aArray = null;
}
 
// dziedziczenie bez wykonywania konstruktora z Iteratora - masz do tego metodę swoją ;]
// dzięki takiej konstrukcji możesz nadpisywać metody w obiekcie potomnym nie zmieniając metod u rodzica ;]
 
InfiniteIterator.prototype = xsApp.util.createObject(Iterator.prototype);
 
//nadpisywanie metod - SAMA LOGIKA DZIEDZICZENIA GDYŻ NIE BARDZO WIEM CO CHCESZ OSIĄGNĄĆ ;]
 
InfiniteIterator.prototype.hasNext = function () {
  return true;
};
 
InfiniteIterator.prototype.hasPrev = function () {
  return true;
};
 
// itd...
// dla obiektów potomnych utworzonych z new InfiniteIterator(array) zadziała także instanceof Iterator
 
// WIADOMO, ŻE KONSTRUKTORY OBIEKTÓW MOŻESZ PODPIĄĆ GDZIEŚ W SWOJEJ PRZESTRZENI NAZW NP. GDZIEŚ W OBIEKTACH STATYCZNYCH {}
 
// TWORZENIE OBIEKTÓW
new Iterator(array);
new InfiniteIterator(array);
[JAVASCRIPT] pobierz, plaintext 

sorki za błędy jeśli wystąpiły ale jestem wstawiony...

Ten post edytował zegarek84 1.05.2011, 10:08:31

[mrok]

Dzieki za pomoc.

Btw, mam wrażenie, iż twoje next() nie powinno zwracać wartości "następnego" elementu (choć i nawet tego nie robi do końca poprawnie)

next() nie zwraca nastepnego elementy tylko aktualny i inkrementuje wartość index. Wiem że nie do końca oznacza to "next" i tez sie nad tym glowilem, ale siegnalem do literatury i tam stosują identyczną konstrujcę dla iteratora.

Jesli masz inne zastrzezenia to daj znac.


@zegarek84
Mam wrazenie ze w temacie JS to jesteś poziom wyżej odemnie - jak zrozumiem Twojego posta i pobawie sie Twoim kodem to dam znac Dzis niedziela wiec sie obijam