Thread: functions within functions?

Would it be possible to call a function from within a function? I have being set a task which requires me to do a calculation in one function and without returning a value from either function, when i try calling the new function from within the current one i get the error: error C2065: 'printout' : undeclared identifier even though I've defined it before main. Any ideas?

That should be possible. Post your code.

This is the whole thing:

Code:

#include <iostream>

using std::cin;
using std::cout;
using std::endl;
using std::fixed;

#include <cmath>



void seperate (double numbertobesplit); // function one //
void printOut (double wholepart, double fractional, int numbersign); // function two // 

int main()
{
    double UsersNumber;
    cout << "Enter a value to analyse:";
    cin >> UsersNumber;
    seperate (UsersNumber);


    return 0;
}

void seperate (double numbertobesplit) // the first function // 

{
    double wholepart, fractional;
    int numberssign;
    
    wholepart = floor (numbertobesplit);
    fractional = fabs (numbertobesplit) - wholepart;
    
    if (wholepart > 0){
        numberssign = 1;}
    else{
        numberssign = 0;}

    printout (wholepart, fractional, numberssign); // this is the call to the second function //

}

void printOut (double wholepart, double fractional, int numberssign) // second function starts here // 

{
    cout << "sign: ";
        if (numberssign = 1){
            cout << "+\n";}
        else{
            cout << "-\n";}

    cout << "Whole number magnitude: " << wholepart << endl;
    cout << "Fractional part: " << fractional << endl;

}

If you are wondering its supposed to split a floating point number into three pieces, one part is the sign, the next is the magnitude followed by the fractional part.

Bugger! well that would teach me to follow our instructions won't it?

Now it actually works, i have another quick problem! Is there something in c++ that says that double length floating point numbers are unable to be negative, i ask this because whenever i put a negative number into the code it never picks it up in the if statement (if wholepart > 0), it always comes out as positive? would it be easier to use a boolean expression?

Code:

if (numberssign == 1){
    cout << "+\n";}
else{
    cout << "-\n";}

if (numberssign = 1); assigns numberssign to the value 1 then returns true. Then else{ numberssign = 0;} assigns numberssign to 0, and once again returns true. Finally, the printOut() function once again assigns it back to 1.

heh, I've just being playing with my code and it seems alot easier to just use booleans, i actually noticed how to fix the errors only a few minutes after i posted, guess i should spend more time coding and less time posting

Anyway, i think I've removed all the bugs from it, it can now handle negative numbers and even remove its sign when its displayed, i don't suppose anyone can test it for me make sure I'm not missing anything daft

Code:

// ICA - Question 3 - Peter Dermott - c3024654 //

#include <iostream>

using std::cin;
using std::cout;
using std::endl;
using std::fixed;

#include <cmath>



void seperate (double numbertobesplit); // function one //
void printOut (double wholepart, double fractional, bool numbersign); // function two // 

int main()
{
	double UsersNumber;
	cout << "Enter a value to analyse:";
	cin >> UsersNumber;
	seperate (UsersNumber);


	return 0;
}

void seperate (double numbertobesplit) // the first function // 

{
	double wholepart, fractional;
	bool numberssign;
	
	wholepart = floor (numbertobesplit);
	fractional = fabs (numbertobesplit) - wholepart;
	
	if (wholepart >  0){
	fractional = fabs (numbertobesplit) - wholepart;
		numberssign = true;}
	else{
	
		fractional = fabs (numbertobesplit) + wholepart + 1;
		wholepart = (wholepart * -1) - 1;

		numberssign = false;}

	printOut (wholepart, fractional, numberssign); // this is the call to the second function //

}

void printOut (double wholepart, double fractional, bool numberssign) // second function starts here // 

{
	cout << "sign: ";
		if (numberssign == true){
			cout << "+\n";}
		else{
			cout << "-\n";}

	cout << "Whole number magnitude: " << wholepart << endl;
	cout << "Fractional part: " << fractional << endl;

}

Thanx

why do you have:

Code:

using std::fixed;

?

because the fixed modifier is in the std namespace...

But he never uses fixed anywhere in his code.

Maybe he is planning on using it later

That's crazy horse! lol

Nah, i thought that i would need it for some reason, thanks for spotting that

Bah way to blow my cover petedee