Удалить [] внутри деструктора, выделяя внутри конструктора

0

Я выделяю память в конструкторе, а затем удаляю [] в деструкторе, но я получаю _Block_Type_Is_Valid (pHead->nBlockUse)" Error. Я столкнулся с правилом три и _Block_Type_Is_Valid (pHead->nBlockUse)" Error с этим. В моей программе у меня только один экземпляр, и я я использую и не копирую конструктор ни Оператор присваивания копий, поэтому я думал, что мне просто нужен явно объявленный деструктор. В конструкторе я копирую значения массива символов, и я не могу понять, что не так. Ниже приведен короткий код программы. Заголовок:

class CTest
{
private:
    char *TestTable;
    int TestTableLength;
    std::chrono::steady_clock::time_point StartPoint;
    std::chrono::steady_clock::time_point EndPoint;
    std::chrono::steady_clock::time_point CheckPoint;
    std::chrono::system_clock::duration d;
public:
    CTest(char *SignTable);
    ~CTest();
    void NewCombination();
    bool AskToPlay();
    bool AskForNewCombination();
    void Play();
};

Источник:

CTest::CTest(char *SignTable)
{
    int SignTableLength = 0;
    for (int i = 0; *(SignTable + i) != '\0'; i++)
    {
        SignTableLength++;
    }
    TestTableLength = SignTableLength ;
    TestTable = new char[TestTableLength];
    for (int i = 0; *(SignTable + i) != '\0'; i++)
    {
        *(TestTable + i) = *(SignTable + i);
    }
}


CTest::~CTest()
{
    delete[] TestTable;
}

void CTest::NewCombination()
{
    int tmpInt; 
    char tmpChar;
    if (TestTable != NULL)
    {
        for (int i = 0; i < TestTableLength - 1; i++)
        {
            tmpInt = rand() % (TestTableLength - i);
            tmpChar = TestTable[TestTableLength - 1 - i];
            TestTable[TestTableLength - 1 - i] = TestTable[tmpInt];
            TestTable[tmpInt] = tmpChar;
        }
    }
}

bool CTest::AskToPlay()
{
    char play;
    std::cout << "Do you want to play again ?" << std::endl << "If yes press 'y' else  press something else" << std::endl;
    std::cin >> play;
    if (play == 'y') return true;
    else return false;
};

bool CTest::AskForNewCombination()
{
    char newcom;
    std::cout << "Do you want me to create new combination?" << std::endl << "If yes press 'y' else press something else" << std::endl;
    std::cin >> newcom;
    if (newcom == 'y') return true;
    else return false;
};

void CTest::Play()
{
    StartPoint = std::chrono::steady_clock::now();
    CheckPoint = std::chrono::steady_clock::now();
    std::cout << "3\t";
    d = CheckPoint - StartPoint;
    while (std::chrono::duration_cast<std::chrono::milliseconds>(d).count() < 1000)
    {
        CheckPoint = std::chrono::steady_clock::now();
        d = CheckPoint - StartPoint;
    }
    std::cout << "2\t";
    while (std::chrono::duration_cast<std::chrono::milliseconds>(d).count() < 2000)
    {
        CheckPoint = std::chrono::steady_clock::now();
        d = CheckPoint - StartPoint;
    }
    std::cout << "1" << std::endl;
    while (std::chrono::duration_cast<std::chrono::milliseconds>(d).count() < 3000)
    {
        CheckPoint = std::chrono::steady_clock::now();
        d = CheckPoint - StartPoint;
    }
    std::cout << "START!" << std::endl;
    StartPoint = std::chrono::steady_clock::now();
    for (int i = 0; i < (TestTableLength ) ; i++)
    {
        std::cout << *(TestTable + i) << " ";
    }
};

Главный:

char Signs[] = { '1', '2', '3', '4', '5', '6', 'q', 'w', 'e', 'r', 'f', 'g' };
CTest Test(Signs);

int _tmain(int argc, _TCHAR* argv[])
{
    srand(time(NULL));
    while (Test.AskToPlay())
    {
        if (Test.AskForNewCombination()) Test.NewCombination();
        Test.Play();
    }

    Test.~CTest();
    return 0;
}
  • 2
    Используйте std::string .
  • 7
    Вероятно, вы используете конструктор копирования, не осознавая этого. Вам нужно будет привести пример, где вы используете этот класс, который вызывает ошибку. Если объект не может быть скопирован, вы должны реализовать это и в коде. Вот механизм, чтобы сделать это .
Показать ещё 9 комментариев
Теги:
delete-operator

1 ответ

2

Вы вызываете деструктор явно в конце main. Это не должно быть сделано, деструктор вызывается автоматически в конце программы (т.е. Когда ваш глобальный Test объект выходит за рамки). Ваш явно называемый деструктор освобождает выделенную память, а после завершения программы деструктор вызывается снова, пытаясь снова освободить память, вызывая наблюдаемый сбой.

  • 0
    Большое спасибо, я просто смотрю на классы и ошибочно считаю, что обязанностью программиста является вызов деструкторов и окончание программы.
  • 0
    @ user3023499 Это обязанность программиста в случае динамического размещения вашего объекта (с помощью вызова оператора delete), а здесь это не так. У вас есть автоматический объект, который означает, что он уничтожается при выходе из области видимости (включая вызов деструктора).

Ещё вопросы

Сообщество Overcoder
Наверх
Меню