Найти слова, повторяющиеся более одного раза, не могу найти ошибку

@Skip · Регистрация: 11.11.2010

C++

#include <iostream>
using namespace std;
 
void obr1(char **s, char **mas, int n, int m)
{
    int i;
    int k;
    char *tm;
    for(i = 0; i < n; i++)
    {
        k = -1;
        tm = s[i];
        for(i = 0; i < n; i++)
        {
            if (strcmp(tm,s[i])==0)
            {
                k++;
            }
        }
        if(k > 0)
        {
            m++;
            mas = new char *[m]();
            mas[m-1]=tm;
        }
    }
    for(i = 0; i < m; i++)
    {
        cout << mas[i] << "\n";
    }
}

s-массив слов.
mas - массив в который надо записать все слова встречающиеся более одного раза.
n - размер массива s.
m - размер массива mas.

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

@Lupus · 27.11.2011, 18:28

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

C++

if(k > 0) 
{ 
     m++; 
     mas = new char *[m](); 
     mas[m-1]=tm; 
}

Смотрится не очень, особенно запись в ячейку с индексом равному размеру массива

@Skip · 27.11.2011, 18:38 **[ТС]**

ну это же динамический массив и заранее я не знаю , сколько повторяющих слов будет

@Lupus · 27.11.2011, 18:48

но если в цикле каждый раз создавать новый массив, данные о старом(ых) будут утрачены...
или я чего-то не догоняю

да и delete не хватает

@Skip · 27.11.2011, 19:46 **[ТС]**

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

но если в цикле каждый раз создавать новый массив, данные о старом(ых) будут утрачены...
или я чего-то не догоняю

да и delete не хватает

да это я уже догнал , щас проблема в другом ,
я создал 2ой массив в который переношу на время , но проблема в том что когда я удаляю первый массив , то значение во втором тоже затирается. тк значение из первого я переношу во 2ой через указатель

@Lupus · 27.11.2011, 19:48

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

да это я уже догнал , щас проблема в другом ,
я создал 2ой массив в который переношу на время , но проблема в том что когда я удаляю первый массив , то значение во втором тоже затирается. тк значение из первого я переношу во 2ой через указатель

Я вообще по указателям не силен, но здесь скорее всего нужен еще один указатель.
А так, вообще, если это не лаба по массивам, я бы смотрел в сторону STL-контейнеров, листа в частности

@Skip · 27.11.2011, 20:04 **[ТС]**

эх если бы -__-

@~~-=ЮрА=-~~ · 27.11.2011, 20:48

Skip, что вообще надо из текста перепсать слова несколько раз в нём встречающиеся в другой 2-х мерный массив?Язык Си/С++?Вообще я бы рекомендовал использовать std::string но это так. Основное - что нужно сделать в программе + ввод текста с клавиатуры или файла, пишем!

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

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

if(k > 0)
* * * * * * * * {
* * * * * * * * * * * * m++;
* * * * * * * * * * * * mas = new char *[m]();
* * * * * * * * * * * * mas[m-1]=tm;
* * * * * * * * }

- это концептуально неверный подход - нужно добавлять к уже имеющемся массиву а не создавать новый, данные о предыдущих словах просто будут уничтожены...

@Skip 3 / 3 / 0 Регистрация: 11.11.2010 Сообщений: 38
	27.11.2011, 18:38 [ТС]	3
	ну это же динамический массив и заранее я не знаю , сколько повторяющих слов будет 0

@Lupus 24 / 24 / 4 Регистрация: 13.03.2011 Сообщений: 326
	27.11.2011, 18:48	4
	но если в цикле каждый раз создавать новый массив, данные о старом(ых) будут утрачены... или я чего-то не догоняю да и delete не хватает 0

@Skip 3 / 3 / 0 Регистрация: 11.11.2010 Сообщений: 38
	27.11.2011, 19:46 [ТС]	5
	Сообщение от Lupus но если в цикле каждый раз создавать новый массив, данные о старом(ых) будут утрачены... или я чего-то не догоняю да и delete не хватает да это я уже догнал , щас проблема в другом , я создал 2ой массив в который переношу на время , но проблема в том что когда я удаляю первый массив , то значение во втором тоже затирается. тк значение из первого я переношу во 2ой через указатель 0

@Lupus 24 / 24 / 4 Регистрация: 13.03.2011 Сообщений: 326
	27.11.2011, 19:48	6
	Сообщение от Skip да это я уже догнал , щас проблема в другом , я создал 2ой массив в который переношу на время , но проблема в том что когда я удаляю первый массив , то значение во втором тоже затирается. тк значение из первого я переношу во 2ой через указатель Я вообще по указателям не силен, но здесь скорее всего нужен еще один указатель. А так, вообще, если это не лаба по массивам, я бы смотрел в сторону STL-контейнеров, листа в частности 0

@Skip 3 / 3 / 0 Регистрация: 11.11.2010 Сообщений: 38
	27.11.2011, 20:04 [ТС]	7
	эх если бы -__- 0

@~~-=ЮрА=-~~ Заблокирован
	27.11.2011, 20:48	8
	Skip, что вообще надо из текста перепсать слова несколько раз в нём встречающиеся в другой 2-х мерный массив?Язык Си/С++?Вообще я бы рекомендовал использовать std::string но это так. Основное - что нужно сделать в программе + ввод текста с клавиатуры или файла, пишем! Добавлено через 1 минуту Сообщение от Skip if(k > 0) * * * * * * * * { * * * * * * * * * * * * m++; * * * * * * * * * * * * mas = new char [m](); * * * * * * * * * * * mas[m-1]=tm; * * * * * * * * } - это концептуально неверный подход - нужно добавлять к уже имеющемся массиву а не создавать новый, данные о предыдущих словах просто будут уничтожены... 0

Опции темы