Матрица.

[Prin53]

Заменить нулевые элементы на сумму прилежащих ему элементов.

#include <iostream>
using namespace std;

int main()
{
	int A[10] [10],n,m;
	cout<<"Enter n and m\n";
	cin>>n>>m;
	for(int j=0; j<n; j++)
	{
		for(int i=0; i<n; i++)
		{
			A[i] [j]  = rand()%10-5;
			cout<<A[i][j]<<" ";
		}
		cout<<"\n";
	}
	 for (int i=0;i<n;i++)
    {
         for (int j=0;j<m;j++)
         {
             if (A[i][j] == 0)
             {
                if ((i > 1 ) && (i < n) && (1 < j) && (j < m))
                    A[i][j]=A[i+1][j]+A[i-1][j]+A[i][j+1]+A[i][j-1]+A[i-1][j-1]+A[i-1][j+1]+A[i+1][j-1]+A[i+1][j+1];
                else
                    if ((i == 1) &&  ((j > 1) && (j < m)))
                        A[i][j]=A[i][j-1]+A[i][j+1]+A[i+1][j-1]+A[i+1][j+1]+A[i+1][j];
                    else
                        if ((i == n) && ((1 < j) && (j < m)))
                            A[i][j]=A[i][j-1]+A[i][j+1]+A[i-1][j]+A[i-1][j-1]+A[i-1][j+1];
                        else
                            if ((j == 1) &&  ((i > 1 ) && (i < n)))
                                A[i][j]=A[i-1][j]+A[i+1][j]+A[i+1][j+1]+A[i-1][j+1]+A[i][j+1];
                            else
                                if ((j == m) && ((i > 1 ) && (i < n)))
                                    A[i][j]=A[i-1][j]+A[i+1][j]+A[i-1][j-1]+A[i][j-1]+A[i+1][j-1];
                                else
                                    if ((j == 1) && (i == 1))
                                        A[i][j]=A[i+1][j]+A[i][j+1]+A[i+1][j+1];
                                    else
                                        if ((j == 1) && (i == n))
                                            A[i][j]=A[i][j+1]+A[i-1][j]+A[i-1][j+1];
                                        else
                                            if ((j == m) && (i == 1))
                                                A[i][j]=A[i][j-1]+A[i+1][j]+A[i+1][j-1];
                                            else
                                                if ((j == m) && (i == n))
                                                    A[i][j]=A[i-1][j]+A[i][j-1]+A[i-1][j-1];
             }
         }
    }
cout<<"\n";
for(j=0; j<n; j++)
	{
		for(i=0; i<n; i++)
		{
			cout<<A[i][j]<<" ";
		}
		cout<<"\n";
	}
return 0;
}

Что в коде не так?

[Granus]

1) Вот это условие неверное (i == n), индексация массива от 0 до n-1, и цикл у Вас так же выполняется, значения n просто никогда не будет
2) Вообще весь код внутри if (A[i][j] == 0) ужасный. Вот более-менее нормальный вариант:

if (i > 0) {A[i][j] += A[i - 1][j];}
if (i < n - 1) {A[i][j] += A[i + 1][j];}
if (j > 0) {A[i][j] += A[i][j - 1];}
if (j < n -1) {A[i][j] += A[i][j + 1];}

[Prin53]

спасибо, а если просто окружить матрицу нулями и считать по одной формуле?

[Granus]

а если просто окружить матрицу нулями и считать по одной формуле?

В принципе никто не запрещает, но если матрица будет больших размеров, вы потратите много дополнительной ненужной памяти. И чем вас четыре условия не устраивают?)

[Prin53]

Сделал как вы сказали. Думаю, и правда, лучший вариант. Еще раз спасибо.