1296 / 469 / 151
Регистрация: 24.08.2011
Сообщений: 2,249
1

Heap corruption detected при удалении динамического массива

01.12.2014, 20:23. Показов 2030. Ответов 6
Метки нет (Все метки)

mystring.cpp - здесь происходит ошибка в функции MyString::operator[] на строке delete[] s. В чем тут может быть дело?
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
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
#include "mystring.h"
#include <string.h>
#include <iostream>
using namespace std;
 
MyString::MyString(unsigned int len)
{
    length = len;
    str = new char[len+1];
}
 
MyString::MyString(char *s)
{
    length = strlen(s);
    str = new char[length+1];
    strcpy(str, s);
}
 
MyString::MyString(MyString &right)
{
    length = right.length;
    str = new char[length + 1];
    strcpy(str, right.str);
}
 
MyString::~MyString()
{
    delete[] str;
}
 
void MyString::Clear()
{
    delete[] str;
    length = 0;
    str = new char[1];
    str[0] = '\0';
}
 
char* MyString::GetChar() const
{
    return str;
}
 
unsigned int MyString::GetLength() const
{
    return length;
}
 
void MyString::SetStr(const char *s)
{
    strncpy(str, s, length+1);
}
 
ostream& operator<<(ostream &out, const MyString &right)
{
    out << right.GetChar();
    return out;
}
 
MyString operator+(MyString &left, MyString &right)
{
    MyString res(left.GetLength() + right.GetLength());
    char *newstr = new char[left.GetLength() + right.GetLength() + 1];
    strcpy(newstr, left.GetChar());
    strcat(newstr, right.GetChar());
    res.SetStr(newstr);
    delete[] newstr;
    return res;
}
 
MyString operator+(MyString &left, const char *right)
{
    MyString res(left.GetLength() + strlen(right));
    char *newstr = new char[left.GetLength() + strlen(right)];
    strcpy(newstr, left.GetChar());
    strcat(newstr, right);
    res.SetStr(newstr);
    delete[] newstr;
    return res;
}
 
MyString operator+(const char *left, MyString &right)
{
    MyString res(strlen(left) + right.GetLength());
    char *newstr = new char[strlen(left) + right.GetLength()];
    strcpy(newstr, left);
    strcat(newstr, right.GetChar());
    res.SetStr(newstr);
    delete[] newstr;
    return res;
}
 
char* MyString::operator[](int n) const
{
    char *s = new char[strlen(str)], *word = new char[strlen(str)], *cur;
    int curn = 0;
    strcpy(s, str);
    cur = strtok(s, " ");
    while (curn < n && cur != NULL)
    {
        cur = strtok(NULL, " ");
        curn++;
    }
    if (curn == n)
        strcpy(word, cur);
    else
        word[0] = '\0';
    delete[] s;
    return word;
}
 
char* MyString::operator()(int b, int n) const
{
    char *newstr = new char[n+1];
    strncpy(newstr, str + b, n);
    newstr[n] = '\0';
    return newstr;
}
 
char** MyString::operator%(int n) const
{
    char *newstr = new char[length + 1];
    strcpy(newstr, str);
    char **wordsarray = new char*[n];
    wordsarray[0] = strtok(newstr, " ");
    int i;
    for (i = 1; wordsarray[i - 1] != NULL && i < n; i++)
        wordsarray[i] = strtok(NULL, " ");
    if (wordsarray[0] == NULL)
        cout << "V stroke net slov!\n";
    else if (wordsarray[i - 1] == NULL)
    {
        cout << "V stroke menishe slov!\n";
        for (; i < n; i++)
            wordsarray[i] = NULL;
    }
    else if (strtok(NULL, " ") != NULL)
        cout << "V stroke bolishe slov!\n";
    return wordsarray;
}
mystring.h:
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
#pragma once
#include <iostream>
using namespace std;
 
class MyString
{
public:
    MyString(unsigned int len = 0);
    MyString(char *s);
    MyString(MyString &right);
    ~MyString();
 
