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 vector
s 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();
};
精彩评论