Ошибка при работе с памятью

[Joose]

Всем привет. Столкнулся с проблемой при работе с malloc

#define H 4
#define V 4

...........

int** GENERATE_FORM()
{
    int key, i, j;
    //Массив SHAPES_TEMP хранит в себе виды возможных фигур
    int SHAPES_TEMP[7][4][4] =
    {
        {
            {1,1,1,1},
            {0,0,0,0},
            {0,0,0,0},
            {0,0,0,0}
        },
        {
            {1,1,0,0},
            {1,1,0,0},
            {0,0,0,0},
            {0,0,0,0}
        },
        {
            {1,0,0,0},
            {1,1,1,0},
            {0,0,0,0},
            {0,0,0,0}
        },
        {
            {0,0,1,0},
            {1,1,1,0},
            {0,0,0,0},
            {0,0,0,0}
        },
        {
            {0,1,1,0},
            {1,1,0,0},
            {0,0,0,0},
            {0,0,0,0}
        },
        {
            {1,1,0,0},
            {0,1,1,0},
            {0,0,0,0},
            {0,0,0,0}
        },
        {
            {0,1,0,0},
            {1,1,1,0},
            {0,0,0,0},
            {0,0,0,0}
        }
    };
    int **shape;
    shape = (int**)malloc(V * sizeof(int*));
    for(i = 0; i < V; i++)
        *shape = (int*)malloc(H * sizeof(int));
    key = (rand() % 7);

    //На основе случайного числа из массива фигур SHAPES_TEMP копируется двоичная последовательность
    for(i = 0; i < V; i++)                                  //Копировать определенную фигуру из SHAPES_TEMP
    {
        for(j = 0; j < H; j++)
        {
           shape[i][j] = SHAPES_TEMP[key][i][j]
        }
    }
    printf("end\n");

    return shape;
}

Проблема в том, что вместо массива 4*4 выделяется память под массив 2*4. Никак не пойму в чем ошибка. Помогите, пожалуйста.




С ошибкой разобрался, тему можно закрыть

[Kukurudza]

у мну работает:

int main() {
	static const size_t V = 4;
	static const size_t H = 4;
	static const size_t M = 7;

	static const int SHAPES_TEMP[M][V][H] = { 
		{
			{1,1,1,1},
			{0,0,0,0},
			{0,0,0,0},
			{0,0,0,0}
		},
		{
			{1,1,0,0},
			{1,1,0,0},
			{0,0,0,0},
			{0,0,0,0}
		},
		{
			{1,0,0,0},
			{1,1,1,0},
			{0,0,0,0},
			{0,0,0,0}
		},
		{
			{0,0,1,0},
			{1,1,1,0},
			{0,0,0,0},
			{0,0,0,0}
		},
		{
			{0,1,1,0},
			{1,1,0,0},
			{0,0,0,0},
			{0,0,0,0}
		},
		{
			{1,1,0,0},
			{0,1,1,0},
			{0,0,0,0},
			{0,0,0,0}
		},
		{
			{0,1,0,0},
			{1,1,1,0},
			{0,0,0,0},
			{0,0,0,0}
		}
	};

	int** shape;
	shape = new int*[V];
	for (size_t i = 0; i < V; ++i) {
		shape[i] = new int[H];
	}

	const size_t key = rand() % M;
	for (size_t i = 0; i < V; ++i) {
		for (size_t j = 0; j < H; ++j) {
			shape[i][j] = SHAPES_TEMP[key][i][j];
		}
	}

	std::cout << "key = " << key << "\n";
	for (size_t i = 0; i < V; ++i) {
		for (size_t j = 0; j < H; ++j) {
			std::cout << shape[i][j] << "\t";
		}
		std::cout << "\n";
	}
	std::cout << "\n";

	return 0;
}

обратите внимание на статические константы. уберите дефайны. это не очень хорошо. и не забудьте потом удалить shape, а то память потечет. а, и не нужно в программе плодить магические константы типа 4, 7. они должны иметь имя и комментарий - что это. иначе человек, который будет читать вашу программу, будет очень долго чесать репу и думать: почему у него здесь: int SHAPES_TEMP[7][4][4] стоят цифры, а здесь:

for(i = 0; i < V; i++)                                  //Копировать определенную фигуру из SHAPES_TEMP
    {
        for(j = 0; j < H; j++)

дефайны (константы)

[Joose]

Спасибо, просто сначала вместо макроконстант были обычные числа, а потом где-то просто забыл заменить. По поводу очищения shape - каким образом можно очистить массив уже после возвращения его в вызывающую функцию?

[Igor95]

возвращения его в вызывающую функцию?

возвратить В вызывающую функцию?

[Kukurudza]

	for (size_t i = 0; i < V; ++i) {
		delete [] shape[i];
	}
	delete [] shape;

[Joose]

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

возвратить В вызывающую функцию?

Ну да, функция GENERATE_FORM возвращает указатель на указатель В вызывающую функцию main. Или я что-то делаю не так?

[Joose]

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

	for (size_t i = 0; i < V; ++i) {
		delete [] shape[i];
	}
	delete [] shape;

Благодарю.