Zwracanie std :: vector - right approach

Question

Próbuję utworzyć metodę klasy, która zwróci std :: vector, i jestem trochę zdezorientowany co do najlepszego sposobu na zrobienie tego.

Podejście Użyłem jest zdefiniować w następujący sposób:

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

iw sposobie przydzielania nowego std :: vector, którą zapełnić danymi. Wracam do tego wskaźnika.

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 to iterator zdefiniowany przez klasę).

W moim main.cpp, robiłem co następuje:

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

wiem z tym podejściem, mam zamiar dostać dangling wskaźnik chyba usunę wskaźnik w kodzie main.cpp , więc jest już trochę "niebezpieczny". Jaki jest najlepszy sposób na zwrócenie nowego wektora std :: vector z metody klasy?

Dzięki chłopaki Pete

Answer 1

Najlepszym sposobem jest powrót przez wartość:

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; 
} 

Następnie chcesz korzystać z funkcji takich jak to:

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

Kompilator będzie prawdopodobnie używać RVO i nie wykonuj żadnych kopii. Jeśli używasz C++ 11, to przeniesienie semantyki zapewni, że nie zostaną wykonane żadne kopie.

Answer 2

Powrót wartością

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

    // ... 

    return rtnVec; 
} 

Jednak jeśli chcesz powrócić przez wskaźnik, można użyć inteligentne kursory:

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

    //... 

    return rtnVec; 
} 

Answer 3

Jeśli chcesz uniknąć kopii/błędnych odwołań/..., inny sposobem jest po prostu zdać std::vector przez odniesienie do metody:

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

wystarczy następnie zdefiniować go wcześniej:

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