[m.....5]

#mikroreklama http://techblog.robertmatusewicz.pl/2015/07/13/array-prototype-find-polyfilling/ #javascript #programowanie

[Marmite]

@matek85: Poprawnym rozwiązaniem jest nie iterować po tablicy używając pętli for..in. Najlepiej w ogóle jej używać.

[regis3]

@Marmite: ah te mikrooptymalizacje. Nic do nich nie mam, ale dopiero jeśli są rzczywistym wąskim gardłem (np programowanie grafiki). W innym wypadku całkowita strata czasu. Szczególnie jeśli w grę wchodzi zarządzanie drzewem.

[Marmite]

@regis3: Ale że jakie mukrooptymalizacje? Optymalizacją w tym wypadku jest juz bardziej użycie właśnie for..in (teoretycznie dużo szybsza niż np. Array.prototype.forEach), więc nie wiem czy pijesz do mojego komentarza czy kodu kolegi.

[regis3]

@Marmite: Bardziej ogólnik, co do tak zwanego poprawnego rozwiązania.

[Marmite]

@regis3: Pętla for..in jest zła, dlatego uważam ze nie powinno się jej używać. Zwłaszcza w przypadku tablic, gdzie boleśnie demaskuje ona ich największą wadę - to, że są obiektami (a raczej to, że indeksy to tak naprawdę własności). Dla programisty to powinno zostać przezroczyste. Dlatego uważam że to powinno być poprawne rozwiązanie w tym wypadku

[regis3]

@Marmite: Co do tablic zgoda.

[regis3]

@Marmite: Odnosiłem się do tego: Najlepiej w ogóle jej używać. Znam tysiąc takich "najlepiej".

[m.....5]

@Marmite: no tak, tylko IMO nie ma sensu poprawiać kodu z zewnętrznym javascripcie, który nie należy do ciebie (i właściwie do którego nie masz fizycznego dostępu). Co do używania "for in" z tablicami, zgadzam się. Mojego code-review by nie przeszło.

[m.....5]

@regis3: jakie optymalizacje?

[Marmite]

@regis3: Z tym że to nie było podyktowane kwetią optymalizacji, tylko tym, że ta pętla jest po prostu schrzaniona. Już prędzej polecam iterowanie Object.getOwnPropertyNames, w 99% przypadków (dane z dupy) i tak nie chcesz iterować po łańcuchu prototypów.

[regis3]

@Marmite: true, lub Object.keys
@matek85: Tutaj my bad. Nie chodziło mi stricte o optymaliacje a raczej a zasady typu: Iteruj zawsze tak, tego nie iteruj nigdy. Nie napisaliście nieprawdy.

[Marmite]

@regis3: Ja tam nigdy niczego nikomu nie zabraniam - jeśli wiesz co robisz, to śmiało. Ale iterowanie for..in po tablicy raczej nie świadczy o wiedzeniu co się robi :P

[Marmite]

tylko IMO nie ma sensu poprawiać kodu z zewnętrznym javascripcie, który nie należy do ciebie (i właściwie do którego nie masz fizycznego dostępu)

@matek85: No no, ostatnio skrobnąłem o tym posta, gdzie podobna rzecz była na trochę większą skalę - polecam: http://www.wykop.pl/wpis/13406109/dopiero-co-ukazala-sie-swieza-jeszcze-ciepla-specy/