Почему Undefined при вырезании элемента из массива?

Question

[nagge101]

Здравствуйте. Познаю прекрасный мир JS и написал свой маленький костыль!

Главная его задача принять 2 аргумента и вырезать элемент из массива!
код:

var arrElement = ['nagge', 'catferq', 'toriluft'];

console.log(arrElement.indexOf('catferq'));

function cuteArr(arr, value) {
    let a = arr.indexOf(value);
    if (a) {
        arr.splice(0, a);
        console.log(arrElement);
    }
}

В консоль при объявлении переменной он выдает undefined, почему, не подскажете?
И как лучше модернизировать код?

Comments

[alex]

выдает undefined, почему, не подскажете?

потому что ваша функция ничего не возвращает, но в вашем случае ей ничего и не нужно возвращать, так что забейте на этот undefined. а вот .splice вы не правильно применяете здесь. смотрите ответ StockholmSyndrome

Answer 1

[Виталий Киренков]

Читайте доку.
https://developer.mozilla.org/ru/docs/Web/JavaScri...

Позиция указывается первым параметром, вторым - кол-во элементов

Comments

[Ярослав Иванов]

вопрос не про это.

Answer 2

[Ярослав Иванов]

Потому что читаем про функции. Если функция ничего не возращает(нет return'a). То возвращается undefined.

https://learn.javascript.ru/function-basics

Значение функции без return и с пустым return
В случае, когда функция не вернула значение или return был без аргументов, считается что она вернула undefined:

P.S. ваш if сработает всегда так как даже если indexOf вернет -1 то он будет преобразован в true. Поменяйте на if(a != -1)

Comments

[Виталий Киренков]

Но и ваше решение не совсем верно. Если индекс элемента 0, то в условие не попадёт)

[Ярослав Иванов]

Виталий Киренков, ну-ка if(0 != -1) это по твоему будет false ?

[Виталий Киренков]

Ярослав Иванов, проглядел, мой косяк.

Answer 3

[Stockholm Syndrome]

function cuteArr(arr, value) {
  let a = arr.indexOf(value); 
  if (a !== -1) {
    arr.splice(a, 1); 
    return arr;
  }
}