c++ - C++ 動的多次元配列の問題

Question

私は2Dプラットフォーマーを開発しています。問題を解決するのが難しいまで、すべてがうまくいきました。レベルマップは動的多次元配列(char **map)に格納されます。再定義するまでは問題なく動作します

コードの一部は次のとおりです。

Map& Map::operator=(const Map& rhs)
{
    if(width!=0||height!=0)
    {
        for(int i=0;i<width;i++)
            delete[] map[i];
        delete[] map;
    } //deleting previously created array

    height=rhs.height;
    width=rhs.width; //receiving other map's size

    map=new char* [width];
    walkmap=new unsigned char* [width];
    objmap=new char* [width];
    for(int i=0;i<width;i++)
    {
        *(map+i)=new char[height];
    } //creating new array

    for(int h=0;h<height;h++)
        for(int w=0;w<width;w++)
        {
            map[w][h]=rhs.map[w][h];
        } //receiving new values

    //...
}

最初はすべて正常に動作しますが、配列を 2 回目に再定義する必要があるとき、配列が別の配列から新しい値を受け取るときに、その部分でプログラムがクラッシュします。私は何かを見逃しているかもしれませんが、私はそれを見つけることができません! 私はこの問題を探していましたが、私が間違っていることを見つけられませんでした。お願い助けて。

Answer 1

いつものように、Boost にはエレガントでメモリ効率の高い多次元配列クラスがあります。

http://www.boost.org/doc/libs/1_39_0/libs/multi_array/doc/user.html

たとえば、bool 値の 10 x 20 配列をセットアップするには、次のようにします。

 
    boost::multi_array  mtaFlagMatrix(boost::extents[10][20]);
 

次に、その要素にアクセスします。

 
    mtaFlagMatrix[2][6] = false; // indexes are zero-based - just like normal C arrays
     ...
    if ( mtaFlagMatrix[2][6] )
    {
       ...
    }
 

次に、この方法で配列のサイズを変更できます (既存の値は保持されます)。

 
typedef boost::multi_array array_type;


array_type::extent_gen extents;
  array_type A(extents[3][3][3]);
  A[0][0][0] = 4;
  A[2][2][2] = 5;
  A.resize(extents[2][3][4]);
  assert(A[0][0][0] == 4);
  // A[2][2][2] is no longer valid.

これにより、極端なケースのテストに多くの時間を節約できました。

Answer 2

2 次元配列が適切に解放されていません。Iliffe の 2 次元配列の割り当て方法を使用することをお勧めします。これは、より高速で安全に使用できます。

char** alloc_array( int h, int w )
{
  typedef char* cptr;
  int i;
  char** m = new cptr[h];
  m[0] = new char[h*w];
  for(i=1;i<h;i++) m[i] = m[i-1]+w;
  return m;
}

void release_array(char** m)
{
  delete[] m[0];
  delete[] m;
}

int main()
{
  int r,c;
  char** tab;
  int width  = 5;
  int height = 3;
  tab = alloc_array(height, width); /* column first */

  for(r = 0;r<height;++r)
   for(c = 0;c<width;++c)
    tab[r][c] = (1+r+c);

  for(r = 0;r<height;++r)
  {
    for(c = 0;c<width;++c)
    {
      printf("%d\t",tab[r][c]);
    }
    puts("");
  }
  release_array(tab);
}

これは、適切な 2 次元配列クラスに簡単にカプセル化することも、生の割り当ての代わりに std::vector を使用するようにすることもできます。キャッシュフレンドリーな2次元配列を行うこの方法を好み、スタイルは[] []アクセスを提供し、多項式1次元アクセスよりも遅くはなく、時には高速です。