Question

我在第二个类中遇到问题，同时从第一个类动态分配对象。有时当n < 3或4程序没有问题地工作但是大多数情况下n> 1。 3或者甚至有时当它较少而我运行代码时它会在第二个输入的对象之后崩溃。crash-screenshot。任何有关如何解决此问题的帮助和提示将不胜感激。

#include <iostream>
#include <cstring>

using namespace std;
class ParkingPlac{
private:
    char adresa[20] ;
    char id[20] ;
    int zarabotka ;
    int cena ;
public:
    ParkingPlac() //first class
    {
        cout<<"Constructor "<<endl ;
    }
    ParkingPlac(char *adr , char *i , int c )
    {
        strcpy(this->adresa,adr) ;
        this-> cena = c ;
        char *id = new char [strlen(i)+1] ;
        strcpy(this->id,i) ;
        cout<<"Konstruktor"<<endl ;
        delete[] id ;
    }
    ~ParkingPlac()
    {
        cout<<"Destruktor" <<endl;
    }
    void print()
    {
        cout<<adresa << " "<<id<<" "<<cena<<endl ;
    }
    char *getid()
    {
        return id;
    }
    void setid(char *i)
    {
        strcpy(this->id,i) ;
    }
    char *getadresa()
    {
        return adresa;
    }
    int getcena()
    {
        return cena ;
    }
    void setadresa(char *a)
    {
        strcpy(this->adresa,a) ;
    }
    void setcena(int i)
    {
        this->cena = i ;
    }
};
class PretprijatieParking{
private:
    char imeprezie[35] ;
    ParkingPlac *placovi;
    int brojplacovi ;
public:
    PretprijatieParking(char *ime)
    {
        strcpy(this->imeprezie,ime) ;
        brojplacovi = 0;
    }
    void pecati()
    {
        cout<<imeprezie<<" "<<brojplacovi<<endl ;
    }
    ~PretprijatieParking()
    {
        cout<<"DestruKtor"<<endl;
        delete []placovi ;
    }
    void dodadiParkingPlac(ParkingPlac p)
    {
        int i ;
        for(i = 0 ; i<brojplacovi;i++)
        {
            if(strcmp(placovi[i].getid(), p.getid()) == 0)
                return  ;
        }
        placovi = new ParkingPlac[brojplacovi] ;
        placovi[brojplacovi].setadresa(p.getadresa()) ;
        placovi[brojplacovi].setid(p.getid());
        placovi[brojplacovi].setcena(p.getcena()) ;

        cout<<placovi[brojplacovi].getadresa()<<endl;
        cout<<placovi[brojplacovi].getid()<<endl  ;
        cout<<placovi[brojplacovi].getcena()<<endl ;
        brojplacovi+=1 ;
    }
 };

 int main(){
     PretprijatieParking gradski("Mirko Mirkovski");
     int n;
     char adresa[50],id[50];
     int brojcasovi,cenacas;
     cin>>n;
     for (int i=0;i<n;i++){
         cin.get();
         cin.getline(adresa,50);
         cin>>id>>cenacas;

         ParkingPlac edna(adresa,id,cenacas);
         gradski.dodadiParkingPlac(edna);
    }
    //plakjanje
    //cin>>n;
    //for (int i=0;i<n;i++){
    //
    //    cin>>id>>brojcasovi;
    //
    //    if(!gradski.platiParking(id,brojcasovi))
    //       cout<<"Ne e platen parking. Greshen ID."<<endl;
    //}

    cout<<"========="<<endl;
    gradski.pecati();
    return 0 ;
}

Answer 1

我看到的问题：

1。为大小为0的数组分配内存

在PretprijatieParking的构造函数中，成员变量brojplacovi初始化为0。然后你继续使用：

     placovi = new ParkingPlac[brojplacovi] ;

2。使用越界索引访问内存

使用上面的行分配内存后，数组的最高有效索引为brojplacovi - 1。但是，您继续使用：

     placovi[brojplacovi].setadresa(p.getadresa()) ;

3。泄漏内存

您没有任何代码可以确保：

存储在先前分配的内存中的数据被复制/移动到新分配的内存中。
您没有取消分配以前分配的内存。

您的代码中可能存在其他问题。我快速浏览后看到了上面的内容。第一个和第二个问题足以导致不确定的行为。

除非您被禁止，否则请使用std::vector而不是动态分配的数组和std::string而不是动态分配的字符串。他们将处理大多数与记忆相关的问题。

类的C ++动态内存分配

1 个答案:

1。为大小为0的数组分配内存

2。使用越界索引访问内存

3。泄漏内存