Работа с динамическим массивом: три варианта инициализации в отдельной функции

@bazJaz · Регистрация: 11.07.2014

Не правильно выводит

C++

1	cout << "1 елемент массива: " << mainArray[0] << endl;

В чем проблема? Спасибо.

C++

#include <iostream>
#include <locale.h>
/* run this program using the console pauser or add your own getch, system("pause") or input loop */
using namespace std;
 
int enterArray(int *, int);
int MaxElement(int *, int);
int MinElement(int *, int);
 
int main(int argc, char** argv) {
    setlocale(LC_ALL,"rus");
    int* mainArray;
    int size;
    cout << "Размер массива: ";cin >> size;
    enterArray(mainArray, size);
    cout << "1 елемент массива: " << mainArray[0] << endl;
    return 0;
}
 
int enterArray(int* array, int size){
    cout << "Размер массива enterArray: " << size << " !\n";
    array = new int[size];
    for(int i = 0 ; i < size ; i++){
        array[i] = i*i;
    }
    for(int i = 0 ; i < size ; i++){
        cout << "Елементы массива: "<<array[i]<<" !\n";
    }
    return 0;
}

@~~IrineK~~ · 11.10.2014, 18:28

Пишем в строке 15

C++

1	mainArray = new int[size];

а строке 22 убираем

C++

1	array = new int[size];

Не забудьте перед return 0 в main удалить

C++

1	delete [] mainArray;

@bazJaz · 11.10.2014, 18:34 **[ТС]**

спасибо!

@taras atavin · 11.10.2014, 18:45

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

@bazJaz · 11.10.2014, 18:57 **[ТС]**

Сообщение от taras atavin

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

если у меня будет несколько функций и массив надо инициализировать в одной из функций а в другой надо найти минимальный елемент массива, как передавать?
надо чтобы инициализация была в enterArray

C++

int main(int argc, char** argv) {
    setlocale(LC_ALL,"rus");
    int* mainArray;
    int size;
    cout << "Размер массива: ";cin >> size;
    mainArray = new int[size];
    enterArray(mainArray, size);
    cout << "1 елемент массива: " << mainArray[2] << endl;
    cout << "Минимальный елемент: " << MinElement(mainArray, size) << " !\n"; 
    cout << "Максимальный елемент: " << MaxElement(mainArray, size) << " !\n";
    
    delete [] mainArray;
    return 0;   
}
 
int enterArray(int* array, int size){   
    for(int i = 0 ; i < size ; i++){
        array[i] = (i+1)*(i+1);
    }
    for(int i = 0 ; i < size ; i++){
        cout << "Елементы массива: "<<array[i]<<" !\n";
    }
    return 0;
}
int MinElement(int* array, int size){
    int min = array[0];
    for(int i = 1 ; i < size; i++){
        if(min > array[i]){
            min = array[i];
        }
    }
    return min;
}
int MaxElement(int* array, int size){
    int max = array[0];
    for(int i = 1 ; i < size; i++){
        if(max < array[i]){
            max = array[i];
        }
    }
    return max;
}

@~~IrineK~~ · 11.10.2014, 19:05

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

и массив надо инициализировать в одной из функций

Тогда функция должна возвращать указатель на массив.

Добавлено через 4 минуты
Как-то так:

C++

int main(int argc, char** argv) {
    setlocale(LC_ALL,"rus");
    int size;
    cout << "Размер массива: ";cin >> size;
    int *mainArray = enterArray(size);
    cout << "1 елемент массива: " << mainArray[2] << endl;
    cout << "Минимальный елемент: " << MinElement(mainArray, size) << " !\n";   
    cout << "Максимальный елемент: " << MaxElement(mainArray, size) << " !\n";
    
    delete [] mainArray;
    return 0;   
}
 
int *enterArray(int size){   
    int *arr = new int[size];
    for(int i = 0 ; i < size ; i++){
        arr[i] = (i+1)*(i+1);
    }
    for(int i = 0 ; i < size ; i++){
        cout << "Елементы массива: "<<arr[i]<<" !\n";
    }
    return arr;
}

@taras atavin · 11.10.2014, 19:07

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

Тогда функция должна возвращать указатель на массив.

Не обязательно. Можно передать указатель на указатель, или ссылку на указатель.

@~~IrineK~~ · 11.10.2014, 19:11

Сообщение от taras atavin

Можно передать указатель на указатель, или ссылку на указатель.

Ваш код?

@taras atavin · 11.10.2014, 19:17

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

