Почему функция возвращает undefined?

Question

[Alex_mersvg]

let arr = [1,1,1,1,1,2,3,4,1,2,3,4,2,3];
function nam(val){
    let array= val;
    let res = [];

    for(let i = 0; i < array.length; i++){
        if(res.includes(array[i]) ) return
        res.push(array[i])
    }

    return res;
}
nam(arr);

как ее написать правильно я знаю, но почему именно эта запись возвращает undefined? Идет же проверка, "если массив содержит елемент Х, то перейти к след. итерации. Но по логике если условие дает false, то цикл должен пойти дальше и добавить элемент в массив?"

Comments

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

if(res.includes(array[i]) ) return

return завершает функцию. Поменяйте на break

[Alex_mersvg]

Ярослав Иванов, Благодарю за ответ. Но и break здесь не поможет. Поможет continue. Совсем вылетело из головы что return завершает функцию.

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

Alex_mersvg, да точно, continue.

[szQocks]

проще решить задачу - console.log([...(new Set([1,1,1,1,1,2,3,4,1,2,3,4,2,3]))])

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

szQocks, и зачем это? Автор учит алгоритмы. Причём тут set?

[szQocks]

Ярослав Иванов, интересно и где же ты тут увидел алгоритм ?

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

szQocks, окей гугл. Что такое алгоритмы.

[szQocks]

Ярослав Иванов, если верить гуглу, тогда чем тебе моя запись не алгоритм?

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

szQocks, ну если для вас вызвать одну функцию для поиска уникальных значений в массиве это есть алгоритм в общем понимании то у меня для вас плохие новости.

Answer 1

[dollar]

Потому что перевод строки после return движок JS воспринимает как границу между операторами. Как будто там точка с запятой.
Если записать в одну строчку, то не будет undefined:

if(res.includes(array[i]) ) return res.push(array[i])

Ну а так, в целом, return здесь вовсе не нужен.
Ведь return - это выход из функции.
А вам нужен выход из текущей итерации цикла - это continue.