Return a pointer to array from a function in C++?

Question

I am a beginning programmer and I have a question about a function that returns a pointer to array of doubles in C++. The function takes two arrays and adds up each element, like in a sum of vectors.

I think the correct way to do is....

double *myfunction(double *x, double *y, int n){
  double *r = new double[n];
  for(int i=0;i<n;i++){
    r[i] = x[i]+y[i];
  }
  return r;
}

The problem is that I use that function in a while-loop in the main function like this

int main(){
  double *x, *y, *s;
  x = new double[2];
  y = new double[2];
  x = {1,1};
  y = {2,2};

  while(/*some condition */){
    /*some process...*/

    s = myfunction(x,y, 2);

    /*some process...*/
  }

  delete[] x;
  delete[] y;
  delete[] s;
}

My question is what about the memory leak? Each time I use "myfunction" (inside the while-loop) I reserve memory for the variable "s", that means that if the while-loop is executed 5 times, then the program reserves 5 times the memory for the variable "s"?

Is there exists a way to do this (return a pointer to arrays from a function and use that function inside a loop)??

Thank you in advanced.

std::unique_ptr<double> s(myfunction(x,y,2));, but honestly I'd use std::vector from the get-go for all the dynamic allocations in this code. — WhozCraig
– WhozCraig, Commented May 26, 2015 at 2:56
@songyuanyao yeah, probably. Just got back from dinner and tipped a pretty good sized margarita =P — WhozCraig
– WhozCraig, Commented May 26, 2015 at 3:03

HadeS · Accepted Answer · 2015-05-26 02:58:28Z

4

To avoid memory leak you need to use s as soon as you get it and delete it once you are done.

 int main(){
      double *x, *y, *s;
      x = new double[2];
      y = new double[2];
      x = {1,1};
      y = {2,2};

      while(//some condition ){
        s = myfunction(x,y, 2);
        //do some process here
        delete[] s;
      }
      delete[] x;
      delete[] y;

    }

answered May 26, 2015 at 2:58

HadeS

2,03820 silver badges38 bronze badges

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

Comments

Barry · Accepted Answer · 2015-05-26 03:01:23Z

3

I'd say the more correct way to write myfunction is:

std::vector<double> myfunction(double *x, double *y, int n){
  std::vector<double> r;
  r.reserve(n);
  for(int i=0;i<n;i++){
    r.push_back(x[i]+y[i]);
  }
  return r;
}

That way, you don't have to worry about memory leaks, and your while loop can just be:

while (/* some condition*/) {
    std::vector<double> s = myfunction(x, y, 2);
    // whatever
}

answered May 26, 2015 at 3:01

Barry

312k32 gold badges735 silver badges1.1k bronze badges

Comments

Community · Accepted Answer · 2017-05-23 11:58:26Z

2

You asked:

Each time I use "myfunction" (inside the while-loop) I reserve memory for the variable "s", that means that if the while-loop is executed 5 times, then the program reserves 5 times the memory for the variable "s"?

The answer is Yes.

You also asked:

Is there exists a way to do this (return a pointer to arrays from a function and use that function inside a loop)??

The answer is Yes. You need to add code to delete the returned memory,

  while( /*some condition */){
    s = myfunction(x,y, 2);

    // Use s

    // Now delete s.
    delete[] s;
  }

A better solution than having to deal with new and delete is to return a std::vector from myfunction. Then, you don't need worry about managing memory. That is the answer by Barray

edited May 23, 2017 at 11:58

CommunityBot

11 silver badge

answered May 26, 2015 at 3:15

R Sahu

207k15 gold badges168 silver badges291 bronze badges

Collectives™ on Stack Overflow

Return a pointer to array from a function 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