Even all above is correct I would see the
correct solution by changing the input type.
void money_convertor(float cash)
Reasons:
1. you read what the code will do: some float calculation
2. it removes the compiler warning
3. currency calculation has always some float values ($ or €)
The third point I see as the most important, but the first is the
computer science main reason.