Копирование из второго массива в первый массив с используя функцию

@Mordoc · Регистрация: 12.02.2020

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

Хотел написать функцию, которая копирует данные из второго массива в первый, при этом данные первого массива удаляются. Написал код, но он работает не стабильно. Если постоянно запускать программу, то она работает как задумано 50/50. Не могу найти ошибку.

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
#include <iostream>
using namespace std;
 
void FillArray(int* const arr, const int size) {
    //массив заполняется случайными значениями
    for (int i = 0; i < size; i++)
    {
        arr[i] = rand() % 10;
    }
 
}
 
void ShowArr(const int* const arr, const int size) {
    //показать массив в консоли
    for (int i = 0; i < size; i++)
    {
        cout << arr[i] << "\t";
    }
    cout << endl;
}
 
void CopyArray( int* arr1, int* const arr2, const int sizeofArr2) {
    //копируем значения из arr2 в arr1 массив 
    //данные arr1 нам не нужны
    //массивы не всегда одинаковы по размеру
    delete[] arr1;
    arr1 = new int[sizeofArr2];
    for (int i = 0; i < sizeofArr2; i++)
    {
        arr1[i] = arr2[i];
    }
 
}
 
void main() {
    //setlocale(LC_ALL, "ru");
 
    //тест с одинаковыми по размеру массивами
    int size = 5;
    int* firstArr = new int[size];
    int* secondArr = new int[size];
 
    //создаю и показаваю
    cout << "first array =\t";
    FillArray(firstArr, size);
    ShowArr(firstArr, size);
 
    //создаю и показаваю
    cout << "second array =\t";
    FillArray(secondArr, size);
    ShowArr(secondArr, size);
 
    //перемещаем данные из второго массива в первый с потерей данных у первого
    CopyArray(firstArr, secondArr, size);
    cout << "first array =\t";
    ShowArr(firstArr, size);
    cout << "second array =\t";
    ShowArr(secondArr, size);
    cout << firstArr << endl;
    cout << secondArr << endl;
 
    //избавляюсь от всех массивов
    cout << "deleting arrays" << endl;
    delete[] firstArr;
    delete[] secondArr;
}

@Mordoc · 11.03.2021, 20:10 **[ТС]**

S7,

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
#include <iostream>
using namespace std;
 
void FillArray(int* const arr, const int size) {
    //массив заполняется случайными значениями
    for (int i = 0; i < size; i++)
    {
        arr[i] = rand() % 10;
    }
 
}
 
void ShowArr(const int* const arr, const int size) {
    //показать массив в консоли
    for (int i = 0; i < size; i++)
    {
        cout << arr[i] << "\t";
    }
    cout << endl;
}
 
void CopyArray(int* arr1, int* arr2, int sizeofArr2) {
    //копируем значения из arr2 в arr1 массив 
    //данные arr1 нам не нужны
    //массивы не всегда одинаковы по размеру
    delete[] arr1;
    arr1 = new int[sizeofArr2];
 
    for (int i = 0; i < sizeofArr2; i++)
    {
        arr1[i] = arr2[i];
    }
 
}
 
void main() {
    /*
    setlocale(LC_ALL, "ru");
 
    //тест с одинаковыми по размеру массивами
    int size1 = 5;
    int size2 = 9;
    int* firstArr = new int[size1];
    int* secondArr = new int[size2];
 
    //создаю и показаваю
    cout << "first array =\t";
    FillArray(firstArr, size1);
    ShowArr(firstArr, size1);
 
    //создаю и показаваю
    cout << "second array =\t";
    FillArray(secondArr, size2);
    ShowArr(secondArr, size2);
 
    //перемещаем данные из второго массива в первый с потерей данных у первого
    CopyArray(firstArr, secondArr, size2);
    cout << "first array =\t";
    ShowArr(firstArr, size2);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
 
    cout << endl;
    cout << firstArr << endl;
    cout << secondArr << endl;
 
    //избавляюсь от всех массивов
    cout << "deleting arrays" << endl;
    delete[] firstArr;
    delete[] secondArr;
    */
 
 
    int size1 = 5;
    int size2 = 9;
    int* firstArr = new int[size1];
    int* secondArr = new int[size2];
 
    FillArray(firstArr, size1);
    FillArray(secondArr, size2);
 
    cout << "first array =\t";
    ShowArr(firstArr, size1);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
    cout << endl;
 
 
    delete[] firstArr;
    firstArr = new int[size2];
    for (int i = 0; i < size2; i++)
    {
        firstArr[i] = secondArr[i];
    }
    cout << "first array =\t";
    ShowArr(firstArr, size2);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
 
    cout << endl;
    cout << "adres firstArr " << firstArr << endl;
    cout << "adres secondArr " << secondArr << endl;
 
 
    delete[] firstArr;
    delete[] secondArr;
}

