Javascript jquery

Question

Witam, jestem początkującym i chciałbym się was spytać czy ten kod można nazwać dobrym. Da się go inaczej napisać ? By robił to samo, ale mniejszą ilością kodu ?

Tutaj jest kod <--- RUN

Answer 1

Ale co to właściwie ma robić?

Sprawdzanie warunku w else if jest zbędne - albo coś wynosi 0 albo 1. Tak samo kod wewnątrz tego warunku - wszystko na wejściu jest widoczne więc show zbędne.

Użycie operatora === zamiast == bezcelowe.

var index = $(this).index();
/* twój
var index = $(".li_class").index(this);
*/

 

Comments

Użycie operatora === zamiast == bezcelowe.

Zawsze powinno się używać === i !==.

http://speakingjs.com/es5/ch01.html#basic_equality_operators

---

powiedzmy ze zamiast hide() damy opacity na 0 zeby "zniknelo", trzeba tworzyć tablicę by sprawdzać czy jest ukryte ? (toggle[index] = 0) czy nie(1) ?