    void Clear();
    unsigned int GetLength() const;
    char* GetChar() const;
    void SetStr(const char *s);
 
    char* operator[](int n) const;
    char* operator()(int b, int n) const;
    char** operator%(int n) const;
 
private:
    unsigned int length;
    char *str;
};
 
ostream& operator<<(ostream &out, const MyString &right);
MyString operator+(MyString &left, MyString &right);
MyString operator+(MyString &left, const char *right);
MyString operator+(const char *left, MyString &right);
main.cpp:
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
85
86
87
88
89
90
91
92
93
// Создайте класс Строка с обязательными член-данными: длина строки, указатель на строку (строка хранится в динамически выделенной памяти).
// Обязательные методы: конструктор по заданной длине строки, конструктор по заданной строке, конструктор по другому объекту класса Строка,
// конструктор копирования, очистка строки, вывод строки.
// Перегрузить операторы: + (конкатенация), [] - выдача слова, () - выдача подстроки, % - разделение строки на слова (возвращать массив слов
// и предусмотреть сообщения: слишком много слов, нет слов, слишком длинное слово).
 
#include "mystring.h"
#include <iostream>
using namespace std;
 
int Menu()
{
    int vybor;
    system("cls");
    cout << "1 - Vyvesti stroki\n"
        << "2 - Konkatenatsia\n"
        << "3 - Vydati slovo po indexu\n"
        << "4 - Vydati podstroku\n"
        << "5 - Razdeliti stroku na slova\n"
        << "6 - Ochistiti stroki\n"
        << "0 - Vyiti iz programmy\n"
        << "Vash vybor: ";
    cin >> vybor;
    cout << endl;
    return vybor;
}
 
int main()
{
    char s[100];
    cout << "Vvedite pervuiu stroku: ";
    gets(s);
    MyString str1(s);
    cout << "Vvedite vtoruiu stroku: ";
    gets(s);
    MyString str2(s);
 
    int n, k;
    char **words;
 
    while (true)
    {
        switch (Menu())
        {
        case 1: cout << str1 << endl << str2 << endl; break;
        case 2: cout << str1 + str2 << endl; break;
        case 3:
            cout << "Vvedite indeks slova iz pervoi stroki (indeksy s 0): ";
            cin >> n;
            cout << str1[n] << endl
                << "Vvedite indeks slova iz vtoroi stroki (indeksy s 0): ";
            cin >> n;
            cout << str2[n] << endl;
            break;
        case 4:
            cout << "Vvedite nachalo podstroki v pervoi stroke: ";
            cin >> n;
            cout << "Vvedite kolichestvo simvolov v podstroke pervoi stroki: ";
            cin >> k;
            cout << str1(n, k) << endl;
            cout << "Vvedite nachalo podstroki vo vtoroi stroke: ";
            cin >> n;
            cout << "Vvedite kolichestvo simvolov v podstroke vtoroi stroki: ";
            cin >> k;
            cout << str2(n, k) << endl;
            break;
        case 5:
            cout << "Na skoliko slov razdeliti pervuiu stroku: ";
            cin >> n;
            words = str1 % n;
            for (int i = 0; i < n; i++)
                if (words[i] != NULL)
                    cout << words[i] << endl;
                else
                    break;
            cout << "Na skoliko slov razdeliti vtoruiu stroku: ";
            cin >> n;
            words = str1 % n;
            for (int i = 0; i < n; i++)
                if (words[i] != NULL)
                    cout << words[i] << endl;
                else
                    break;
            break;
        case 6:
            str1.Clear();
            str2.Clear();
            break;
        case 0: return 0;
        }
        system("pause");
    }
}
Добавлено через 1 час 1 минуту
актуально
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
01.12.2014, 20:23
Ответы с готовыми решениями:

Heap corruption detected при добавлении нового элемента в однонаправленный список
Метод add на вход получает строку, затем ищет в списке такой элемент и добавляет новое звено...

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

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

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

