二维数组中的用户输入 (C++)

Question

不知道为什么当我打印检查时 for 循环不会在二维数组中保存正确的值。 有任何想法吗？

#include <iostream>
using namespace std;

int row, col; 

int main()
{
int num;
int val[row][col];

cout << "How many rows are there?" << endl;    
cin >> row;
cout << "How many columns are there?" << endl;
cin >> col;
cout << "Enter values for the matrix: " << endl;

for (int i = 0; i < row; i++)                 
{
    for (int j = 0; j < col; j++)
    {
        cin >> val[i][j]; 
    }   
}
return 0;
}

Answer 1

#include <iostream>
using namespace std;

int main()
{
    int row, col; 

    cout << "How many rows are there?" << endl;    
    cin >> row;
    cout << "How many columns are there?" << endl;
    cin >> col;
    cout << "Enter values for the matrix: " << endl;

    // check if row and col > 0
    int* val = new int[row * col];

    for (int i = 0; i < row; i++)                 
    {
        for (int j = 0; j < col; j++)
        {
            cin >> val[i * col + j]; 
        }   
    }
    delete[] val;
    return 0;
}

Answer 2

这并没有按照您的想法做。 首先，在程序启动时将row和col初始化为零。 然后你有int val[row][col]; 这不是有效的C ++，而是C可变长度数组。 由于此时row和col均为0，因此此数组的长度为零。

然后在循环中读取一堆值，覆盖堆栈中的内容并导致未定义的行为。

相反，您应该使用动态分配的内容，例如std::vector或所选数学库中的适当矩阵类。 通常不建议手动使用动态分配（如Ali建议的那样使用new int[row * col] ），因为这样很容易导致内存泄漏，尤其是在涉及异常的情况下。

Answer 3

如果C ++支持可变长度数组，则只需移位即可编写所需内容：

int val[row][col];

直到知道row和col为止。 检查一下这篇文章： 可变长度数组为什么不属于C ++标准？ 否则，您的代码将具有未定义的行为。 您应该使用动态分配。

Answer 4

首先，你不能

int val[row][col];

row和col之前的值已知。 其次，这种2D数组初始化仅在C语言中是标准的。

您将需要使用C ++运算符new []（类似于C中的malloc函数）在堆上手动分配数组。 但这不是C ++惯用的语言，该语言的全部重点是避免这样做，这就是为什么我不解释如何做到这一点的原因。

完成目标所需的正确C ++方法是使用std :: vector，它是一个非常强大的C样式数组包装器，可以自动为您分配（和取消分配）内存（以及许多其他东西）。

这是最简单的方法：

#include <iostream>
#include <vector>

using namespace std;

int main()
{
    int row;
    cout << "How many rows are there?" << endl;
    cin >> row;

    int col;
    cout << "How many columns are there?" << endl;
    cin >> col;

    int num;
    cout << "Enter values for the matrix: " << endl;
    cin >> num;

    vector<vector<int>> values(col); //initialize outer vector with col inner vectors

    for (int i = 0; i < row; i++)
    {
        for (int j = 0; j < col; j++)
        {
            values[i].push_back(num);
        }
    }

    for (int i = 0; i < row; i++)
    {
        for (int j = 0; j < col; j++)
        {
            cout << values[i][j];
        }

        cout << endl;
    }

    return 0;
}

另外，我建议您使用更多含义来命名变量，并避免使用命名空间std。

编辑：从您处理程序的方式来看，我认为您对C熟悉。如果不是这种情况，并且您正在学习书籍或教程，则应该找到更好的书籍或教程。

Answer 5

#include <iostream>
#include <cstdlib>
#include <fstream>

using namespace std;

int main() {
int squareOrder;
const int MAX_SIZE = 5;
int squareArray[MAX_SIZE][MAX_SIZE] = {0};
int i;
int j;


cout << "What is the order of the square to be tested?";

cin >> squareOrder;

cout << endl << "Please enter the " << squareOrder*squareOrder << " values for the magic square in row major order: ";

for (i = 0; i < MAX_SIZE; i++) {
    for (j = 0; j < MAX_SIZE; j++) {
            cin >> squareArray[i][j];
    }
}


for (i = 0; i < MAX_SIZE;  i++) {
    for (j = 0; j < MAX_SIZE; j++) {
            cout << squareArray[i][j];
    }
}

return 0;
}

我有如下代码，当我用go运行程序时，用户输入好像是死循环？ 程序停止接收用户输入的唯一方法是输入无效（除整数之外的任何内容）。 在继续之前，我正在尝试查看这些数字是否按照我设想的方式存储。 提前感谢您的帮助！