Passing variables between functions in C++

Question

My task is to type and read() several double variables and store them in vector, then the function compute() should calculate the sum of all variables stored in the vector and their average. I have fixed these functions and they work well. The problem is with the final function print() .. the function should print out the result - 'sum' and 'average' variables. But my code for print() is not working properly and prints out wrong numbers. Here is my code:

#include <iomanip>
#include <cstdlib>
#include <iostream>
#include <vector> 
using namespace std;

vector<double> read(){//this function works fine
    cout << "Write some numbers with space between them?" << endl;
    double numz;
    vector<double> myvector;
    do {
    cin >> numz;
    myvector.push_back (numz);
    } while (numz);
    return myvector;
}  

void compute(double average, double sum, vector<double> &myvector){//this function works fine
//if i put cout in this function it calculates correctly
    vector<double>::iterator it;
    for ( it=myvector.begin() ; it < myvector.end()-1; it++ ){
    sum += *it;
    }
    average = sum/myvector.size();

}

void print(double average, double sum){//this is printing out wrong numbers
    cout.precision(2); 
    cout << sum << endl;
    cout << fixed  << average << endl;
}


int main (){
    vector<double> myvector = read();
    double average;
    double sum;
    compute(average, sum, myvector);
    print(average, sum);//this is printing out wrong numbers

    system("pause");
  return 0;
}

Thanks in advance for any help, i understand that my mystake is in variable passing between functions but i have spent the whole day debugging and reading tutorials without any luck.

emartel · Accepted Answer · 2012-11-12 01:11:39Z

5

Just add & in front of your variable in the function declaration and it will pass a reference to your variable!

void compute(double& average, double& sum, vector<double> &myvector)

answered Nov 12, 2012 at 1:11

emartel

7,7731 gold badge32 silver badges59 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Community · Accepted Answer · 2017-05-23 11:48:53Z

0

There are several flaws in your program:

The declaration of compute maybe better to changed to:
```
void compute(const vector<double> &myvector, double &average, double &sum);
```
Passing a const vector<double>& makes it not modifiable in the function, and takes the references of average/sum makes changing the passing in arguments possible, see more of "formal argument" and "actual argument" here.
You'd better declares function read as:
```
void read(vector<double>& myVector);
```
as this will avoid copying of the vector when function read returns.

edited May 23, 2017 at 11:48

CommunityBot

11 silver badge

answered Nov 12, 2012 at 2:23

Zhi Wang

1,1685 gold badges21 silver badges35 bronze badges

Comments

Johannes Thoma · Accepted Answer · 2012-11-12 01:14:59Z

0

You forgot the reference operator (&) in the declaration of the compute function: Without it the variables in the main() function will not be modified: Write

void compute(double &average, double &sum, vector<double> &myvector){//this function works fine

instead of what you wrote and it should work.

answered Nov 12, 2012 at 1:14

Johannes Thoma

1,08611 silver badges22 bronze badges

Collectives™ on Stack Overflow

Passing variables between functions in C++

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related