Zagnieżdżenie pętli Opcje

[Krisu]

[PHP] pobierz, plaintext 
handlers.each(function(index){
 
for (i = 2; i <= items.length; i++) {
 
	this.cls = (index === 0 ) ? 'tab first-tab active' : 'tab tab'+ [i] + ''
 
}	
 
title += '<li class="' + this.cls + '">'
title += $(this).html()
title += '</li>'
 
});
[PHP] pobierz, plaintext 

Mam taką oto funkcję, docelowo chodzi mi o to by kolejne listy li przyjmowały w nazwie klasy numery kolejne. W chwili gdy pętlę zrobię w ten sposób zwraca mi tylko końcową wartość pętli. Niestety na JS za bardzo się nie znam i nie mam pomysłu jak zagnieździć pętle by dobrze zliczała.

[trueblue]

Zwraca końcową, bo umieściłeś fragment kodu poza pętlą.

Jeśli znasz się na PHP, to tam działanie pętli jest analogiczne.

[Krisu]

[PHP] pobierz, plaintext 
handlers.each(function(index){
 
title += '<li class="' + for (i = 2; i <= items.length; i++) { this.cls = (index === 0 ) ? 'tab first-tab active' : 'tab tab'+ [i] + '' } + '">'
title += $(this).html()
title += '</li>'
 
});
[PHP] pobierz, plaintext 

Wiem o tym, najchętniej zbudowałbym to w ten sposób jak powyżej i pewnie byłoby po problemie, ale nie wiem jak w JS składniowo poprawnie to zapisać...

[trueblue]

A nie myślałeś, aby wstawić te 3 linijki przed zamknięciem klamry pętli?

[Krisu]

Wówczas rzeczywiście działa, jednak funkcja także powiela same zakładki i zamiast listy o nazwie artykuł 1, artykuł 2, artykuł 3, mam listy artykuł 1, artykuł 1, artykuł 1, artykuł 2, artykuł 2, artykuł 2 itd.

Ten post edytował Krisu 23.08.2016, 09:50:39

[trueblue]

Co masz w handlers, co masz w items?
Sam each to już struktura pętli, a w niej tworzysz drugą.

Zobacz ten przykład:
https://jsfiddle.net/waawL47y/

Nie wiem czy jest poprawny, bo Ty tworząc pętlę w jednej iterujesz chyba po wszystkich elementach <li> (zakładam, że w handlers są wszystkie), a w drugiej iterujesz od 3-ego elementu (for).

Ten post edytował trueblue 23.08.2016, 10:01:42

[Krisu]

Dziękuję bardzo za pomoc Wystarczyło zrezygnować z fora i skorzystać ze zmiennej index

[PHP] pobierz, plaintext 
 
handlers.each(function(index){
 
this.cls = (index === 0 ) ? 'tab first-tab active' : 'tab tab'+ [index] + ''
 
title += '<li class="' + this.cls + '">'
title += $(this).html()
title += '</li>'	
 
});
[PHP] pobierz, plaintext 

Ten post edytował Krisu 23.08.2016, 10:14:31

[trueblue]

A co Ty robisz potem z title?

[Krisu]

[PHP] pobierz, plaintext 
$(tabs_buttons).find('.tabs_container').append( title )
$(items).find('.title').remove()
[PHP] pobierz, plaintext 

[trueblue]

Nie wiem co masz w tabs_buttons oraz w items.
Jednym zdaniem, jeśli dodajesz klasy do istniejących elementów (ew. usuwasz inne klasy z tych elementów), to robisz to na okrętkę.

[Krisu]

Jeżeli chcesz dokładnie wiedzieć do czego było mi to potrzebne napiszę na PW, ale w 100% mi pomogłeś i osiągnąłem zamierzony efekt. Po prostu nie znam się na JS, wolę pisać w PHP.