Передача двумерного массива в функцию

[Megapixar]

Решил вот я написать программу которая составляет магические квадраты. Там две Функции первая составляет квадрат, вторая проверяет на равенство сумм, вот только не получаеться передать значение из первой функции во вторую...

#include "stdafx.h"
#include <stdio.h>
#include <time.h>
#include <stdlib.h>
#include <malloc.h>
void generkv();
void proverka(int **,int);
int _tmain(int argc, _TCHAR* argv[])//10
{ 
	
	generkv();
	

		return 0;
}
void generkv()
{const int n1=4;
	int **kv=NULL;
	int *p2,i,j,h,n2,k=0,ind,*mas,kvadrat[n1][n1];
	n2=n1*n1;
	mas=(int*)malloc(n2*sizeof(int));                
		 if(!mas)
			 {printf("error");
			  exit(1);
			 }
		// kvadrat=(int*)malloc(n2*sizeof(int));               
		// if(!kvadrat)
			// {printf("error");
			//  exit(1);
			// }
for(i=0;i<n2;i++)
{mas[i]=i+1;
printf(" %d ",mas[i]);
}
srand(time(NULL));

						for(i=0;i<n1;i++)
							{ for(j=0;j<n1;j++)
								{	
									ind=rand()%(n2-k);
									k++;
									kvadrat[i][j]=mas[ind];
										for(h=ind+1;h<n2;h++)
											mas[h-1]=mas[h];
									
									p2=(int*)realloc(mas,(n2-1)*sizeof(int));
										if(p2)
											mas=p2;
										else 
											return;
									
								}
						    }
						
						proverka(kvadrat,n1); // вот здесь пишет ошибку  не возможно перевести из int[][] в int**
}
void proverka(int **kvadrat,int n1)
{ int i,j;
  double Sum,s1,s2,s3,s4;
	Sum=n1*(n1*n1+1)/2;
	s1=Sum;
	s2=Sum;
	s3=Sum;
	s4=Sum;
	printf("Sum=%f",Sum);
	for(i=0;i<n1;i++)
	{	if(s1!=Sum || s2!=Sum)
			generkv();
		else
		{s1=0;
		s2=0;}
		for(j=0;j<n1;j++)
			{	s1=s1+*(*(kvadrat+i)+j);
				s2=s2+*(*(kvadrat+j)+i);
			}

	}
				for(i=0;i<n1;i++)
					{	if (s3!=Sum || s4!=Sum)
								generkv();
							else
								{s3=0;
								s4=0;}
								for(j=0;j<n1;j++)
								{if(i==j)
									s3=s3+*(*(kvadrat+i)+j);
								else if ((i+j)==(n1+1))
									s4=s4+*(*(kvadrat+i)+j);
								}
					}	
				for(i=0;i<n1;i++)
							{ printf("\n");
								for(j=0;j<n1;j++)
							 printf(" %d ",kvadrat[i][j]);
							}
}

[RUSt88]

proverka(kvadrat,n1); // вот здесь пишет ошибку не возможно перевести из int[][] в int**

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

посмотри, мож поможет

	int **pmat; // объявление указателей на массивы
	int R; // размерность матрицы
	pmat = new int *[R];  // создаём массив указателей
	for (int i = 0; i < R; i++)
	{
		*(pmat + i) = new int[R]; //создаём матрицу указателей
	}
	randomize();
	for (int i = 0; i < R; i++)  // цикл по строкам
	{
		for (int j = 0; j < R; j++) //цикл по столбцам
		{
			*(*(pmat + i) + j) = random(9); //заполняем случайными числами 0..9
		}
           }

вместо *(*(pmat + i) + j) можно писать p[i][j]

[Megapixar]

Вроде понял, спасибо

[evgn]

Можно было просто написать так:

proverka((int **)kvadrat,n1); // вот здесь пишет ошибку не возможно перевести из int[][] в int**

Т.е. сделать приведение типов.
Либо описать kvadrat как

int **kvadrat = (int **)new int[n1*n1];

Либо еще как-нибудь... есть много способов

[Megapixar]

Ребята новая проблема.....по ходу, при генерации квадрата он все время генерирует один и тот же.

