行列の積を関数を使って求める・・？

No5 bf109e さんに補足です。 No5 bf109e さんのご回答の、first()、second() のそれぞれの引数を、以下の3つのうちのいずれかに変更してみてください。 # a だけでなく、b も同様です。 int a[][] →int a[NUMBER][NUMBER] 　int a[][NUMBER] 　int (*a)[][NUMBER] 一番目は、配列をそのまま渡すイメージでわかりやすいと思います。 二番目は、配列をそのまま渡すイメージではありますが、前の要素数が空っぽになっています。 要するに書いても意味がないってことです。 その理由が三番目です。 これは「配列を指すポインタ」の定義になります。 配列はその先頭要素を指すポインタに読み替えられるのはご存知でしょうか。 配列の配列でも同様の読替えが起こります。 参考URL の No5 (私のアドバイスですけど。。。) にも書いていますので、そちらもご参照ください。 # 少々長いのはご勘弁ください。。。 これでいかがでしょうか。

#include<stdio.h> #define NUMBER 10 void first(int a[][], int b[][]) { /*行列a,bの要素を入力*/ second(a, b, x1, x2, y1, y2); } void second(int a[][], int b[][], int x1, int x2, int y1, int y2) { int c[NUMBER][NUMBER] = {0}; /*a,bの積を計算,表示*/ } int main(void) { int a[NUMBER][NUMBER] = {0}; int b[NUMBER][NUMBER] = {0}; printf("行列の積を計算します\n"); first(a,b); return 0; } 大まかな形はこんなもんでどうでしょう。

#1さんの様に、 式中にc[i][j]を埋め込まないで、一時変数を使うというのは、 ループの中で、[i][j]によるアドレスの計算をいちいちさせないという効果があります。 コンパイラによっては自分で最適化してくれるかもしれませんが、書いても損にはなりません。（むしろ推奨です） ただ、使う前に初期化しないといけないのは同じです。

＃２さんに賛成。 よく考えたら、同じでした。 ごめんなさい。

c[i][k]がちゃんと０で初期化されていないのが原因だと思います。 適当なタイミング for(j=0; j<x2; j++)の前で c[i][k]=0をしてやればいいような気がします

for(j=0; j<x2; j++) c[i][k] = c[i][k] + a[i][j]*b[j][k]; } ここで、足し過ぎてませんか？ 変数をもうひとつ用意して、 ｗ=ｗ + a[i][j]*b[j][k]; ｝ c[i][k] = ｗ; では、いかがでしょう。