@S7 · 11.03.2021, 20:10

Потестите на своих пк, должно работать.

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
#include <iostream>
#include <vector>
#include <string>
#include <algorithm>
#include <cmath>
#include <set>
#include <iomanip>
#define ll long long
#define d double
using namespace std;
void FillArray(vector <ll> &a, ll size) {
    //массив заполняется случайными значениями
    for (int i = 0; i < size; i++)
    {
        a[i] = rand() % 10;
    }
    return;
}
void ShowArr(vector <ll> &a) {
    //показать массив в консоли
    for (int i = 0; i < a.size(); i++)
    {
        cout << a[i] << " ";
    }
    cout << endl;
}
void makecopyofvec(vector <ll>& from, vector<ll>& to, ll size) {
    ll i;
    for (i = 0; i < size; i++) {
        to[i] = from[i];
    }
    return;
}
int main() {
    ll n;
    setlocale(LC_ALL, "ru");
    cout << "Напишите размер массива" << endl;
    cin >> n;
    vector <ll> a(n);
    FillArray(a, n);
    cout << "Массив который будет копирован" << endl;
    ShowArr(a);
    vector <ll> b(a.size());
    makecopyofvec(a, b, n);
    cout << "Копия массива" << endl;
    ShowArr(b);
    cout << "Производим обратное копирование" << endl;
    makecopyofvec(b, a, n);
    cout << "Первый массив" << endl;
    ShowArr(a);
    cout << "Второй массив" << endl;
    ShowArr(b);
    ll* adrofa = &a[0];
    ll* adrofb = &b[0];
    cout << adrofa << endl << adrofb;
    return 0;
}

@Mordoc · 11.03.2021, 20:14 **[ТС]**

S7,

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
#include <iostream>
using namespace std;
 
void FillArray(int* const arr, const int size) {
    //массив заполняется случайными значениями
    for (int i = 0; i < size; i++)
    {
        arr[i] = rand() % 10;
    }
 
}
 
void ShowArr(const int* const arr, const int size) {
    //показать массив в консоли
    for (int i = 0; i < size; i++)
    {
        cout << arr[i] << "\t";
    }
    cout << endl;
}
 
void CopyArray(int* arr1, int* arr2, int sizeofArr2) {
    //копируем значения из arr2 в arr1 массив 
    //данные arr1 нам не нужны
    //массивы не всегда одинаковы по размеру
    delete[] arr1;
    arr1 = new int[sizeofArr2];
 
    for (int i = 0; i < sizeofArr2; i++)
    {
        arr1[i] = arr2[i];
    }
 
}
 
void main() {
    int size1 = 5;
    int size2 = 9;
    int* firstArr = new int[size1];
    int* secondArr = new int[size2];
 
    FillArray(firstArr, size1);
    FillArray(secondArr, size2);
 
    cout << "first array =\t";
    ShowArr(firstArr, size1);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
    cout << endl;
 
 
    delete[] firstArr;
    firstArr = new int[size2];
    for (int i = 0; i < size2; i++)
    {
        firstArr[i] = secondArr[i];
    }
    cout << "first array =\t";
    ShowArr(firstArr, size2);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
 
    cout << endl;
    cout << "adres firstArr " << firstArr << endl;
    cout << "adres secondArr " << secondArr << endl;
 
 
    delete[] firstArr;
    delete[] secondArr;
}

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

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

Mordoc, кое-что заменил, проверьте работу:

Проблема в том, что моя функция должна быть void и работать с указателями на память, чтобы всё на лету происходило, создать новый массив и вернуть его у меня уже идеи были, но это не то.

@Yetty · 11.03.2021, 20:17

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

моя функция должна быть void

понятно. просто сообщите мой вариант работает или нет

@S7 · 11.03.2021, 20:20

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

S7,

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
#include <iostream>
using namespace std;
 
void FillArray(int* const arr, const int size) {
    //массив заполняется случайными значениями
    for (int i = 0; i < size; i++)
    {
        arr[i] = rand() % 10;
    }
 
}
 
