Код будет выполнять как операторы if, так и else. Javascript

Question

У меня есть массив с именем used.I есть функция, которая добавляет код в массив, но проверяет, был ли добавлен код снова. Если он был добавлен, он должен сказать «False» else True проблема в том, что он скажет как ложные, так и истинные. Вот мой код:

var used=[];//use that to check if the user has added the code to favourites again 

var x='',y='',blur='',r="",g="",b="",op="",color="rgb(0,0,0)",shadow=''; //create the variables that will be used 
function update(){ 
    x=$("#x").val(); 
    y=$("#y").val(); 
    op=$("#o").val(); 
    r=$("#red").val(); 
    g=$("#green").val(); 
    b=$("#blue").val(); 
    blur=$("#blur").val(); 
    shadow = x+"px "+y+"px "; 


} 
$(document).ready(function(){ 
$("#addC").click(function(){ 
    if(used.indexOf(shadow) != -1){ 
    alert("false"); 
    } 
    else{ 
    used.push(shadow); 
    alert("True"); 

    } 
}); 
    $('.shadow-slider').change(function(){ 

    update(); 

    if (op != 1){ 
    if(op.length > 3){op=op.substring(0, 3);} 
    color="rgba("+r+","+g+","+b+","+op+")"; 
    } 
    else{ 
    color="rgb("+r+","+g+","+b+")"; 
    } 
    if (blur != 0){shadow += blur+"px ";} 
    shadow += color; 
    $('#object').css("text-shadow",shadow); 
    $('#code-output').html("text-shadow:"+shadow+";"); 

    }); 



}); 

Тень представляет собой глобальную переменную, которая изменяется в зависимости от некоторых ползунков моих page.Any идей?

Answer 1

Я имею в виду, вы связали обработчик щелчка дважды, так что в первый раз он не найден, и вы получите второй случай (и добавить его), но второй раз, когда он нашел (добавил в первый раз) и показывает первый случай.