как сделать код более качественным

[rUs_LAN]

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

п.с. перевод гугл транслейт
п.п.с. visual c# wpf

[Prizrak86]

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

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

п.с. перевод гугл транслейт
п.п.с. visual c# wpf

что то у меня как то с ошибками открывается проект, выложи сюда исходник.

[Скарам]

То, что бросилось в глаза:
Дублирование кода, код можно сократить раза в 2, к примеру:

w1[i] = Math.Pow(a1[i, 1] * a1[i, 2] * a1[i, 3] * a1[i, 4], 1 / 4.0) / (Math.Pow(a1[1, 1] * a1[1, 2] * a1[1, 3] * a1[1, 4], 1 / 4.0) +
Math.Pow(a1[2, 1] * a1[2, 2] * a1[2, 3] * a1[2, 4], 1 / 4.0) + Math.Pow(a1[3, 1] * a1[3, 2] * a1[3, 3] * a1[3, 4], 1 / 4.0) +
Math.Pow(a1[4, 1] * a1[4, 2] * a1[4, 3] * a1[4, 4], 1 / 4.0));

Вынести в функцию, передавать в нее параметры, а не писать одно и то же 5 раз подряд. Еще пример дублирования:

 h = 1;

            for (i = 1; i < 5; i++)
            {
                for (j = 1; j < 5; j++)
                {
                    if (i == j) a5[i, j] = 1;
                    if (j > i) { a5[i, j] = nm[5,h]; h++; }
                    if (j < i) { a5[i, j] = 1 / a5[j, i]; }

                }
            }

Так же написан 5 раз подряд. Вот еще:

 for (i = 1; i < 5; i++)
            {
                for (j = 1; j < 5; j++)
                {
                    Console.Write("{0:f2}\t", a3[i, j]);
                    row[i] = row[i] + "   " + string.Format("{0:0.000}", a3[i, j]);
                }
                row[i] = row[i] + "\t w[" + i + "]=" + string.Format("{0:0.0000}", w3[i]);
                vMatrix.Items.Add(row[i]);

            }

Вместо массивов, я бы использовал коллекцию, проще, эффективней, можно с linq побаловаться. После того, как все лишнее из кода уйдет можно разобрать остальное.

[rUs_LAN]

Создал 2 функции, но пришлось использовать массивы массивов. Вы не могли показать простенький пример работы с коллекциями.

файл .7z.zip проблемы с кодрыровкой в zip.