помогите с кодом!

[Entera]

В С нужно из файла вызвать матрицу. В этой матрицы найти все ненулевые элементы и написать все значения и индексы этих значений
Вот код..ток в нем что то не так...посмотрите!!!

#include<stdio.h>
#include<alloc.h>

Код:

matrica(int **a, int *b, int *c, int m, int n, int i, int j, int *k)
{
for(i=0;i<m;i++)
for(j=0;j<n;j++)
if(a[i][j]!=0)
{
b[*k]=a[i][j];
c[*k]=(i*n)+j;
(*k)++;
}
}
main()
{
int **a, *b, *c, m, n, i, j, k;
FILE *in;
if((in=fopen("1.txt","r"))==NULL)
{
printf("\n net faila"); exit(1);
}
fscanf(in, "%d%d",&m,&n);
k=m*n/2;
a=(int**)malloc(m*sizeof(int*));
for(i=0;i<m;i++)
a[i]=(int*)malloc(n*sizeof(int));
b=(int*)malloc(k*sizeof(int));
c=(int*)malloc(k*sizeof(int));
for(i=0;i<m;i++)
for(j=0;j<n;j++)
fscanf(in,"%d",&a[i][j]);
matrica(a,b,c,m,n,i,j,&k);
{
printf("\n OTVET \n");
printf("Element Indeks");
for(i=0;i<k;i++)
printf("\n %d %d",b[i],c[i]);
}
}

[Aveic]

Что сразу бросается в глаза: функции не возвращают значений, void не написан
Во-первых неправильно создается матрица:

Код:

a=(int**)malloc(m*sizeof(int*));
for(i=0;i<m;i++)
 a[i]=(int*)malloc(n*sizeof(int));

Что естественно, означает что и обращение к элементам матрицы тоже неверно в функции matrica(...)
Вместо

Код:

if(a[i]==0)...

надо видимо писать

Код:

if(a[i][j]==0)...

Потом несуразица какая-то с использованием переменной k. Почему она равна m*n/2 ? Потом еще много ошибок, о которых уже впадлу писать, особенно вроде последнего printf когда в %d подставляется указатель
в общем могу предложить следующий вариант:

Код:

struct element { int val,x,y; };

int matrixf(int** matrix,element* elems,int m,int n)
{
	int k=0;
	for(int i=0;i<m;++i)
		for(int j=0;j<n;++j)
			if(matrix[i][j]!=0)
			{
				element el={matrix[i][j],i,j};
				elems[k++]=el;
			}
	return k;			
}

int main()
{
	int** v,m,n;
	FILE* in;
	in=fopen("1.txt","r");
	if(in==NULL)
	{
		printf("\n net faila \n");
		exit(1);
	}
	fscanf(in,"%d%d",&m,&n);
	v=(int**)malloc(m*sizeof(int*));
	int i=0,j=0;
	for(;i<m;++i)
		v[i]=(int*)malloc(n*sizeof(int));
	for(i=0;i<m;++i)
		for(;i<n;++j)
			fscanf(in,"%d",&v[i][j]);
	int k=m*n;
	element* els=(element*)malloc(k*sizeof(element));
	k=matrixf(v,els,m,n);
	printf("\n OTVET :) \n");
	for(i=0;i<k;++i)
		printf("Element {%d,%d} with value=%d\n",els[i].x,els[i].y,els[i].val);
	printf("\n END \n");
	return 0;
}

[Антон (LogRus)]

с ним много чего не так.

кое-что поправил.
Посмотри. сравни. спрашивай.

Пока писал уже меня опередили. Два примера лучше чем один

Код:

struct mindex
{
	int x;
	int y;
	int * value;
};

void matrica(int **a, mindex *b, int m, int n)
{
	for(int i=0; i < m; ++i)
		for(int j=0; j < n; j++)
			if(a[i][j]!=0)
			{
				b->value = &a[i][j];
				b->x = i;
				b->y = j;
				++b;
			}
}

int _tmain(int argc, _TCHAR* argv[])
{
	FILE *in;
	if((in=fopen("1.txt","r")) == NULL)
	{
		printf("\n net faila");
		exit(1);
	}

	int m, n;
	fscanf(in, "%d%d",&m,&n);

	int ** a = (int**)malloc(m*sizeof(int*));

	for(int i=0; i<m; ++i)
		a[i]=(int*)malloc(n * sizeof(int));

	mindex * b = (mindex*)malloc(m * n * sizeof(struct mindex));
	memset(b, 0, m * n * sizeof(struct mindex));

	for(int i = 0; i < m; ++i)
		for(int j = 0; j < n ; ++j)
			fscanf(in,"%d",&a[i][j]);

	matrica(a, b, m, n);

	printf("\n OTVET \n");
	printf("Element Indeks");
	mindex * b_pos = b;
	for(int i=0; i < m*n && b_pos->value != NULL; ++i, ++b_pos)
	{
		printf("\n Value=%d x=%d y=%d", *b_pos->value, b_pos->x,  b_pos->y);
	}

	return 0;
}

[Dimka]

Во-первых неправильно создается матрица:

Код:

a=(int**)malloc(m*sizeof(int*));
for(i=0;i<m;i++)
 a[i]=(int*)malloc(n*sizeof(int));

А что тут неправильно?

[Dimka]

LogRus, а кто память чистить будет? Непедагогично.

[Антон (LogRus)]

dimka,
Но могу сказать, что в данном случае не критично, т.к. всё равно при выходе прибьются.

[Aveic]

dimka, я написал правильный вариант, на тот момент у него было неправильно написано. Автор исправил сообщение. Было написано a=(int*)malloc(n*sizeof(int)). Он уже много чего исправил.

LogRus, а кто память чистить будет? Непедагогично.

Да очистку мы забыли , но все равно для такой программы это не приведет к особым проблемам.

[Dimka]

LogRus, Aveic, вот потому я и говорю "Непедагогично." В этой программе проблем не будет, но приучать к хорошему, светлому, вечному надо сразу.

[Aveic]

Согласен, тогда Entera, освобождение ресурсов остается тебе на домашнее упражнение