搜索

首页  >  问答  >  正文

windows - C++ CopyConstructor 引发的异常!!

代码如下:

void CopyStr(char *&destination, char *&source) {
    int sz = strlen(source) + 1;//此处引发异常!!
    destination = new char[sz];
    for (unsigned i = 0; source[i] != 'rrreee'; i++)
        destination[i] = source[i];
    destination[sz - 1] = 'rrreee';
    return;
}

Data::Data(Data &adata) {
    CopyStr(adata.P_name, P_name);
    CopyStr(adata.address, address);
    CopyStr(adata.number, number);
}

代码说明:Data类现在有三个char*成员,分别是P_name,address和number。

编译图片:

求大神解答,谢谢!!

过去多啦不再A梦过去多啦不再A梦2723 天前830

全部回复(3)我来回复

  • 伊谢尔伦

    伊谢尔伦2017-05-31 10:41:26

    首先char *&source这种参数改为char *const &source,其次CopyStr(adata.P_name, P_name),这种改为CopyStr(P_name, adata.P_name)。
    这是我测试的代码,直接可以运行:

    #include <iostream>
    #include <cstring>
    using std::cout;
    using std::endl;
    using std::strlen;
    class Data
    {
    public:
      char *a = "";
      char *b = "";
      char *c = "";
      Data(char *a, char *b, char *c):a(a),b(b),c(c) {};
      Data(Data const &data);
    };
    
    
    int CopyStr(char * &des,  char *  const &source)
    {
      int sz = strlen(source) + 1;
      des = new char(sz);
      for (unsigned i = 0; source[i] != 'rrreee'; i++)
        *(des +i)  = source[i];
      *(des+sz - 1) = 'rrreee';
      return 0;
    }
    
    Data::Data(Data const  &data)
    { 
      CopyStr(a, data.a);
      CopyStr(b, data.b);
      CopyStr(c, data.c);
    }
    
    int main()
    {
      Data data = Data("abcd", "b", "c");
      char * p = "232323";
      Data data2 =  Data(data);
      char * p2 = "1232";
      CopyStr(p2, p);
      cout << p2 << endl;
      cout << data.a << endl;
      cout << data2.a << endl;
      return 0;
    }

    回复
    0
  • 我想大声告诉你

    我想大声告诉你2017-05-31 10:41:26

    调用CopyStr时的第二个参数(比如P_name)是哪里来的?

    回复
    0
  • phpcn_u1582

    phpcn_u15822017-05-31 10:41:26

    vs会用0xCCCCCCCC填充未初始化的内存区域(这也是烫烫烫的由来);
    考虑到你访问到了0xCCCCCCCC,所以要不就是你传进去了个非法指针,要不就是这个char*指针对应的字符串没有以'0'结尾;
    可以试着输出一下source的值,然后逐字节输出指针所指向的数据试试

    回复
    0
  • 取消回复