二次元配列の出力

Question

二次元配列を使ってデータを出力したいのですがこのソースをコンパイルすると次のようになってしまい正しい値が出力されません。どこがいけないか教えて頂けますでしょうか。言語はC++です。

ソース

#include <iostream>
#include <math.h>
using namespace std;
int main(){
	int nofc,i;
	double imm[nofc][2];
    nofc=2;
    i=0;
    imm[0][0]=145;
    imm[0][1]=50;
    imm[1][0]=160;
    imm[1][1]=60;

    
   while(i<nofc){
    cout<<imm[i][0]<<endl;
    cout<<imm[i][1]<<endl;
    i=i+1;
    }
}

コンパイル結果
$ ./a.out
145
50
6.95324e-310
6.95324e-310

Answer 1

nofc の値設定のタイミングのことの指摘は ryunix さんの回答で指摘されてますので。
配列のサイズをなるべく明示的に指定したいですむ方法を示します。

#include <iostream>
#include <math.h>

using namespace std;
int main() {
  double imm[][2] = {
    {145, 50},
    {160, 60},
    {161, 61},
    {162, 62},
  };

  int nofc = sizeof(imm) / sizeof(imm[0]);
  for (int i = 0; i < nofc; i++) {
    cout<<imm[i][0] << endl;
    cout<<imm[i][1] << endl;
  }
}

実行結果は次のようになります。

$ g++ array.c++
$ ./a.out 
145
50
160
60
161
61
162
62

Comments

[ristori]

sizeof演算子を使うやり方があるのですね！便利です！今後使おうと思います。ありがとうございます。

Answer 2

二次元配列の宣言時に nofc の値が初期値(0)のままになっている為だと思われます。
以下のように値の設定を先に行うことで、正常に出力されました。

int main(){
    int nofc,i;
    nofc=2;
    double imm[nofc][2];

参考: Ideone.com - Zf09p6 - Online C++0x Compiler & Debugging Tool

Comments

[ristori]

ありがとうございます。順番を変えたら確かに正常に出力出来ました。大変助かりました。

[ryunix]

お役に立てて良かったです :)