c++ 类：在公共函数中使用私有数组？答案

【问题标题】：c++ class: using a private array in a public function?c++ 类：在公共函数中使用私有数组？
【发布时间】：2015-04-02 07:42:54
【问题描述】：

这是我在头文件中使用的代码：

#ifndef TETRIS_TETRIMINO
#define TETRIS_TETRIMINO

const int TETRIMINO_GRID_SIZE = 4;

struct Location {
    int row;
    int col;
};

class Tetrimino {
private:
    int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE];
    char color;
    Location location;
public:
    // constructor
    Tetrimino(int type = 7); // valid type values are 0-6

//---------------------------------------------
//accessors
    char getColor();
    Location getLocation();
    void getGrid(int gridOut[][TETRIMINO_GRID_SIZE]);

//---------------------------------------------
//mutators
    void setLocation(Location newLocation);
    void setLocation(int row, int col);

    void rotateLeft();
    void rotateRight();
    void moveLeft();
    void moveRight();
    void moveDown();
    void moveUp();

//---------------------------------------------
//others
    void dataDump();
    };
    #endif

这是.cpp：

#include "tetrimino.h"
#include <iostream>
#include <ctime>
using namespace std;

//random number generator
int randNum()
{
    int randNum;
    int high = 6;
    int low = 0;
    srand(static_cast<unsigned int>(time(NULL)));
    randNum = rand() % (high - low + 1) + low;
    return randNum;

}

Tetrimino::Tetrimino(int type)
{
    //check to see if type is 0-6, if not set to 7
    if (type < 0 || type >= 7)
    {
        type = randNum();
    }

    //set associated type to a tetrimino
    if (type == 0)
    {
        //set grid to i tetro
        int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE] =
        {
            { 0, 0, 0, 0 },
            { 1, 1, 1, 1 },
            { 0, 0, 0, 0 },
            { 0, 0, 0, 0 }
        };
        //set color to teal
        color = 't';

        //initialize starting position
        location.row = 0;
        location.col = 0;
    }
    else if (type == 1)
    {
        //set grid to j tetro
        int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE] =
        {
            { 0, 0, 0, 0 },
            { 0, 1, 0, 0 },
            { 0, 1, 1, 1 },
            { 0, 0, 0, 0 }
        };
        //set color to blue
        color = 'b';

        //initialize starting position
        location.row = 0;
        location.col = 0;
    }
    else if (type == 2)
    {
        //set grid to L tetro
        int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE] =
        {
            { 0, 0, 0, 0 },
            { 0, 0, 1, 0 },
            { 1, 1, 1, 0 },
            { 0, 0, 0, 0 }
        };
        //set color to orange
        color = 'o';

        //initialize starting position
        location.row = 0;
        location.col = 0;
    }
    else if (type == 3)
    {
        //set grid to o tetro
        int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE] =
        {
            { 0, 0, 0, 0 },
            { 0, 1, 1, 0 },
            { 0, 1, 1, 0 },
            { 0, 0, 0, 0 }
        };
        //set color to yellow
        color = 'y';

        //initialize starting position
        location.row = 0;
        location.col = 0;
    }
    else if (type == 4)
    {
        //set grid to s tetro
        int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE] =
        {
            { 0, 0, 0, 0 },
            { 0, 1, 1, 0 },
            { 1, 1, 0, 0 },
            { 0, 0, 0, 0 }
         };
        //set color to green
        color = 'g';

        //initialize starting position
        location.row = 0;
        location.col = 0;
    }
    else if (type == 5)
    {
        //set grid to T tetro
        int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE] =
        {
            { 0, 0, 0, 0 },
            { 0, 1, 0, 0 },
            { 1, 1, 1, 0 },
            { 0, 0, 0, 0 }
        };
        //set color to purple
        color = 'p';

        //initialize starting position
        location.row = 0;
        location.col = 0;
    }
    else if (type == 6)
{
    //set grid to z tetro
    int grid[TETRIMINO_GRID_SIZE][TETRIMINO_GRID_SIZE] =
    {
        { 0, 0, 0, 0 },
        { 0, 1, 1, 0 },
        { 0, 0, 1, 1 },
        { 0, 0, 0, 0 }
    };
    //set color to red
    color = 'r';

    //initialize starting position
    location.row = 0;
    location.col = 0;
}
  };

  //accessors
  char Tetrimino::getColor()
  {
      return color;
  }
  Location Tetrimino::getLocation()
  {
      return location;
  }
  void Tetrimino::getGrid(int gridOut[][TETRIMINO_GRID_SIZE])
  {
      //loop goes through each row
      for (int row = 0; row < TETRIMINO_GRID_SIZE; row++)
      {
          //goes through each col of current row
          for (int column = 0; column < TETRIMINO_GRID_SIZE; column++)
          {
              cout << gridOut[row][column] << " ";
          }
          //new line between rows
          cout << endl;
      }
  }

//mutators
//leaving these out of this for sanity

  void main()
  {
      Tetrimino test(0);

      cout << test.getColor() << endl;
      test.getGrid(test.grid);

  }

好的，显然代码不完整。我对如何使用公共函数 getGrid 从 Tetrimino 类中打印出网格数组感到非常困惑和困惑。头文件是预制给我的（虽然我理解）所以我不想编辑它。为什么 getGrid 函数首先需要一个参数？

