Простенькая программа С++ работает неверно. Как исправить?

Question

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

По задумке, программа должна принимать номер строки в двойном квадратном массиве и сравнивать строку со столбом, имеющим соответствующий номер, но что-то пошло не так.

#include <iostream>
using namespace std;
int main() {
const int n = 1000;
int a,b, i, j;
bool str= false;
float arr[n][n];
cout << "Введите размерность:\n";
cin >> a;
cout << "Заполнение массива:\n";

for (int i = 0; i < a; i++) {
    for (int j = 0; j < a; j++) {
        cout <<"Введите элемент ["<<i+1<<","<<j+1<<"]:";
        cin >> arr[i][j];
        }
    }
for (int i = 0; i < a; i++) {
    for (int j = 0; j < a; j++)
        cout << arr[i][j]<<" ";
        cout << "\n";
        }
cout << "Введите номер строки: \n";
cin >> b;
    for (int i = 0; i < a; i++) {
        for (int j = 0; j < a; j++)
            if (arr[b-1][j] == arr[i][b-1]) str = true;

    }
    if (str == false) cout <<"Строка и столбец не совпадают";
    if (str == true) cout <<"Строка и столбец совпадают";
}

Что я делаю не так?

Answer 1

[Роман]

Исправил ответ. Заменил ваш двойной цикл с i,j в сравнении строки со столбцом на одинарный цикл. Матрица ведь квадратная?)

...
bool str= true;
...
for (int i = 0; i < a; i++) {
    if (arr[i][b-1] != arr[b-1][i]) str = false;
}
...

Comments

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

Насчёт доп. переменной-счётчика понял, а изначальное положение str как нибудь влияет на результат?

[Роман]

Протопоп Метапоп, да. изначально ставим str= true, что значит что строка и столбец совпадают по всей длине. Далее, если хоть один из соответствующих друг другу по индексу элементов строки и столбца не совпадают то значит и вся строка со столбцом не совпадают. Для этого проверяем каждый элемент строки с соответствующим элементом столбца и если не равны то str= false.

Если же сделать наоборот (изначально str=false а при совпадении элементов строки и столбца str=true) то это будет означать "совпал минимум один из элементов столбца и строки")))

[Роман]

Протопоп Метапоп, и еще, сразу не заметил что n у вас уже есть и еще она константа, просто хотел подчеркнуть что в данном цикле n не строки и не столбцы а и то и то сразу, но так как n уже используется заменил в ответе на i

[Роман]

Протопоп Метапоп, полный код должен получится как то так:

код

#include <iostream>
using namespace std;
int main() {
const int n = 1000;
int a,b, i, j;
bool str= true;
float arr[n][n];
cout << "Введите размерность:\n";
cin >> a;
cout << "Заполнение массива:\n";

for (int i = 0; i < a; i++) {
    for (int j = 0; j < a; j++) {
        cout <<"Введите элемент ["<<i+1<<","<<j+1<<"]:";
        cin >> arr[i][j];
    }
}
for (int i = 0; i < a; i++) {
    for (int j = 0; j < a; j++)
        cout << arr[i][j]<<" ";
    }
    cout << "\n";
}
cout << "Введите номер строки: \n";
cin >> b;
for (int i = 0; i < a; i++) {
   if (arr[b-1][i] != arr[i][b-1]) str = false;
}
if (str == true){
  cout <<"Строка и столбец совпадают";
} else {
  cout <<"Строка и столбец не совпадают";
}

[res2001]

Протопоп Метапоп, Не стоит сравнивать вот так в лоб числа с плавающей точкой никогда. Погуглите на эту тему и больше так не делайте. Хотя применительно к этой конкретной задаче это, возможно, и не является проблемой, (а может и является), но в других более сложных задачах это встанет в полный рост и будете потом гадать из-за чего программа не работает.
Для вашей задачи просто замените тип массива на int. Ну и входные данные надо поменять.
Поставьте break после str = false в том же ifе, т.к. дальше нет смысла продолжать сравнения.

[Роман]

res2001 написал

Поставьте break после str = false в том же ifе, т.к. дальше нет смысла продолжать сравнения.

Протопоп Метапоп, это в предложенном мной варианте, если что, в вашем изначальном break не поможет.

[Роман]

res2001, нет проблемы сравнивать флоаты, нужно просто ограничить число знаков после запятой до конкретного, это решит проблему

[res2001]

Роман, В данной задаче, я думаю, что это вообще не будет проблемой, т.к. никаких вычислений с элементами массива не проводится после ввода. Но я бы, на всякий случай, изменил тип массива на int.
Я эту тему затронул, чтоб Протопоп Метапоп знал, что есть такая проблема. Округление может помочь в простых случаях. Но часто бывает, что вы не знаете какого порядка числа в переменной, или нужно выжать максимально возможную точность и т.п. На тему сравнения чисел с плавающей точкой в инете много информации и обычно округление даже не предлагается как вариант решения проблемы.

[Роман]

res2001, если нельзя округлять, то задаем нормальный (или какой другой) закон распределения, описывающий допустимые отклонения (считай нужная точность), и прогоняя через него разницу между числами смотрим, вписался или не вписался.