JavaScript, если переменная имеет indexOf элемент в массиве

Question

У меня есть переменная, которая динамически задается на странице. Затем у меня есть список цветов в массиве. Мне нужно увидеть, содержит ли переменная цвета один из элементов массива.

Вот код:

var colorlist = ['Silver', 'Gray', 'Black', 'Red', 'Purple', 'White']; 
var col1 = ""; 
var color1 = 'Titanium Silver'; 
for (var c = 0; c < colorlist.length; c++) 
{ 
    if(color1.indexOf(colorlist[c])) 
    { 
     col1 = colorlist[c]; 
    } 
    else 
    { 

    } 
} 

То, что я бы ожидать, что это возвращение является «Серебро», но он постоянно возвращался последний элемент в массиве. Что я делаю не так?

Answer 1

Измените сравнение с:

if(color1.indexOf(colorlist[c]) > -1) 

-1 возвращается, когда элемент не найден, но -1 является truthy. Единственным номером - 0. Поэтому ваше сравнение не будет работать так, как вы ожидаете.

Посмотрите на эту скрипку, который печатает результат indexOf и truthyness значения: http://jsfiddle.net/R3Xb3/

Обратите внимание, как первый один возвращается true, потому что «Silver» находится в строке (в действительной индекс). Но тогда остальные не найдены, возвращаемый индекс - -1, а результат - true.