Как реализовать функцию inArray?

Question

[Ella Fitzgerald]

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

Требуется реализовать функцию, примеры вызова которой представлены ниже:

inArray(15, [1, 10, 145, 8]) === false;
[23, 674, 4, 12].inArray(4) === true;

Я так понимаю, функция inArray проверяет наличие элемента в выбранном массиве и возвращает true в случае успеха, иначе – false. Как ее написать, чтобы срабатывал первый вызов из примера, мне понятно. Вот так например:

function inArray(value, array) {
  for (var i = 0; i < array.length; i++) {
    if (value === array[i]) {
      return true;
    } 
  }
  return false;
}

inArray(15, [1, 10, 145, 8]); // false

Но как мне ее нужно переписать, чтобы работал и второй вариант вызова?

Comments

[OVK2015]

она уже есть
https://developer.mozilla.org/ru/docs/Web/JavaScri...

[Ella Fitzgerald]

спасибо, почитаю)

[OVK2015]

Елизавета Озерова: Не за что, вроде

Answer 1

[DTX]

function inArray(value, array) {
  if (!array)
    array = this;
  for (var i = 0; i < array.length; i++) {
    if (value === array[i]) {
      return true;
    } 
  }
  return false;
}
Array.prototype.inArray = inArray;
console.log([1, 10, 145, 8].inArray(10));
console.log(inArray(100, [1, 10, 145, 8]));

Comments

[Ella Fitzgerald]

спасибо, работает!)

[Дмитрий Беляев]

DTX изменение аргументов функции ведет к плохо поддерживаемому коду и отключению оптимизатора

Answer 2

[Алексей Уколов]

Array.prototype.inArray = function (item) {return this.indexOf(item) > -1}

Только, патчить глобальные объекты - плохая практика, не стоит этого делать за пределами образовательных проектов.
Ну и вашу функцию стоит переписать на использование indexOf(), ни к чему изобретать велосипед.

Comments

[DTX]

А как лучше сделать, чтобы одна функция функция была и там и там? Можно вторым аргументом массив передавать и писать
if (!array)
array = this;
Или есть вариант поудобнее сделать?

И непонятно, почему это плоха практика. Что плохого в том, что я одну функцию допишу?
Вот как тут например:
stackoverflow.com/questions/149055/how-can-i-forma...

Number.prototype.formatMoney = function(c, d, t){
var n = this,
c = isNaN(c = Math.abs(c)) ? 2 : c,
d = d == undefined ? "." : d,
t = t == undefined ? "," : t,
s = n < 0 ? "-" : "",
i = parseInt(n = Math.abs(+n || 0).toFixed(c)) + "",
j = (j = i.length) > 3 ? j % 3 : 0;
return s + (j ? i.substr(0, j) + t : "") + i.substr(j).replace(/(\d{3})(?=\d)/g, "$1" + t) + (c ? d + Math.abs(n - i).toFixed(c).slice(2) : "");
};

[Ivan Bogachev]

Если все это в образовательных целях, то получится что-то такое (никогда так не делайте в реальности):

function inArray(value, array) {
    return (array || this).indexOf(value) > -1;
}
Array.prototype.inArray = inArray;

[Ella Fitzgerald]

большое спасибо за помощь!

Answer 3

[yogurt1]

function inArray(key, arr) {
  return arr.map(i => i === key).length > 0
}

Answer 4

[nikoyar]

Для ES6

function inArray(key, arr) {
  return arr.some((id) => id == key)
}

Можно и просто arr.includes(key), но тут внутри идет строгое сравнение === и можно поиметь проблемы с разными типами данных