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

Работа с одномерным целочисленным массивом в функции - C++

Восстановить пароль Регистрация
 
Kenny95
0 / 0 / 0
Регистрация: 26.02.2013
Сообщений: 15
04.07.2016, 19:27     Работа с одномерным целочисленным массивом в функции #1
Здравствуйте. Я написал несколько функций по работе с массивом. В dimentInt() я объявляю массив и передаю его в addNewElIDA, которая добавляет новый элемент в массив на указанное место. Дело в том, что функция addNewElIDA работает как надо. но когда я из меню функции dimentInt() вызываю функцию shawIDA, то выводится массив IDA, но новый элемент остаётся на последнем месте и вместо значения выводится белеберда. Что я делаю не так?

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
void shawIDA(int *IDA, int sizeIDA) {
    system("cls");
    for (int i = 0; i < sizeIDA; i++) {
        cout << IDA[i] << " ";
    }
    _getch();
}
 
void addNewElIDA(int *IDA, int &sizeIDA) {
    int number, pos, temp;
    int *tempIDA = new int(sizeIDA);
    for (int i = 0; i < sizeIDA; i++) {
        tempIDA[i] = IDA[i];
    }
    sizeIDA++;
    IDA = new int(sizeIDA);
    for (int i = 0; i < sizeIDA-1; i++) {
        IDA[i] = tempIDA[i];
    }
    system("cls");
    cout << "Введите номер нового элемента: ";
    while (!(cin >> pos) || (cin.peek() != '\n') || ((pos < 0) || (pos > sizeIDA))) {
        cin.clear();
        while (cin.get() != '\n');
        cout << endl << "Введено не корректное значение!" << endl;
        cout << "Введите номер нового элемента: ";
    }
    cout << "Введите значение нового элемента: ";
    while (!(cin >> number) || (cin.peek() != '\n')) {
        cin.clear();
        while (cin.get() != '\n');
        cout << endl << "Введено не корректное значение!" << endl;
        cout << "Введите значение нового элемента: ";
    }
    IDA[sizeIDA - 1] = number;
    for (int i = sizeIDA-1; i > pos; i--) {
        temp = IDA[i-1];
        IDA[i-1] = IDA[i];
        IDA[i] = temp;
    }
    cout << endl;
    for (int i = 0; i < sizeIDA; i++) {
        cout << IDA[i] << " ";
    }
    _getch();
}
 
