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作为原始数字是不必要的风险。它们作为命名常量值更安全。为什么？因为如果所有使用都引用相同的值，您只需要对常量进行一次代码更改，以更改数组的大小。这也可以防止反转，忘记更改至少一个值。你要么尝试在数组的边界外打印，可能会崩溃或打印得太少。

这稍微改变了类定义：

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

您的行和列声明在哪里？你需要：

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;
    }

稍后尝试一个动态数组，e。 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";
 }

如何在类函数中打印2D数组

5 个答案: