C++ Возврат и вставка объекта 2D-массива

Похоже, вам нужно больше узнать об указателях в C++ и о передаче по ссылке или передаче по значению.

Ваш метод returnPiece определяется как возвращающий значение одной ячейки. Учитывая индекс (например, [4] [4]), вы возвращаете копию содержимого этой ячейки, поэтому вы не сможете его изменить, или, вернее, изменение его приведет к изменению копии.

Я уверен, что кто-то предоставит вам правильный синтаксис, но я бы очень рекомендовал изучить этот материал, поскольку в противном случае вы можете использовать код, который вы получили неправильно.

Вот как бы я это сделал:

class Array {
  public:

    Array() {
      for (int i = 0; i < 4; ++i)
      {
         for (int j = 0; j < 4; ++j)
         {
            (*this)(i, j) = 0;
         }
      }
    }


    int &operator()(int i, int j)
    {
      return pieceArray[i][j];
    }

  private:
    int pieceArray[4][4];
};

Затем вы можете сделать что-то вроде:

  Array x; // create 4x4 array
  x(1, 2) = 3; // modify element

Передавая свой массив, вы должны решить, хотите ли вы сделать копию массива или просто хотите вернуть указатель на массив. Для возврата массивов вы не можете (легко) вернуть копию - вы можете вернуть только указатель (или ссылку в C++). Например:

// Piece::returnPiece is a function taking no arguments and returning a pointer to a
// 4x4 array of integers
int (*Piece::returnPiece(void))[4][4]
{
    // return pointer to the array
    return &pieceArray;
}

Чтобы использовать его, назовите его так:

int (*arrayPtr)[4][4] = myPiece->returnPiece();
int cell = (*arrayPtr)[i][j];  // cell now stores the contents of the (i,j)th element

Обратите внимание на сходство между объявлением типа и его использованием - круглые скобки, оператор разыменования * и квадратные скобки находятся в одних и тех же местах.

Ваше объявление для Grid::InsertArray верное - он принимает один аргумент, который представляет собой массив целых чисел 4x4. Это вызов по значению: всякий раз, когда вы вызываете его, вы делаете копию своего массива 4x4, поэтому любые вносимые вами изменения не отражаются в переданном массиве. Если вы вместо этого хотите использовать вызов по ссылке, вы можете вместо этого передайте указатель на массив:

// InsertArray takes one argument which is a pointer to a 4x4 array of integers
void Grid::InsertArray(int (*arr)[4][4])
{
     for(int i = 0; i < x_ROWS; i++)
     {
         for(int j = 0; j < y_COLUMNS ; j++)
             squares[i][j] = (*arr)[i][j];
     }
}

Эти объявления типов с указателями на многомерные массивы могут быстро запутать. Рекомендую сделать для него typedef вот так:

// Declare IntArray4x4Ptr to be a pointer to a 4x4 array of ints
typedef int (*IntArray4x4Ptr)[4][4];

Затем вы можете объявить свои функции более читаемыми:

IntArray4x4Ptr Piece::returnPiece(void) { ... }
void Grid::InsertArray(IntArray4x4Ptr arr) { ... }

Вы также можете использовать программу cdecl, чтобы помочь расшифровать сложные типы C / C++.

Беда с Адама РозенфилдаarrayPtr в том, что Piece :: pieceArray может вылететь из-под вас. (Копирование по ссылке vs Копирование по значению.)

Копирование по значению неэффективно. Но если вы действительно хотите это сделать, просто обманите:

struct FOO { int  piece [4][4]; };

FOO Piece::returnPiece()
{
  FOO f;
  memcpy( f.piece, pieceArray, sizeof(pieceArray) );
  return f;
}

void Grid::InsertArray( const FOO & theFoo )
{
  // use theFoo.piece[i][j]
}

Конечно, лучшим, более объектно-ориентированным решением было бы создание returnPiece () и возврат объекта Кусок или Множество. (Как предложил Хуан ...)