Js Jquery - vysuvanie/zasuvanie

Myslíš něco takového?

Kvůli tobě ne. :–) Už je to staršího data.

Funkční to sice je, ale programátorksy to není dobře.

1. Vytváření stálé anonymní funkce pro kliknutí
2. V případě, že rozevřený má být jen jeden blok, pak stačí ukládat jen ten otevřený a neprojíždět všechny.
3. Místo nastavování celé třídy se hodí funkce jako addClass a removeClass

Ad 1) A čemu to vadí?
Ad 2) Nešlo by výchozí blok rozbalit přidáním třídy do HTML. Musel by se tedy předat parametr, který blok má JS rozbalit => musel by se tedy vymyslet další identifikátor pro najití a navíc by to způsobilo hrozbu probliknutí. Nejprve by byl blok vidět, po přidání třídy js by zmizel a po initu by se zase zobrazil.
Ad 3) Nemyslím si, že by to něco zlepšilo.

Str4wberry:
„Ad 1) A čemu to vadí?“
Možno myslel priradenie v cykle objekt.onclick=názovFunkcie; lebo objekt.onclick=function(){...}; vyrába nový objekt funkcie. Ušetrila by sa tým nejaká pamäť.

Str4wberry:
1 - jak napsal _es, setkával jsem se tady s tím hodně, kdy mi to taky bylo vytýkáno (a od té doby se to tak snažím dělat) a zkrátka dobrým způsobem je, že se ta funkce jen předá a nevytvoří se znovu
2 - proč by nedal, ten který má být ve výchozím stavu rozevřen, bude mít třídu "show" (jinak nevím jestli jsi se někdy setkal s problikáváním při podobných řešení, já ne, pouze v případě, že se kód vykonává až v document ready). Navíc to bude efektivnější, protože se pole objektů bude procházet jen jednou a ne při každém kliknutí. Může se uchovávat pole rozevřených elementů (pokud jich má být více)
3 - zlepšílo, protože pokud se k tomu divu nastaví nějaká třída (úplně jiná, nesouvisející s touto funkcí), tak se bude přepisovat a bude to vést k neočekávaným stavům. Samozřejmě že tady žádný problém není, ale proč to nedělat tak, aby se ani v budoucnu neobjevil žádný problém

hugo123:
Zakomentuj řádek volání funkce clearAll();

Nějak takto:
http://teststranek.kvalitne.cz/rozbalovaci-menu/

Reakce na joa:
Ad 1) Podle mě je zase dobrým způsobem psát kód, aby byl dobře čitelný. Pro mě je nejčitelnější, jak jsem ho zapsal a otázka výkonu je v obou případech naprosto marginální.
Ad 2) Zbytečně složité. A výkonově opět zanedbatelné. Jinak ano, pokud bude mezi obsahem, co se má skrývat, a JS kódem, který ten obsah bude skrývat, něco zdržujícího, což může být cokoliv, tak se probliknutí projeví.
Ad 3) Zbytečně složité. A bude žrát hromady výkonu. :–)

Str4wberry:
1 - možná, ale špatným způsobům se člověk hůř odnaučuje
2 - ok, osobně problikávání řeším voláním kódu hned za elementem co kde chci udělat
3 - v tomhle případě se to zesložití, ale odpadne některými takzvaný "wtf faktor" :-)

Já bych to napsal nějak takto. Tak jestli se někomu na tom něco nelíbí, sem s připomínkami :)

joe:
Mě se to třeba nelíbí. Je to roztahané a dlouhé. Já bych zas použil jQuery a napsal bych to ve 3 řádcích. Též neshledávám na Str4wberry-ho ukázce nic závadného a všechny tebou zmíněné výkonnostní problémy jsou absolutně zanedbatelné.

PS. V tvém případě bych se nějak zabavil prací či jiným koníčkem než neustálým sjížděním diskusního fóra a hledáním problémů tam kde nejsou ;-).

Kcko:
Já bych ale řekl, že tam problém je a myslím, že jsem to dobře argumentoval ;-) Pokud něco dělám, tak myslím na použitelnost a efektivnost. Ten kód samozřejmě může být jiný (i lepší), jen jsem přepisoval to, co Str4wberry dal k dispozici, tak jak mi to přišlo pod ruku. Samozřejmě by to mohlo vypadat taky takhle:


Ale i tento kód má své nevýhody.

„všechny tebou zmíněné výkonnostní problémy jsou absolutně zanedbatelné“
Až budeš používat tento způsob na jiný velikostní rozměr dat, tak se ozvi ;-)

„PS. V tvém případě bych se nějak zabavil prací či jiným koníčkem než neustálým sjížděním diskusního fóra a hledáním problémů tam kde nejsou ;-).“
Prosím o důkaz.

„Já bych zas použil jQuery a napsal bych to ve 3 řádcích“
A že jsi to nenapsal.

hugo123:
Kód je úplně stejný, jen se v tomto případě používá jQuery. Celý kód JavaScriptu


Nebo živá ukázka na jsFiddle

Je, ale před uživateli bez JavaScriptu bude obsah navždy schován.

Reaguji na huga123:
Pokud je to tak triviální jako v tomto případě, tak je tvá otázka snad úplně zbytečná…

„Nie je to krajsie?“
Můžeš toho smazat ještě mnohem víc a skončíš u jednoho krátkého krásného nefunkčního řádku :-)
Mně se mnohem víc lídí kódy v [#2] a [#13]. Mají menší riziko, že se rozbijí v nových prohlížečích.


Reaguji na joa:
„$("html").addClass("js");“
Proč dáváš tento řádek do $(document).ready?

Chamurappi:
Má být mimo (kvůli problikávání) - moje chyba, obvykle to řeším už v HTML (s nastavenou třídou nojs u tagu html) jako prvním skriptem
nebo pro detailisty

„Mají menší riziko, že se rozbijí v nových prohlížečích.“
Sdělil bys nám to riziko, prosím?

Ahoj, nebude náhodou stačit například toto? :)