0

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

我得到错误：

'std::cout << * moj 中的 'operator<<' 不匹配

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

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;
}
4

2 回答 2

1

你的问题是：

在内存分配函数未返回的任何地址上调用解除分配函数是未定义的行为。你的代码中有一个未定义的行为，因为你从不使用 destructor() 分配内存，new []而是调用delete []destructor( delete[] this->napis;)。

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

你应该有： 

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;

这是您的程序的在线版本，经过上述修改，它工作得很好。

于 2012-05-26T12:04:19.893 回答
1

问题是那moj是一个String *，而不是一个String。所以moj[0]不调用你的operator <<，它只是取消引用指针。

于 2012-05-26T11:02:23.577 回答