void ShowArr(const int* const arr, const int size) {
    //показать массив в консоли
    for (int i = 0; i < size; i++)
    {
        cout << arr[i] << "\t";
    }
    cout << endl;
}
 
void CopyArray(int* arr1, int* arr2, int sizeofArr2) {
    //копируем значения из arr2 в arr1 массив 
    //данные arr1 нам не нужны
    //массивы не всегда одинаковы по размеру
    delete[] arr1;
    arr1 = new int[sizeofArr2];
 
    for (int i = 0; i < sizeofArr2; i++)
    {
        arr1[i] = arr2[i];
    }
 
}
 
void main() {
    int size1 = 5;
    int size2 = 9;
    int* firstArr = new int[size1];
    int* secondArr = new int[size2];
 
    FillArray(firstArr, size1);
    FillArray(secondArr, size2);
 
    cout << "first array =\t";
    ShowArr(firstArr, size1);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
    cout << endl;
 
 
    delete[] firstArr;
    firstArr = new int[size2];
    for (int i = 0; i < size2; i++)
    {
        firstArr[i] = secondArr[i];
    }
    cout << "first array =\t";
    ShowArr(firstArr, size2);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
 
    cout << endl;
    cout << "adres firstArr " << firstArr << endl;
    cout << "adres secondArr " << secondArr << endl;
 
 
    delete[] firstArr;
    delete[] secondArr;
}

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

Проблема в том, что моя функция должна быть void и работать с указателями на память, чтобы всё на лету происходило, создать новый массив и вернуть его у меня уже идеи были, но это не то.

У меня так и работает, я в функцию передаю именно ссылку на массив , если бы я писал вместо: vector <ll> &a писал:
vector <ll> a, то у меня бы копировался весь массив. Мы же только в самом начале добавляем массив и рехервируем память, а потом используем их с помощью указателей

@Mordoc · 11.03.2021, 20:32 **[ТС]**

S7, Я нашел одну проблему:

Находится она в функции CopyArray, когда я передаю firstArr , то функция удаляет из памяти элементы этого массива, потом
я говорю, чтобы у firstArr создался новый пустой массив на sizeofArr2 элементов. Происходит, что-то не понятное точно создается в памяти массив, его даже наполняют элементами в цикле из массива secondArr. По хорошему праметр arr1 превращается в локальный массив функции CopyArray и остается там. В итоге из функции выходит firstArr, который указывает в пустоту (то есть ссылка есть, а данных нет), в конце когда пытаются delete[] firstArr, удалять нечего и выпадает ошибка.

Вопрос как спасти firstArr и помешать arr1 стать локальным массивом.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
void CopyArray(int* arr1, int* arr2, int sizeofArr2) {
    //копируем значения из arr2 в arr1 массив 
    //данные arr1 нам не нужны
    //массивы не всегда одинаковы по размеру
    delete[] arr1;
    arr1 = new int[sizeofArr2];
 
    for (int i = 0; i < sizeofArr2; i++)
    {
        arr1[i] = arr2[i];
    }
 
}

@S7 · 11.03.2021, 20:34

Точно не знаю, но помоему не имеет смысла внутри функции делать новый массив, ты же должен просто идти от 0 до размера массива и ставить в один массив элементы другого.

@Mordoc · 11.03.2021, 20:37 **[ТС]**

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

@S7 · 11.03.2021, 20:43

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

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

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

@Mordoc · 11.03.2021, 20:47 **[ТС]**

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

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

он не может работать с разноразмерными массивами, во всяком случае эта версия, и зачем так много библиотек <iostream> достаточно. Я еще не знаю как реализуются вектора в c++.

@Mordoc · 12.03.2021, 08:24 **[ТС]**

Нашел окончательное решение проблемы функции CopyArray, надо было передавать массив по ссылке в виде

C++
1
 *&имя_массива

Таким образом в функции не появляются фантомные массивы и существующие массивы не выходят пустые
Код работает только на ссылках массивов, присвоений массива к массиву нет, всё работает в функции, массивы могут быть разного размера. Главное массивы должны быть одного типа данных.

Окончательный код с тестированием на разноразмерных массивах.

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
#include <iostream>
using namespace std;
 
void FillArray(int* const arr, const int size) {
    //массив заполняется случайными значениями
    for (int i = 0; i < size; i++)
    {
        arr[i] = rand() % 10;
    }
}
 
