指定しているのにコンパイルできない！どうして？

Question

前提・実現したいこと

N×3と3×Mの行列の積を求めるプログラムを作っています
コンパイルが出来ない為、コンパイルできるようにしたいです。

発生している問題・エラーメッセージ

error: redeclaration of 'a' with no linkage
previous declaration of 'a' was here
error: redeclaration of 'b' with no linkage
previous declaration of 'b' was here



#include <stdio.h>
#include <math.h>
int main(void){
int n,m,i,j,k,l,x,y,z; 
double a[i][j],b[k][l];

    printf("Aの列数\n"); 
    scanf("%d", &n);
    printf("Bの行数\n"); 
    scanf("%d", &m);

    for(i=1; i<n; i++){
      for(j=1; j<4; j++){
      printf("A[%d][%d]を入力",i,j);
      scanf("%d", a[i][j] );
      }
    }
    
    for(k=1; k<4; k++){
      for(l=1; l<m; l++){
      printf("B[%d][%d]を入力",k,l);
      scanf("%d", b[k][l] );
      }
    }

double C[x][y], a[x][z], b[z][y];  
 
    for(x=1; x<n; x++){
      for(y=1; y<m; y++){
        for(z=1; z<4; z++){
        C[x][y] = a[x][z]*b[z][y];
        printf("結果＝%d",C[x][y]);
        }
      }
    }
return 0;
}

試したこと

double型とうで指定したがうまくいかない

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

前提・実現したいこと

C言語でN×3と3×Mのプログラムを作成しています。
コンパイルがどうしても出来ない状態です。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

ソースコード

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[BeatStar]

カテ違い。本当にPHPなの？
ガチでPHPなの？

[y_waiwai]

PHPのタグ付いてるけど、どー見てもPHPじゃないですね。
修正しよう

[matsu333]

失礼しました。間違えました。コンパイルエラーに関する質問です。
よろしくお願いします。

[y_waiwai]

その言語は何でしょうかｗ

＃笑ってしまったｗ

[matsu333]

C言語です

[y_waiwai]

なら、C言語のタグがありますんで、それつけてくださいｗ

[matsu333]

始めて使用するので。。。失礼しました。

[BeatStar]

あと、
> ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。

のような関係のない部分は削除しましょう。余計なノイズです。

[matsu333]

失礼しました。更新いたしましたのでよろしければご教授お願いいたします。

[dodox86]

なぜこう、基礎をすっ飛ばして目的第一で突っ走る方が後を絶たないのだろうか。とても不思議です。何か、研究などの道具としての扱いなのですかね。

[K_3578]

Cだと結構色んな所から触れる可能性があるので何かを便利にしようとしてコピペで動かそうとしたら
動かないって人が多いのではという推測をしてます

Answer 1

コンパイルは、当たり外れのくじを引くわけではないのです。失敗すれば、必ず対応するエラーメッセージを返してくれるはずです。

エラーメッセージを読むのが第一歩です。それもせずに改善しようとしているのでは、当て物と何も変わりません。

Answer 2

とりあえず、基礎からやり直しましょう。
基礎ができていればありえない問題です。

ｰｰｰｰ

一つ目。エラーメッセージぐらいは読みましょう。
エラーメッセージは暴言や罵倒ではありません。
コンパイラからのメッセージです。
ヒントが隠されています。

error: redeclaration of 'a' with no linkage
previous declaration of 'a' was here

[訳]
エラーだぜ〜: “a”ってのが重複して宣言されってっぞ？
この行より先に、すでに”a”ってのが宣言されてっぞ？

つまり、重複宣言が問題なっているわけです。

いや、わかるやん。ってなりそうですが、
コンパイラは機械です。

コンパイラの気持ちになってみましょう。

最後の、『結果』があるようなfor文内で、
配列aが使われていますが、これの定義はなんでしょうか。
でもコードを見ると、なぜか二箇所でaが宣言されています。
そうなると、コンパイラは○○なので「え？ どっちのaなん？」と混乱します。
まあ、人間でも混乱しますが。

そうなると、解釈できずに、『おいおい、重複してんぞ！？』と言うわけです。

これが今回のコンパイルエラー。

ほら、基礎と理由がわかればあとは簡単ですね。
どちらかを別名にするか、一方(おそらく意味的に二番目の方)を削除すればいいだけです。

このようにヒントが隠されています。

二つ目。コードを読みましょう。

プログラミングは『こう書けばいい』というものではありません。

なのでコードを読みましょう。
コードを読むコツは『一行レベルで、その行が何をしているかを考えながら読む』です。

