在类构造函数中设置std :: vector

时间:2012-07-10 14:24:54

标签: c++ constructor new-operator stdvector

我正在设计一个具有std::vector<int>作为实例变量的类。我正在使用std::vector因为我需要在运行时设置它的大小。以下是我的代码的相关部分:

my_class.h:

#include <vector>
using std::vector;
class MyClass {
    int size;
    vector<int> vec;
}

my_class.cc:

#include "my_class.h"
using std::vector
MyClass::MyClass(int m_size) : size(m_size) {
     vec = new vector<int>(size,0);
}

当我尝试编译时,我收到以下错误消息:

g++ -c -Wall my_class.cc -o my_class.o

my_class.cc: In constructor ‘MyClass::MyClass(int):

  my_class.cc:4 error: no match for ‘operator=’ in ‘((MyClass*)this)->My_Class::vec = ((*(const allocator_type*)(& std::allocator<int>())), (operator new(24u), (<statement>, ((std::vector<int>*)<anonymous>))))’

make: *** [my_class.o] Error 1

但是,当我将违规行更改为:

vector<int> temp(size,0);
vec = temp;

它现在编译顺利,我得到了所需的行为,可以访问我的矢量

vec[i]  // i having been defined as an int yada yada yada

这种解决方法没问题,但我想了解它的工作原理,第一种方法失败了。提前谢谢。

3 个答案:

答案 0 :(得分:30)

只是做:

MyClass::MyClass(int m_size) : size(m_size), vec(m_size, 0)

您似乎已经了解初始化列表,为什么不直接初始化向量?

vec = new vector<int>(size,0);

是非法的,因为new会返回一个指针,在您的情况下vec是一个对象。

您的第二个选择:

vector<int> temp(size,0);
vec = temp;
虽然它编译,但额外的工作没有收获。到达作业时,已经构建了两个向量,然后将其丢弃。

答案 1 :(得分:10)

在你的班级中使用vector是合法的,问题在于你如何初始化它:

#include <vector>

class MyClass {
public:
    MyClass(int m_size);

    // ... more things...
private:
    int size;
    vector<int> vec;
}

您正在指定一个指向新矢量对象的指针,就好像此矢量对象未初始化一样。

vec = new vector<int>(size,0);

如果您真的想要这个,那么您应该将vec对象声明为:

vector<int> * vec;

不要忘记添加析构函数:

MyClass::~MyClass {
    delete vec;
}

为什么在放弃new粒子时它会起作用?因为您正在创建一个新对象vector,并且覆盖您的类中的一个(但这并不能保证原始对象被正确消除)。

你实际上不需要这样做。当你到达MyClass的构造函数时,你的vector对象已经被初始化(它的默认构造函数被调用)。如果您只是想确保为size项保留内存:

MyClass::MyClass(int m_size): size(m_size) {
    vec.reserve( size );
}

如果您希望向量具有size元素,则:

MyClass::MyClass(int m_size): size(m_size), vec(m_size, 0)
    {}

最后,正如其中一位评论者指出的那样,一旦构建了载体,实际上并不需要大小。所以你可以摆脱size成员:

class MyClass {
public:
    MyClass(int m_size): vec(m_size, 0)
        {}

    unsigned int getSize() const
        { return vec.size(); }

    // ... more things...
private:
    vector<int> vec;
}

希望这有帮助。

答案 2 :(得分:2)

#include <vector>
#include <iostream>
#include <string>
#include <typeinfo>

using std::cout;
using std::endl;
using std::string;
using std::vector;
using std::to_string;

class Parse
{
private:
    string         m_str;
    vector<string> m_vec;
public:
    // Constructor with all defaults (1 of 4 constructors)
    Parse(){ 
        cout << "\ncreating class with all default values\n";
        m_str = "";
        m_vec.push_back("");    
    }

    // Constructor with all cases used
    Parse  (string         &tmp_str,
            vector<string> tmp_vec):

            m_str          (tmp_str),
            m_vec          (tmp_vec)
    {
        cout << "Your vector contains " + to_string(m_str.size()) + " arguments\n";
    }

    // Constructor with other contents given but not vector
    Parse  (string         &tmp_str): 
            m_str          (tmp_str)
    {
        m_vec.push_back("");
    }
    // Constructor with only Vector given but not other contents
    Parse  (vector<string>   tmp_vec):
            m_vec           (tmp_vec)
    {
        m_str = "";
    }

    string get_str_var(){return m_str;}

    void classed_print_vector_strings()
    {
        for (string i : m_vec){ cout << i << " \n";}
    }

};



// rm ./class_vector; g++ class_vector.cpp -o class_vector -std=c++17; ./class_vector arg1 arg2 arg3

int main(int argc, char *argv[])
{
    // turn **argv to a vector
    vector<string> args(argv, argv + argc);
    // iterate from argv through argv+argc

    // initialize with default arguments.
    Parse tracker1;
    // initalize with all used arguments
    Parse tracker2(args[0], args);
    // initalize with only the vector
    Parse tracker3(args);
    // initalzie without the vector, but with another arg
    Parse tracker4(args[0]);

    cout << "\nTracker 1 ---------------------\n";
    tracker1.classed_print_vector_strings();
    cout << "\nTracker 2 ---------------------\n";
    tracker2.classed_print_vector_strings();
    cout << "\nTracker 3 ---------------------\n";
    tracker3.classed_print_vector_strings();
    cout << "\nTracker 4 ---------------------\n";
    tracker4.classed_print_vector_strings();


    return 0;
}

// rm ./class_vector; g++ class_vector.cpp -o class_vector -std=c++17; ./class_vector arg1 arg2 arg3

// This will show you how to create a class that will give 
// you the option to initilize the class with or without 
// the vector with other arguments present and/or not present. 

//  My Background. . .  
//  github.com/Radicalware  
//  Radicalware.net  
//  https://www.youtube.com/channel/UCivwmYxoOdDT3GmDnD0CfQA/playlists