2017-03-07 38 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