#include <stdio.h>
#include <math.h>
int main(void){
// 変数 n等を宣言
int n,m,i,j,k,l,x,y,z;
// 配列a, bを宣言 
double a[i][j],b[k][l];
    // ユーザ入力を受け付ける
    printf("Aの列数\n"); 
    scanf("%d", &n);
    printf("Bの行数\n"); 
    scanf("%d", &m);

     // 変数i の初期値: 1, iがn未満の間, +1しながらループ
    for(i=1; i<n; i++){
      /// 変数j の初期値: 1, jが4未満の間, +1しながらループ
      for(j=1; j<4; j++){
      // ユーザ入力を受け付ける
      printf("A[%d][%d]を入力",i,j);
      scanf("%d", a[i][j] );
      }
    }

   // 〜中略〜

// 新たに配列C, a, b を宣言
double C[x][y], a[x][z], b[z][y];  

    // 変数 x の初期値: 1, xがn未満の間, +1しながらループ
    for(x=1; x<n; x++){
      for(y=1; y<m; y++){
        for(z=1; z<4; z++){
        C[x][y] = a[x][z]*b[z][y];
        printf("結果＝%d",C[x][y]);
        }
      }
    }
return 0;
}

のようにやります。
実際には細かくやるべきですが、今回の問題はそこじゃないので割愛します。

そして、このコメントを疑似コードとして切り出します。

1. 変数n,m 等を宣言する
2. 配列a, bを宣言する
3. ユーザ入力を受け付けて取得
4. 変数iの初期値: 1, iがn未満の間, ループ
    4.1. 変数jの初期値: 1, jが4未満の間, ループ
         4.1.1. ユーザ入力を受け付けて取得
〜中略〜
7. 配列C,a,b を宣言する
8. 変数xの初期値: 1, xがn未満の間, ループ
    8..1. 変数…, ループ
       8.1.1. ループ
          8.1.1.1. 計算！
          8.1.1.2. 出力

みたいな感じですね。(途中から手ぬきな感じですが)

これを現実世界で手作業でやってみてください。
つまり、算数か数学かなんかの手順としてあると考えるのです。

そして、実際のデータを入れて考えみるとか。

現実世界だと、『変数宣言』とかは、ホワイトボードかなんかに、

m: 1
n: 2

みたいに書いていく感じです。

でも、(7)で、『新たに』書く状態です。

n: 1
n: 2
…
a: { 1, 2, 4 … }
b: { 3, 3, 4, … }
C: { … }
a: { 0, 0, 0, … } <= これ
b: { 0, 0, 0, … } <= これ

みたいな状態です。で、いざ読もうとすると、『あれ？a ってどっちのだっけ？』となります。

そのため、エラーが出ているのです。

プログラミングは簡単に言えば、現実世界のシミュレーションです。

[質問者さんが学ぶべきこと]
■ 変数
■ 配列
■ (変数等の)スコープ
■ プログラミングとは
■ エラーメッセージを読んで対処すること
■ コードを読むこと

Comments

[matsu333]

本当に失礼しました。他のユーザー様からも同様のご指摘をいただきました。基礎固めが出来ていない状態でこのプログラムの作成をしようとしたことが誤りであったと痛感しています。長文で丁寧に回答してくださりありがとうございます。より力を付けてから中身のある質問をしていくよう心がけます。

Answer 3

int n,m,i,j,k,l,x,y,z;

double a[i][j],b[k][l];

配列の宣言は、宣言時にそのサイズが決まってなければいけません。
i,j,k,lには値が入ってません

Comments

[matsu333]

回答ありがとうございます。i,j,k,lに最初に値を入れるにはどのようにすればいいでしょうか？
#defineで書き込むのですか？

[y_waiwai]

そもそもの話になりますが、まずはC言語の基本的な文法を学ぶところから始めましょう。
あなたのそのコードの意味を理解しない限りは、どんなアドバイスしたところでそのコードは動きませんよ

[matsu333]

分かりました。ありがとうございます。

Answer 4

エラーメッセージのとおりです。
同じ関数スコープ内にaとbが2回宣言・定義されています。

Comments

[matsu333]

回答ありがとうございます。2回宣言定義されているということはa[i][j]とa[x][z]の二つにaが含まれた状態で宣言されてしまっているということでしょうか？この場合はどちらかを別の文字に置き換えないとなりませんか？？

[itagagaki]

同じスコープに同じ名前の変数を複数宣言することができないのは理解できますか？

[matsu333]

すいません。C言語初心者なモノで、、、理解していませんでした。複数宣言できない"a"はどのようにすればよいでしょうか。