Инициализировать вектор-член в конструкторе C++

Question

У меня есть класс, который имеет элемент с типом vector<CCPoint>. Я хочу инициализировать этот член в вызове конструктора, как его можно достичь?

Я сделал это так:

.h

class A{ 
    public: 
     A(vector<CCPoint> *p); 
    private: 
     vector<CCPoint> *p; 
} 

.cpp

A:A(){ 
    this->p = p; 
} 

вызов

Vector<CCPoint> *p = new Vector<CCPoint>; 
A a = new A(p); 

Answer 1

Это утечка памяти, потому что никто не удаляет вектор, который вы «новый».

Кроме того, почему есть указатель на вектор вообще? Вы беспокоитесь о том, что копировать его в конструктор дорого?

Изменить элемент, чтобы быть вектором:

class A{ 
    public: 
     A(vector<CCPoint> p); 
    private: 
     vector<CCPoint> p; 
} 

Изменить конструктор использовать список Инициализатора:

A:A(vector<CCPoint> newP) : p(newP){ 
    // Empty 
} 

И не называйте, как это:

Vector<CCPoint> p; 
A a(p); 

Никогда, создайте объект с «новым», если вы точно не знаете, почему вы это делаете, и даже тогда пересматриваете.

Замечание об исполнении: Да, это может привести к возникновению векторной копии, в зависимости от копирования экземпляра компилятором. Альтернативным решением для модных брюк C++ 11 было бы использовать перемещение:

class A{ 
    public: 
     A(vector<CCPoint> p); 
    private: 
     vector<CCPoint> p; 
} 

A:A(vector<CCPoint> newP) : p(std::move(newP)){ 
    // Empty 
} 

Vector<CCPoint> p; 
A a(std::move(p)); // After this completes, 'p' will no longer be valid. 

Answer 2

Там есть ошибка в файле CPP, вы» повторно отсутствует вторая толстая кишка:

A::A() { 

Кроме того, вы можете напрямую инициализировать р с помощью списка инициализации, как так:

A::A(vector<CCPoint>* _p) : 
p(_p) 
{} 

Не то, что есть какое-либо реальное преимущество в использовании этого для примитивных типов, таких как указатели, но это хорошо конвенция. Отвечает ли это на ваш вопрос? Я не совсем понимаю, в чем проблема, именно на основе вашего сообщения.