Question

我在重载[]时遇到了一个很大的问题，我完全按照示例中的说明使用它，它不起作用，编译器甚至看不到它。

我收到错误：

与'operator＆lt;＆lt;'不匹配在'std :: cout＆lt;＆lt; * moj

第二个问题是，即使我使用复制构造，如果我删除原始对象，复制的对象也会消失。但现在当我添加析构函数程序时崩溃了。

C:\Documents and Settings\Duke\Moje dokumenty\MaciekK\String\main.cpp|90|error: no match for 'operator<<' in 'std::cout << * moj'|
#include <iostream>
#include <cstdio>
#include <stdio.h>
#include <cstring>
using namespace std;

class String{
public:
 char* napis;
 int dlugosc;

    char & operator[](int el) {return napis[el];}
    const char & operator[](int el) const {return napis[el];}
    String(char* napis){
    this->napis = napis;
    this->dlugosc = this->length();
 }

String(const String& obiekt){

    int wrt = obiekt.dlugosc*sizeof(char);
    //cout<<"before memcpy"<<endl;
    memcpy(this->napis,obiekt.napis,wrt);

    //cout<<"after memcpy"<<endl;
    this->dlugosc = wrt/sizeof(char);
}

~String(){
    delete[] this->napis;
}

int length(){
    int i = 0;
    while(napis[i] != '\0'){
        i++;
    }
    return i;
}


void show(){
    cout<<napis<<" dlugosc = "<<dlugosc<<endl;
}



 };




int main()
{

String* moj = new String("Ala ma kota");
 //  cout<< moj[0] <<endl;  // I GETT ERROR NO MATH FOR OPERATO<< IN STD:: COUTN<< * MOJ
String* moj2  = new String(*moj);

moj->show();
delete moj;
moj2->show();
return 0;
}

Answer 1

问题是moj是String *，而不是String。因此moj[0]不会调用您的operator <<，它只是取消引用指针。

Answer 2

你的问题是：

在内存分配函数未返回的任何地址上调用释放函数是未定义行为。您的代码中有未定义的行为，因为您永远不会使用new []分配内存，而是在析构函数（delete []）中调用delete[] this->napis;。

您没有实现构造函数＆amp;复制构造函数正确。
您需要在构造函数中以及复制构造函数中分配动态内存。目前，您不在构造函数和复制构造函数中分配内存，而是执行浅拷贝而不是深拷贝。

你应该：

String(char* napis)
{
    //I put 20 as size just for demonstration, You should use appropriate size here.
    this->napis = new char[20];   <-------------- This is Important!
    memcpy(this->napis,napis,12);
    this->dlugosc = this->length();
}

String(const String& obiekt)
{

    int wrt = obiekt.dlugosc*sizeof(char);
    this->napis = new char[wrt];  <-------------- This is Important!
    memcpy(this->napis,obiekt.napis,wrt);
    this->dlugosc = wrt/sizeof(char);
}

此外，您需要在delete上调用moj2以避免在程序结束时发生内存泄漏。

delete moj2;

Here 是您的程序的在线版本，具有上述修改，并且工作正常。

重载[]运算符和复制构造函数不起作用

2 个答案: