Heap corruption detected

@MayaNash · Регистрация: 24.08.2011

Студворк — интернет-сервис помощи студентам

Возникает эта ошибка при выполнении функции 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");
    }
}

@Max Dark · 30.11.2014, 11:16

Керра, 73я строка - утечка памяти,
87я строка - не резервируется память под '\0'

@MayaNash · 30.11.2014, 11:52 **[ТС]**

Теперь еще одна проблема - в конце функции Strings::Merge удаляется временная переменная, которая создается при return, и res. И опять та же ошибка, при вызове деструктора от res, на строке delete[] str[i] на первом же элементе массива. Что опять?

@Max Dark · 30.11.2014, 12:10

Керра, Отсутствует конструктор копирования(Strings::Strings(const Strings& cpy))
в Merge Вы не выделяете память под сами строки, только под массив длин и массив указаделей

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

Не по теме:

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

@MayaNash · 30.11.2014, 12:20 **[ТС]**

Не по теме:

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

Strings(Strings &right, int n = 0) - это копировщик, разве он не сработает как обычный, в нем же второе значение по умолчанию

@Max Dark · 30.11.2014, 12:25

Не по теме:

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

Сообщение от Керра

Strings(Strings &right, int n = 0)

не совпадает с сигнатурой конструктора копирования type::type(const type&)

@MayaNash · 30.11.2014, 12:44 **[ТС]**

однако, именно этот копировщик и вызывается

Добавлено через 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");
    }
}

@Max Dark · 30.11.2014, 13:37

так вот хотя бы не падает...

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");
    }
}

@MayaNash · 30.11.2014, 13:57 **[ТС]**

это задание надо сделать на элементарном уровне, без библиотек. аа известно где конкретно ошибка?

@Max Dark · 30.11.2014, 14:06

Сообщение от Керра

известно где конкретно ошибка?

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

@MayaNash · 30.11.2014, 14:17 **[ТС]**

ну, здесь all warnings being treated as errors. а ошибка, которая приводит к этому heap corruption detected?

@Max Dark · 30.11.2014, 14:29

Керра, я веду к тому что Ваш код тупо не собирается
Даже если не включать преобразование предупреждений в ошибки.

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я строка - утечка памяти

@MayaNash · 30.11.2014, 14:42 **[ТС]**

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 минут
вот только с этой утечкой непонятно что делать. разве что создавать лишнюю пустую строку, чтобы она потом удалилась с остальными.

@Max Dark · 30.11.2014, 14:59

Керра, проверяйте индекс до обращения к массиву и эта "пустая строка" вам не понадобится

@MayaNash · 30.11.2014, 15:11 **[ТС]**

это же часть функционала класса

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,793 Записей в блоге: 4
	30.11.2014, 11:16
	Керра, 73я строка - утечка памяти, 87я строка - не резервируется память под '\0' 1

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	30.11.2014, 11:52 [ТС]
	Теперь еще одна проблема - в конце функции Strings::Merge удаляется временная переменная, которая создается при return, и res. И опять та же ошибка, при вызове деструктора от res, на строке delete[] str[i] на первом же элементе массива. Что опять? 0

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,793 Записей в блоге: 4
	30.11.2014, 12:10
	Керра, Отсутствует конструктор копирования(Strings::Strings(const Strings& cpy)) в Merge Вы не выделяете память под сами строки, только под массив длин и массив указаделей Добавлено через 1 минуту Не по теме: хочется нажать ctrl+A, DEL ... 1

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	30.11.2014, 12:20 [ТС]
	Не по теме: Cra3y, ну не надо так, у студентов код не может быть качественным, за редкими исключениями, учимся жеж Strings(Strings &right, int n = 0) - это копировщик, разве он не сработает как обычный, в нем же второе значение по умолчанию 0

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	30.11.2014, 13:57 [ТС]
	это задание надо сделать на элементарном уровне, без библиотек. аа известно где конкретно ошибка? 0

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	30.11.2014, 14:17 [ТС]
	ну, здесь all warnings being treated as errors. а ошибка, которая приводит к этому heap corruption detected? 0

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,793 Записей в блоге: 4
	30.11.2014, 14:59
	Керра, проверяйте индекс до обращения к массиву и эта "пустая строка" вам не понадобится 0

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	30.11.2014, 15:11 [ТС]
	это же часть функционала класса 0