[C++] Нужна помощь с кодом

[Сергей9898]

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


Задача: Дан файл, в котором в первой строчке задана размерность матрицы, а далее задана сама матрица.
Найти локальные минимумы и записать в файл значений локального минимума, а так же его расположение в матрице.
Значение в матрице считается локальным минимумом, если оно меньше всех соседних элементов.

#include <iostream> 
 
#include <fstream> 
 
using namespace std;
 
 
int main()
{
 
    int min = 0;
    int ni = 0;
    int i, j;
    int n, m;
    bool s;
 
 
    ifstream f("in.txt");
    f >> n >> m;
    int **a = new int *[n]; 
    for (int i = 0; i < n; ++i)
        a[i] = new int[m];
 
    
        for (int i = 0; i < n; ++i)
            for (int j = 0; j < m; ++j)
                f >> a[i][j];
    
 
    
 
 
    setlocale(LC_ALL, "russian");
 
 
    ofstream fout("cpp.txt");
 
    fout << "Матрица до обработки: " << endl;
 
    for (i = 0; i < n; i++)
    {
        for (j = 0; j < m; j++)
            fout << a[i][j] << "\t "; fout << "\n";
 
    }
 
 
 
    for (i = 0; i < n; i++)
        for (j = 0; j < m; j++)
        {
            s = true;
            if (i > 0 && a[i - 1][j] <= a[i][j])
                s = false;
            if (j > 0 && a[i][j - 1] <= a[i][j])
                s = false;
            if (i < n - 1 && a[i + 1][j] <= a[i][j])
                s = false;
            if (j < m - 1 && a[i][j + 1] <= a[i][j])
                s = false;
            if (s)
            {
 
                fout << ni + 1 << "-й элемент : " << a[i][j] << " \tСтрока : " << i + 1 << " \tСтолбик : " << j + 1 << endl;
                ni++;
                min++;
            }
        }
    fout << "==============================================" << endl;
    fout << "Кол-во минимумов: " << min << endl;
    fout << "==============================================" << endl;
    fout.close();
 
    system("pause");
 
    for (int i = 0; i < n; i++)
        delete[] a[i];
    delete[] a;
 
    return 0;
 
 
}

Если я заменю ту часть вот этим, будет ли выходит за границы массива?

if (i + 1 <= n) { if (a[i][j] > a[i + 1][j]) { s = false; }; }
if (i - 1 >= 0) { if (a[i][j] > a[i - 1][j]) { s = false; }; }
if (j + 1 <= n) { if (a[i][j] > a[i][j + 1]) { s = false; }; }
if (j - 1 >= 0) { if (a[i][j] > a[i][j - 1]) { s = false; }; }
if ((i + 1 <= n)&&(j + 1 <= n)) { if (a[i][j] > a[i + 1][j + 1]) { s = false; }; }
if ((i + 1 <= n)&&(j - 1 >= 0)) { if (a[i][j] > a[i + 1][j - 1]) { s = false; }; }
if ((i - 1 >= 0)&&(j + 1 <= n)) { if (a[i][j] > a[i - 1][j + 1]) { s = false; }; }
if ((i - 1 >= 0)&&(j - 1 >= 0)) { if (a[i][j] > a[i + 1][j - 1]) { s = false; }; }

[p51x]

Будет. В первой строчке. У вас нет элементов a[n][j].

[Сергей9898]

Сообщение от p51x

Будет. В первой строчке. У вас нет элементов a[n][j].

Не подскажите, как мне это исправить?

[min@y™]

бросается в глаза сразу:

000034.png