Question

我想存储从文件中读取的稀疏矩阵，但是当构造函数完成时，参数未存储在对象中。

文件具有“行col”结构，其中row是矩阵行，col是矩阵列。如果文件的坐标存在，则在此位置存储1，否则存储0。

这是构造函数：

MatriuSparse::MatriuSparse(const string nomFitxer) {
        fstream fitxer;
        fitxer.open(nomFitxer);

        int m_columna;
        int m_fila;
        m_Ncolumnes = 8;
        m_Nlinies = 8;
        vector<vector<int> > matriu;
        for (int i = 0; i < m_Nlinies; i++)
        {
            vector<int> temporal;
            for (int j = 0; j < m_Ncolumnes; j++) {
                temporal.push_back(0);
            }
            matriu.push_back(temporal);
        }
        fitxer >> m_fila >> m_columna;

        while (!fitxer.eof())
        {
            matriu[m_fila][m_columna] = 1;
            fitxer >> m_fila >> m_columna;
        }
        fitxer.close();
        //Here matrix has size 8
    }

这是主要内容：

 string nomFitxer = "Xarxa1.txt";
    MatriuSparse m1(nomFitxer);
    // Here m1 matrix has size 0
    cout << m1;

Answer 1

除m_Ncolumnes和m_Nlinies设置为常量值外，构造函数不会在类中存储任何内容。其他所有内容都存储在构造函数本地的变量中，并在构造函数完成后消失。

您很有可能隐藏具有相同名称的成员向量。只需删除此行：

    vector<vector<int> > matriu;   // hiding the member matriu ?

不相关1：，您可以轻松地初始化向量而无需嵌套循环：

    matriu = vector<vector<int>>(m_Nlinies, vector<int>(m_Ncolumnes, 0));

不相关2 ：在eof()上循环无法正常工作。您必须循环提取。因此，一旦您的Matriu初始化，就执行：

    while (fitxer >> m_fila >> m_columna)
    {
        matriu[m_fila][m_columna] = 1;
    }

Answer 2

您的主要问题是您需要使matriu成为该类的成员，而不是在函数中本地声明它。

此外，我还有其他评论。

如果您事先知道矩阵的大小是8x8，那么使用vector和array实在是过头了。

命名约定有点奇怪。我假设您使用m_表示成员变量，但是m_columna和m_fila是局部变量。

您提到的是稀疏矩阵，但是您要填充的matriu矩阵是密集格式。

稀疏矩阵构造函数不在对象中存储参数

2 个答案: