Поиск наименьшего делителя, почему мой код не работает?

Question

[mirikkingg]

Задача найти наименьший делитель. Мне не нужен готовый код, я хочу понять, что у меня не так?

let num = 4; // задается функцией .
const smallestDivisor = (num) => {
  let divider = 1;
if (num % 2 === 0 && num > 0) {
  return num;
} else if (num <= 0) {
  return NaN;
} else {
  while (num % divider === 1) {
    divider = divider + 1;
    return divider;
  }
}
}

Почему код выведет undefined

Comments

[Lynn «Кофеман»]

Всё

Answer 1

[mirikkingg]

const smallestDivisor = (num) => {
let divider = 2;
if (num % 2 === 0 && num > 0) {
return 2;
} else if (num <= 0) {
return NaN;
} else {
if (num === 1) { return num; }
while (num % divider !== 0) {
divider++;
}
return divider;
}
}
Все ломалось из-за единицы.

Answer 2

[Володимир Паламар]

Вынесите return из while

let num = 4; // задается функцией .
const smallestDivisor = (num) => {
let divider = 1;
if (num % 2 === 0 && num > 0) {
//return num;
return 2
} else if (num <= 0) {
return NaN;
} else {
while (num % divider === 1) {
divider = divider + 1;
//return divider;
}
return divider;
}
}
//console.log(smallestDivisor(num)) //=>4

Comments

[mirikkingg]

да ошибка, но код выведет 4, когда должен вывести 2.
while (num % divider === 1) {
divider = divider + 1;
}
Он еще 2 раза выполнит цикл, почему? Ведь остаток от деления 4 % 2 = 0, а цикл не завершился.

[galaxy]

mirikkingg, потому что хрень, а не код. Возьми на листочке разбери, что он делает на нескольких примерах.

if (num % 2 === 0 && num > 0) { - это что за условие?

while (num % divider === 1) { это что еще? ты же в курсе, что кроме 0 и 1 бывают еще и другие остатки?

[Володимир Паламар]

mirikkingg,

if (num % 2 === 0 && num > 0) {
//return num;
return 2

[mirikkingg]

galaxy, if (num % 2 === 0 && num > 0) // если число без остатка делится на 2 то 2 и есть наименьший делитель, ну и число соответственно должно быть больше 0.
Ну второе условие глупость, согласен.

[galaxy]

mirikkingg, так там на строчку ниже было return num;
Зачем вообще отдельно обрабатывать 2?

} else if (num <= 0) { вдруг вспомнил, что еще и отрицательные числа бывают... Ну ок, а если на вход 3.14 придет?

[mirikkingg]

galaxy, ну это тесты, таких чисел там нет по этому и не предусматривал.
Все пофиксил, вопрос лишь в том, что при 121 // => 1, а нужно 11.
Отдельно 2 не обрабатывается, я ошибся, вместо num, 2 // как наименьший делитель.

[galaxy]

mirikkingg,

const smallestDivisor = (num = 121) => {
	let divider = 2;
	if (num <= 0) {
		return NaN;
	} else {
		while (divider * divider <= num) {
			if (num % divider === 0)
				return divider;
			divider++;
		}
		return num;
	}
}

Answer 3

[evgeniy-kolmak]

const smallestDivisor = (num) => {
  if (num % 2 === 0 && num > 0) {
    return 2;
  } else if (num <= 0) {
    return NaN;
  } else if (num === 1) {
    return num;
  }
  // Проверки на корректный ввод данных для значения "num"
  const devider = (count = 2) => {
    if (num % count !== 0) { /* Если "Num" делится на значение "devider" с остатком (остаток от деления не равен 0,  к значению делителя прибовляем (+1) */
      return devider(count + 1);
    } else {
      return count; /* Дефолтное значение (если остаток от деления равен 0, count остается прежним) */
    }
  }
  return devider();
}