Javascript - si la déclaration ne fonctionne pas?

Question

J'essaie de définir une classe à active en fonction de l'URL. J'essaie d'utiliser le code ci-dessous, mais dans tous les cas, il active la classe active pour le second onglet.

var pathname = window.location.pathname; 

if(pathname = '/learn/subsection2') { 
      $("ul.tabs li:eq(1)").addClass("active").show(); //Activate second tab 
      $(".tab_content:eq(1)").show(); //Show second tab content 
    } else { 
      $("ul.tabs li:first").addClass("active").show(); //Activate first tab 
      $(".tab_content:first").show(); //Show first tab content 
    } 

Answer 1

Vous attribuez plutôt que de vérifier l'égalité dans votre instruction if.

if(pathname == '/learn/subsection2') { 
... 

Answer 2

if(pathname = '/learn/subsection2') { // assignment 
if(pathname == '/learn/subsection2') { // test for equality 
if(pathname === '/learn/subsection2') { // test for equality without type coercion 

Answer 3

Vous utilisez = au lieu de ==, une erreur de programmation commune. = est l'affectation, == est la comparaison.

if (pathname == '/lean/subsection2') { // ... 

Lorsque vous utilisez =, il attribue la chaîne /lean/subsection2 à la variable pathnameet comme il évalue la valeur booléenne, ce qui est toujours vrai (il faudrait être faux ou non défini), il prend toujours le bloc de condition positive.

Answer 4

Utilisez == au lieu de = dans l'instruction if.

Answer 5

Vous avez utilisé un = au lieu d'un == ou === dans votre comparaison. C'est la raison pour laquelle de nombreux programmeurs modifient la déclaration pour qu'elle génère une erreur par rapport au code de fonctionnement involontaire ... c'est une erreur très courante!

Voici un exemple de la même instruction if g commutée. Si vous aviez utilisé ce format, mais que vous faisiez la même erreur, vous auriez lancé une erreur qui vous aurait aidé à la localiser plus rapidement:

if('/learn/subsection2' == pathname){ ... }