Thread: Determinant Calculation Program.... HELP PLZ

Well did you call it 3 times?
read( mat1, n, fp );
read( mat2, n, fp );
read( mat3, n, fp );
?
You're not returning n.

By the way, n should be a local variable. It's useless as a parameter.

Code:

void read(double a[N][N], FILE* inp)

{
	int i,j,n;
	fscanf(inp,"%d", &n);
	for (i=0; i<n; i++)
	{ 
		for (j=0;j<n;j++)
		fscanf(inp, "%lf", &a[i][j]);			
	}
    return n;
}

Then
n1 = read( mat1, fp );

How many times do you call this new read function of yours ?

EDIT : Beaten to the punch

hm ok, I just replaced the read function with the code above and it gave me errors:

c:19: warning: `return' with a value, in function returning void
c:54: warning: passing arg 2 of `read' makes pointer from integer without a cast
c:54: too many arguments to function `read'

So if Im inputing from a single txt file that contains data of 3 matrices, I still have to call read function 3 times in the main?

PS. Also the thing I noticed if I include the fscanf in the read function to read n, it wont read the n which is the first value but rather the first element of the matrix row.

# 1 Don't bump up your threads.

# 2 your function is declared as void, yet it returns an int. So it should be declared to return an int. That's sort of Salem's bad. But I don't think he expected you to copy and paste the code. Further, Salem's function takes in a FILE *, whereas you're probably sending it an int. Bad news. Lastly, your original function took 3 arguments. Salem's takes 2. I've noted this error in your code before. By now you should know that when your compiler complains about "too many arguments to function foo" it means that you're sending too many arguments to function foo...

> c:19: warning: `return' with a value, in function returning void
So make it return something

void read(double a[N][N], FILE* inp)
becomes
int read(double a[N][N], FILE* inp)

Simple innit?