Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 5.00/4: Рейтинг темы: голосов - 4, средняя оценка - 5.00
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
1

Heap corruption detected

30.11.2014, 11:10. Просмотров 711. Ответов 14
Метки нет (Все метки)

Возникает эта ошибка при выполнении функции Strings::Append (строка 81, в меню это пункт 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
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
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
#include <iostream>
using namespace std;
 
class Strings
{
public:
    Strings(); // конструктор со вводом с клавиатуры
    Strings(int n); // создается массив строк размера n, и массив длин, память под строки не выделяется
    Strings(Strings &right, int n = 0); // конструктор-копировщик для нормальной передачи нового массива из функции, n - количество копируемых элементов
    ~Strings();
    char*& at(int i);
    Strings Append(Strings &right); // сцепление
    Strings Merge(Strings &right); // слияние
    bool Found(char *s); // есть ли в данном массиве эта строка
    void Print();
    void Print(int i);
private:
    int number, *length;
    char **str;
};
 
Strings::Strings()
{
    cout << "Kolichestvo strok: ";
    cin >> number;
    length = new int[number];
    str = new char*[number];
    for (int i = 0; i < number; i++)
    {
        cout << "Stroka nr. " << i+1 << endl
            << "Dlina: ";
        cin >> length[i];
        str[i] = new char[length[i]+1]; // +1 для завершающего нулевого символа
        cout << "Stroka: ";
        fflush(stdin); // чистим входной поток, т.к. там застревает конец строки после ввода числа, и строка не читается
        cin.getline(str[i], length[i]+1);
    }
}
 
Strings::Strings(int n)
{
    number = n;
    length = new int[n];
    str = new char*[n];
}
 
Strings::Strings(Strings &right, int n)
{
    if (n == 0)
        number = right.number; // копируем количество строк
    else
        number = n;
    length = new int[number]; // выделяем память под длины
    str = new char*[number]; // выделяем память под строку
    for (int i = 0; i < number; i++)
    {
        length[i] = right.length[i]; // копируем длину текущей строки
        str[i] = new char[length[i]+1]; // выделяем память под текущую строку (+1 для завершающего нулевого символа)
        strcpy(str[i], right.str[i]); // копируем строку
    }
}
 
Strings::~Strings()
{
    delete[] length;
    for (int i = 0; i < number; i++)
        delete[] str[i];
    delete[] str;
}
 
char*& Strings::at(int i)
{
    char *newchar = new char;
    *newchar = '\0';
    if (i < 0 || i >= number) // если выходит за границы
        return newchar; // возвращаем пустую строку
    else
        return str[i];
}
 
Strings Strings::Append(Strings &right)
{
    Strings res(number > right.number? number : right.number); // берем количество элементов в большем массиве
    for (int i = 0; i < number || i < right.number; i++) // пока не закончился больший массив
    {
        res.length[i] = strlen(at(i)) + strlen(right.at(i)); // at вернет пустую строку, если выйдем за границы массива
        res.str[i] = new char[res.length[i]];
        strcpy(res.at(i), at(i)); // копируем i-тую строку из первого массива
        strcat(res.at(i), right.at(i)); // и добавляем к ней строку из втрого массива
    }
    return res;
}
 
bool Strings::Found(char *s)
{
    for (int i = 0; i < number; i++)
        if (strcmp(str[i], s) == 0)
            return true;
    return false;
}
 
Strings Strings::Merge(Strings &right)
{
    Strings res(number + right.number); // выделяем память для максимально возможного количества элементов
    int realnumber = 0; // запоминаем сколько на самом деле скопировалось
    for (int i = 0; i < number; i++)
        if (!res.Found(str[i]))
        {
            res.length[realnumber] = length[i];
            strcpy(res.str[realnumber++], str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
        }
    return Strings(res, realnumber); // скопировались не все элементы, возвращаем массив строк без лишних элементов
}
 
void Strings::Print()
{
    cout << "V massive soderjitsa " << number << " stroki:\n";
    for (int i = 0; i < number; i++)
        cout << str[i] << endl;
}
 
void Strings::Print(int i)
{
    cout << at(i) << endl;
}
 
int Menu()
{
    int vybor;
    system("cls");
    cout << "1 - Vyvesti massivy na ekran\n"
        << "2 - Vyvesti otdelinuiu stroku po indeksu\n"
        << "3 - Poelementnoe stseplenie\n"
        << "4 - Sliyanie s iskliucheniem povtorenii\n"
        << "0 - Vyhod iz programmy\n"
        << "Vash vybor: ";
    cin >> vybor;
    cout << endl;
    return vybor;
}
 
int main()
{
    cout << "Pervyi massiv\n";
    Strings s1;
    cout << "\nVtoroi massiv\n";
    Strings s2;
    int idx;
    char *str;
 
    while (true)
    {
        switch (Menu())
        {
        case 1: s1.Print();
            s2.Print();
            break;
        case 2: cout << "Vvedite index stroki v pervom massive: ";
            cin >> idx;
            str = s1.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            cout << "Vvedite index stroki vo vtorom massive: ";
            cin >> idx;
            str = s2.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            break;
        case 3: s1.Append(s2).Print();
            break;
        case 4: s1.Merge(s2).Print();
            break;
        case 0: return 0;
        }
        system("pause");
    }
}
0
Миниатюры
Heap corruption detected   Heap corruption detected  
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
30.11.2014, 11:10
Ответы с готовыми решениями:

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

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

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

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

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

14
Max Dark
шКодер самоучка
1992 / 1760 / 869
Регистрация: 09.10.2013
Сообщений: 3,887
Записей в блоге: 6
Завершенные тесты: 2
30.11.2014, 11:16 2
Керра, 73я строка - утечка памяти,
87я строка - не резервируется память под '\0'
1
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
30.11.2014, 11:52  [ТС] 3
Теперь еще одна проблема - в конце функции Strings::Merge удаляется временная переменная, которая создается при return, и res. И опять та же ошибка, при вызове деструктора от res, на строке delete[] str[i] на первом же элементе массива. Что опять?
0
Max Dark
шКодер самоучка
1992 / 1760 / 869
Регистрация: 09.10.2013
Сообщений: 3,887
Записей в блоге: 6
Завершенные тесты: 2
30.11.2014, 12:10 4
Керра, Отсутствует конструктор копирования(Strings::Strings(const Strings& cpy))
в Merge Вы не выделяете память под сами строки, только под массив длин и массив указаделей

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

Не по теме:

хочется нажать ctrl+A, DEL ...

1
30.11.2014, 12:10
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
30.11.2014, 12:20  [ТС] 5

Не по теме:

Cra3y, ну не надо так, у студентов код не может быть качественным, за редкими исключениями, учимся жеж


Strings(Strings &right, int n = 0) - это копировщик, разве он не сработает как обычный, в нем же второе значение по умолчанию
0
Max Dark
шКодер самоучка
1992 / 1760 / 869
Регистрация: 09.10.2013
Сообщений: 3,887
Записей в блоге: 6
Завершенные тесты: 2
30.11.2014, 12:25 6

Не по теме:

я вообще неуч, меня из универа выкинули 10 лет назад


Цитата Сообщение от Керра Посмотреть сообщение
Strings(Strings &right, int n = 0)
не совпадает с сигнатурой конструктора копирования type::type(const type&)
0
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
30.11.2014, 12:44  [ТС] 7
однако, именно этот копировщик и вызывается

Добавлено через 1 минуту
при return создается временное значение, с помощью этого копировщика, потом это значение копируется куда надо, при этом так же вызывается этот копировщик, хотя вызывается он без n, т.е. используется то значение по умолчанию, как и предполагалось. то есть нет необходимости именно в той сигнатуре, главное чтобы был возможен вызов без дополнительных параметров

Добавлено через 2 минуты
и память выделяется в 113, 121 и 126 строках... а, кажется это устаревшая версия тут
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
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
#include <iostream>
using namespace std;
 
class Strings
{
public:
    Strings(); // конструктор со вводом с клавиатуры
    Strings(int n); // создается массив строк размера n, и массив длин, память под строки не выделяется
    Strings(Strings &right, int n = 0); // конструктор-копировщик для нормальной передачи нового массива из функции, n - количество копируемых элементов
    ~Strings();
    char*& at(int i);
    Strings Append(Strings &right); // сцепление
    Strings Merge(Strings &right); // слияние
    bool Found(char *s, int frombeginning); // есть ли в данном массиве эта строка
    void Print();
    void Print(int i);
private:
    int number, *length;
    char **str;
};
 
Strings::Strings()
{
    cout << "Kolichestvo strok: ";
    cin >> number;
    length = new int[number];
    str = new char*[number];
    for (int i = 0; i < number; i++)
    {
        cout << "Stroka nr. " << i+1 << endl
            << "Dlina: ";
        cin >> length[i];
        str[i] = new char[length[i]+1]; // +1 для завершающего нулевого символа
        cout << "Stroka: ";
        fflush(stdin); // чистим входной поток, т.к. там застревает конец строки после ввода числа, и строка не читается
        cin.getline(str[i], length[i]+1);
    }
}
 
Strings::Strings(int n)
{
    number = n;
    length = new int[n];
    str = new char*[n];
}
 
Strings::Strings(Strings &right, int n)
{
    if (n == 0)
        number = right.number; // копируем количество строк
    else
        number = n;
    length = new int[number]; // выделяем память под длины
    str = new char*[number]; // выделяем память под строку
    for (int i = 0; i < number; i++)
    {
        length[i] = right.length[i]; // копируем длину текущей строки
        str[i] = new char[length[i]+1]; // выделяем память под текущую строку (+1 для завершающего нулевого символа)
        strcpy(str[i], right.str[i]); // копируем строку
    }
}
 
Strings::~Strings()
{
    delete[] length;
    for (int i = 0; i < number; i++)
        delete[] str[i];
    delete[] str;
}
 
char*& Strings::at(int i)
{
    char *newchar;
    if (i < 0 || i >= number) // если выходит за границы
    {
        newchar = new char;
        *newchar = '\0';
        return newchar; // возвращаем пустую строку
    }
    else
        return str[i];
}
 
Strings Strings::Append(Strings &right)
{
    Strings res(number > right.number? number : right.number); // берем количество элементов в большем массиве
    for (int i = 0; i < number || i < right.number; i++) // пока не закончился больший массив
    {
        res.length[i] = strlen(at(i)) + strlen(right.at(i)); // at вернет пустую строку, если выйдем за границы массива
        res.str[i] = new char[res.length[i]+1];
        strcpy(res.at(i), at(i)); // копируем i-тую строку из первого массива
        strcat(res.at(i), right.at(i)); // и добавляем к ней строку из втрого массива
    }
    return res;
}
 
bool Strings::Found(char *s, int frombeginning)
{
    for (int i = 0; i < frombeginning && i < number; i++)
        if (strcmp(str[i], s) == 0)
            return true;
    return false;
}
 
Strings Strings::Merge(Strings &right)
{
    Strings res(number + right.number); // выделяем память для максимально возможного количества элементов
    int realnumber = 0; // запоминаем сколько на самом деле скопировалось
    for (int i = 0; i < number; i++)
        if (!res.Found(str[i], realnumber))
        {
            res.length[realnumber] = length[i];
            res.str[realnumber] = new char[res.length[realnumber]+1];
            strcpy(res.str[realnumber], str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
            realnumber++;
        };
    for (int i = 0; i < right.number; i++)
        if (!res.Found(right.str[i], realnumber))
        {
            res.length[realnumber] = right.length[i];
            res.str[realnumber] = new char[res.length[realnumber]+1];
            strcpy(res.str[realnumber], right.str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
            realnumber++;
        };
    for (int i = realnumber; i < res.number; i++)
        res.str[i] = new char[1];
    // заполняем оставшиеся строки, т.к. res в конце функции удаляется, а значит все строки удаляются. чтобы нормально сработал delete[], надо выделить память.
    return Strings(res, realnumber); // скопировались не все элементы, возвращаем массив строк без лишних элементов
}
 
void Strings::Print()
{
    cout << "V massive soderjitsa " << number << " stroki:\n";
    for (int i = 0; i < number; i++)
        cout << str[i] << endl;
}
 
void Strings::Print(int i)
{
    cout << at(i) << endl;
}
 
int Menu()
{
    int vybor;
    system("cls");
    cout << "1 - Vyvesti massivy na ekran\n"
        << "2 - Vyvesti otdelinuiu stroku po indeksu\n"
        << "3 - Poelementnoe stseplenie\n"
        << "4 - Sliyanie s iskliucheniem povtorenii\n"
        << "0 - Vyhod iz programmy\n"
        << "Vash vybor: ";
    cin >> vybor;
    cout << endl;
    return vybor;
}
 
int main()
{
    cout << "Pervyi massiv\n";
    Strings s1;
    cout << "\nVtoroi massiv\n";
    Strings s2;
    int idx;
    char *str;
 
    while (true)
    {
        switch (Menu())
        {
        case 1: s1.Print();
            s2.Print();
            break;
        case 2: cout << "Vvedite index stroki v pervom massive: ";
            cin >> idx;
            str = s1.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            cout << "Vvedite index stroki vo vtorom massive: ";
            cin >> idx;
            str = s2.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            break;
        case 3: s1.Append(s2).Print();
            break;
        case 4: s1.Merge(s2).Print();
            break;
        case 0: return 0;
        }
        system("pause");
    }
}
0
Max Dark
шКодер самоучка
1992 / 1760 / 869
Регистрация: 09.10.2013
Сообщений: 3,887
Записей в блоге: 6
Завершенные тесты: 2
30.11.2014, 13:37 8
так вот хотя бы не падает...
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
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
/**
аццкая смесь сишного кода и с++11
*/
#include <iostream>
#include <memory>
#include <algorithm>
#include <cstring>
#include <cstdio>
 
using namespace std;
 
class Strings
{
public:
    Strings(); // конструктор со вводом с клавиатуры
    Strings(int n); // создается массив строк размера n, и массив длин, память под строки не выделяется
    Strings(const Strings &right, int n = 0); // конструктор-копировщик для нормальной передачи нового массива из функции, n - количество копируемых элементов
    ~Strings();
    char* at(int i) const;
    Strings Append(const Strings &right); // сцепление
    Strings Merge(const Strings &right); // слияние
    bool Found(const char *s, int frombeginning); // есть ли в данном массиве эта строка
    void Print();
    void Print(int i);
private:
    int number, *length;
    char **str;
    static std::shared_ptr<char> empty;
};
std::shared_ptr<char> Strings::empty = std::make_shared<char>('\0');
 
Strings::Strings()
{
    cout << "Kolichestvo strok: ";
    cin >> number;
    length = new int[number];
    str = new char*[number];
    for (int i = 0; i < number; i++)
    {
        cout << "Stroka nr. " << i+1 << endl
            << "Dlina: ";
        cin >> length[i];
        str[i] = new char[length[i]+1]; // +1 для завершающего нулевого символа
        cout << "Stroka: ";
        //fflush(stdin); // чистим входной поток, т.к. там застревает конец строки после ввода числа, и строка не читается
        cin.getline(str[i], length[i]+1);
    }
}
 
Strings::Strings(int n)
{
    number = n;
    length = new int[n];
    std::fill_n(length, n, 0);
    str = new char*[n];
    std::fill_n(str, n, nullptr);
}
 
Strings::Strings(const Strings &right, int n)
{
    if (n == 0)
        number = right.number; // копируем количество строк
    else
        number = n;
    length = new int[number]; // выделяем память под длины
    str = new char*[number]; // выделяем память под строку
    for (int i = 0; i < number; i++)
    {
        length[i] = right.length[i]; // копируем длину текущей строки
        str[i] = new char[length[i]+1]; // выделяем память под текущую строку (+1 для завершающего нулевого символа)
        strcpy(str[i], right.str[i]); // копируем строку
    }
}
 
Strings::~Strings()
{
    delete[] length;
    for (int i = 0; i < number; i++)
        delete[] str[i];
    delete[] str;
}
 
char* Strings::at(int i) const
{
    //static char newchar[1] = "";
    if (i < 0 || i >= number) // если выходит за границы
    {
        return empty.get(); // возвращаем пустую строку
    }
    return str[i];
}
 
Strings Strings::Append(const Strings &right)
{
    Strings res(number > right.number? number : right.number); // берем количество элементов в большем массиве
    for (int i = 0; i < number || i < right.number; i++) // пока не закончился больший массив
    {
        res.length[i] = strlen(at(i)) + strlen(right.at(i)); // at вернет пустую строку, если выйдем за границы массива
        res.str[i] = new char[res.length[i]+1];
        strcpy(res.at(i), at(i)); // копируем i-тую строку из первого массива
        strcat(res.at(i), right.at(i)); // и добавляем к ней строку из втрого массива
    }
    return res;
}
 
bool Strings::Found(const char *s, int frombeginning)
{
    for (int i = 0; i < frombeginning && i < number; i++)
        if (strcmp(str[i], s) == 0)
            return true;
    return false;
}
 
Strings Strings::Merge(const Strings &right)
{
    Strings res(number + right.number); // выделяем память для максимально возможного количества элементов
    int realnumber = 0; // запоминаем сколько на самом деле скопировалось
    for (int i = 0; i < number; i++)
        if (!res.Found(str[i], realnumber))
        {
            res.length[realnumber] = length[i];
            res.str[realnumber] = new char[res.length[realnumber]+1];
            strcpy(res.str[realnumber], str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
            realnumber++;
        };
    for (int i = 0; i < right.number; i++)
        if (!res.Found(right.str[i], realnumber))
        {
            res.length[realnumber] = right.length[i];
            res.str[realnumber] = new char[res.length[realnumber]+1];
            strcpy(res.str[realnumber], right.str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
            realnumber++;
        };
    for (int i = realnumber; i < res.number; i++)
        res.str[i] = nullptr;
    // заполняем оставшиеся строки, т.к. res в конце функции удаляется, а значит все строки удаляются. чтобы нормально сработал delete[], надо выделить память.
    return Strings(res, realnumber); // скопировались не все элементы, возвращаем массив строк без лишних элементов
}
 
void Strings::Print()
{
    cout << "V massive soderjitsa " << number << " stroki:\n";
    for (int i = 0; i < number; i++)
        cout << str[i] << endl;
}
 
void Strings::Print(int i)
{
    cout << at(i) << endl;
}
 
int Menu()
{
    int vybor;
    //system("cls");
    cout << "1 - Vyvesti massivy na ekran\n"
        << "2 - Vyvesti otdelinuiu stroku po indeksu\n"
        << "3 - Poelementnoe stseplenie\n"
        << "4 - Sliyanie s iskliucheniem povtorenii\n"
        << "0 - Vyhod iz programmy\n"
        << "Vash vybor: ";
    cin >> vybor;
    cout << endl;
    return vybor;
}
 
int main()
{
    cout << "Pervyi massiv\n";
    Strings s1;
    cout << "\nVtoroi massiv\n";
    Strings s2;
    int idx;
    char *str;
 
    while (true)
    {
        switch (Menu())
        {
        case 1: s1.Print();
            s2.Print();
            break;
        case 2: cout << "Vvedite index stroki v pervom massive: ";
            cin >> idx;
            str = s1.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            cout << "Vvedite index stroki vo vtorom massive: ";
            cin >> idx;
            str = s2.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            break;
        case 3: s1.Append(s2).Print();
            break;
        case 4: s1.Merge(s2).Print();
            break;
        case 0: return 0;
        }
        //system("pause");
    }
}
0
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
30.11.2014, 13:57  [ТС] 9
это задание надо сделать на элементарном уровне, без библиотек. аа известно где конкретно ошибка?
0
Max Dark
шКодер самоучка
1992 / 1760 / 869
Регистрация: 09.10.2013
Сообщений: 3,887
Записей в блоге: 6
Завершенные тесты: 2
30.11.2014, 14:06 10
Цитата Сообщение от Керра Посмотреть сообщение
известно где конкретно ошибка?
Bash
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
max@debi:~/projects$ g++ -std=c++03 -Wall -pedantic -Werror slist.cpp
slist.cpp: In constructor ‘Strings::Strings()’:
slist.cpp:35:16: error: ‘stdin’ was not declared in this scope
         fflush(stdin); // чистим входной поток, т.к. там застревает конец строки после ввода числа, и строка не читается
                ^
slist.cpp:35:21: error: ‘fflush’ was not declared in this scope
         fflush(stdin); // чистим входной поток, т.к. там застревает конец строки после ввода числа, и строка не читается
                     ^
slist.cpp: In copy constructor ‘Strings::Strings(Strings&, int)’:
slist.cpp:59:36: error: ‘strcpy’ was not declared in this scope
         strcpy(str[i], right.str[i]); // копируем строку
                                    ^
slist.cpp: In member function ‘char*& Strings::at(int)’:
slist.cpp:73:11: error: reference to local variable ‘newchar’ returned [-Werror=return-local-addr]
     char *newchar;
           ^
slist.cpp: In member function ‘Strings Strings::Append(Strings&)’:
slist.cpp:89:37: error: ‘strlen’ was not declared in this scope
         res.length[i] = strlen(at(i)) + strlen(right.at(i)); // at вернет пустую строку, если выйдем за границы массива
                                     ^
slist.cpp:91:32: error: ‘strcpy’ was not declared in this scope
         strcpy(res.at(i), at(i)); // копируем i-тую строку из первого массива
                                ^
slist.cpp:92:38: error: ‘strcat’ was not declared in this scope
         strcat(res.at(i), right.at(i)); // и добавляем к ней строку из втрого массива
                                      ^
slist.cpp: In member function ‘bool Strings::Found(char*, int)’:
slist.cpp:100:29: error: ‘strcmp’ was not declared in this scope
         if (strcmp(str[i], s) == 0)
                             ^
slist.cpp: In member function ‘Strings Strings::Merge(Strings&)’:
slist.cpp:114:47: error: ‘strcpy’ was not declared in this scope
             strcpy(res.str[realnumber], str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
                                               ^
slist.cpp:122:53: error: ‘strcpy’ was not declared in this scope
             strcpy(res.str[realnumber], right.str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
                                                     ^
slist.cpp:128:35: error: no matching function for call to ‘Strings::Strings(Strings)’
     return Strings(res, realnumber); // скопировались не все элементы, возвращаем массив строк без лишних элементов
                                   ^
slist.cpp:128:35: note: candidates are:
slist.cpp:47:1: note: Strings::Strings(Strings&, int)
 Strings::Strings(Strings &right, int n)
 ^
slist.cpp:47:1: note:   no known conversion for argument 1 from ‘Strings’ to ‘Strings&’
slist.cpp:40:1: note: Strings::Strings(int)
 Strings::Strings(int n)
 ^
slist.cpp:40:1: note:   no known conversion for argument 1 from ‘Strings’ to ‘int’
slist.cpp:22:1: note: Strings::Strings()
 Strings::Strings()
 ^
slist.cpp:22:1: note:   candidate expects 0 arguments, 1 provided
slist.cpp: In function ‘int Menu()’:
slist.cpp:146:17: error: ‘system’ was not declared in this scope
     system("cls");
                 ^
slist.cpp: In function ‘int main()’:
slist.cpp:177:27: error: ‘strlen’ was not declared in this scope
             if (strlen(str) == 0)
                           ^
slist.cpp:184:27: error: ‘strlen’ was not declared in this scope
             if (strlen(str) == 0)
                           ^
slist.cpp:195:23: error: ‘system’ was not declared in this scope
         system("pause");
                       ^
slist.cpp: In member function ‘Strings Strings::Merge(Strings&)’:
slist.cpp:129:1: error: control reaches end of non-void function [-Werror=return-type]
 }
 ^
cc1plus: all warnings being treated as errors
0
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
30.11.2014, 14:17  [ТС] 11
ну, здесь all warnings being treated as errors. а ошибка, которая приводит к этому heap corruption detected?
0
Max Dark
шКодер самоучка
1992 / 1760 / 869
Регистрация: 09.10.2013
Сообщений: 3,887
Записей в блоге: 6
Завершенные тесты: 2
30.11.2014, 14:29 12
Керра, я веду к тому что Ваш код тупо не собирается
Даже если не включать преобразование предупреждений в ошибки.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
max@debi:~/projects$ g++ slist.cpp
slist.cpp: In member function ‘char*& Strings::at(int):
slist.cpp:77:11: warning: reference to local variable ‘newchar’ returned [-Wreturn-local-addr]
     char *newchar;
           ^
slist.cpp: In member function ‘Strings Strings::Merge(Strings&):
slist.cpp:132:35: error: no matching function for call to ‘Strings::Strings(Strings)’
     return Strings(res, realnumber); // скопировались не все элементы, возвращаем массив строк без лишних элементов
                                   ^
slist.cpp:132:35: note: candidates are:
slist.cpp:51:1: note: Strings::Strings(Strings&, int)
 Strings::Strings(Strings &right, int n)
 ^
slist.cpp:51:1: note:   no known conversion for argument 1 from ‘Strings’ to ‘Strings&’
slist.cpp:44:1: note: Strings::Strings(int)
 Strings::Strings(int n)
 ^
slist.cpp:44:1: note:   no known conversion for argument 1 from ‘Strings’ to ‘int’
slist.cpp:26:1: note: Strings::Strings()
 Strings::Strings()
 ^
slist.cpp:26:1: note:   candidate expects 0 arguments, 1 provided
Добавлено через 7 минут
опять же 78я строка - утечка памяти
0
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
30.11.2014, 14:42  [ТС] 13
Cra3y, нашла ошибку - в Found передавалось значение сколько первых строк сравнивать, ибо не для всех еще выделена память. но если передавался 0, который передавался по умолчанию, то значение менялось на number. хотя вначале выделена память только под 0 строк. исправила значение по умолчанию на -1, и в условии, соответственно, на -2 - работает без нового копировщика. копировщик попробовала - благодаря нему почему-то компилятор наконец-то указал на нужную строку в Found. не знаю как это связано... но теперь все работает как надо
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
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
#include <iostream>
using namespace std;
 
class Strings
{
public:
    Strings(); // конструктор со вводом с клавиатуры
    Strings(int n); // создается массив строк размера n, и массив длин, память под строки не выделяется
    Strings(Strings &right, int n = -1); // конструктор-копировщик для нормальной передачи нового массива из функции, n - количество копируемых элементов
    ~Strings();
    char*& at(int i);
    Strings Append(Strings &right); // сцепление
    Strings Merge(Strings &right); // слияние
    bool Found(char *s, int frombeginning = -1); // есть ли в данном массиве эта строка
    void Print();
    void Print(int i);
private:
    int number, *length;
    char **str;
};
 
Strings::Strings()
{
    cout << "Kolichestvo strok: ";
    cin >> number;
    length = new int[number];
    str = new char*[number];
    for (int i = 0; i < number; i++)
    {
        cout << "Stroka nr. " << i+1 << endl
            << "Dlina: ";
        cin >> length[i];
        str[i] = new char[length[i]+1]; // +1 для завершающего нулевого символа
        cout << "Stroka: ";
        fflush(stdin); // чистим входной поток, т.к. там застревает конец строки после ввода числа, и строка не читается
        cin.getline(str[i], length[i]+1);
    }
}
 
Strings::Strings(int n)
{
    number = n;
    length = new int[n];
    str = new char*[n];
}
 
Strings::Strings(Strings &right, int n)
{
    if (n == -1)
        number = right.number;
    else
        number = n;
    length = new int[number]; // выделяем память под длины
    str = new char*[number]; // выделяем память под строку
    for (int i = 0; i < number; i++)
    {
        length[i] = right.length[i]; // копируем длину текущей строки
        str[i] = new char[length[i]+1]; // выделяем память под текущую строку (+1 для завершающего нулевого символа)
        strcpy(str[i], right.str[i]); // копируем строку
    }
}
 
Strings::~Strings()
{
    delete[] length;
    for (int i = 0; i < number; i++)
        delete[] str[i];
    delete[] str;
}
 
char*& Strings::at(int i)
{
    char *newchar;
    if (i < 0 || i >= number) // если выходит за границы
    {
        newchar = new char;
        *newchar = '\0';
        return newchar; // возвращаем пустую строку
    }
    else
        return str[i];
}
 
Strings Strings::Append(Strings &right)
{
    Strings res(number > right.number? number : right.number); // берем количество элементов в большем массиве
    for (int i = 0; i < number || i < right.number; i++) // пока не закончился больший массив
    {
        res.length[i] = strlen(at(i)) + strlen(right.at(i)); // at вернет пустую строку, если выйдем за границы массива
        res.str[i] = new char[res.length[i]+1];
        strcpy(res.at(i), at(i)); // копируем i-тую строку из первого массива
        strcat(res.at(i), right.at(i)); // и добавляем к ней строку из втрого массива
    }
    return res;
}
 
bool Strings::Found(char *s, int frombeginning)
{
    if (frombeginning == -1)
        frombeginning = number;
    for (int i = 0; i < frombeginning && i < number; i++)
        if (strcmp(str[i], s) == 0)
            return true;
    return false;
}
 
Strings Strings::Merge(Strings &right)
{
    Strings res(number + right.number); // выделяем память для максимально возможного количества элементов
    int realnumber = 0; // запоминаем сколько на самом деле скопировалось
    for (int i = 0; i < number; i++)
        if (!res.Found(str[i], realnumber))
        {
            res.length[realnumber] = length[i];
            res.str[realnumber] = new char[res.length[realnumber]+1];
            strcpy(res.str[realnumber], str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
            realnumber++;
        };
    for (int i = 0; i < right.number; i++)
        if (!res.Found(right.str[i], realnumber))
        {
            res.length[realnumber] = right.length[i];
            res.str[realnumber] = new char[res.length[realnumber]+1];
            strcpy(res.str[realnumber], right.str[i]); // копируем и сразу увеличиваем реальное количество элементов в результативном массиве
            realnumber++;
        };
    for (int i = realnumber; i < res.number; i++)
        res.str[i] = new char[1];
    // заполняем оставшиеся строки, т.к. res в конце функции удаляется, а значит все строки удаляются. чтобы нормально сработал delete[], надо выделить память.
    return Strings(res, realnumber); // скопировались не все элементы, возвращаем массив строк без лишних элементов
}
 
void Strings::Print()
{
    cout << "V massive soderjitsa " << number << " stroki:\n";
    for (int i = 0; i < number; i++)
        cout << str[i] << endl;
}
 
void Strings::Print(int i)
{
    cout << at(i) << endl;
}
 
int Menu()
{
    int vybor;
    system("cls");
    cout << "1 - Vyvesti massivy na ekran\n"
        << "2 - Vyvesti otdelinuiu stroku po indeksu\n"
        << "3 - Poelementnoe stseplenie\n"
        << "4 - Sliyanie s iskliucheniem povtorenii\n"
        << "0 - Vyhod iz programmy\n"
        << "Vash vybor: ";
    cin >> vybor;
    cout << endl;
    return vybor;
}
 
int main()
{
    cout << "Pervyi massiv\n";
    Strings s1;
    cout << "\nVtoroi massiv\n";
    Strings s2;
    int idx;
    char *str;
 
    while (true)
    {
        switch (Menu())
        {
        case 1: s1.Print();
            s2.Print();
            break;
        case 2: cout << "Vvedite index stroki v pervom massive: ";
            cin >> idx;
            str = s1.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            cout << "Vvedite index stroki vo vtorom massive: ";
            cin >> idx;
            str = s2.at(idx);
            if (strlen(str) == 0)
                cout << "Vy vveli nekorrektnyi index!\n";
            else
                cout << "\"" << str << "\"\n";
            break;
        case 3: s1.Append(s2).Print();
            break;
        case 4: s1.Merge(s2).Print();
            break;
        case 0: return 0;
        }
        system("pause");
    }
}
Добавлено через 7 минут
вот только с этой утечкой непонятно что делать. разве что создавать лишнюю пустую строку, чтобы она потом удалилась с остальными.
0
Max Dark
шКодер самоучка
1992 / 1760 / 869
Регистрация: 09.10.2013
Сообщений: 3,887
Записей в блоге: 6
Завершенные тесты: 2
30.11.2014, 14:59 14
Керра, проверяйте индекс до обращения к массиву и эта "пустая строка" вам не понадобится
0
MayaNash
1291 / 460 / 151
Регистрация: 24.08.2011
Сообщений: 2,248
30.11.2014, 15:11  [ТС] 15
это же часть функционала класса
0
30.11.2014, 15:11
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
30.11.2014, 15:11

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

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

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


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

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

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