현재 위치

홈 ›› Forums ›› 공부 ›› 프로그래밍 QnA

생성자와 소멸자에서 메모리 해제와 관련된 질문입니다.

gugudan의 이미지
글쓴이: gugudan / 작성시간: 월, 2005/03/07 - 3:24오전 

#include <stdio.h>


#include <deque>



class test
{
public:
    char * temp;
    test()
    {
        printf("test 생성자입니다.\n");
        temp = new char[10];
    }
virtual        ~test()
    {
        printf("test 소멸자입니다.\n");
        delete []temp;
    }

};



int main()
{

    deque<test> aaa;
    test bbb;

    aaa.push_back(bbb);
}

이걸 실행시키면 에러가 발생합니다..
소멸자가 두번 호출이 되어서 Segmentation fault
가 발생하더군요..

한번만 호출하게 할려면 어떻게 해야 하나요?

Forums: 
프로그래밍 QnA
bugiii의 이미지

글쓴이: bugiii / 작성시간: 월, 2005/03/07 - 4:25오전

deque 의 내부 동작을 잘 생각해보시고 test 라는 클래스가 복사될 때 멤버로 있는 포인터가 어떻게 복사되는지도 조사해보시면 왜 2번 delete [] 가 되는지 아실 수 있을 겁니다.

이것을 회피하기 위해서 여러가지 방법이 있겠지만 stl 을 사용하시는 것 같으니까 vector< char > temp 로 멤버를 선언하시고 사용하시면 new [], delete [] 하실 필요도 없고 복사시 생기는 문제도 일단은 피할 수 있을 것 같습니다. 상황에 맞는 좀 더 나은 방법이나 최적화는 좋은 책들을 참고하시길 바랍니다.

girneter의 이미지

글쓴이: girneter / 작성시간: 월, 2005/03/07 - 9:37오전 

#include <stdio.h>


#include <deque>



class test
{
public:
    char * temp;
    test()
    {
        printf("test 생성자입니다.\n");
        temp = new char[10];
    }
    test(const test& copy)
    {
        printf("test 복사생성자입니다.\n");
        temp = new char[10];
        memcpy(temp, copy.temp, 10);
    }
virtual        ~test()
    {
        printf("test 소멸자입니다.\n");
        delete []temp;
    }

};



int main()
{

    deque<test> aaa;
    test bbb;

    aaa.push_back(bbb);
}

위와 같이 복사생성자를 추가하면 해결되겠지요.
더 정확히는 대입연산자도 추가해야겠지만.

저는 C++ 기초플러스 라는 책의 11장을 참조했습니다
맘에 드는 책인데 어느 분은 쓰레기라고도 하시더군요

개념없는 초딩들은 좋은 말로 할때 DC나 웃대가서 놀아라. 응?

둘러보기