// Генерация магических квадратов.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include <stdio.h>
#include <time.h>
#include <stdlib.h>
#include <malloc.h>
void generkv();
void proverka(int *kvadrat,int);

int _tmain(int argc, _TCHAR* argv[])
{ 
	
	generkv();
	

		return 0;
}
void generkv()
{const int n1=3;
	int *p2,i,j,h,n2,k=0,ind,*mas,kvadrat[n1][n1];
	n2=n1*n1;
	mas=(int*)malloc(n2*sizeof(int));                
		 
	if(!mas)
			 {printf("error");
			  exit(1);
			 }
		
for(i=0;i<n2;i++)
	mas[i]=i+1;

	
			for(i=0;i<n1;i++)
					{ for(j=0;j<n1;j++)
						{	srand(time(NULL));
							ind=rand()%(n2-k);
							k++;
							kvadrat[i][j]=mas[ind];
								for(h=(ind+1);h<n2;h++)
										mas[h-1]=mas[h];
									
							p2=(int*)realloc(mas,(n2-1)*sizeof(int));
									if(p2)
											mas=p2;
										else 
											return;
									
						}
				    }
				//free(mas);
//free(p2);
proverka(*kvadrat,n1);
}


void proverka(int *kvadrat,int n1)
{ int i,j;
  double Sum,s1,s2,s3=0,s4=0;
	Sum=n1*(n1*n1+1)/2;
	
	for(i=0;i<n1;i++)	
		{	s1=0;
	
				for(j=0;j<n1;j++)
					s1=s1+ *(kvadrat + i*n1 + j);
						
			if(s1!=Sum)
			generkv();// здесь если ставить printf("%f", s1); то все печатает, а если вызывать функцию то не работает.....
	// printf("%f", s1);
		}


	for(i=0;i<n1;i++)	
	{
		s2=0;
				for(j=0;j<n1;j++)
					s2=s2+ *(kvadrat + j*n1 + i);
		
			
		if(s2!=Sum) 
		generkv();// здесь если ставить printf("%f", s2); то все печатает, а если вызывать функцию то не работает.....
//printf("%f", s2);
	}
	
		
			for(i=0;i<n1;i++)
					{	
							for(j=0;j<n1;j++)
									{if(i==j)
									s3=s3+ *(kvadrat + i*n1 + j);}
					}
if (s3!=Sum)
generkv();// здесь если ставить printf("%f", s3); то все печатает, а если вызывать функцию то не работает.....
//printf("%f", s3);				
			for(i=0;i<n1;i++)
					{
								for(j=0;j<n1;j++)
									{if((i+j)==(n1-1))
									s4=s4+ *(kvadrat + i*n1 + j);}
					}
		
if (s4!=Sum)
generkv();// здесь если ставить printf("%f", s4); то все печатает, а если вызывать функцию то не работает.....
//printf("%f", s4);
							
							if(s1==Sum && s2==Sum && s3==Sum && s4==Sum)				
				{for(i=0;i<n1;i++)
									{ printf("\n");
										for(j=0;j<n1;j++)
									 printf(" %d ",*(kvadrat + i*n1 + j));
									}

				}
				else 
		
printf("  f%ff    %f    %f   %f    %f",Sum,s1,s2,s3,s4);
}

[evgn]

srand(time(NULL));

перемести в начало _tmain().
srand вообще обычно один раз вызывается.

[Megapixar]

а вот в этом кусочке что за проблема??
if(s1!=Sum)
generkv();// здесь если ставить printf("%f", s1); то все печатает, а если вызывать функцию то не работает.....
// printf("%f", s1);

[evgn]

А как оно не работает?
Скорее всего у тебя просто происходит переполнение стека, т.к. функция generkv вызывается рекурсивно неизвестно сколько раз.

[Megapixar]

Ну да, она вызываеться до тех пор пока не сгенерится квадрат, у которого суммы элементов по строкам, столбцам и диагоналям будут равны. Так что это нереально осуществить?

[evgn]

Почему не реально? Реально. Но надо подумать над алгоритмом. Избежать такой дикой рекурсии. И память у тебя выделяется, но нигде не освобождается.