-1

我试图auto &在 ranged for 循环中使用来修改值。但我很震惊，它“一直”都不起作用。我在下面附上我的原始代码。它只是在矩阵中找到元素 0，并将相应的行和列设置为全零。

    #include<iostream>
    #include<vector>
    using namespace std;

    void zerolify(vector<vector<int>>& m)
    {
        int row=m.size();
        int col=col>0?m[0].size():0;

        vector<int> r(row,0);
        vector<int> c(col,0);

        for(int i=0;i<row;i++)
        {
            for(int j=0;j<col;j++)
            {
                if(m[i][j]==0)
                {
                    r[i]=1;
                    c[j]=1;
                }
            }
        }

        for(int i=0;i<row;i++)
        {
            if(r[i])
            {
                //cout<<"row: "<<i<<endl;
                for(auto &e : m[i])
                {
                    e=0;
                }
                //for(int j=0;j<col;j++)
                //{
                //  m[i][j]=0;
                //}
            }
        }
        for(int i=0;i<col;i++)
        {
            if(c[i])
            {
                for(int j=0;j<row;j++)
                {
                    m[j][i]=0;
                }
            }
        }
    }
    void printMatrix(vector<vector<int>> m)
    {
        for(auto i:m)
        {
            for(auto j:i)
            {
                cout<<j<<" ";
            }
            cout<<endl;
        }
    }
    int main(int argn, char** argv)
    {
        vector<vector<int>> m(5,vector<int>(5));
        m[0]={1,2,3,0,4};
        m[1]={1,1,3,5,4};
        m[2]={1,2,3,0,4};
        m[3]={1,2,3,5,4};
        m[4]={1,2,3,5,4};
        cout<<"Original matrix:"<<endl;
        printMatrix(m);
        zerolify(m);
        cout<<"Zerolify"<<endl;
        printMatrix(m);
        return 0;
   }

我正在使用 Mac OS 10.12.2 和 Apple LLVM 版本 8.0.0 (clang-800.0.42.1) 编译器。这是我的结果： 我的命令行输出

您可以看到它第一次没有工作，保持矩阵不变。第二次成功了。我也试过auto &&了，它给了我同样的“不确定”行为。但是带有直接索引的 for 循环一直都在工作（在代码中注释掉了）。目前，我无法在任何其他情况下复制该问题，但是这里的逻辑已经足够简单了。

会不会是编译器问题？任何人都可以运行代码并查看是否可以复制问题吗？

4

1 回答 1

5

auto是无罪的。

以下初始化 (in zerolify()) 是错误的 (未定义的行为)

int col=col>0?m[0].size():0;

col因为您使用 的值进行初始化col。

不知道你想要什么，但我想正确的初始化是

int col { row > 0 ? m[0].size() : 0 };
于 2017-03-07T01:25:04.327 回答