TheAlgorithms / JavaScript

Algorithms and Data Structures implemented in JavaScript for beginners, following best practices.

Home Page:https://the-algorithms.com/language/javascript

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

calcFactorial(n) in Factorial.js must return integer value instead of a string value

itsAkshayDubey opened this issue · comments

Hi,

As I can see, the current implementation of calculating the factorial of a number returns a string like "The factorial of 5 is 120" instead this function must return an integer value of 120, so that we can use this function in other algorithms to calculate factorial without having to write logic to calculate factorial and thereby increasing code redundancy in this repo. Currently, I working on an algorithm that requires me to calculate the factorial of a number but I am unable to use calcFactorial(n) from Factorial.js as it returns a string.

Regards,
Akshay

Agreed. Feel free to fix it.