Check if number is not integer in recursion js [duplicate]

Question

I have a recursion function which calculates factorial for a given number.

function findFactorial(num) {
  if (num < 1 || !num.isInteger) {
    return "error";
  } else {
    return num * findFactorial(num - 1);
  }
}

console.log(findFactorial(7));

I want to check if number is less then 1 OR is not integer - and return "error" for it. Code I tried above isn't working and giving me "error" every time function is called.

What is isInteger?

Dave Newton
– Dave Newton

2021-04-16 13:30:55 +00:00
Commented Apr 16, 2021 at 13:30 — Dave Newton
– Dave Newton, Commented Apr 16, 2021 at 13:30
Almost got it by accident, it's Number.isInteger.

ASDFGerte
– ASDFGerte

2021-04-16 13:34:08 +00:00
Commented Apr 16, 2021 at 13:34 — ASDFGerte
– ASDFGerte, Commented Apr 16, 2021 at 13:34

Jozott · Accepted Answer · 2021-04-16 13:42:59Z

1

You have three issues in your code.

First your code will always return "error", since you will always come to the point where num == 0.
Second you have no termination condition (except for the wrong return "error"), so the recursion will never return a valid value.
And last but not least, you try to call a function isInteger() but this function does not exist for the num object.

Here is the valid code:

function findFactorial(num) {
  if (isNaN(num) || num < 1) {
    return "error";
  }
  if (num == 1) {
    return 1;
  }
  return num * findFactorial(num - 1);
}

console.log(findFactorial(7));

answered Apr 16, 2021 at 13:42

Jozott

2,7322 gold badges18 silver badges32 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Som Shekhar Mukherjee · Accepted Answer · 2021-04-16 14:01:53Z

1

num.isInteger function is not availble in js. We have Number.isInteger(val).

Some code changes needed.

When a number is not an integer or is a negative number then only we need to return the error.
When a number reaches 1 then we need to return 1 itself to get then factorial values.

See the below code.

function findFactorial(num) {
  if (!Number.isInteger(num) || num < 0) {
    return "error";
  } else if (num <= 1) {
    return 1;
  }
  else {
    return num * findFactorial(num - 1);
  }
}

console.log(findFactorial(7));
console.log(findFactorial(-5));
console.log(findFactorial("six"));

edited Apr 16, 2021 at 14:01

Som Shekhar Mukherjee

8,1982 gold badges16 silver badges34 bronze badges

answered Apr 16, 2021 at 13:35

Kaustubh Khare

3,5203 gold badges38 silver badges51 bronze badges

Collectives™ on Stack Overflow

Check if number is not integer in recursion js [duplicate]

2 Answers 2

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Linked

Related