static_cast<float> for division of int

[shaLL]

Simple calculator. How to get the static_cast to change from int to float?

#include <iostream>
using namespace std;

int Calculate(int x, int y, char chOperation)
	{	
		switch(chOperation)
		{
		case '/': 
			return  x = (x % y != 0) ? 
			(static_cast<float>(x) / y) : (x / y);
		case '*': 
			return x * y;
		case '+': 
			return x + y;
		case '-': 
			return x - y;
	
		default:	
			cout << "Error." << endl;
			break;
		}
	}
 

int main()
{
while(1)

	{
	cout << "\n--------------------------------\n" << endl;
	int x, y; char chOperation;
	cout << "Calculator: ";
	cin >> x;
	cin >> chOperation;
	cin >> y;

	cout << "\n--------------------------------\n" 
	<< x << " " << chOperation<< " "
	<< y << " = " << Calculate(x, y, chOperation);
	}
	
	return 0;
}

[mutexe]

firstly i'd cast the denomitor.
secondly that wont work in the current state as your function returns an int.
You'd have to make it return a float.

[shaLL]

firstly i'd cast the denomitor.
secondly that wont work in the current state as your function returns an int.
You'd have to make it return a float.

So, if I were to just give it a return ternary instead of "x =" ? Need a little more clarification. Below doesn't work either.

return (x % y != 0) ? 
(x / static_cast<float>(y)) : 
(x / y);

[MiiNiPaa]

Your function itself returns int. int Calculate(. No matter what you return, it will be casted to int, because this is what you told it to do.

[shaLL]

I see, thanks MiiNiPaa.

float Calculate(float x, int y, char chOperation)
{
	
	switch(chOperation)
	{
	case '/': 
		return x / y; 
	case '*': 
		return x * y;
	case '+': 
		return x + y;
	case '-': 
		return x - y;
	
	default:	
		cout << "Error." << endl;
		break;
	}
}

This works.

I had to change both: float Calculate(float x ... into a float. If it was just "Calculate" or "x" it would not work.