IF statement doenst work

-4

У меня есть эта функция, но я не знаю, будет ли «переключатель» лучше?IF statement doenst work

(function ($) { 
    var doc = $.urlParam('doc'); 
    if (doc) { 
     if (doc = 'new') { 
      alert(doc); 
     } 
     if (doc = 'new2') { 
      alert(doc); 
     } 
     if (doc = 'new3') { 
      alert(doc); 
     } 
    } 
})(jQuery);

Предупреждение должно появляться только в том случае, если параметр в URL-адресе является правильным, как в инструкции IF.

Полный код можно найти здесь: https://jsfiddle.net/yc5f9ct7/4/

источник

2015-05-12 fr3d

Во-первых, используйте '==' для проверки равенства. Во-вторых, учитывая, что все три условия выполняют одну и ту же логику, в чем смысл выражения 'if' в первую очередь? –

Используйте '==' или '===' вместо '=', помните '=' - оператор присваивания – Satpal

Также обратите внимание: он * выглядит *, как ваш код хочет обернуть эту последнюю часть в DOM-готовом обработчике, но вы обертывая его в IIFE. Измените оболочку только на это: 'jQuery (function ($) {YOUR CODE});' Этот ярлык обеспечивает DOM-готовый и локально ограниченный '$' –

Как все отметили, вы используете назначение = вместо от сравнения == или точное сравнение ===.

Это в стороне, если вы тестируете одну переменную для нескольких значений, и намерены иметь различный код на каждый, то коммутатор более логично:

var doc = $.urlParam('doc'); 
switch (doc){ 
    case 'new': 
     alert(doc); 
     break; 
    case 'new2': 
     alert(doc); 
     break; 
    case 'new3': 
     alert(doc); 
     break; 
}

Также обратите внимание: это выглядит как код хочет также обернуть эту последнюю часть в обработчике DOM, но вместо этого вы оберните его в IIFE. Измените обертку на jQuery(function($){ YOUR CODE HERE });

например.

jQuery(function($){ 
    var doc = $.urlParam('doc'); 
    switch (doc){ 
     case 'new': 
      alert(doc); 
      break; 
     case 'new2': 
      alert(doc); 
      break; 
     case 'new3': 
      alert(doc); 
      break; 
    } 
});

Этот удобный ярлык для DOM готов, обеспечивает локальную область $.

источник

2015-05-12 10:00:08

Вы должны использовать == вместо = проверить равенство в случае утверждения:

(function ($) { 

var doc = $.urlParam('doc'); 


if (doc) { 
     if (doc == 'new') { 
      alert(doc); 

     } 
     if (doc == 'new2') { 
      alert(doc); 

     } 
     if (doc == 'new3') { 
      alert(doc); 

     } 
    } 

})(jQuery);

источник

2015-05-12 09:47:38

Спасибо, вчера было немного поздно :) – fr3d

Это выглядит с JSFiddle, как будто они тоже имели * DOM готовый обработчик *, но вместо этого используют IIFE. –

Как уже упоминалось, двойные РАВНО требуется, но вы можете также сократить это:

(function ($) { 
    var doc = $.urlParam('doc'); 
    if (doc) { 
     if (doc == 'new' || doc == 'new2' || doc == 'new3') { 
      alert(doc); 
     } 
    } 
})(jQuery);

источник

2015-05-12 09:50:16 Paddy

Может быть укорочен до 'if (doc && (doc == 'new' || doc == 'new2' || doc == 'new3')) {' ... –

IF statement doenst work

ответ

Смежные вопросы