Форум программистов, компьютерный форум CyberForum.ru

Программа которая изменяет динамический массив строк - Написал, но есть ошибка - C++

Восстановить пароль Регистрация
 
dimcoder
Полярный
 Аватар для dimcoder
449 / 422 / 66
Регистрация: 11.09.2011
Сообщений: 1,108
23.05.2012, 10:48     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #1
Всем доброго времени суток. В общем сделал программу, которая вкратце делает следующее:
1) Создает динамических массив строк char **m
2) Просит юзера вводить строки, когда закончит нажать ctrl+z и Enter
a) После ввода строки, вызывается функция добавления этой строки в массив m
3) После ввода программа выводит полученный массив

Но раз уж я здесь, значит есть косяк. Когда ввожу строки (ввел строку, нажал ввод), после некоторого числа строк (варьируется 3-15), программа просто виснет. Стоит секунд 20-30, как будто зациклилась(см. скриншот). Вводить ничего не получается. Потом вылетает. Работаю в DevCpp, на Borland'е тоже испытал - та же хрень. Думаю что что-то намудрил с памятью, хотя проверял неоднократно. Собственно код:

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
#include <cstdlib>
#include <iostream>
#include <ctime>
 
using namespace std;
 
void add_element(char ***m, int &k, const char *s)
{
    char **buf = new char*[k];                //Delaem copiyu
    for (int i = 0; i < k; i++)
        buf[i] = new char[strlen((*m)[i])];
    for (int i = 0; i < k; i++)
        strcpy(buf[i], (*m)[i]);
        
    for (int i = 0; i < k; i++)              //Ochischaem pamyat'
        delete [] ((*m)[i]);  
    delete [] (*m);
    
    (*m) = new char*[k+1];                   //Sozdaem novyu massiv
    for (int i = 0; i < k; i++)
        (*m)[i] = new char[strlen(buf[i])];
    (*m)[k] = new char[strlen(s)];
    
    for (int i = 0; i < k; i++)               //Zapolnyaem ego iz kopii
        strcpy((*m)[i], buf[i]);
    strcpy((*m)[k], s);
    
    for (int i = 0; i < k; i++)                //Ochischaem pamyat'
        delete [] (buf[i]);  
    delete [] buf;
    k++;
}
 
int main(int argc, char *argv[])
{
    int k = 0;
    char **m = new char*[k];               
    for (int i = 0; i < k; i++)
        m[i] = new char[k];
    char s[200];
    
    while (cin >> s)                      //Vvodym stroku...
    {
        add_element(&m, k, s);           //Zapysyvaem ee v massiv
    }
    
    cout << k << endl;            //Skol'ko v obschem strok vveli 
    
    for (int i = 0; i < k; i++)         //Vyvod samih strok
        cout << m[i] << endl; 
                  
    for (int i = 0; i < k; i++)          //Osvobozhdaem pamyat'
        delete [] (m[i]);  
    delete [] m;
    
    system("PAUSE > NULL");
    return EXIT_SUCCESS;
}
Надеюсь на помощь. Буду очень благодарен.
Миниатюры
Программа которая изменяет динамический массив строк - Написал, но есть ошибка  
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
23.05.2012, 10:48     Программа которая изменяет динамический массив строк - Написал, но есть ошибка
Посмотрите здесь:

C++ Двухмерный динамический массив строк
C++ Динамический массив строк
C++ динамический массив строк
Программа, которая помещает в массив строк текст, введенный пользователем C++
C++ Динамический массив для строк
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
antoha398
155 / 155 / 3
Регистрация: 29.03.2012
Сообщений: 418
23.05.2012, 11:30     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #2
зачем вам в функции такой параметр char ***m? почему бы не сделать так char **m, тогда обращеие к строке будет m[i], и вызов функции будет add_elem(m,k,s)
go
Эксперт C++
3582 / 1362 / 128
Регистрация: 16.04.2009
Сообщений: 4,528
23.05.2012, 11:55     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #3
Цитата Сообщение от dimcoder Посмотреть сообщение
char **buf = new char*[k]; * * * * * * * *//Delaem copiyu
* * for (int i = 0; i < k; i++)
* * * * buf[i] = new char[strlen((*m)[i])];
Есть strdup

Да и код не хорошо написан. Вам нужно написать, что-то вроде realloc в Си.
antoha398
155 / 155 / 3
Регистрация: 29.03.2012
Сообщений: 418
23.05.2012, 11:59     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #4
еще если не ошибаюсь strlen возвращает длину без завершающего символа, strcpy копирует строку вместе с ним.
dimcoder
Полярный
 Аватар для dimcoder
449 / 422 / 66
Регистрация: 11.09.2011
Сообщений: 1,108
23.05.2012, 21:03  [ТС]     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #5
Цитата Сообщение от antoha398 Посмотреть сообщение
зачем вам в функции такой параметр char ***m?
Если передать не так, то получиться копия массива, который был в main(), и следовательно массив в main() функция не изменит. Только память получиться очистить так.
Цитата Сообщение от go Посмотреть сообщение
Да и код не хорошо написан.
Почему?
Цитата Сообщение от go Посмотреть сообщение
Вам нужно написать, что-то вроде realloc в Си.
Вот за это спасибо!
Цитата Сообщение от antoha398 Посмотреть сообщение
еще если не ошибаюсь strlen возвращает длину без завершающего символа
Во, точно - как я так это пропустил?! В этом и была ошибка! Спасибо большое!
antoha398
155 / 155 / 3
Регистрация: 29.03.2012
Сообщений: 418
23.05.2012, 21:33     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #6
Цитата Сообщение от dimcoder Посмотреть сообщение
Если передать не так, то получиться копия массива, который был в main(), и следовательно массив в main() функция не изменит. Только память получиться очистить так.
**m это указатель, никакой копии массива не будет. Функция будет работать с тем же массивом.
go
Эксперт C++
3582 / 1362 / 128
Регистрация: 16.04.2009
Сообщений: 4,528
23.05.2012, 21:50     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #7
Цитата Сообщение от dimcoder Посмотреть сообщение
Если передать не так, то получиться копия массива, который был в main(), и следовательно массив в main() функция не изменит.
Копия чего-чего? Может быть копия указателя? Так передавать нужно по ссылке. Нужное замечание, я его и недосмотрел.
C++
1
void add_element(char** &m, int &k, const char *s)
И почему бы не использовать std::string, std::vector?
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
24.05.2012, 07:53     Программа которая изменяет динамический массив строк - Написал, но есть ошибка
Еще ссылки по теме:

C++ Динамический массив строк
Класс динамический массив строк C++
Двумерный динамический массив строк C++

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

Или воспользуйтесь поиском по форуму:
dimcoder
Полярный
 Аватар для dimcoder
449 / 422 / 66
Регистрация: 11.09.2011
Сообщений: 1,108
24.05.2012, 07:53  [ТС]     Программа которая изменяет динамический массив строк - Написал, но есть ошибка #8
Цитата Сообщение от go Посмотреть сообщение
Так передавать нужно по ссылке.
Когда программа не работала, было как то не важно, по ссылке или нет. Главное что бы заработала, а потом поправки. Но все же спасибо.
Цитата Сообщение от go Посмотреть сообщение
И почему бы не использовать std::string, std::vector?
В том то и задача, что бы без них. Практикуюсь, так сказать.

Не по теме:

PS Мдааа, вчера в последнем моем сообщении набредил я малость

Yandex
Объявления
24.05.2012, 07:53     Программа которая изменяет динамический массив строк - Написал, но есть ошибка
Ответ Создать тему
Опции темы

Текущее время: 06:20. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru