JS код возвращает Undefined, почему?

Question

[Haykeloff]

Хотел написать код который находит минимальный делитель числа. Вроде написал, да вот только последний блок кода возвращает Undefined, и я так и не смог понять почему. Вот кстати код:

const smallestDivisor = (num) => {
let n = 3;
if (num === 1) {
    return num;
} else if ( num % 2 === 0) {
    return 2;
} else if (num % 3 === 0) {
    return 3;
} else {
    while(n > Math.pow(num, 0.5)){
        n = n + 2;
        if (num % n === 0) {
        return n;
        }
     }
}
};

Answer 1

[Виталий]

3 > Math.pow(11, 0.5)
собственно вот здесь ответ, твой алгоритм не корректен, у тебя в блоке элз нет входа в луп, и соответственно выход из функции без явного return что равно return undefined

Comments

[Haykeloff]

Прошу прощения, исправил. Однако все ровно возвращает undefined

[Ivan Bogachev]

Haykeloff, У вас функция не всегда возвращает значение. Подумайте, что будет, если число простое? Выполнение программы дойдет до последнего else, цикл покрутится немного, но делитель все еще не будет найден. Цикл заканчивается и... и все. Функция закончилась, ничего не вернула.

P.S.: имхо Math.sqrt(num) понятнее глазу, чем Math.pow(num, 0.5)

Answer 2

[Somewhere Intech]

const smallestDivisor = (num) => {
	if ( num % 2 === 0) return 2;
	let m = Math.abs(num), max = m > 9 ? Math.sqrt(m) : m;
	for(n=3; n < max; n+=2) if (m % n === 0) return n;
	return m;
}