如何在类函数中打印二维数组

Question

所以我对类编程有点陌生，我不确定我的代码做错了什么，因为我试图打印一个二维数组，但在尝试构建它时出现错误。这里是：

#include <iostream>

using namespace std;

class PlaneSeats
{
private:
    char seatAvailability[7][4];

public:
    PlaneSeats();
    void displayAvailability();
};

PlaneSeats::PlaneSeats()
{
    seatAvailability[7][4] = {{'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'}};
}

void PlaneSeats::displayAvailability()
{
    cout << row+1 << " ";
    for (int column = 0; column<4; column++)
    {
        cout << seatAvailability[row][column] << " ";
    }
    cout << endl;
}

int main()
{
    PlaneSeats plane;
    plane.displayAvailability();
    return 0;
}

构建时出现的错误是：

'row' was not declared in this scope    line 27

Symbol 'row' could not be resolved    line 27

Symbol 'row' could not be resolved    line 30

Multiple markers at this line    line 22
    - cannot convert '<brace-enclosed initializer list>' to 'char' in assignment
    - extended initializer lists only available with -std=c++0x or -std=gnu++0x [enabled by
    default]

Answer 1

我基本同意 Blake 的评估，但我认为 OP 对 displayAvailability 的意图更像是这样：

void PlaneSeats::displayAvailability()
{
    for (int row = 0; row<7; row++)
    {
        cout << row+1 << " ";
        for (int column = 0; column<4; column++)
        {
            cout << seatAvailability[row][column] << " ";
        }
        cout << endl;
    }
}

在列的打印周围使用额外的 for 循环。

接下来，OP 不能在构造函数中进行静态初始化，至少不是那样，但是感谢现代 C++ 的奇迹，他们可以这样做：

class PlaneSeats
{
private:
    char seatAvailability[7][4] =
    {
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'}
    };
public:
    PlaneSeats();
    void displayAvailability();
};

但是请注意，OP 声明了一个包含 7 行的数组，并且只为其中的六个分配了值。这不太可能崩溃，因为数组条目已被分配，但会冒险进入未定义的行为，并且没有人知道结果将在那些未初始化的插槽中放置什么值。

现在是唠叨。

使用 7 和 4 作为原始数字是不必要的风险。它们作为命名常量值更安全。为什么？因为如果所有使用引用相同的值，您只需要对常量进行一次代码更改，即可更改数组的大小。这也防止了相反的情况，忘记更改至少一个值。您要么尝试在数组边界之外打印并可能崩溃，要么打印的太少。

这会稍微改变类定义：

class PlaneSeats
{
private:
    static constexpr int numrows = 7;
    static constexpr int numcols = 4;
    char seatAvailability[numrows][numcols] =
    {
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'},
        {'A', 'B', 'C', 'D'}
    };
public:
    PlaneSeats();
    void displayAvailability();
};

和 displayAvailability 类似的变化

void PlaneSeats::displayAvailability()
{
    for (int row = 0; row<numrows; row++)
    {
        cout << row+1 << " ";
        for (int column = 0; column<numcols; column++)
        {
            cout << seatAvailability[row][column] << " ";
        }
        cout << endl;
    }
}

这也使代码可读性更好，因为 7 只是一个数字，而 numrows 包含一些上下文提示。

当然还有大家最喜欢的唠叨：Why is "using namespace std" considered bad practice?

Answer 2

'row' was not declared in this scope    line 28

你在哪里声明 row 和 column ？ 你需要：

private:
int row;
int column;

在你的类中，然后在构造函数或使用它们的函数中初始化它们。

PlaneSeats::PlaneSeats()
{
    seatAvailability[7][4] = {{'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'},
                   {'A', 'B', 'C', 'D'}};
}

你不能这样初始化。您必须手动初始化每个维度。

Answer 3

row 是什么？局部变量，成员？它没有被宣布。 （并且只有 6 行被初始化。）试试这个遍历数组：

static const int NUM_ROW = 7;
static const int NUM_COL = 4;
char seatAvailability[NUM_ROW][NUM_COL];
/* .. */
    for( int row = 0; row < NUM_ROW; ++row )
    {
        for( int col = 0; col < NUM_COL; ++col )
        {
            std::cout << "\t" << seatAvailability[row][col];
        }
        std::cout << std::endl;
    }

稍后尝试动态数组，例如。 g.：

std::vector< char > seats( NUM_ROW * NUM_COL, 0 );

不能以seats[row][col]的方式访问，而是seats[col+NUM_COL*row]。但是，它对静态数组有其自身的优势。

Answer 4

数组基础：大括号初始化器仅在声明数组变量时可用。所以你不能按照自己的方式去做。

所以最好在声明的时候赋值，如果值早知道的话。

Follow This link : Modified Code

Answer 5

这里说明了使用矢量并在不知道其长度的情况下迭代元素并打印它的最佳方式：

    vector<vector<int>> mat {{11, 12, 13},{21, 22, 23},{31, 32, 33}};

for(auto line : mat)
{
    for(auto element : line)
    {
        cout<<element<< " ";
        }
   cout<<"\n";
 }