C++ 菜鸟的问题（1）

njkzkj

#include <iostream>
#include <cmath>
#include <vector>

using namespace std;

struct dates_t {
        int p;
        int e;
        int i;
        int d;
};

//Biorhythms
vector <dates_t>  get_input(void)
{
         vector <dates_t>  dates;
         dates_t date_item;
         int p,e,i,d;
         do{
                 cin >> p >> e >> i >>d ;
                 if (p == -1 && e== -1 && i == -1 && d == -1)
                         return dates;
                 date_item.d = d;
                 date_item.e = e;
                 date_item.i = i;
                 date_item.p = p;

                 dates.push_back(date_item);
         }while(1);
}

int main()
{
        vector <dates_t>  something;
        something = get_input();
        for (vector <dates_t>::iterator it = something.begin() ; it < something.end() ; it++) {
                cout << (*it).d<< (*it).e << (*it).i << (*it).p << endl;

        }
        return 0;
}

这个是我写的一个小程序，问题是
vector <dates_t>  dates;好象是局部变量，为什么在主函数下可以显示出来？

gamedragon

你在get_input里面已经把return dates了，dates里面的数据已经被copy出来到something里面了。

njkzkj

也就是说调用了一次copy constructor?
好象在something时又调用了一次 = operator ,
如何改正呐？

jeff_yecn

其实多数情况下，这不算什么太大的问题。我想现代的编程哲学里面有两条：
1. 开发的时候，代码的可读性比榨取最后一滴性能更重要。
2. 优化是基于 profile 的结果，只针对需要的部分进行，而不要基于想象。多数时候，常见的问题，编译器已经帮你进行调整了，或者说，这个优化对系统性能的影响完全忽略不计。

如果假设这里多一次 copy/assign 很重要的话，你可以修改 get_input，把 something 作为参数传给它：
void get_input(vector<dates_t> & value);