6
7175 / 6147 / 2805
Регистрация: 14.04.2014
Сообщений: 26,494
01.12.2014, 20:44 2
В SetStr() почему не учитываешь вариант с передачей строки большей длины? Но даже если такого не происходит, где установка конца строки?
0
1296 / 469 / 151
Регистрация: 24.08.2011
Сообщений: 2,249
01.12.2014, 20:50  [ТС] 3
nmcf, s всегда завершается нулевым символом, и для него всегда хватает места. Понятно, что функция, мягко говоря, не универсальная, но она вообще для внутренних нужд, надо ее в private. Хотя, исправить конечно надо.

Ошибку нашла - в MyString::operator[] забыла выделить +1 байт для нулевого символа

Добавлено через 1 минуту
А, так там почти правильно. В s не может не быть нулевого символа, а копируется просто не более length+1 символов, то есть не больше, чем возможно. Вот последний действительно надо нулем всегда делать.
0
7175 / 6147 / 2805
Регистрация: 14.04.2014
Сообщений: 26,494
01.12.2014, 22:20 4
У тебя все строки одной длины, что ли?
0
1296 / 469 / 151
Регистрация: 24.08.2011
Сообщений: 2,249
02.12.2014, 07:26  [ТС] 5
Нет, length - это член класса
0
7175 / 6147 / 2805
Регистрация: 14.04.2014
Сообщений: 26,494
02.12.2014, 09:19 6
Я говорю про работу со строками в основной программе. Вызов SetStr() всегда выполняется со строкой той же длины, что и length, что ли?
0
Эксперт по математике/физикеЭксперт С++
1996 / 1326 / 379
Регистрация: 16.05.2013
Сообщений: 3,435
Записей в блоге: 6
02.12.2014, 09:30 7
Керра, конструктор копирования должен принимать константную ссылку:
C++
1
    MyString(const MyString &right);
иначе не будет работать к примеру такой простой код:
C++
1
2
3
4
5
6
MyString func(const char* s) {
    return MyString(s);
}
...
char s[] = "blablabla";
MyString s1 = func(s);
Аналогично для конструктора принимающего указатель на си строку. Ведь она может оказаться read-only.
А вот эти грабли еще страшнее:
C++
1
2
3
4
5
class MyString
{
public:
    MyString(unsigned int len = 0);
    char* operator[](int n) const;
Вот немного надуманный пример, но суть должна быть ясна:
Кликните здесь для просмотра всего текста
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
#include <iostream>
#include <cstring>
#include <cmath>
using std::cin;
using std::cout;
using std::endl;
class MyString
{
public:
    MyString(unsigned int len = 0);
    MyString(char *s);
    MyString(MyString &right);
    ~MyString();
    char* operator[](int n) const;
private:
    unsigned int length;
    char *str;
};
 
MyString::MyString(unsigned int len)
{
    length = len;
    str = new char[len+1];
}
 
MyString::MyString(char *s)
{
    cout << "Сюрпрайз! Я сработал\n";
    length = strlen(s);
    str = new char[length+1];
    strcpy(str, s);
}
 
MyString::~MyString()
{
    delete[] str;
}
 
char* MyString::operator[](int n) const
{
    char *s = new char[strlen(str)], *word = new char[strlen(str)], *cur;
    int curn = 0;
    strcpy(s, str);
    cur = strtok(s, " ");
    while (curn < n && cur != NULL)
    {
        cur = strtok(NULL, " ");
        curn++;
    }
    if (curn == n)
        strcpy(word, cur);
    else
        word[0] = '\0';
    delete[] s;
    return word;
}
 
bool operator== (const MyString& s1, const MyString& s2) {
    return true;
}
 
int main() {
    MyString s1(10);
    MyString s2(10);
    if(s1 == s2[3])
        cout << "Ups!\n";
 
    return 0;
}


Мораль конструктор с одним агрументом сопровождайте ключевым словом explicit.
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
02.12.2014, 09:30
Помогаю со студенческими работами здесь

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

HEAP CORRUPTION DETECTED
Доброго времени суток, господа. Только начал изучать ООП и столкнулся с проблемой. Нужно...

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

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


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

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

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