указатели vs ссылки

[Aistn]

Ребята есть две функции... та что через ссылки сделана работает та что через указатели нет... что я делаю не правильно?????

void result (float array[I][J], float &min, float &max, int &count) {
	max = min = array[0][0]; 
	count = 0;
	for (int i=0; i<I; i++) {
		for(int j=0; j<J; j++) {
			
			if (array[i][j] == 0) {
				count++;
			}
			if (min > array[i][j]) {
				min = array[i][j];
			}
			if (max < array[i][j]) {
				max = array[i][j];
			}	
		}
     }

void result (float **array, float *min, float *max, int *count) {
	*max = *min = array[0][0]; 
	*count = 0;
	for (int i=0; i<I; i++) {
		for(int j=0; j<J; j++) {
			
			if (array[i][j] == 0) {
				*count++;
			}
			if (*min > array[i][j]) {
				*min = array[i][j];
			}
			if (*max < array[i][j]) {
				*max = array[i][j];
			}	
		}
     }

при выполнении второй функции выскакивает в консоли ошибка сегментации а программа компилится... при этом из всех элементов работает нормально max указатель остальные нет

[Granus]

I,J объявлены глобально?

здесь вроде все правильно, покажите код создания массива и вызова самой функции

[Aistn]

#include <iostream>

#define I 5
#define J 6

using namespace std;

void outputArray(float**, char);
float **inputArray(char);
void result (float**, float*, float*, int*);

int main(int argc, char **argv) {
    float **a;
	float *min, *max;
	int *count;
	a = inputArray('A');
	outputArray(a, 'A');
	result(a, min, max, count);
	cout << "\nНаименьшее значение в массиве А: " << *min << endl;
	cout << "Наибольшее значение в массиве А: " << *max << endl;
    cout << "Количество нулевых значений в массиве А: " << *count << endl;
    return 0;
}

float **inputArray(char name) {
	float **p = new float *[I];
	cout << "Заполните массив: " << name << "["<< I << "][" << J <<"]"<< endl;
	for (int i = 0; i < I ;i++) {
		p[i] = new float [J];
		for(int j = 0; j < J ;j++) {
			cout << "Введите элемент " << name << "[" << i << "][" << j <<"] : ";
			cin >> p[i][j];
		}
	}
	return p;
}

// определение функции вывода массива на экран
void outputArray(float **array, char name) {
	int i,j;
	cout.setf( ios::showpoint | ios::fixed);
	cout << "\nМассив " << name ;
	for (i=0; i<I; i++) {
		cout << "\n";
		for(j=0; j<J; j++) {
			cout.width(8);     // минимальное кол-во позиций
			cout.precision(2);
			cout << array[i][j];
		}
     }

}

void result (float **array, float *min, float *max, int *count) {
	*max = *min = array[0][0]; 
	*count = 0;
	for (int i=0; i<I; i++) {
		for(int j=0; j<J; j++) {
			
			if (array[i][j] == 0) {
				*count++;
			}
			if (*min > array[i][j]) {
				*min = array[i][j];
			}
			if (*max < array[i][j]) {
				*max = array[i][j];
			}	
		}
     }
}

[Granus]

	float *min, *max;
	int *count;
	...
	result(a, min, max, count);

Вы создаете min, max и count как указатели. при создании они ни на что не указывают (в крайнем случае уж точно не на область памяти, Вашей программе выделенную).
Вам нужно либо передавать не указатели, а адреса переменных

	float min, max;
	int count;
	...
	result(a, &min, &max, &count);
	cout << "\nНаименьшее значение в массиве А: " << min << endl;
	cout << "Наибольшее значение в массиве А: " << max << endl;
	cout << "Количество нулевых значений в массиве А: " << count << endl;

...либо выделить память в указателях под переменные:

	float *min=new float, *max=new float;
	int *count=new int;
	...
	result(a, min, max, count);
	cout << "\nНаименьшее значение в массиве А: " << *min << endl;
	cout << "Наибольшее значение в массиве А: " << *max << endl;
	cout << "Количество нулевых значений в массиве А: " << *count << endl;

[Aistn]

Спасибо большое)))) я просто недопонимал момента что под указатели нужно выделять память... я считал что при объявлении они резирвируют память как и обычные переменные))) спасибо))

[Granus]

При объявлении они как и обычные переменные просто создаются и в них записан какой-либо мусор. А так как указатель - это адрес в памяти, ни к чему хорошему это не приведет) Создавая что-то в памяти оператором new и записывая адрес созданного в указателем, мы обеспечиваем возможность работы с ним)

да, еще
если создаете в памяти оператором new, то потом обязательно удалите из памяти оператором delete

float *min=new float;
... // работа с указателем
delete min;

[Aistn]

буду иметь ввиду, спасибо...
То есть как я понял я не могу сделать так....
int *i;
*i=12;
???
Только
int *i;
i = new int;
*i =12;
??

[Granus]

Да, Вы абсолютно правы)
Компилятор это примет, но во время выполнения будет ошибка

кстати, так как вы таким же образом создаете массив

float **p = new float *[I];
...

то его тоже нужно бы удалить

for(i=0;i<I;++i)delete [] p[i];
delete [] p;

delete [] - оператор, удаляющий массив, адрес которого записан в указателе

[Aistn]

Спасибо ещё раз за помощь)))

[Granus]

Получилось хорошее руководство по указателям :D