если у меня будет несколько функций и массив надо инициализировать в одной из функций а в другой надо найти минимальный елемент массива, как передавать?
надо чтобы инициализация была в enterArray

Указатель на параметр надо передать по ссылке, или указателю, а сам параметр (массив) по ссылке на указатель, или по указателю на указатель.

C++

1	int MinElement(int **, int);

, или

C++

1	int MinElement(int *&, int);

. Кстати, почему функция возвращает int? Если уж надо возвращать значение, зависящее от того, была ли ошибка времени исполнения, то лучше bool. Но зачем? Неудача выделения памяти даст указатель, равный NULL, или null_ptr, так что достаточно void. Или надо обязательно вызов функции запихать в условие ифа, или цикла?

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

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

Ваш код?

C++

#include <iostream>
#include <locale.h>
/* run this program using the console pauser or add your own getch, system("pause") or input loop */
using namespace std;
 
int enterArray(int *, int);
int MaxElement(int *, int);
int MinElement(int **, int);
 
int main(int argc, char** argv) {
    setlocale(LC_ALL,"rus");
    int* mainArray;
    int size;
    cout << "Размер массива: ";cin >> size;
    enterArray(&mainArray, size);
    cout << "1 елемент массива: " << mainArray[0] << endl;
    delete [] mainArray;
    return 0;
}
 
int enterArray(int **array, int size){
    cout << "Размер массива enterArray: " << size << " !\n";
    *array = new int[size];
    for(int i = 0 ; i < size ; i++){
        (*array)[i] = i*i;
    }
    for(int i = 0 ; i < size ; i++){
        cout << "Елементы массива: "<<(*array)[i]<<" !\n";
    }
    return 0;
}

, или

C++

#include <iostream>
#include <locale.h>
/* run this program using the console pauser or add your own getch, system("pause") or input loop */
using namespace std;
 
int enterArray(int *, int);
int MaxElement(int *, int);
int MinElement(int *&, int);
 
int main(int argc, char** argv) {
    setlocale(LC_ALL,"rus");
    int* mainArray;
    int size;
    cout << "Размер массива: ";cin >> size;
    enterArray(mainArray, size);
    cout << "1 елемент массива: " << mainArray[0] << endl;
    delete [] mainArray;
    return 0;
}
 
int enterArray(int *&array, int size){
    cout << "Размер массива enterArray: " << size << " !\n";
    array = new int[size];
    for(int i = 0 ; i < size ; i++){
        array[i] = i*i;
    }
    for(int i = 0 ; i < size ; i++){
        cout << "Елементы массива: "<<array[i]<<" !\n";
    }
    return 0;
}

.

@~~IrineK~~ · 11.10.2014, 19:19

bazJaz, у вас три варианта - выбирайте )

@bazJaz · 11.10.2014, 21:07 **[ТС]**

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

у вас три варианта - выбирайте )

спасибо))

@bazJaz 35 / 32 / 21 Регистрация: 11.07.2014 Сообщений: 390
	11.10.2014, 18:34 [ТС]	3
	спасибо! 0

@taras atavin 4197 / 1790 / 211 Регистрация: 24.11.2009 Сообщений: 27,563
	11.10.2014, 18:45	4
	bazJaz, проблема в том, что сам указатель передан по указателю, то есть функция enterArray имеет его копию, когда она выделяет память, меняет значение копии, но не оригинала, так как к нему функция enterArray вообще не имеет доступа, поэтому память выделяется по параметру функции enterArray , а не по переменной, взятой за основу его значения, в функции main память по прежнему не выделена. 1

@taras atavin 4197 / 1790 / 211 Регистрация: 24.11.2009 Сообщений: 27,563
	11.10.2014, 19:07	7
	Сообщение от IrineK Тогда функция должна возвращать указатель на массив. Не обязательно. Можно передать указатель на указатель, или ссылку на указатель. 1

@~~IrineK~~ Заблокирован
	11.10.2014, 19:11	8
	Сообщение от taras atavin Можно передать указатель на указатель, или ссылку на указатель. Ваш код? 1

@~~IrineK~~ Заблокирован
	11.10.2014, 19:19	10
	bazJaz, у вас три варианта - выбирайте ) 1

@bazJaz 35 / 32 / 21 Регистрация: 11.07.2014 Сообщений: 390
	11.10.2014, 21:07 [ТС]	11
	Сообщение от IrineK у вас три варианта - выбирайте ) спасибо)) 0

Работа с динамическим массивом: три варианта инициализации в отдельной функции

Решение