Проблемы с деструктором

@dota · Регистрация: 20.09.2010

Author24 — интернет-сервис помощи студентам

Вот код :

C++

#include<iostream.h>
#include<string.h>
 
struct Value {
    char val_1[31];
    int val_2;
    bool val_3;
};
 
class Object{
private :
    char * name;
    Value value;
public :
    Object(char Name[],Value V){  // îñíîâíîé êîíñòðóêòîð
        name = new char[strlen(Name)];      
        strcpy(name,Name);
        strcpy(value.val_1,V.val_1);
        value.val_2 = V.val_2;
        value.val_3 = V.val_3;
    }
    Object(const Object & ob){  // êîíñòðóêòîð êîïèðîâàíèÿ
        name = new char[strlen(ob.name)];       
        strcpy(name,ob.name);   
        strcpy(value.val_1,ob.value.val_1);
        value.val_2 = ob.value.val_2;
        value.val_3 = ob.value.val_3;
    } 
    bool is_empty(){return name==0;}
    //~Object(){ if(!is_empty()) delete [] name;}
    bool operator = (const Object& ob){
        bool flag = false;
        if((!strcmp(name,ob.name))&&(!strcmp(value.val_1,ob.value.val_1))&&
            (value.val_2==ob.value.val_2)&&(value.val_3==ob.value.val_3)) flag = true;
        return flag;
    }
};
 
int main() {
    Value V1 = {"Student",19,true};
    Value V2 = {"Student",19,true};
    Object A("Petrov",V1);
    Object B("Ivanov",V2);
    if(A=B) cout<<"Objects are equal"<<endl;
    else cout<<"Objects are different"<<endl;
    return 0;
}

Без деструктора программа работает , но когда убираю значки комментария , то ошибок все равно нет , вылетает системная ошибка. Не могу понять , в чем проблема .

@soon · 06.04.2012, 19:06

никаких ошибок

Bash

soon@desktop:~/Src/C++/main$ make && ./main
g++ -c -Wall -D_GLIBCXX_USE_NANOSLEEP -O3 --std=gnu++11 main.cpp -o main.o
main.cpp: In function ‘int main()’:
main.cpp:44:25: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
main.cpp:45:25: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
g++ main.o -lm -lboost_regex -o main
Objects are different
soon@desktop:~/Src/C++/main$

dota, оператор присваивания - =. Оператор стравнения - ==. Разберитесь с тем, что вы перегружаете.

@zss · 06.04.2012, 19:23

Попробуйте

C++

1	~Object(){ if(name!=NULL) delete[] name;}

@Toshkarik · 06.04.2012, 20:01

Конструктор должен принимать тип const char * в данном случае а не char []. Проверил с ним и запустил, все вроде работает.

Добавлено через 1 минуту
То есть попробуйте просто заменить char Name[] на const char *Name в параметрах конструктора.

@dota · 07.04.2012, 15:48 **[ТС]**

услышал всех и исправил все , как сказали , но все равно та же фигня (

C++

#include<iostream.h>
#include<string.h>
 
struct Value {
    char val_1[31];
    int val_2;
    bool val_3;
};
 
class Object{
private :
    char * name;
    Value value;
public :
    Object(const char * Name,const Value V){  // îñíîâíîé êîíñòðóêòîð
        name = new char[strlen(Name)];      
        strcpy(name,Name);
        strcpy(value.val_1,V.val_1);
        value.val_2 = V.val_2;
        value.val_3 = V.val_3;
    }
    Object(const Object & ob){  // êîíñòðóêòîð êîïèðîâàíèÿ
        name = new char[strlen(ob.name)];       
        strcpy(name,ob.name);   
        strcpy(value.val_1,ob.value.val_1);
        value.val_2 = ob.value.val_2;
        value.val_3 = ob.value.val_3;
    } 
    ~Object(){ if(name!=NULL) delete[] name;}
    bool operator == (const Object& ob){
        bool flag = false;
        if((!strcmp(name,ob.name))&&(!strcmp(value.val_1,ob.value.val_1))&&
            (value.val_2==ob.value.val_2)&&(value.val_3==ob.value.val_3)) flag = true;
        return flag;
    }
};
 
int main() {
    Value V1 = {"Student",19,true};
    Value V2 = {"Student",19,true};
    Object A("Petrov",V1);
    Object B("Ivanov",V2);
    if(A==B) cout<<"Objects are equal"<<endl;
    else cout<<"Objects are different"<<endl;
    return 0;
}

@DU · 07.04.2012, 17:19

http://www.cplusplus.com/refer... ng/strcpy/

char * strcpy ( char * destination, const char * source );
Copy string
Copies the C string pointed by source into the array pointed by destination, including the terminating null character.

поэтому в строке 16 нужно добавить еденичку для этого нуль символа:

C++

1	name = new char[strlen(Name) + 1];

остальные места тоже нужно проверить на подобный глюк. а еще лучше использовать std::string.

@dota · 07.04.2012, 17:24 **[ТС]**

спс большое , теперь заработало . я просто думал , что strlen учитывает нуль-символ .

@Toshkarik 1181 / 894 / 94 Регистрация: 03.08.2011 Сообщений: 2,461
	06.04.2012, 20:01	4
	Конструктор должен принимать тип const char * в данном случае а не char []. Проверил с ним и запустил, все вроде работает. Добавлено через 1 минуту То есть попробуйте просто заменить char Name[] на const char *Name в параметрах конструктора. 1

@dota 3 / 3 / 1 Регистрация: 20.09.2010 Сообщений: 100
	07.04.2012, 17:24 [ТС]	7
	спс большое , теперь заработало . я просто думал , что strlen учитывает нуль-символ . 0