При использовании memcpy временами выводит полный бред

@billy121 · Регистрация: 15.10.2019

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

У меня задается динамический массив. Где элементы - динамические массивы, где индексация с 1 элемента, а 0 элемент - кол-во (размер) тех элементов что с 1.
Затем я циклично применяю к этим внутренним массивам функции.
( к первому массиву - 1 функцию
ко второму массиву - 2 функцию
к третьему массиву - 3 функцию
к 4 массиву - 4 функцию
к 5 массиву - 1 фунцию
и тд.)

Так вот. Во второй фун-ии я в начало массива должен добавить K случайных чисел. Для этого я сначала меняю размер массива.
Затем копирую со смещением все элементы(начиная с 1 индекса) и в цикле до K заполняю также с 1 индекса случайными числами.

Первоначально я сделал зачем-то с возвратом массива
int AddK(int* A, int k, int UpLimit, int DownLimit)

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
{
    int size, i, new_size;
    size = *A;
    new_size = size + k;
    //(int*)realloc(A, (new_size+1) * sizeof(int));
    A = (int*)realloc(A, (new_size+1) * sizeof(int)); // размер
    memcpy(A + k + 1, A + 1, size * sizeof(int));
    for (i = 0; i < k + 1; ++i)
        {
            if (i == 0)
                    {
                        *A = new_size;
                    }
            else 
                {
                    *(A + i) = DownLimit + rand()%(UpLimit - DownLimit + 1);    
                }
        }
    return A;
}

и соответственно приравнивал при вызове

C
1
BigArr[i] = AddK(BigArr[i], k, UpLimit, DownLimit);

Вот так все работало без косяков. Но потом сделал, как нужно, без возвращений и присваиваний.
И временами выдается строка с муссором.
проблеманая функция с 53 строк, ее вызов на 178
Вот полный код:

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <mem.h>
 
 
int DeleteMax(int* A)
    {
        int size, Max, i, k, new_size;
        size = *A;
        Max = *(A + 1); k = 1;
        for (i = 2; i < size; ++i)
            {
                if (Max < *(A + i))
                    {
                        k = 1;
                        Max = *(A + i); 
                    }
                else if (Max == *(A + i))
                    {
                        k += 1;
                    }
            }
        new_size = size - k;
        k = 0;
        for (i = 0; i < size + 1; ++i)
            {
                if (i == 0)
                    {
                        *A = new_size;
                    }
                else if ((i > 0) & (*(A+i) != Max))
                    {
                         *(A + i - k) = *(A+i);
                    }
                else if ((i > 0) & (*(A+i) == Max))
                    {
                        k += 1;
                    }
            }
        A = (int*)realloc(A, (new_size+1) * sizeof(int));
        //for(i = 0; i < new_size + 1; ++i)
        //  printf("%d ", *(A+i));  
        //printf("\n %d %d", Max, k);
        if ((*A == 1) & (*(A + 1) == Max))
            {
                A = (int*)realloc(A, sizeof(int));
                *A = 0;
            }
                
    }
 
int AddK(int* A, int k, int UpLimit, int DownLimit)
{
    int size, i, new_size;
    size = *A;
    new_size = size + k;
    //(int*)realloc(A, (new_size+1) * sizeof(int));
    A = (int*)realloc(A, (new_size+1) * sizeof(int)); // размер
    memcpy(A + k + 1, A + 1, size * sizeof(int));
    for (i = 0; i < k + 1; ++i)
        {
            if (i == 0)
                    {
                        *A = new_size;
                    }
            else 
                {
                    *(A + i) = DownLimit + rand()%(UpLimit - DownLimit + 1);    
                }
        }
    //return A;
}
 
 
int ReverseArr(int* A)
{
    int size, i;
    size = *A;
    int tmp_arr[size];
    for (i = 1; i < size + 1; ++i)
        {
            tmp_arr[size - i + 1] = *(A + i);
        }
    for (i = 1; i < size + 1; ++i)
        {
            *(A + i) = *(tmp_arr + i);
        }       
}
 
int NullArr(int* A)
{
    int size, i, k, f;
    size = *A;
    f = 1;
    k = 0;
    for (i = 1; i < size + 1; ++i)
        {
            if (!(*(A + i) % 2) & f)
                {
                    f = 0;
                    k = i;
                }
        }
    if (k != 0)
        {
            memset(A + k, 0, sizeof(int));
        }
    //printf("k = %d", k);  
}
 
