Thread: Where am I going wrong here?

Trying to use an inline function to do a factorial, yet won't compile:

Code:

#include<iostream>using namespace std;


inline int factorial(int n);
int computeFactorials(int, int);






int main()
{
	factorial(1, 8);
	return 0;
}


int factorial(int n)
{
return (n == 1) ? 1: ( factorial(n - 1) * n) ;}


int computeFactorials(int num, int max)
{
	cout << "Factorial of " << num << ": ";
	cout << factorial(num) << endl;
	num ++;
	if (num > max ) return 0;
	else return factorial(num, max);
}

It looks like you may need two functions named factorial(), one with one parameter, and one with two parameters.

The compiler's warnings should be a big clue:

Code:

181267.cc: In function ‘int main()’:
181267.cc:16:16: error: too many arguments to function ‘int factorial(int)’
   16 |  factorial(1, 8);
      |                ^
181267.cc:6:12: note: declared here
    6 | inline int factorial(int n);
      |            ^~~~~~~~~
181267.cc: In function ‘int computeFactorials(int, int)’:
181267.cc:33:32: error: too many arguments to function ‘int factorial(int)’
   33 |  else return factorial(num, max);
      |                                ^
181267.cc:21:5: note: declared here
   21 | int factorial(int n)
      |     ^~~~~~~~~