Странное повреждение кучи

@NickCoolberg · Регистрация: 12.03.2017

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

Доброго времени суток, форумчане!

Есть проект, в нем реализована функцию дублирования всех элементов статического массива с исключением:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
bool CheckAndDubl(double *arr, int &ArrLen) {
 
    // проверка на возможность дублирования
    if (ArrLen * 2 > N) {
        throw "Ошибка выполнения: удлиненный массив больше, чем его длинна";
    }
    int NewLen = ArrLen;
    for (int i = 0; i < ArrLen * 2; i += 2) {
        for (int j = NewLen - 1; j > i; j--) {
            arr[j + 1] = arr[j];
        }
        arr[i + 1] = arr[i];
        NewLen++;
    }
    ArrLen = NewLen;
    return 0;
}

Сама функция работает и свое дело выполняет.
Далее я тестирую эту функция при помощи библиотеки cassert:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
bool test_CheckAndDubl() {
 
    int Len = 2;
    double d[] = { 4, 3, 1 }; // Результат {4, 4, 3, 3, 1, 1}
 
    CheckAndDubl(d, Len);
    assert((d[0] = 4) && (d[1] = 4) && (d[2] = 3) && (d[3] = 3) && (d[4] = 1) && (d[5] = 1));
 
    // В случаях, где длина нового массива больше, чем глобальная константа, выбрасывается исключение
 
    cout << "Test CheckAndDubl = OK" << endl;
    
    return true;
}

И в данном случае происходит повреждение кучи, которое я не могу как-то оправдать:

Run-Time Check Failure #2 - Stack around the variable 'Len' was corrupted.

Помогите, пожалуйста, как исправить данную проблему?

@likehood · 30.03.2017, 17:16

1) Статический массив нельзя расширить, так что функция CheckAndDubl некорректна.
2) В ассерте происходит выход за границы массива, длина которого равна трём.
3) Вместо сравнения (==) у вас идут присваивания (в 7 строке).

@NickCoolberg · 30.03.2017, 17:38 **[ТС]**

1) Задумался на Вашей подсказкой и понял, что я задаю неверные массивы
Переписал это:

C++
1
2
int Len = 3;
double d[] = { 4, 3, 1 };

На вот это:

C++
1
2
3
int Len = 3;
double d[N];
d[0] = 4; d[1] = 3; d[2] = 1;

Теперь задаю такой массив, который мне был нужен и функция работает верно
3) Да, точно, я присваивал значения элементам, а не сравнивал их

Теперь тесты проходят верно, спасибо!

@likehood · 30.03.2017, 17:42

Сообщение от NickCoolberg

Переписал это:

Сообщение от NickCoolberg

На вот это:

Что-то не вижу особой разницы, оба кода делают одно и то же. Первый как-то покомпактнее.

@NickCoolberg · 30.03.2017, 17:48 **[ТС]**

Второй код задает массив длинной N (N=20) и после я его заполняю, а первый задавал массив уже определенной длинны

@TheCalligrapher · 30.03.2017, 17:51

Delete

@likehood · 30.03.2017, 17:52

Сообщение от NickCoolberg

Второй код задает массив длинной N

Точно, не заметил.
Хотя, эти подходы можно объединить:

C++
1
2
3
const int N = 20;
int Len = 3;
double d[N] = { 1, 2, 3 }; // размер равен 20

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера 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. Пошагово создадим проект для загрузки изображения. . .

@likehood 1272 / 1029 / 470 Регистрация: 25.12.2016 Сообщений: 3,333
	30.03.2017, 17:16
	1) Статический массив нельзя расширить, так что функция CheckAndDubl некорректна. 2) В ассерте происходит выход за границы массива, длина которого равна трём. 3) Вместо сравнения (`==`) у вас идут присваивания (в 7 строке). 0

@NickCoolberg 0 / 0 / 0 Регистрация: 12.03.2017 Сообщений: 6
	30.03.2017, 17:48 [ТС]
	Второй код задает массив длинной N (N=20) и после я его заполняю, а первый задавал массив уже определенной длинны 0

@TheCalligrapher Вездепух 12938 / 6805 / 1821 Регистрация: 18.10.2014 Сообщений: 17,224
	30.03.2017, 17:51
	Delete 0