Debug Assertion Failed

@andreyananas · Регистрация: 15.10.2013

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

Ошибка. Не могу понять что не так. Как я понял с дебагера, проблема во втором освобождении памяти. Вообщем хелп ми)

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
// hw_3.cpp : Defines the entry point for the console application.
// Пользователь вводит строки пока не надоест
 
#include "stdafx.h"
#include <iostream>
#include <iomanip>
 
using namespace std;
 
int _tmain(int argc, _TCHAR* argv[])
{
     int str_size = 100, n_str = 4, Index_pc(0);
     char ** pStr = new char*[n_str];
     for (int i(0); i < n_str; ++i)
         pStr[i] = new char[str_size];
 
     while (true)
     {
         cin.getline(pStr[Index_pc++], str_size);
         if (Index_pc == n_str)
         {
             char ** pTemp = new char*[n_str + n_str / 2];
            for (int i(0); i < n_str + n_str / 2; ++i)
                 pTemp[i] = new char[str_size];
 
             memcpy(pTemp, pStr, sizeof(int)*n_str);
 
             for (int i(0); i < n_str; ++i)
                 delete[] pStr[i];
             delete[] pStr;
 
             pStr = pTemp;
             cout << " memalloc old size " << n_str << " new " << n_str + n_str / 2 << endl;
             n_str += n_str / 2;
         }
     }
    return 0;
}

Croessmah · 20.07.2014, 06:29

C++
1
memcpy(pTemp, pStr, sizeof(int)*n_str);

точно int? Может char?

@uglyPinokkio · 20.07.2014, 07:23

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

Не могу понять что не так.

Заводим массив указателей на строки и строки.

C++
1
2
3
 char ** pTemp = new char*[n_str + n_str / 2];
            for (int i(0); i < n_str + n_str / 2; ++i)
                 pTemp[i] = new char[str_size];

Копируем старый массив указателей, при этом теряем память, отведенную выше

C++
1
memcpy(pTemp, pStr, sizeof(int)*n_str);

После чего старые строки удаляем, но указатели на них уже скопированы в pTemp.

C++
1
2
for (int i(0); i < n_str; ++i)
                 delete[] pStr[i];

На следующей итерации эти же строки будут удаляться еще раз.

Вто тут:

C++
1
memcpy(pTemp, pStr, sizeof(int)*n_str);

Копировать надо не указатели, а сами строки.

@andreyananas · 20.07.2014, 08:02 **[ТС]**

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

точно int? Может char?

я так и не понял тут int или char???

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

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

Копировать надо не указатели, а сами строки.

Ничего я не понял, напиши как по твоему правильно будет, плс

@uglyPinokkio · 20.07.2014, 08:16

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

Ничего я не понял

Вот тут ты копируешь только адреса строк:

C++
1
memcpy(pTemp, pStr, sizeof(int)*n_str);

Вместо этого надо копировать данные:

C++
1
2
for (int i=0; i < n_str; ++i)
   strncpy(pTemp[i],pStr[i],str_size);

@andreyananas · 20.07.2014, 08:43 **[ТС]**

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

strncpy

??? Почему не memcpy?

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

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

memcpy(pTemp, pStr, sizeof(int)*n_str);

Так это вообще не нужно делать?

