[解決済み】魔法の四角形プログラム(C++)

質問

古典的なマジックスクエアのアルゴリズムに馴染みのない人のために。マジックスクエアは2次元の配列 (n x n) で、各位置に1からn^2までの数値が含まれます。各値は一度だけ現れる。さらに、各行、列、対角線の和は同じでなければならない。 奇数魔法陣の解を書くので、入力は奇数でなければならない。

問題は完成したのですが、今のところ未知のバグ(論理？出力？)があり、この1時間悩まされ続けています。 出力される値が非常に的外れなんです。 何か手助けがあれば、とてもありがたいです。

#include<iostream>
#include<iomanip>
using namespace std;

int main()
{
  int n;

  cout<< "Please enter an odd integer: ";
  cin>>n;

  int MagicSquare[n][n];


  int newRow,
  newCol;

  // Set the indices for the middle of the bottom i
  int i =0 ;
  int j= n / 2;

  // Fill each element of the array using the magic array
  for ( int value = 1; value <= n*n; value++ )
  {
     MagicSquare[i][j] = value;
     // Find the next cell, wrapping around if necessary.
     newRow = (i + 1) % n;
     newCol = (j + 1) % n;
     // If the cell is empty, remember those indices for the
     // next assignment.
     if ( MagicSquare[newRow][newCol] == 0 )
     {
        i = newRow;
        j = newCol;
     }
     else
     {
        // The cell was full. Use the cell above the previous one.
        i = (i - 1 + n) % n;
     }

  }


  for(int x=0; x<n; x++)
  {
     for(int y=0; y<n; y++)
         cout << MagicSquare[x][y]<<" ";
     cout << endl;
  }
}

解決方法は？

を初期化するのを忘れています。 MagicSquare をすべてゼロにする。

  for(int i = 0; i < n; i++) {
    for(int j = 0; j < n; j++) {
      MagicSquare[i][j] = 0;
    }
  }

したがって、このチェックはほとんど失敗します。

if ( MagicSquare[newRow][newCol] == 0 ) {
   i = newRow;
   j = newCol;
}

C/++では0に初期化されないので。