Динамическое выделение памяти под массив

Да, естественно, Ax обЪявлен, как

double **Ax;

delete[] Ax[i];

хотя это здесь не существенно.

Не так? "с C под *nix на C++ под M$"

> Не так? "с C под *nix на C++ под M$"

Ах-ха. Я б уж если и переделывал, то делал бы кроссплатформенным.

Для интереса, налабал такой вот код

int main(int argc, char** argv)
{

	//Allocate memory for arrays
	int i,j;

	

	Ax = (double **)malloc(N * sizeof(double *));
	if(!Ax) 
		err_exit(1);
	else{
		for(i=0;i<N;i++){
			Ax[i] = (double *)malloc((N-1) * sizeof(double));
			if(!Ax[i]){
				cleanup(Ax,N);
				err_exit(1);
			}
		}
	}

	Ax[3][1] = 5.0;
	//free arrays memory
	cleanup(Ax,N);

	
	return 0;
}

void cleanup(double **A, int x)
{
	int i;
	for(i=0;i<x;i++)
		free(A[i]);
	free(A);
}

Тоже - консольное выражение при занесении значения в Ax[i][j] - нафиг вылетает :( Может кто привести Пример КАК надо выделить память для динамического массива N*M? А то уже голова пухнет.

for(int i=0;i<N;i++){ delete Ax[i]; } Теперь подумай, как ты память выделял?

Да, ошибка такая

Unhandled exception at 0x0041149f in parvich.exe: 0xC0000005: Access violation writing location 0xfdfdfe05.

Я может чего-то не понимаю, но почему у тебя память под матрицу выделяется в разых местах (имеются ввиду разные куски памяти), а обращаешься ты к элементам матрицы как будто матрица лежит в одном выделенном(непрерывном, замечу) сегменте памяти?

> Для интереса, налабал такой вот код...

N = ...?

Как я понимаю методику выделкния памяти:

1)Объвляем указатель на указатель double **Ax

2)Выделяем массив из N-ного количества указателей типа double на double

3)Для каждого указателя выделяем строку матрицы из M элементов.

Эти действия производятся последовательно, почему в разных местах?

k_andy

/****************************************************
parvich.h
Defines global structures and functions declarations
*****************************************************/

#define N 3			//number of nodes
#define M 5			//number of brunches

Попробуй представить карту памяти при таком выделении. Вся память у тебя будет сотоять из N отрезков длинной N*sizeof (double). Они не будут идти последовательно друг за другом в общем случае! Обращение вида: A[i][j] - это обращение к памяти которая сидит по адресу: A + i*N+j. В случае выделения памяти по твоему алгоритму - залет на SIGSEGV гарантирован.

Хм, тогда может так:

doulbe *Ax = (double *) malloc(M * N * sizeof(double))

единственный выход?

Не единственный, но выход. Так работать будет. Кстати я слегка намудрил выше (комментировал не тот код, который надо было бы), но ты внимательно смотри на то, что делаешь и следи за памятью - тогда все будет просто зашибись.

> Ax[3][1] ...

> #define N 3

Ну всё понятно...

> Как я понимаю методику выделкния памяти:

Так выделяют память первокурсники, еще не освоившие Си.

> #define N 3 //number of nodes

Тебе не кажется, что в таком случае A[3] из "налабанного кода" выходит за пределы массива из 3-х элементов?

Вот б..! Такое пропустить... Позор на мою седую голову, а я тут демагогию развел..., черт....

Мда, позор на мою(неседую голову) :) Просто до этого N и M были по полтиннику :(((

p.s. Остановлися на последнем варианте с malloc, всем thx

> Остановлися на последнем варианте с malloc

Это зря. Уж лучше тогда std::vector<std::vector<double> >. Не пишите C-код на C++.

+1

1. exception unsafe by design
2. местами попросту некорректно (проверка на NULL)

вывод: продолжаем читать Страуструпа и gotw.ca

// wbr

>то я делаю не так?

сам же отвечаешь на свой вопрос:

>Переношу одну программулину с C под *nix на C++

может не стоит на С++ её переносить, если ты сам толком его не знаешь?

Reset

Может быть, но не от меня это зависит. :(

>Обращение вида: A[i][j] - это обращение к памяти которая сидит по адресу: A + i*N+j. В случае выделения памяти по твоему алгоритму - залет на SIGSEGV гарантирован.

только если у тебя статический массив вида double A[100][100] //N=100