2017-03-07 39 views
-1

我正在尝试使用auto &在ranged for循环中修改该值。但我感到非常震惊,以至于“无所不在”这一行为无效。我在下面附上我的原始代码。它只需在矩阵中找到元素0,并将相应的行和列设置为全零。自动进入循环的C++ 11不工作

#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与苹果LLVM版本8.0.0(铛 - 800.0.42.1)编译器。以下是我的结果: My command line output

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

难道是编译器问题吗?任何人都可以运行代码并查看问题是否可以复制?

回答

5

auto是无罪的。

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

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

,因为你使用的col值初始化col

不是一定要了解你想要什么,但我想正确的初始化

int col { row > 0 ? m[0].size() : 0 }; 
+0

尼斯抓,愚蠢的错误,谢谢! – tengi