en.javascript.info/1-js/4-data-structures/7-array/7-array-find/solution.md
2015-01-11 01:54:57 +03:00

1.8 KiB
Raw Blame History

Возможное решение:

function find(array, value) {

  for(var i=0; i<array.length; i++) {
    if (array[i] == value) return i;
  }
   
  return -1;
}

Однако, в нем ошибка, т.к. сравнение == не различает 0 и false.

Поэтому лучше использовать ===. Кроме того, в современном стандарте JavaScript существует встроенная фунция Array#indexOf, которая работает именно таким образом. Имеет смысл ей воспользоваться, если браузер ее поддерживает.

//+ run
function find(array, value) {
  if (array.indexOf) { // если метод существует
    return array.indexOf(value);
  }

  for(var i=0; i<array.length; i++) {
    if (array[i] === value) return i;
  }
   
  return -1;
}

var arr = ["a", -1, 2, "b"];

var index = find(arr, 2);

alert(index);

... Но еще лучшим вариантом было бы определить find по-разному в зависимости от поддержки браузером метода indexOf:

// создаем пустой массив и проверяем поддерживается ли indexOf
if ( [].indexOf ) { 

  var find = function(array, value) {
    return array.indexOf(value);
  }

} else {
  var find = function(array, value) {
    for(var i=0; i<array.length; i++) {
      if (array[i] === value) return i;
    }
   
    return -1;
  }

}

Этот способ - лучше всего, т.к. не требует при каждом запуске find проверять поддержку indexOf.