Intro-CSharp-Book-v2015

902 Въведение в програмирането със C#
Правилно използване на изрази
При работата с изрази има едно много просто правило: не ползвайте
сложни изрази! Сложен израз наричаме всеки израз, който извършва
повече от едно действие. Ето пример за сложен израз:
for (int i = 0; i < xCoord.Length; i++)
{
for (int j = 0; j < yCoord.Length; j++)
{
matrix[i][j] =
matrix[xCoord[FindMax(i) + 1]][yCoord[FindMin(i) + 1]] *
matrix[yCoord[FindMax(i) + 1]][xCoord[FindMin(i) + 1]];
}
}
В примерния код имаме сложно изчисление, което запълва дадена матрица
спрямо някакви изчисления върху някакви координати. Всъщност е много
трудно да се каже какво точно се случва, защото е използван сложен израз.
Има много причини, заради които трябва да избягваме използването на
сложни изрази като в примера по-горе. Ще изброим някои от тях:
- Кодът трудно се чете. В нашия пример няма да ни е лесно да разберем
какво прави този код и дали е коректен.
- Кодът трудно се поддържа. Помислете, какво ще ни струва да
поправим грешка в този код, ако не работи коректно.
- Кодът трудно се поправя, ако има дефекти. Ако примерният код погоре
даде IndexOutOfRangeException, как ще разберем извън границите
на кой точно масив сме излезли? Това може да е масивът xCoord или
yCoord или matrix, а излизането извън тези масиви може да е на
няколко места.
- Кодът трудно се дебъгва. Ако намерим грешка, как ще дебъгнем
изпълнението на този израз, за да намерим грешката?
Всички тези причини ни подсказват, че писането на сложни изрази е
вредно и трябва да се избягва. Вместо един сложен израз, можем да
напишем няколко по-прости израза и да ги запишем в променливи с
разумни имена. По този начин кодът става по-прост, по-ясен, по-лесен за
четене и разбиране, по-лесен за промяна, по-лесен за дебъгване и полесен
за поправяне. Нека сега пренапишем горния код, без сложни изрази:
for (int i = 0; i < xCoord.Length; i++)
{
for (int j = 0; j < yCoord.Length; j++)
{
int maxStartIndex = FindMax(i) + 1;