Как сделать вывод в цикле один раз?

Question

[tera1004]

Код проверяет является ли массив симметричным относительно главной диагонали.
Проблема у меня с выводом. Выводится каждый цикл. Нужно чтобы вывелось единственный раз в соответствии с результатом. Если каждый раз цикл If является true ,то массив "симметричен". Если же хоть раз выдал false ,то вывод "не симметричен"

int n = 3;
            int[,] a = { { 1, 1, 1 }, { 1, 1, 1 }, { 1, 1, 1 } };
            for (int i = 0; i < a.GetLength(0); ++i)
             for (int j = 0; j < a.GetLength(1); ++j)
                 if (a[i, j] == a[j, i])
                Console.WriteLine("Симметрична");
                else Console.WriteLine("Ne simmetrichna");

Answer 1

[#]

ну тогда уж:

int[,] a = { { 1, 1, 1 }, { 1, 1, 1 }, { 1, 1, 1 } };
bool s = true;
for (int i = 0; i < a.GetLength(0) && s; ++i)
  for (int j = 0; j < a.GetLength(1) && s; ++j)
    s &= a[i, j] == a[j, i];
Console.WriteLine(s ? "Симметрична" : "Ne simmetrichna");

при этом можно и просто присваивать контрольке значение

int[,] a = { { 1, 1, 1 }, { 1, 1, 1 }, { 1, 1, 1 } };
bool s = true;
for (int i = 0; i < a.GetLength(0) && s; ++i)
  for (int j = 0; j < a.GetLength(1) && s; ++j)
    s = a[i, j] == a[j, i];
Console.WriteLine(s ? "Симметрична" : "Ne simmetrichna");

а можно проверять квадратность матрицы до начала циклов, а так же перебирать "треугольник"

spoiler

using System;

namespace simm_array
{
    class Program
    {
        static void Main(string[] args)
        {
            var m = new int[,] { { 1, 1, 1 }, { 1, 1, 1 }, { 1, 1, 1 } };
            var w = m.GetLength(0);
            var h = m.GetLength(1);
            var s = w == h;
            for (var x = 0; x < w && s; ++x)
                for (var y = x; y < h && s; ++y)
                    s = m[x, y] == m[y, x];
            Console.WriteLine(s ? "Симметрична" : "Ne simmetrichna");
        }
    }
}

в каментах предложено не сканировать главную диагональ.. и это логично! )) .. ну собственно и просто делается

spoiler

using System;

namespace simm_array
{
    class Program
    {
        static void Main(string[] args)
        {
            var m = new int[,] { { 1, 1, 1 }, { 1, 1, 1 }, { 1, 1, 1 } };
            var w = m.GetLength(0);
            var h = m.GetLength(1);
            var s = w == h;
            for (var x = 0; x < w && s; ++x)
                for (var y = x + 1; y < h && s; ++y)
                    s = m[x, y] == m[y, x];
            Console.WriteLine(s ? "Симметрична" : "Ne simmetrichna");
        }
    }
}

ну и немного функционального стиля+синтаксического сахара

spoiler

using System;

namespace simm_array
{
    static class Program
    {
        static void Main(string[] args)
        {
            var m = new int[,] { { 1, 1, 1 }, { 1, 1, 1 }, { 1, 1, 1 } };
            m.dump();
            m.isSymmetrical().report();
        }

        static bool isSymmetrical(this int[,] m)
        {
            var w = m.GetLength(0);
            var h = m.GetLength(1);
            var s = w == h;
            for (var x = 0; x < w && s; x++)
                for (var y = x + 1; y < h && s; y++)
                    s = m[x, y] == m[y, x];
            return s;
        }

        static void dump(this int[,] m)
        {
            var w = m.GetLength(0);
            var h = m.GetLength(1);
            for (var x = 0; x < w; x++)
            {
                for (var y = 0; y < h; y++)
                {
                    Console.Write($"\t{m[x, y]}");
                }
                Console.WriteLine();
            }
            Console.WriteLine();
        }

        static void report(this bool s) => Console.WriteLine(s ? "Симметрична" : "Ne simmetrichna");
    }
}

выхлоп

Comments

[Дмитрий Беляев]

при этом можно и просто присваивать контрольке значение

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

[lam0x86]

Дмитрий Беляев, но ведь там так же, как и в вашем решении, есть выход из цикла, если !s.

[lam0x86]

Можно ещё немного оптимизировать, не проверяя основную диагональ.

[#]

Дмитрий Беляев, можно. и вот почему

флаг будет иметь значение последней проверки

и это хорошо. это повод прекратить сканирование. достаточно одного несовпадения (нарушения симметрии), что бы признать матрицу не симметричной

[#]

lam0x86,

но ведь там так же, как и в вашем решении, есть выход из цикла, если !s.

.. мм я конечно допускаю что мог не заметить.. но я копировал первую правку из ответа коллеги с внедрением s в условие цикла, а версию из студии привел целиком. исторически, были еще каменты к вопросу, с предложением брейка, а затем критикой, что брейк не лечит 2 цикла. нашел где это было, вы и так в курсе )))
... ответ Дмитрий Беляев показывал как найти решение без брейка, но с полным сканированием (а так же он пытался в каменте выше оспаривать присвоение результата проверки ))) ... но по той же причине - немножечко недогона..)

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

[#]

Дмитрий Беляев, а еще, приведенный код проверен в студии, на примерах и симметричного, и не симметричного набора данных.... не поверите - работает ;)))

Answer 2

[Дмитрий Беляев]

int[,] a = { { 1, 1, 1 }, { 1, 1, 1 }, { 1, 1, 1 } };
bool s = true;
for (int i = 0; s && i < a.GetLength(0); ++i)
  for (int j = 0; s && j < a.GetLength(1); ++j)
    s = s && a[i, j] == a[j, i];
Console.WriteLine(s ? "Симметрична" : "Ne simmetrichna");

Comments

[#]

???
&&= - проверьте компилятором

[Дмитрий Беляев]

поправил, а заодно добавил ранний выход из цикла

Answer 3

[Farwisdomer]

Если хоть 1 раз a [i, j] <> a[j, i] - значит мартица несимметрична. Значит достаточно поменять условие

if (a[i, j] <> a[j, i])
Console.WriteLine("Ne simmetrichna")
break;

если условие не выполнилось за циклом вывести - симметрична.

Comments

[tera1004]

Ах
Спасибо,сам что-то не додумался до такого простого варианта.

[lam0x86]

tera1004, неправильного варианта, кстати. break сработает для внутреннего цикла. Из двух циклов так не выйти. Ну и фигурные скобки не забываем ставить.

[tera1004]

lam0x86, Это понятно,что <> заменяется на !=. И завершение циклов по другому происходит. Но мне самое главное нужно было узнать как условие поменять. Что-то моя голова не сообразила насчет этого

[#]

tera1004, ну тут пока подсказка на половину. в моем ответе, последний фрагмент из студии - и компилируется, и работает, и правильно определяет отсутствие симметрии (но только относительно главной диагонали)