void ShowArr(const int* const arr, const int size) {
    //показать массив в консоли
    for (int i = 0; i < size; i++)
    {
        cout << arr[i] << "\t";
    }
    cout << endl;
}
 
void CopyArray(int*& arr1, int* const arr2, const int sizeofArr2) {
 
    delete[] arr1;
    arr1 = new int[sizeofArr2];
    for (int i = 0; i < sizeofArr2; i++)
    {
        arr1[i] = arr2[i];
    }
    
}
 
 
void main() {
    setlocale(LC_ALL, "ru");
 
    //тест с массивами
    int size1 = 5;
    int size2 = 9;
    int* firstArr = new int[size1];
    int* secondArr = new int[size2];
 
    //заполняю и показаваю
    cout << "first array =\t";
    FillArray(firstArr, size1);
    ShowArr(firstArr, size1);
 
    //заполняю и показаваю
    cout << "second array =\t";
    FillArray(secondArr, size2);
    ShowArr(secondArr, size2);
 
    //перемещаем данные из второго массива в первый с потерей данных у первого
    CopyArray(firstArr, secondArr, size2);
    
    cout << "first array =\t";
    ShowArr(firstArr, size2);
    cout << "second array =\t";
    ShowArr(secondArr, size2);
    
    cout << endl;
    //адреса
    cout <<"firstArr adress =\t"<<firstArr << endl;
    cout <<"secondArr aresss =\t"<< secondArr << endl;
 
    //избавляюсь от всех массивов
    cout << "deleting arrays" << endl;
    delete[] firstArr;
    delete[] secondArr;
}

Новые блоги и статьи Все статьи Все блоги /
Кредитный калькулятор Maks 05.08.2026 Решение задачи по прикладной информатике средствами 1С. Задача: Напишите приложение-калькулятор, которое помогает рассчитывать параметры кредита для аннуитетного и дифференцированного видов. . .	У нас сейчас поговорку "Опять 25" нужно переделать на "Опять +35". kumehtar 04.08.2026 С ностальгией вспоминаю времена моего детства, когда у нас и правда +25 - была максимальная температура летом. Раньше +25 °C реально казались вершиной жары, когда можно было весь день пропадать на. . .	Как ИИ начал спорить и врать (возможно почуяв опасность для себя от индустрии - уход от электроники). Hrethgir 04.08.2026 Недельный диалог, на фоне событий с НПЗ. Да, из спирта можно получать бензин, и это не сложно. Но потом в схеме я решил избавиться от насоса, при этом полностью сделав контроль подачи спирта в. . .	Термопринтер QR701 Argus19 03.08.2026 Термопринтер QR701 Купил два термопринтера QR701. На сэлф-тесте написано: Language: PC936 (GB18030). Что означает, что принтеры могут печатать только латиницу и китайские иероглифы. Так же. . .
Создание формы заимствованного документа Maks 03.08.2026 Задача: Необходимо создать собственную форму заимствованного документа. На форме должен быть реквизит "Покупатель", а также табличная часть со следующими реквизитами: - Расчетный счет покупателя. . .	Задача предоставления скидок покупателям Maks 03.08.2026 Задача: В документе "Продажи" необходимо реализовать функционал предоставления скидок покупателям. Скидка должна автоматически рассчитываться и подставляться в соответствующее поле при выборе. . .	Почему SEO не начинается с ключевых слов: что проверить до написания текстов Neotwalker 01.08.2026 Когда владельцу сайта предлагают заняться SEO, первым шагом часто становится сбор запросов и написание текстов. Логика кажется понятной: 1. Находим ключевые слова. 2. Добавляем их на. . .	Знание — сила: Доктрина интенциональности знаний, углубление в формулу Hrethgir 01.08.2026 https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11957&stc=1&d=1785567302 Знаменитый афоризм Фрэнсиса Бэкона «Знание — сила» (Scientia potentia est) в массовой культуре принято понимать. . .

@S7 10 / 6 / 4 Регистрация: 08.04.2020 Сообщений: 61
	11.03.2021, 20:34
	Точно не знаю, но помоему не имеет смысла внутри функции делать новый массив, ты же должен просто идти от 0 до размера массива и ставить в один массив элементы другого. 0

@Mordoc 7 / 5 / 2 Регистрация: 12.02.2020 Сообщений: 19
	11.03.2021, 20:37 [ТС]
	S7, по хорошему решения еще нет, попробую набраться опыта с функциями, ссылками и указателями. Вернусь, чтобы дать нормальное решение. 0