Симметрия массива относительно главной диагонали. Что не так?

Question

[Протопоп Метапоп]

Массив arr[i][j] с вещественными числами.
a и b - размеры.

//проверяем симметрию//
                                       
cout << "\n";                                                 
//переменная-показатель симметрии//                           
sim= true;                                                    
//прогоняем массив//                                          
for (int i = 0; i < a; i++)                                   
{                                                             
    for (int j = 0; j < b;j++)                                
    {                                                         
        //если зеркальные значения не равны sim становится ложным //
        if (i=j) break;
        if (arr[i][j] != arr[j][i])                           
            {                                                 
            sim = false;                                      
            break;                                            
            }                                                 
    if (sim=false) break;                                     
    }                                                         
}      
//выводим ответ//                                                       
if (sim=true)                                                 
    cout<<"Массив симметричен"<<endl;                         
if (sim=false)                                                
    cout<<"Массив несимметричен"<<endl;

Где я ошибся?

Comments

[alexalexes]

Вы не показали, на каком примере массива тестили.
Вы учли в тестах, что матрица должна быть квадратной (a = b)?
Если не квадратная, то предупреждать пользователя.
if (sim=false) break;
У вас тут не сравнение, а присваивание.

[Протопоп Метапоп]

alexalexes, почему матрица должна быть квадратной? Разве мы не можем сравнивать два прямоугольных треугольника? Тестил я на массиве 2х2 с числами 1,2,3,4. Программа выдала что он симметричный.

[Протопоп Метапоп]

alexalexes, в этой строчке мне нужно выйти из цикла, если хоть одна "зеркальная" пара не равна.

[Тимур Покровский]

Протопоп Метапоп, = - это оператор присвоения, а не сравнения

[Протопоп Метапоп]

Тимур Покровский, по идее, если sim=false происходит выход из цикла .

[alexalexes]

1. Предположим, есть матрица 5 на 2, координата [4,0] - допустима, а вот [0,4] - попадете в адрес, не принадлежащий массиву. Так с чем сравнивать?
2.

if (i==j)  // тут сравнение, используется ==
  break;
        if (arr[i][j] != arr[j][i])                           
            {                                                 
            sim = false;  // тут присваивание, используется =                                    
            break;                                            
            }                                                 
    if (sim==false)  // тут снова сравнение, используется ==
    break;

[Тимур Покровский]

Протопоп Метапоп, не происходит, потому что = - это оператор присвоения, а не сравнения

[Протопоп Метапоп]

alexalexes, я поменял код, но программа до сих пор работает неправильно.

//проверяем симметрию//                                                                          
   sim= false;                                                                                      
   //прогоняем массив//                                                                             
                                                                                                    
           if (a==b) {                                                                              
               for (int i = 0; i < a; i++) {                                                        
                   for (int j = 0; j < b; j++) {                                                                                                                            
                       if (i == j) break;                                                                   
                       if (arr[i][j] != arr[j][i]) {                                                
                           sim = true;                                                              
                           break;                                                                   
                       }                                                                            
                       if (sim == true) break;                                                      
                   }                                                                                
               }                                                                                    
           }                                                                                        
                                                                                                    
   if (sim==true)                                                                                    
       cout<<"Массив симметричен"<<endl;                                                            
   if (sim==false)                                                                                   
       cout<<"Массив несиметричен"<<endl;

[Протопоп Метапоп]

Тимур Покровский, я поменял код, но программа всё ещё работает неверно.

//проверяем симметрию//                                                                          
   sim= false;                                                                                      
   //прогоняем массив//                                                                             
                                                                                                    
           if (a==b) {                                                                              
               for (int i = 0; i < a; i++) {                                                        
                   for (int j = 0; j < b; j++) {                                                                                                                            
                       if (i == j) break;                                                                   
                       if (arr[i][j] != arr[j][i]) {                                                
                           sim = true;                                                              
                           break;                                                                   
                       }                                                                            
                       if (sim == true) break;                                                      
                   }                                                                                
               }                                                                                    
           }                                                                                        
                                                                                                    
   if (sim==true)                                                                                    
       cout<<"Массив симметричен"<<endl;                                                            
   if (sim==false)                                                                                   
       cout<<"Массив несиметричен"<<endl;

[alexalexes]

Верно, только sim показывает обратный результат.
Поменяйте местами вывод результата, когда sim==false, то массив симметричен, и все.

Answer 1

[Александр Ананьев]

Что не так:
1. Выражение if(a=b) необходимо заменить на if(a==b)
2. Вещественные числа a и b равны, если fabs(a - b) < EPSILON. Следовательно, выражение if(a!=b) надо заменить на if(!(fabs(a - b) < EPSILON))

Comments

[Протопоп Метапоп]

Я поменял код, но программа до сих пор работает неверно.

//проверяем симметрию//                                                                          
   sim= false;                                                                                      
   //прогоняем массив//                                                                             
                                                                                                    
           if (a==b) {                                                                              
               for (int i = 0; i < a; i++) {                                                        
                   for (int j = 0; j < b; j++) {                                                                                                                            
                       if (i == j) break;                                                                   
                       if (arr[i][j] != arr[j][i]) {                                                
                           sim = true;                                                              
                           break;                                                                   
                       }                                                                            
                       if (sim == true) break;                                                      
                   }                                                                                
               }                                                                                    
           }                                                                                        
                                                                                                    
   if (sim==true)                                                                                    
       cout<<"Массив симметричен"<<endl;                                                            
   if (sim==false)                                                                                   
       cout<<"Массив несиметричен"<<endl;