int main() 
{
    int i, j, A, B, UpLimit, DownLimit, d, k;
    do 
        {
            printf("Input AxB SIZE Arr.\n");
            scanf("%d %d", &A, &B); 
        }
    while ((A<1) || (B<1)); 
    
    do
        {
            printf("How many K elements to add?\n");
            scanf("%d", &k);
        }
    while (k < 1);
    printf("Enter the random number generation range.\n");
    printf("UpLimit, DownLimit\n");
    scanf("%d %d",&UpLimit, &DownLimit);
    if (UpLimit < DownLimit)
    {
        d = UpLimit;
        UpLimit = DownLimit;
        DownLimit = d;
    }
    B += 1;
    // массив указателей
    int** BigArr = (int**) malloc(A * sizeof(int*));
    // создаем и заполняем массивы указателям
    for (i = 0; i < A; ++i)
        BigArr[i] = (int*) malloc(B * sizeof(int));
    srand(time(NULL));
    for (i = 0; i < A; ++i)   
        for (j = 0; j < B; ++j)
            {
                if (j == 0)
                    {
                        BigArr[i][j] = B - 1;
                    }
                else
                    {
                        BigArr[i][j]=DownLimit + rand()%(UpLimit - DownLimit + 1);
                    }
             } 
    // вывод массива
    printf("\n");
    printf("Array before changes.\n");
    printf("---------------------\n");
    for (i = 0; i < A; ++i)   
        {
            for (j = 0; j < B; ++j)
                {
                    printf(" %4d", BigArr[i][j]);   
                }
                
            printf("\n");
        }
    printf("---------------------\n");
    for (i = 0; i < A; i++)
        {
            if (i%4 == 0)
                {
                    DeleteMax(BigArr[i]);
                }
            else if (i%4 == 1)
                {
                    AddK(BigArr[i], k, UpLimit, DownLimit);
                }
            else if (i%4 == 2)
                {
                    ReverseArr(BigArr[i]);
                }
            else if (i%4 == 3)
                {
                    NullArr(BigArr[i]);
                }
            }   
    
    printf("New array.\n");
    printf("---------------------\n");
    for (i = 0; i < A; ++i)   
        {
            for (j = 0; j < BigArr[i][0] + 1; ++j)
                {
                    printf(" %4d", BigArr[i][j]);   
                }
                
            printf("\n");
        }
    printf("---------------------\n");
    // очистка памяти
    for (i = 0; i < A; ++i)  
        free(BigArr[i]);
    
    //потом очищаем основной массив
    free(BigArr); 
    BigArr = NULL;
    
    return 0;
}

в чем можешь быть моя проблемма?

@billy121 · 20.04.2022, 08:50 **[ТС]**

Вот такой вывод.
То есть в первый раз, все добавляет нормально.
А потом уже с мусором

@billy121 · 20.04.2022, 09:03 **[ТС]**

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

Первоначально я сделал зачем-то с возвратом массива

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
int AddK(int* A, int k, int UpLimit, int DownLimit)
{
    int size, i, new_size;
    size = *A;
    new_size = size + k;
    //(int*)realloc(A, (new_size+1) * sizeof(int));
    A = (int*)realloc(A, (new_size+1) * sizeof(int)); // размер
    memcpy(A + k + 1, A + 1, size * sizeof(int));
    for (i = 0; i < k + 1; ++i)
        {
            if (i == 0)
                    {
                        *A = new_size;
                    }
            else 
                {
                    *(A + i) = DownLimit + rand()%(UpLimit - DownLimit + 1);    
                }
        }
    return A;
}

и соответственно приравнивал при вызове

C
1
BigArr[i] = AddK(BigArr[i], k, UpLimit, DownLimit);

вот так выводит все хорошо, но это же неправильно я оформил

@billy121 · 20.04.2022, 09:38 **[ТС]**

А я ведь могу скопировать из массива и вставить в этот массив? Может ли в этом быть моя ошибка?

C
1
memcpy(A + k + 1, A + 1, size * sizeof(int));

Добавлено через 27 минут
Подумал, что может это из за наложения элементов друг на дружку. Т.к. где вычитал, что при memcpy никак нельзя накладывать элементы.Но зато это можно делать при memmove. Я его использовал и получил тот же самый мусор в массиве.
Также я заметил, что у меня данная функция вобще не работает без возвращений и присваиваний.
На самом первом скрине, который я отправил, видно. Что да - два элемента добавились, но 2 последующих не равны тем 2-ум первоначальным в исходном массиве.

@Annemesski · 20.04.2022, 10:08

А не задумывались зачем realloc возвращает указатель? Вы передаете в функцию указатель по значению и если realloc изменил расположение исходного массива, то внешний по отношению к функции Addk код никак не сможет узнать куда был перенесен массив.
Вот тут

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

Вот так все работало без косяков. Но потом сделал, как нужно, без возвращений и присваиваний.

Вы сами на свой вопрос и ответили, так что либо возвращайте указатель, либо передавайте в функцию указатель на указатель.

@billy121 · 20.04.2022, 10:12 **[ТС]**

Annemesski, точняк...
Спасибо, что вразумил

Новые блоги и статьи Все статьи Все блоги /
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.
Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .	Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .	Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .

@billy121 29 / 24 / 5 Регистрация: 15.10.2019 Сообщений: 268
	20.04.2022, 08:50 [ТС]
	Вот такой вывод. То есть в первый раз, все добавляет нормально. А потом уже с мусором Миниатюры 0

@billy121 29 / 24 / 5 Регистрация: 15.10.2019 Сообщений: 268
	20.04.2022, 10:12 [ТС]
	Annemesski, точняк... Спасибо, что вразумил 0

При использовании memcpy временами выводит полный бред

Решение