Я тоже добрый сегодня.
Общее впечатление что
код - пенопласт. В нем очень много строк но очень мало бизнес логики.
1) Во первых непонятно
где фокусироваться. Смотреть формочки - не интересно они состоят в
основном из шумящих событий которые радуют глаз но делают очень мало действий. Это как смотреть
байткод LLVM или Java.
2) Вот такой комментарий я-бы не писал. Он звучит как капитан-очевидность. И ежу понятно что это Main.
/// <summary>
/// Главная точка входа для приложения.
/// </summary>
[STAThread]
static void Main()
Если твой преподаватель требует - то пускай будет. Но я-бы убрал.
3) Вот такие название как
Classes, надо убрать. Они считаются дурным тоном. У тебя-же есть понимание
что внутри? Вот прояви фантазию и дай какое-то бизнесовое название. У тебя в названии вообще висит
газо-анализатор. Используй это.
4) Вот здесь имеет смысл вводить поясняющую переменную. Это основы рефакторинга
о которых писал Кериевски.
Mixtures[mixture].measurementResults[cycle] = gasType == "mixture" ? concentration : Mixtures[mixture].measurementResults[cycle] - concentration;
}//сохранение текущего результата в конкретный цикл конкретной смеси
И комментарий должен стоять над кодом а не внизу.
5) Я не знаю
зачем в 21 веке так сложно конкатенировать строку. Это не Си и не Java1.1.
table.GetControlFromPosition(i, table.RowCount - 1).Text = String.Concat("СКО", ":", Mixtures[i].sko.ToString());
Вот я специально посмотрел в learn портал
https://learn.microsoft.com/en-us/dotnet/csharp/ho... и там пишут что вполне можно брать
операцию плюс или стринговую интерполяцию.
Вобщем старайтесь писать меньше слов. Это - хорошая точка приложения сил для развития как разработчика.
