Functions With Parameters Full Source Code

[Micronomics]

I have a sample code project that I would like to offer the community for their use. Full source provided. (Teaching what I've learned helps me learn).

#include "stdafx.h"
#include <iostream>

using namespace std;

//our custom function with two parameters (float x and float y);
float myFunc(float x, float y){

	//multiplies the parameters;
	float answer = x*y;
	//returns and stores the answer for later use in our main function;
	return answer;
	
	}

//the main function;
int main ()
{
	//declare floats;
	float a, b;
	//user prompt;
	cout << "Enter two numbers to multiply \n";
	//user input;
	cin >> a;
	cin >> b;
	//print the returned value;
	cout << myFunc(a, b) << "\n";
	//pause the computer system;
	cin.get();
	
}

[vlad from moscow]

This expression statement

//function with user inputted parameters;
myFunc(a, b);
//print the returned value;
cout << myFunc(a, b) << "\n";

is superfluous and should be removed.

[Micronomics]

What if I put the output (cout) in the function itself?

[Micronomics]

I see what you were saying now, you're right. Thanks.

[Codeez]

Just to elaborate what vlad is saying, your function returns a value, so simply calling it was pointless as the value was returned, then lost. Using cout, you print that value.

[Micronomics]

How would you change it?

[vlad from moscow]

What are you going to change?

[jkevin]

You can use a variable to store the function.

 float X;
X = myFunct(a,b);
cout << X <<"\n";

[LB]

@jkevin: the variable stores the value returned by the function, not the function itself.

@Micronomics: I assume you know why system("PAUSE"); is bad - if not, some Google searches will clear it up. I have the correct way memorized.

[Micronomics]

Yes, I don't like the system pause method, but I needed the console open long enough for me to see it. Usually, in a longer program, I will just return the entire script to the main function.

[Micronomics]

jKevin, that is a much cleaner way of doing things.

I am usually under the impression that the end user doesn't see my code and therefore, never mattered.

But these methods are much more productive.

[Micronomics]

I have a search algorithm I completed last month, do you guys want to review the source? The user enters a search item, and it displays the result as well as the index location of its array.

[Micronomics]

Wow, that system pause method is ridiculously expensive. I changed it to cin.get();