Почему return не срабатывает в функции?

Question

[junior81]

function multiply(){
  let calc = 1;
  for(let i = 0, len = arguments.length; i < len; i++){
     if(arguments === 0 ) return 0;
    calc = calc * arguments[i];
  }

  return calc
}

Answer 1

[myspace]

Вы делаете return в цикле, просто прерывая цикл, а не останавливая функцию. Вынесите это условие за цикл

function multiply(){

  if( arguments.length === 0 ) {
     return 0; 
  }

  let calc = 1;

  for(let i = 0, len = arguments.length; i < len; i++){
    calc = calc * arguments[i];
  }

  return calc
}

Comments

[junior81]

Большое спасибо.

Answer 2

[Сергей Соколов]

Исправьте эту строчку:
if(arguments[i] === 0 ) return 0;
Надо сравнивать с нулём не весь массив аргументов, а значение очередного аргумента.

Upd.

function multiply() {
	let len = arguments.length;
	if( 0 === len) return 0;
	
	for (let i = 0, calc = 1; i < len; i++) {
		if (arguments[i] === 0) return 0;
		calc = calc * arguments[i];
	}
	return calc;
}

Comments

[junior81]

Дело в том что при отсутствии аргумента функция возвращает 1, а должна прерваться и вывести 0.

Answer 3

[Islam Ibakaev]

просто убери 4-ю строчку