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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
dimcoder
Полярный
458 / 431 / 67
Регистрация: 11.09.2011
Сообщений: 1,129
#1

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

23.05.2012, 10:48. Просмотров 569. Ответов 7
Метки нет (Все метки)

Всем доброго времени суток. В общем сделал программу, которая вкратце делает следующее:
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++
Вот такое дело есть а как зделать так чтобы я мог ввести то количество строк что я хочю ? а не 3. Тоесть сделать динамический массив строк....

Динамический массив строк - C++
Здравствуйте нужна помощь по коду. Моя задача: создать динамический массив строк( по поводу терминов, возможно ошибка, но делать нужно то,...

Динамический массив строк - C++
Добрый вечер. Подскажите, пожалуйста, в чем может быть проблема. Первый элемент массива почему-то автоматически пропускается: ...

Динамический массив строк - C++
Мне нужно считать строки из файла но я не знаю сколько будет строк! и занести их в массив указателей на строки. чтобы было так *a =...

Динамический массив для строк - C++
Добрый вечер! Мне нужно обработать текстовый файл, длину строк я заранее не знаю. Язык Си. Знаю, что нужно использовать malloc, а как...

После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
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++
3586 / 1366 / 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
Полярный
458 / 431 / 67
Регистрация: 11.09.2011
Сообщений: 1,129
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++
3586 / 1366 / 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++
Здравствуйте! Есть программа, создающая динамический массив строк. Помогите пожалуйста разобраться, что значат эти строки: (Если кому не...

Функция которая возвращает динамический массив - C++
Здравствуйте! у меня такой глупый вопрос. Я все время путаюсь и никак не могу четко сформулировать на него ответ. Помогите мне раз и...


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

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

Не по теме:

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

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

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