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

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 行的数组，并且只为其中的 6 行赋值。这不太可能崩溃，因为数组条目已分配，但会冒险进入未定义的行为，并且没有人知道结果会在那些未初始化的槽中放置什么值。

现在开始唠叨了。

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

这稍微改变了类定义:

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?