Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
0 / 0 / 0
Регистрация: 21.10.2010
Сообщений: 27

Странное поведение функции работы с массивом(шаблон)

16.03.2011, 01:29. Показов 834. Ответов 5
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Нужно написать функцию, которая из динамического массива удаляет все повторяющиеся элементы(остается только первый), массив сжимается.То есть , если наш массив 3 3 3 4 0 4 0 0 0, то результат 3 4 0.
Функцию реализовать шаблонной.
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
59
60
61
62
63
64
65
66
67
68
69
70
71
#include<iostream>
#include<time.h>
using namespace std;
template<typename T>
 
void szhatie(T* mas,unsigned &size)
{
    int s=size,q=size;
    T *temp=new T[size];
    for(unsigned i=0;i<size;i++)
    {
        temp[i]=mas[i];
        for(unsigned j=i+1;j<size;j++)
        {
            if(mas[i]==mas[j])
            {
                q--;
                break;
            }
        }
    }
    size=q;
    delete[]mas;
    mas=new T[size];
    unsigned index=0;
    for(unsigned i=0;i<s,index<size;i++)
    {
        bool a=true;
        for(unsigned j=0;j<i;j++)
        {
            if(temp[i]==temp[j])
            {
                a=false;
                break;
            }
        }
        if(a)
        {           
            mas[index++]=temp[i];
        }
    }
    
    delete[]temp;
//      for(int i=0;i<size;i++)
//  {
//      cout<<"Element"<<mas[i]<<endl;
//  }
//}
 
int main()
{
    setlocale(LC_ALL,".1251");
    unsigned size;
    //srand(time(0));
    cout<<"Введите размер массива ";
    cin>>size;
    int *mas=new int[size];
    for(unsigned i=0;i<size;i++)
    {
        //mas[i]=rand()%5;
        cin>>mas[i];
        //cout<<mas[i]<<" ";
    }
        szhatie(mas,size);
    cout<<endl<<"Сжатый массив"<<endl;
    for(unsigned i=0;i<size;i++)
    {
        cout<<mas[i]<<" ";
    }
    return 0;
}
Если я ввожу числа 3 3 3 4 0 4 0 0 0, то результат выводит -572662307 1 0
Если раскомментировать последние строки в функции(которые выводят массив), то результат с теми же числами будет 1433284136 1 0
Если же я прогоняю программу построчно (ctrl+f10) или текст функции засовываю в мэйн, то результат 3 4 0, то есть то что надо.

Добавлено через 3 часа 44 минуты
Иногда вообще какую-то ерунду выдает. Может быть проблема с удалением памяти?
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
16.03.2011, 01:29
Ответы с готовыми решениями:

Странное поведение функции
Здравствуйте! Я взялась за реализацию метода Крылова для нахождения коэффициентов характеристического полинома. Написала программу, но...

Странное поведение map в объкт-функции
есть класс class writer { map &lt;string, list&lt;string&gt; &gt; &amp;dat; list&lt;string&gt; &amp;all; bool flag; //true = familiya public: ...

Шаблон класса для работы с массивом
помогите пожалуйста! Нужно создать шаблон класса для работы с одномерным массивом. Выполнить тестирование путем создания и обработки...

5
221 / 135 / 48
Регистрация: 12.04.2010
Сообщений: 248
16.03.2011, 09:58
Ты используешь C++, поэтому можно применить контейнер list и его метод unique (правда в этом случае массив нужно будет отсортировать). И для изменения твоего исходного указателя на массив нужно именно его и менять. См. код.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
void szhatie(T** masp,unsigned &size)
{
  T * mas = *masp;
  list<T> mylist (mas, mas + size);
  mylist.sort();
  mylist.unique();           
  delete[] *masp;
  mas = new T[size = mylist.size()];
  int i = 0;
//  cout << "mylist:";
  for (list<T>::iterator it=mylist.begin(); it!=mylist.end(); ++it){
    mas[i++] = *it;
//  cout << " " << *it;
  }
//  cout << endl;
  *masp = mas;
}
А твоя функция соответственно будет выглядеть:
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
void szhatie(T** masp,unsigned &size)
{
        T* mas = *masp;
        int s=size,q=size;
        T *temp=new T[size];
        for(unsigned i=0;i<size;i++)
        {
                temp[i]=mas[i];
                for(unsigned j=i+1;j<size;j++)
                {
                        if(mas[i]==mas[j])
                        {
                                q--;
                                break;
                        }
                }
        }
        size=q;
        delete[] *masp;
        mas=new T[size];
        unsigned index=0;
        for(unsigned i=0;i<s,index<size;i++)
        {
                bool a=true;
                for(unsigned j=0;j<i;j++)
                {
                        if(temp[i]==temp[j])
                        {
                                a=false;
                                break;
                        }
                }
                if(a)
                {
                        mas[index++]=temp[i];
                }
        }
 
        delete[]temp;
        *masp = mas;
//      for(int i=0;i<size;i++)
//      {
//              cout<<"Element"<<mas[i]<<endl;
//      }
}
1
0 / 0 / 0
Регистрация: 21.10.2010
Сообщений: 27
16.03.2011, 15:39  [ТС]
Спасибо, все заработало. А через STL нельзя писать, так как это тема еще не пройдена
0
Evg
Эксперт CАвтор FAQ
 Аватар для Evg
21281 / 8305 / 637
Регистрация: 30.03.2009
Сообщений: 22,660
Записей в блоге: 30
16.03.2011, 17:05
В функцию szhatie у тебя передаётся указатель на массив. Далее ты в строке 23 ты этот массив освобождаешь. В строке 24 ты его создаешь по новой. Но "mas" - это указатель, который теперь живёт только внутри процедуры szhatije. Когда ты возвращаешься в main, то указатель "mas" (который живёт в функции main) указывает на "старое" место, которое ты выделил в строке 57, а затем освободил в функции szhatije. Т.е. у тебя попросту тухлый указатель.

С виду тебе надо попросту удалить строки 23 и 24
0
0 / 0 / 0
Регистрация: 21.10.2010
Сообщений: 27
16.03.2011, 19:55  [ТС]
Изменил только вот эту строку и все работает
C++
1
void szhatie(T*& masp,unsigned &size)
0
Evg
Эксперт CАвтор FAQ
 Аватар для Evg
21281 / 8305 / 637
Регистрация: 30.03.2009
Сообщений: 22,660
Записей в блоге: 30
16.03.2011, 22:16
Цитата Сообщение от РагнаР Посмотреть сообщение
Изменил только вот эту строку и все работает
Работать-то работает, только идеологически оно неправильно. Память желательно освобождать на том же уровне, где и выделялась. В твоём случае так вообще незачем её освобождать в процедуре szhatije. У тебя был выделен массив, к примеру, на 100 элементов. Пусть после сжатия там должно остаться 25 элементов. Ну оставь ты свой массив размером в 100, но заполни только 25, ведь в переменной size у тебя эта величина записана. А то, что осталось ещё 75 свободных лементов - ну так фиг с ним. А если всё-таки хочется выделить массив именно на 25, то логичнее было бы поместить это в другой массив, а не в оригинальный. А такая кривая работа с памятью всегда будет служить источником трудноуловимых ошибок

Ну или работать со стандартными контейнерами, поскольку в этом случае ты не занимаешься выделением памяти вручную.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
16.03.2011, 22:16
Помогаю со студенческими работами здесь

Шаблон класса для работы с одномерным массивом
Создать шаблон класса для работы с одномерным массивом. Выполнить тестирование путем создания и обработки массивов, со- ...

Шаблон функции, не совсем понятное условие с массивом
Дан массив А. Составьте программу, которая прибавила бы каждому элементу данной строки элемент принадлежащий этой строке и главной...

Написать функции для работы с динамическим массивом
Дали вот эти задачи никак не могу решить, помогите. 1. Написать следующие функции для работы с динамическим массивом: a. Функция...

Функции для работы с одномерным динамическим массивом
Написать функции для работы с одномерным динамическим массивом: 1. Заполнение массива случайными значениями. 2. Вывод элементов массива...

Программа содержащая функции для работы с массивом структур
Для структуры struct marks { int fiz, mat, pr; }; struct student { char fio; struct marks m;


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

Или воспользуйтесь поиском по форуму:
6
Ответ Создать тему
Новые блоги и статьи
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru