0 / 0 / 0
Регистрация: 17.09.2018
Сообщений: 6
1

HEAP CORRUPTION DETECTED

14.10.2019, 16:33. Показов 2498. Ответов 4
Метки нет (Все метки)

Доброго времени суток, господа.
Только начал изучать ООП и столкнулся с проблемой. Нужно написать класс, содержащий строку символов, строка должна выделяться динамически и заканчиваться нулем. Длину строки хранить не нужно. В этом классе я реализовал (худо-бедно) private-метод генерации строки заданной длины из латинских букв, конструкторы(по умолчанию(с использованием генератора), конструктор с параметром "строка" и конструктор с параметром "длина"), а также конструктор копий. А вот с деструктором возникли некоторые неприятности. Программа вылетает с ошибкой(под спойлером). Буду очень признателен, если кто-то поможет.
Кликните здесь для просмотра всего текста

---------------------------
Microsoft Visual C++ Runtime Library
---------------------------
Debug Error!

Program: ...s\C++\OOP\1\ConsoleApplication1\Debug\ConsoleAp plication1.exe

HEAP CORRUPTION DETECTED: after Normal block (#184) at 0x005CD5C0.
CRT detected that the application wrote to memory after end of heap buffer.


Сам класс:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
#include "pch.h"
#include <iostream>
#include <time.h>
using namespace std;
class CStr
{
    char *A;
    char* Gen(int len)
    {
        char *s, alpha[] = "abcdefghijklmnopqrstuvwxyz";
        s = new char[len];
        for (int i = 0; i < len; ++i) {
            s[i] = alpha[rand() % (sizeof(alpha) - 1)];
        }
        s[len] = 0;
         return s;
    }
public:
    void Print()
    {
        cout << "This string: " << A << endl;
    }
    CStr() 
    {
        int len = (1+rand() % 20);
        A = Gen(len);
        cout <<"String generated: "<< A << endl;
    }
    CStr(const char *str) 
    {
        A = new char[strlen(str)];
        strcpy(A, str);
    }
    CStr(int len)
    {
        if ((len>20)or(len<=0))
        {
            cout << "Wrong value of lenght. Retry..\n";
            return;
        }
        A=Gen(len);
    }
    ~CStr()
    {
        cout << "String " << A << " deleted!\n";
        delete[] A;
        
    }
    CStr(const CStr &other)
    {
        cout << "Copy costructor started for " << this << endl;  
        this->A = new char[strlen(other.A)];
        strcpy(this->A, other.A);
    }
    
};
__________________
Помощь в написании контрольных, курсовых и дипломных работ, диссертаций здесь
0
Лучшие ответы (1)
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
14.10.2019, 16:33
Ответы с готовыми решениями:

HEAP CORRUPTION DETECTED
Собственно такую ошибку выдаёт: &quot;HEAP CORRUPTION DETECTED: after Normal block (#220) at 0x001970B8....

Heap Corruption detected (
что не правильно? запускаю и ошибка вылетает как на предидущей картинке template&lt;typename TT&gt;...

Heap corruption detected
Heap corruption на строке 93 при первом же выполнении цикла. Из-за чего? #include &lt;locale.h&gt; //...

Heap corruption detected
Ошибка возникает после отработки программы Заголовочный файл: #pragma once #include&lt;iostream&gt;...

4
858 / 447 / 112
Регистрация: 06.07.2013
Сообщений: 1,495
14.10.2019, 16:39 2
Цитата Сообщение от HarrisonFord Посмотреть сообщение
C++
1
2
3
4
5
CStr(const char *str) 
{
A = new char[strlen(str)];
strcpy(A, str);
}
Размера выделенной строки "A" должно быть достаточно для того чтобы содержать текст и '\0' в конце. А strlen берет только размер текста.
0
0 / 0 / 0
Регистрация: 17.09.2018
Сообщений: 6
14.10.2019, 17:11  [ТС] 3
Цитата Сообщение от Raali Посмотреть сообщение
Размера выделенной строки "A" должно быть достаточно для того чтобы содержать текст и '\0' в конце. А strlen берет только размер текста.
Эту ошибку можно исправить так?
C++
1
2
3
4
5
CStr(const char *str) 
{
A = new char[strlen(str)+1];
strcpy(A, str);
}
Я исправил все подобные ошибки, но ошибка с кучей не исчезла(
0
858 / 447 / 112
Регистрация: 06.07.2013
Сообщений: 1,495
14.10.2019, 17:17 4
Лучший ответ Сообщение было отмечено HarrisonFord как решение

Решение

Цитата Сообщение от HarrisonFord Посмотреть сообщение
C++
1
2
3
4
5
s = new char[len];
for (int i = 0; i < len; ++i) {
s[i] = alpha[rand() % (sizeof(alpha) - 1)];
}
s[len] = 0;
вот тут еще
C++
1
s[len] = 0;
хотя размер массива равен len (15 строка)

Добавлено через 1 минуту
Цитата Сообщение от HarrisonFord Посмотреть сообщение
Эту ошибку можно исправить так?
ага
1
0 / 0 / 0
Регистрация: 17.09.2018
Сообщений: 6
14.10.2019, 17:28  [ТС] 5
Цитата Сообщение от Raali Посмотреть сообщение
вот тут еще s[len] = 0; хотя размер массива len (15 строка)
Да, в этом и была проблема... Спасибо за помощь!
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
14.10.2019, 17:28
Помогаю со студенческими работами здесь

Heap corruption detected
При попытке очистить память выдает &quot;Heap corruption detected&quot;. Помогите пожалуйста( //...

Heap Corruption Detected
Выдает ошибку HEAP CORRUPTION DETECTED, судя по всему где то ошиблась с распределением памяти...

HEAP CORRUPTION DETECTED
Не могу понять, почему выдает такую ошибку, когда я освобождаю память: &quot;HEAP CORRUPTION DETECTED:...

heap corruption detected
имеется класс CStr - строка, в нём есть конструктор CStr CStr::CStr(char *s) { length = 0;...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
5
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2023, CyberForum.ru