void dimentInt() {
 
    int choiceAct, sizeIDA;
    system("cls");
    cout << "Введите размер одномерного целочисленного массива массива: ";
 
    while (!(cin >> sizeIDA) || (cin.peek() != '\n') || (sizeIDA < 1)) {
 
        cin.clear();
        while (cin.get() != '\n');
 
        system("cls");
        cout << "Введено не корректное значение!" << endl;
        cout << "Введите размер одномерного целочисленного массива массива: ";
    }
 
    int *IDA = new int(sizeIDA);
 
    do {
        system("cls");
        cout << "Выберете действие: " << endl;
        /*...*/
        cout << "\t2) Вывести массив." << endl;
                /*...*/
        cout << "\t10) Добавить новый элемент массива." << endl;
        /*...*/
        cout << "Ваш выбор: ";
        while (!(cin >> choiceAct) || (cin.peek() != '\n') || ((choiceAct < 1) || (choiceAct > 12))) {
            cin.clear();
            while (cin.get() != '\n');
            system("cls");
            cout << "Введено не корректное значение!" << endl;
            cout << "Выберете действие: " << endl;
            /*...*/
            cout << "\t2) Вывести массив." << endl;
                        /*...*/
            cout << "\t10) Добавить новый элемент массива." << endl;
            /*...*/
            cout << "Ваш выбор: ";
        };
        switch (choiceAct) {
            /*...*/
            case 10: {
                addNewElIDA(IDA, sizeIDA);
                break;
            }
                        /*...*/
        }
    }
    while (choiceAct != 12);
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
zss
Модератор
Эксперт С++
 Аватар для zss
5946 / 5551 / 1784
Регистрация: 18.12.2011
Сообщений: 14,175
Завершенные тесты: 1
04.07.2016, 19:50     Работа с одномерным целочисленным массивом в функции #2
Цитата Сообщение от Kenny95 Посмотреть сообщение
void addNewElIDA(int *IDA, int &sizeIDA) {
Передайте IDA по ссылке. Иначе при новом выделении памяти
Цитата Сообщение от Kenny95 Посмотреть сообщение
IDA = new int(sizeIDA);
Вы измените адрес массива только в копии указателя.
C++
1
void addNewElIDA(int& *IDA, int &sizeIDA) {
Кстати, а почему у Вас в операциях типа new int(sizeIDA) круглые скобки, а не квадратные?
Это, ведь, выделение ОДНОГО элемента и инициализация его значением sizeIDA.

И ЕЩЁ я не вижу ни одного оператора delete[] ни для IDA ни для TempIDA
Kenny95
0 / 0 / 0
Регистрация: 26.02.2013
Сообщений: 15
08.07.2016, 04:54  [ТС]     Работа с одномерным целочисленным массивом в функции #3
Цитата Сообщение от zss Посмотреть сообщение
Сообщение от Kenny95
void addNewElIDA(int *IDA, int &sizeIDA) {
Передайте IDA по ссылке. Иначе при новом выделении памяти
Сообщение от Kenny95
IDA = new int(sizeIDA);
Вы измените адрес массива только в копии указателя.
Сделал как вы сказали, но все осталось по прежнему.

Цитата Сообщение от zss Посмотреть сообщение
Кстати, а почему у Вас в операциях типа new int(sizeIDA) круглые скобки, а не квадратные?
Это, ведь, выделение ОДНОГО элемента и инициализация его значением sizeIDA.
Всё время ставил круглые и все нормально работало. даже и не знал, что это большая разница.

Цитата Сообщение от zss Посмотреть сообщение
И ЕЩЁ я не вижу ни одного оператора delete[] ни для IDA ни для TempIDA
Спасибо, что-то совсем забыл про это.

Добавлено через 1 час 38 минут
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
void addNewElIDA(int* IDA, int &sizeIDA) {
    int number, pos, temp;
    int *tempIDA = new int[sizeIDA];
    for (int i = 0; i < sizeIDA; i++) {
        tempIDA[i] = IDA[i];
    }
    sizeIDA++;
    IDA = new int[sizeIDA];
    for (int i = 0; i < sizeIDA-1; i++) {
        IDA[i] = tempIDA[i];
    }
    system("cls");
    cout << "Введите номер нового элемента: ";
    while (!(cin >> pos) || (cin.peek() != '\n') || ((pos < 0) || (pos > sizeIDA))) {
        cin.clear();
        while (cin.get() != '\n');
        cout << endl << "Введено не корректное значение!" << endl;
        cout << "Введите номер нового элемента: ";
    }
    cout << "Введите значение нового элемента: ";
    while (!(cin >> number) || (cin.peek() != '\n')) {
        cin.clear();
        while (cin.get() != '\n');
        cout << endl << "Введено не корректное значение!" << endl;
        cout << "Введите значение нового элемента: ";
    }
    IDA[sizeIDA - 1] = number;
    for (int i = sizeIDA-1; i > pos; i--) {
        temp = IDA[i-1];
        IDA[i-1] = IDA[i];
        IDA[i] = temp;
    }
    cout << endl;
    for (int i = 0; i < sizeIDA; i++) {
        cout << IDA[i] << " ";
    }
    _getch();
}
C++
1
addNewElIDA(IDA, sizeIDA);
Так тут же и происходит передача по ссылке, разве нет?
zss
Модератор
Эксперт С++
 Аватар для zss
5946 / 5551 / 1784
Регистрация: 18.12.2011
Сообщений: 14,175
Завершенные тесты: 1
08.07.2016, 08:39     Работа с одномерным целочисленным массивом в функции #4
Цитата Сообщение от Kenny95 Посмотреть сообщение
разве нет?
Нет, передается копия указателя.
Соответственно при присваивании новое значение записывается в копию.
Для того, чтобы адрес передавался по ссылке, надо написать
C++
1
void addNewElIDA(int& *IDA, int &sizeIDA) {
Kenny95
0 / 0 / 0
Регистрация: 26.02.2013
Сообщений: 15
12.07.2016, 02:39  [ТС]     Работа с одномерным целочисленным массивом в функции #5
Так использовать указатель на ссылку нельзя же в случае с массивами.
Даже компилятор ругается.
zss
Модератор
Эксперт С++
 Аватар для zss
5946 / 5551 / 1784
Регистрация: 18.12.2011
Сообщений: 14,175
Завершенные тесты: 1
12.07.2016, 06:45     Работа с одномерным целочисленным массивом в функции #6
Пардон, нужен не указатель на ссылку, а ссылка на указатель
C++
1
void addNewElIDA(int* &IDA, int &sizeIDA)
Yandex
Объявления
12.07.2016, 06:45     Работа с одномерным целочисленным массивом в функции
Ответ Создать тему
Опции темы

Текущее время: 20:49. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru