операции с матрицей с#

[Natok]

мне нужно проверять матрицу таким способом:
у меня есть матрица, например, 4Х4. и каждий елемент - 0,1 или 2.
если среди 8 соседних елементов какой-либо единицы есть 1, то общее число единиц увеличивается на 1.

int  k=0;
            for (int i = 0; i < r; i++)
                for (int j = 0; j < r; j++)
                    if (m[i][j]==1 && (m[i-1][j-1]==1 || m[i-1][j]==1 || m[i-1][j+1]==1 || m[i][j-1]==1 || m[i][j+1]==1 || m[i+1][j-1]==1 || m[i+1][j]==1 || m[i+1][j+1]==1))
                        k++;

ето ведь уж слишком, да? должно бить что-то более рациональное...

[Sazary]

Может, что-нибудь такое:

int k=0;
for (int i = 0; i < r; i++)
 for (int j = 0; j < r; j++)
  for (int a=-1; a<=1; a++)
   for (int b=-1; b<=1; b++)
    if (!(a==0&&b==0) && m[i+a][j+b]==1) k++;

[Natok]

спасибо большое
но здесь надо еще разобратся с индексами...Visual Studio говорит тут о проблемах с индексировании...)))
шас попробеум

[Sazary]

но здесь надо еще разобратся с индексами...Visual Studio говорит тут о проблемах с индексировании...)))

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

[Natok]

int k = 0;
            for (int i = 0; i < r; i++)
                for (int j = 0; j < r; j++)
                    for (int a = -1; a <= 1; a++)
                        for (int b = -1; b <= 1; b++)
                            if (!(a == 0 && b == 0) && ((i+a)>=0) && ((i+a)<r)) && ((i+b)>=0) && ((i+b)<r)) && m[i + a][j + b] == 1 ) k++;

значит так должно бы работать, но в етом случае " сер Visual Studio 2008" говорит, что слишком много "&&"...

[Natok]

но, даже если осуществить проверку на границы в if, то как ми проверим уже в цыкле первый елемент, ведь он не будет проверятся, так как не виполняется условие...

помогите, пожалйста, найти ошибку!

[Sazary]

Вот так, вроде, работает:

int k=0;
bool flag;
for (int i = 0; i < r; i++)
 for (int j = 0; j < r; j++)
  {
   flag = false;
   for (int a=-1; a<=1; a++)
    for (int b=-1; b<=1; b++)
     {
      if(i+a<0 || i+a >=r ||j+b<0 || j+b>=r) continue;
      if (!(a==0&&b==0) && m[i+a][j+b]==1) flag = true;
     }
   if(flag) k++;
  }

ps Писал на c++, но не суть.

[Natok]

спасибо большое,Sazary!

но оно заработало и с предидущим кодом, просто надо было добавить условие m[i][j] == 1

int k = 0;
            for (int i = 0; i < r; i++)
                for (int j = 0; j < r; j++)
                    for (int a = -1; a <= 1; a++)
                        for (int b = -1; b <= 1; b++)
                            if (!(a == 0 && b == 0) && i+a>=0 && i+a<r && j+b>=0 && j+b<r && m[i + a][j + b] == 1 && m[i][j] == 1) k++;

а самое главное - можно было двигатся по i и j с 1 до r-1,и код был бы намного проще

[Sazary]

а самое главное - можно было двигатся по i и j с 1 до r-1,и код был бы намного проще

Но тогда бы не учитывались элементы на границе ) А ведь их нужно учитывать..

[Natok]

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

а можно вопрос?
что делает оператор continue?
я так понимаю, что когда виполняется условие, то цикл просто двигается дальше, да?
да, кажется так и есть...