Форум программистов, компьютерный форум CyberForum.ru
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
delias
4 / 4 / 0
Регистрация: 16.07.2009
Сообщений: 87
#1

HEAP CORRUPTION DETECTED не могу найти ошибку - C++

09.12.2009, 10:41. Просмотров 813. Ответов 4
Метки нет (Все метки)

код весь прорабатывает, но в конце выдаёт HEAP CORRUPTION
через дебаг прогнал но так и не нашел ничего
может вы подскажите на что ругается ну и в целом прошу высказаться всё ли правильно сделал (интересует правильность кода с 40й строки) или где-то можно было бы по другому:
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
#include<iostream>
#include <time.h>
#include <string.h>
#include <windows.h>
using namespace std;
 
class CStrings {
    char **str;
    int size, count; //размер и кол-во заполненных строк
public:
    CStrings (int n);
    ~CStrings();
    void Add(char *s);
    void print();
    void del(int n);
    void deleteall (char **str);
};
 
CStrings::CStrings(int n){
    size=n;
    str=new char*[n];
    count=0;
}
void CStrings::Add (char *s){
    str[count]=new char[strlen(s)+1];
    strcpy(str[count],s);
    count++;
}
 
CStrings::~CStrings(){
    for (int i=0;i<count;i++)
    delete[]str[i];
    delete[]str;
}
void CStrings::print(){
    for (int i=0; i<count;i++)
        cout<<str[i]<<endl;
}
void CStrings::del(int n){  //как в дебаге посмотреть количество массива??  переделать ретурном!!
    char **strcopy = new char *[count-1];
    for (int i=0,j=0;j<count;j++)
    {
        if (j!=n){
    strcopy[i]=new char [strlen(str[j])];
    strcpy(strcopy[i],str[j]);
    i++;
        }
    }
    
    deleteall(str);
    count--;
    char **str = new char *[count];
    for (int i=0;i<count;i++){
    str[i]=new char [strlen(strcopy[i])];
    strcpy(str[i],strcopy[i]);
    }
    //for (int i=0;i<count;i++)
    //delete[]strcopy[1];
    delete[]strcopy;
}
 
void CStrings::deleteall (char **tr){
    
    for (int i=0;i<count;i++)
    delete[]tr[i];
    delete[]tr;
    
}
 
    void main()
    {
        CStrings s(20);
        s.Add("First");
        s.Add ("Second");
        s.Add ("Third");
        s.Add ("Four");
        s.Add ("Five");
        s.print ();
        s.del(1);
        cout<<endl;
        s.print ();
        
    }
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
09.12.2009, 10:41
Здравствуйте! Я подобрал для вас темы с ответами на вопрос HEAP CORRUPTION DETECTED не могу найти ошибку (C++):

HEAP CORRUPTION DETECTED не могу найти причину - C++
вот фрагмент кода int size; char *TempChar; MyStream.read((char*)&amp;size, sizeof size); TempChar = new char(size + 1); ...

Найти причины возникновения ошибки "heap corruption detected after normal block #331" и исправить эту ошибку - C++
добрый день,нужна ваша помощь. при выводе массива выдает такую ошибку &quot;heap corruption detected after normal block #331&quot;, подскажите где...

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

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

Heap corruption detected - C++
Добрый день! Проблема вот в чём: когда выполнение кода доходит до 163 строки, появляется ошибка: &quot; HEAP CORRUPTION DETECTED: after Normal...

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

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
niXman
Эксперт C++
3135 / 1447 / 49
Регистрация: 09.08.2009
Сообщений: 3,441
Записей в блоге: 2
09.12.2009, 11:00 #2
Цитата Сообщение от delias Посмотреть сообщение
for (int i=0;i<count;i++) delete[]str[i]; delete[]str;
попробуй с закомментированными.
0
ISergey
Maniac
Эксперт С++
1374 / 885 / 52
Регистрация: 02.01.2009
Сообщений: 2,658
Записей в блоге: 1
09.12.2009, 11:05 #3
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
#include<iostream>
using namespace std;
 
class CStrings {
    char **str;
    int size, count; //размер и кол-во заполненных строк
public:
    CStrings (int n);
    ~CStrings();
    void Add(const char *s);
    void print();
    void del(int n);
    void deleteall (/*char **str*/);
};
 
CStrings::CStrings(int n){
    size = n;
    str = new char*[n];
    count = 0;
}
void CStrings::Add (const char *s){
    str[count] = new char[strlen(s)+1];
    strcpy(str[count],s);
    count++;
}
 
CStrings::~CStrings(){
    for (int i = 0;i<count;i++)
        delete[]str[i];
    delete[]str;
}
void CStrings::print(){
    for (int i = 0; i < count; i++)
        cout << str[i] << endl;
}
void CStrings::del(int n){  //как в дебаге посмотреть количество массива??  переделать ретурном!!
    char **strcopy = new char *[count-1];
    for (int i = 0, j = 0; j < count; j++)
    {
        if ( j != n ){
            strcopy[i] = new char [strlen(str[j]) + 1];
            strcpy(strcopy[i],str[j]);
            i++;
        }
    }
 
    deleteall(/*str*/);
    count--;
    /*char ***/str = new char *[count];
    for (int i = 0; i < count; i++){
        str[i] = new char [strlen(strcopy[i]) + 1];
        strcpy(str[i],strcopy[i]);
    }
    for (int i = 0; i< count; i++)
      delete[] strcopy[i];
    delete[] strcopy;
}
 
void CStrings::deleteall (/*char **tr*/){
 
    for (int i = 0; i < count; i++){
        delete[] str[i];
        str[i] = 0;
    }
    delete[] str;
    str = 0;
 
}
 
int main()
{
    CStrings s(20);
    s.Add("First");
    s.Add ("Second");
    s.Add ("Third");
    s.Add ("Four");
    s.Add ("Five");
    s.print ();
    s.del(1);
    s.del(1);
    cout<<endl;
    s.print ();
 
}
0
delias
4 / 4 / 0
Регистрация: 16.07.2009
Сообщений: 87
09.12.2009, 12:16  [ТС] #4
for (int i=0;i<count;i++) delete[]str[i];
delete[]str;
мда, ошибка тут где-то.
не понимаю, судя по книге так вообще можно delete[]str; написать и массив будет удалён.

Добавлено через 1 минуту

и почему в деструкторе ~CStrings это удаление работает?
0
ISergey
Maniac
Эксперт С++
1374 / 885 / 52
Регистрация: 02.01.2009
Сообщений: 2,658
Записей в блоге: 1
10.12.2009, 00:35 #5
Цитата Сообщение от delias Посмотреть сообщение
C++
1
2
for (int i=0;i<count;i++) delete[]str[i]; 
delete[]str;
мда, ошибка тут где-то.
Придуриваешься или код влом скомпилировать.. http://codepad.org/zqQpomQy
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
10.12.2009, 00:35
Привет! Вот еще темы с ответами:

Heap corruption detected - C++
Возникает эта ошибка при выполнении функции Strings::Append (строка 81, в меню это пункт 3). Входные данные на всякий случай прикрепила....

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

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

Heap corruption detected - C++
При попытке очистить память выдает &quot;Heap corruption detected&quot;. Помогите пожалуйста( // labanext.cpp : Defines the entry point for the...


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
10.12.2009, 00:35
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru