开发者

returning a pointed to an object within a std::vector

I have a very basic question on returning a reference to an element of a vector .

There is a vector vec that stores instances of class Foo. I want to access an element from this vector . ( don't want to use the vector index) . How should I code the method getFoo here?

#include<vector>
#include<stdio.h>
#include<iostream>
#include<math.h>

using namespace std;
class Foo {      
      public:
             Foo(){};
             ~Foo(){};
};


class B {
      public:
             vector<Foo> vec;
             Foo* getFoo();
             B(){};
             ~B(){};
};


Foo* B::getFoo(){
int i;
vec.push_back(Foo());
i = vec.size() - 1;

// how to re开发者_高级运维turn a pointer to vec[i] ??

return vec.at(i);

};

int main(){
    B b;
    b = B();
    int i  = 0;
    for (i = 0; i < 5; i ++){
        b.getFoo();   
        }

    return 0;
}


Why use pointers at all when you can return a reference?

Foo& B::getFoo() {
    vec.push_back(Foo());
    return vec.back();
}

Note that references, pointers and iterators to a vectors contents get invalidated if reallocation occurs.

Also having member data public (like your vec here) isn't good practice - it is better to provide access methods for your class as needed.


Why are you adding a new Foo object in your getFoo() method? Shouldn't you just be retrieving the i'th one?

If so, you can use something like

Foo *getFoo(int i) {
  return &vec[i];  // or .at(i)
}

If you want the last element in the vector, use the back() method.


Use the back method. You can do return &vec.back();


I'd suggest to use references instead of pointers. Like this

Foo& B::getFoo(){
  vec.push_back(Foo());
  return vec.back();
};

of course, you will also have to change the declaration for getFoo() in your class B:

class B {
      public:
             vector<Foo> vec;
             Foo& getFoo();
};
0

上一篇:

下一篇:

精彩评论

暂无评论...
验证码 换一张
取 消

最新问答

问答排行榜