我不能像在 main() 中那样简单地调用我想要打印的网格，因为它是私有的。我真的……是的，我知道这是错误的，但我不知道如何正确地去做。

编辑/更新：我从 getGrid() 中删除了参数，并将 gridOut 更改为函数中的简单网格。但是，当我使用 test.getGrid() 调用函数时，打印的数组是：

-858993460 -858993460 -858993460 -858993460
-858993460 -858993460 -858993460 -858993460
-858993460 -858993460 -858993460 -858993460
-858993460 -858993460 -858993460 -858993460

我更改的代码是：

void Tetrimino::getGrid()
{
    //loop goes through each row
    for (int row = 0; row < TETRIMINO_GRID_SIZE; row++)
    {
        //goes through each col of current row
        for (int column = 0; column < TETRIMINO_GRID_SIZE; column++)
        {
            cout << grid[row][column] << " ";
        }
        //new line between rows
        cout << endl;
    }
}

getGrid 已更改为网格。

我现在这样调用函数：

void main()
{
    Tetrimino test(0);

    test.getGrid();

}

【问题讨论】：

“我不想编辑它” - 如果您想打印私人成员，恐怕您将不得不这样做。我不能说为什么函数是这样写的，但它不能用于外部打印类的成员数组。说了这么多，dataDump 是做什么的？
getGrid 可用于打印任何grid。所以这个函数不应该是类的成员。如果此函数仅用于打印内部grid，则根本不需要输入参数gridOut
@MikeSeymour dataDump 只是我计划用来打印内部变量以用于我尚未创建的测试目的的函数。我同意即使我不想编辑，我也只能编辑它。对我来说，为什么首先要争论是没有意义的。
“我对如何从 Tetrimino 类中打印出网格数组感到非常困惑和困惑” - 我认为你会更加“卡住”为什么你的成员网格仍然不确定，并且所有那些本地范围的自动grid 声明都无法改变这一点。使用加速警告进行编译应该会显示 7 个不同的“未使用的变量 'grid'”实例或类似的词。
@WhozCraig 是的，我现在在那儿。我更加困惑为什么会这样。我正在努力找出为什么会发生这种情况，但我明白你指出了什么。

标签： c++ arrays class private public

【解决方案1】：

正如你提到的，头文件是提供给你的，我猜这是一个任务。 getGrid 方法旨在为外部调用者提供接口以获取 Tetrmino 对象网格的副本。由于不能从函数返回数组，getGrid 方法提供了一个输出参数。

示例用法：

void Tetrimino::getGrid(int gridOut[][TETRIMINO_GRID_SIZE]) {
   for(int i = 0; i < TETRMINO_GRID_SIZE; i++) { 
      for(int j = 0; j < TETRMINO_GRID_IZE; j++ ) { 
          gridOut[i][j] = grid[i][j];
      }
   }
}

...
...

Tetrmino obj(3);

... 
... 

int grid[TETRIMINO_GRID_SIZE][TETRMINO_GRID_SIZE];
obj.getGrid(grid);
// now grid holds the copy of interal grid
for(int i = 0; i < TETRMINO_GRID_SIZE; i++) { 
   for(int j = 0; j < TETRMINO_GRID_IZE; j++ ) { 
       std::cout << grid[i][j] << " ";
   }
   std::cout << "\n";
}
std::cout << std::flush;

编辑： 扩展答案：为什么不分配网格？

问题是，在您的构造函数中，您要声明一个与类成员同名的新 int 数组。这意味着，您没有初始化成员变量。 C++ 不允许在初始化后分配给原始数组，你只剩下复制了。

将新变量更改为gridNew 或类似的值，然后从gridNew 逐个复制到grid，就像您现在在getGrid 方法中从grid 复制到gridOut 一样。

【讨论】：

你是正确的，我从哪里得到了标题。所以如果我理解正确的话，在这个解决方案中，getGrid 的公共功能只存在于提取网格信息？
getGrid 与getColor 的方法相同。唯一不同的是，因为使用的是原始数组，所以返回一个副本就没有那么简单了。
啊，我明白了。感谢您的解释！现在，您不会碰巧知道为什么 grid 没有被设置为正确的值吧？
我扩展了答案
非常感谢！真的很棒的解释，你的反应很快！

【解决方案2】：

那个方法是完全错误的，它被设计为打印一个参数而不是内部数据，只需删除输入参数并使用内部矩阵。

顺便说一句，您不应该只使用 std::endl 打印 \n，而是直接使用 \n。 std::endl 将刷新 fd 并花费更多时间。

【讨论】：

直接使用\n是什么意思？您能否详细说明我将如何使用 \n 而不是 endl？
@Zearia 写std::cout << ... << '\n' 而不是std::cout << ... << std::endl，除非您绝对希望它立即出现在屏幕上。

【解决方案3】：

不要从您的 main 调用您的私人成员，也不要从您的 main 发送任何参数。它是您的成员，您不需要任何打印出来，您只需使用它，如果您对直接使用它感到困惑，请在您的函数中使用 this 关键字

for (int row = 0; row < TETRIMINO_GRID_SIZE; row++)
  {
      //goes through each col of current row
      for (int column = 0; column < TETRIMINO_GRID_SIZE; column++)
      {
          cout << this->grid[row][column] << " ";
      }
      //new line between rows
      cout << endl;
  }

【讨论】：