help with void functions

[JakOrp]

hi i know im calling these functions incorrectly in some way, but shouldnt my first two functions in main call correctly? sorry if this seems trivial!

#include <iostream>
using namespace std;

double rate;
int curr_selection;
double input;


void currencycheck()
{

	cout<<"Please enter the currency you would like to convert to\n";
	cout<<"(1)Euros\n";
	cout<<"(2)Pounds\n";
	cin>>curr_selection;
	if (curr_selection!=1&&curr_selection!=2)
	{
		cout<<"Invalid selection\n";
		cout<<"Please enter your selection again\n";
		cin>>curr_selection;
	}
}

void ratecheck()
{
	
	cout<<"Please enter the exchange rate you would like to use\n";
	cout<<"Enter (-1) for default rate\n";
	cin>>rate;
	if (rate==-1)
	{
		rate=1.144;
	}
}

double amount(double)
{
	cout<<"Please enter the amount you would like to convert\n";
	cin>>input;
	return input;
}

double conversion(double)
{
	double result;
	result=input*rate;
	if (curr_selection==1)
	{ 
		cout<<1/result;
	}
	else
	{
	cout<<result;
	}
	return result;
}

int main()
{
	currencycheck;
	ratecheck;
	amount;
	conversion;
	system("pause");
	return 0;
}

[mausy131]

currencycheck;
ratecheck;
amount;
conversion;

You forgot the ()
Fixed code below

int main()
{
	currencycheck();
	ratecheck();
	amount();
	conversion();
	system("pause");
	return 0;
}

[mirec]

so I am begginer too but things i see are 1) you need to put brackets in the main function when you call some function.. so it should be like this:

int main()
{
	currencycheck();
	ratecheck();
	amount();
	conversion();
	system("pause");
	return 0;
}

and when you do amount function and conversion you don t need to put

(double) 

in the brackets.

and second i am not realy sure that your conversion from pound to euro is ok.. i would do it like this:

double conversion()
{
	double result;
	
	if (curr_selection==1)
	{ 
                result = input/rate
		cout<<result;
	}
	else
	{
	result=input*rate;
        cout << result;
	}
	return result;
}

i hope i helped..