Добавлено через 15 минут
ВООБЩЕМ ВСЕ ПОПРОБОВАЛ, НИЧЕГО НЕ ПОМОГАЕТ(((
ХЕЛП)

@uglyPinokkio · 20.07.2014, 08:47

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

Почему не memcpy?

Ну можешь memcpy, для копирования строк обычно используют strncpy, она не копирует лишнего.

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

Так это вообще не нужно делать?

Конечно не нужно

@andreyananas · 20.07.2014, 09:04 **[ТС]**

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

Конечно не нужно

Заработало, спс) правда это все меня немного запутало.
Почему cin.getline не ставит в конце строки '\0' ??? Самому нужно ставить???

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

C++
1
2
3
             for (int i(0); i < n_str; ++i)
                 delete[] pStr[i];
             delete[] pStr;

Правильное это освобождение памяти?

@uglyPinokkio · 20.07.2014, 09:16

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

Почему cin.getline не ставит в конце строки '\0'

Кто сказал?

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

Правильное это освобождение памяти?

Правильное.

@andreyananas · 20.07.2014, 09:22 **[ТС]**

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

Кто сказал?

когда вывожу скопированные строки, после слов идет какая то линия...

@uglyPinokkio · 20.07.2014, 09:33

Должен добавлять:

http://www.cplusplus.com/refer... m/getline/

A null character ('\0') is automatically appended to the written sequence if n is greater than zero, even if an empty string is extracted.

@andreyananas · 20.07.2014, 09:35 **[ТС]**

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

Должен добавлять:
http://www.cplusplus.com/refer... m/getline/
A null character ('\0') is automatically appended to the written sequence if n is greater than zero, even if an empty string is extracted.

Может копирование удаляет?

@zss · 20.07.2014, 09:47

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

memcpy(pTemp, pStr, sizeof(int)*n_str);

Эта операция копирует указатели из pStr в pTemp и, следовательно,
стирает указатели, которые были выделены для pTemp -> потеряли эту память.

@andreyananas · 20.07.2014, 10:46 **[ТС]**

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

Эта операция копирует указатели из pStr в pTemp и, следовательно,
стирает указатели, которые были выделены для pTemp -> потеряли эту память.

Ну да, и что? Я же вроде не об этом спрашивал.

Добавлено через 40 минут
Объясните что с '\0'
плс

@uglyPinokkio · 20.07.2014, 11:54

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

Объясните что с '\0'

Ну так код исправленный где? В исходном я вывода не вижу.

@andreyananas · 20.07.2014, 13:16 **[ТС]**

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
// hw_3.cpp : Defines the entry point for the console application.
// Пользователь вводит строки пока ненадоест
 
#include "stdafx.h"
#include <iostream>
#include <iomanip>
 
using namespace std;
 
int _tmain(int argc, _TCHAR* argv[])
{
     int str_size = 100, n_str = 4, Index_pc(0);
     char ** pStr = new char*[n_str];
     for (int i(0); i < n_str; ++i)
         pStr[i] = new char[str_size];
 
     while (true)
     {
         cin.getline(pStr[Index_pc++], str_size);
         if (Index_pc == n_str)
         {
             char ** pTemp = new char*[n_str + n_str / 2];
             for (int i(0); i < n_str + n_str / 2; ++i)
                 pTemp[i] = new char[str_size];
 
             for (int i(0); i < n_str; ++i)
                 memcpy(pTemp[i], pStr[i], sizeof(char)*str_size);
 
 
             for (int i(0); i < n_str; ++i)
                 delete[] pStr[i];
             delete[] pStr;
 
             pStr = pTemp;
             for (int i(0); i < n_str; ++i)
             {
                 for (int j(0); j < 20; ++j)
                     cout << pStr[i][j];
                 cout << endl;
             }
             cout << " memalloc old size " << n_str << " new " << n_str + n_str / 2 << endl;
             n_str += n_str / 2;
         }
 
     }
 
         return 0;
}

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

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

Ну так код исправленный где? В исходном я вывода не вижу

Вот

@uglyPinokkio · 20.07.2014, 17:32

C++
1
2
 for (int j(0); j < 20; ++j)
   cout << pStr[i][j];

Вот тут выводится 20 символов независимо от длины строки. Причем тут ноль?

C++
1
   cout << pStr[i];

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера 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, то после закрытия окошка. . .

@uglyPinokkio 327 / 230 / 55 Регистрация: 30.05.2014 Сообщений: 682
	20.07.2014, 09:33
	Должен добавлять: http://www.cplusplus.com/refer... m/getline/ A null character ('\0') is automatically appended to the written sequence if n is greater than zero, even if an empty string is extracted. 0