Возврат std :: vector - правильный подход

Question

Я пытаюсь создать метод класса, который вернет std :: vector, и я немного смущен о наилучшем способе сделать это.

подход я использовал, чтобы определить следующий метод:

std::vector<double>* GetBins(void); 

и в методе выделения нового зЬй :: вектор, который я заполняю данными. Я возвращаю указатель на это, т.е.

std::vector<double>* Frequency::GetBins(void) { 
    std::vector<double> *rtnVec = new std::vector<double>(); 
    for (_itMap = _mapFreq.begin(); _itMap != _mapFreq.end(); _itMap++) { 
     rtnVec->push_back((*_itMap).first); 
    } 
    return rtnVec; 
} 

(_itMap - это итератор, определенный классом).

В моей main.cpp, я сделал следующее:

std::vector<double>* myBins; 
myBins = myFreq3->GetBins(); 
delete myBins; 

Я знаю, что с этим подходом, я собираюсь получить оборванный указатель, если я удалить указатель в коде main.cpp , так что это уже немного «опасно». Каков наилучший способ вернуть новый метод std :: vector из метода класса?

Спасибо, ребята Пит

Answer 1

Самый лучший способ, чтобы вернуться к значению:

std::vector<double> Frequency::GetBins() { 
    std::vector<double> rtnVec; 
    rtnVec.reserve(_mapFreq.size()); // reserve enough size, no reallocations 
    for (_itMap = _mapFreq.begin(); _itMap != _mapFreq.end(); ++_itMap) { 
     rtnVec.push_back(_itMap->first); 
    } 
    return rtnVec; 
} 

Тогда вы бы использовать функцию, как это:

std::vector<double> myBins = myFreq3->GetBins(); 
// no need for delete! 

Компилятор, вероятно, использовать RVO и не выполнять копии. Если вы используете C++ 11, то перемещение семантики гарантирует, что копии не будут выполнены.

Answer 2

Возврат по значению

std::vector<double> Frequency::GetBins(void) { 
    std::vector<double> rtnVec; 

    // ... 

    return rtnVec; 
} 

Однако, если вы хотите вернуться к указателю, вы можете использовать смарт-указатели:

std::unique_ptr<std::vector<double>> Frequency::GetBins(void) { 
    std::unique_ptr<std::vector<double>> rtnVec(new std::vector<double>()); 

    //... 

    return rtnVec; 
} 

Answer 3

Если вы хотите, чтобы избежать копий/повисшие указатели/..., другой способ просто передать свой std::vector по ссылке к методу:

void Frequency::GetBins(std::vector<double>& bins) { 
    for (_itMap = _mapFreq.begin(); _itMap != _mapFreq.end(); _itMap++) { 
     bins->push_back((*_itMap).first); 
    } 
} 

Вам просто нужно то для его определения:

std::vector<double> myBins; 
myFreq3->GetBins(myBins);