复制构造函数中的C ++向量数组

时间:2012-04-03 21:23:17

标签: c++ pointers vector constructor

我是复制构造函数的新手,当我开始使用向量时似乎无法让它们工作。

//  ROBOT CLASS
class Robot {
    private:
        char *name;
        int size;

        int cmdCount;

        command *cmdList;
        char items[8][16];
        int resources[3]; // silver, gold, then platinum

    public: 
        Robot( );
        Robot(int num_of_cmds, const char *nm);
        Robot(const Robot &);
        ~Robot( );

        const char *get_name( );
        command get_command(int pos) const;
        void set_item(const char item[ ], int pos);
        const char *get_item(int pos);

};

//  ROBOT CONSTRUCTOR default
Robot::Robot( ) {

    //  Load Robot name
    cmdCount = 5;
    try {
        name = new char[11];
    }
    catch(std::bad_alloc) {
        cout << "Error allocating " << 11+1 << " bytes of memory\n";
        name = NULL;
    }

    if (name) {
        strcpy (name, "univac.dat");
    }

    //  Allocate memory for command array
    vector <command> cmdList[5];

};

//  ROBOT COPY CONSTRUCTOR
Robot::Robot(const Robot &from) {
    cmdCount = from.cmdCount;
    //  Load Robot name
    try {
        name = new char[11];
    }
    catch(std::bad_alloc) {
        cout << "Error allocating " << 11+1 << " bytes of memory\n";
        name = NULL;
    }

    if (name) {
        strcpy (name, from.name);
    }

    //  Allocate memory for command array
    vector <command> cmdList[5];

    for (int i=0; i < cmdCount;i++) {
        cmdList[i] = from.cmdList[i];
    }

    for (int i=0; i < 8; i++) {
        strcpy(items[i], from.items[i]);
    }

    for (int i=0; i < 3; i++) {
        resources[i] = from.resources[i];
    }

}    

编译时遇到的错误是:

  

robot.cpp:在复制构造函数'Robot :: Robot(const Robot&amp;)'中:   robot.cpp:117:错误:'cmdList [i] = 中的'operator ='不匹配(((命令)from-&gt; Robot :: cmdList)+((unsigned int)( ((unsigned int)i)* 172u)))'   /usr/include/c++/4.4/bits/vector.tcc:156:注意:候选人是:std :: vector&lt; _Tp,_Alloc&gt;&amp; std :: vector&lt; _Tp,_Alloc&gt; :: operator =(const std :: vector&lt; _Tp,_Alloc&gt;&amp;)[with _Tp = command,_Alloc = std :: allocator]

我如何在复制构造函数中复制矢量数组?

1 个答案:

答案 0 :(得分:5)

在您的班级中,您声明一个成员变量:

command *cmdList;

但是在每个构造函数中,都声明了一个具有相同名称的局部变量:

vector <command> cmdList[5];

实际上,您需要成员变量的类型为vector<command>

//  ROBOT CLASS
class Robot {
    private:
…
        std::vector<command> cmdList;    
…
};

然后,在默认构造函数中,您可以为它分配内存。您不一定要这样做,具体取决于您以后如何使用它。

//  ROBOT CONSTRUCTOR default
Robot::Robot( ) :cmdList(5) {
… // no mention of cmdList in the body required    
};

最后,在复制构造函数中,复制它:

//  ROBOT COPY CONSTRUCTOR
Robot::Robot(const Robot &from) : cmdList(from.cmdList) {
… // no mention of cmdList in the body required
}  

<小时/> 替代:作为替代方案,如果您选择不在构造函数中使用初始化列表,则可以执行以下操作:

Robot::Robot() {
…
    cmdList = std::vector<command>(5);
    // or cmdList.resize(5);
…
}

Robot::Robot(const Robot &from) {
… 
    cmdList = from.cmdList;
…
}  

<小时/> 额外信用:如果您进行了以下更改,那么可能根本不需要复制构造函数!您也不需要析构函数或赋值运算符:

 class Robot {
    private:
        std::string name;
        int size;

        int cmdCount;

        std::vector<command> cmdList;
        char items[8][16];
        int resources[3]; // silver, gold, then platinum

    public: 
        Robot( );
        Robot(int num_of_cmds, const char *nm);

        const char *get_name( ) { return name.c_str(); }
        command get_command(int pos) const { return cmdList[pos]; }
        void set_item(const char item[ ], int pos);
        const char *get_item(int pos);
};