Авто & amp; в диапазоне для цикла C ++ 11 не работает

Я пытался использовать auto & в диапазоне для цикла, чтобы изменить значение. Но я был так потрясен, что не сработало «ВСЕ ВРЕМЯ». Я прилагаю свой оригинальный код ниже. Он просто находит элемент 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 &&Это дало мне то же «неопределенное» поведение. Но цикл с прямой индексацией работает постоянно (закомментировано в коде). В настоящее время я не мог воспроизвести проблему в любом другом случае, однако логика здесь уже достаточно проста.

Может ли это быть проблемой компилятора? Может кто-нибудь запустить код и посмотреть, может ли проблема быть воспроизведена?

-1

Решение

auto невиновен

Следующая инициализация (в zerolify()) неправильно (неопределенное поведение)

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

потому что вы инициализируете col используя значение col,

Не уверен, что понимаю, что вы хотите, но я полагаю, правильная инициализация

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

Другие решения

Других решений пока нет …

По вопросам рекламы [email protected]