Реорганизовать код: вызовы методов в зависимости от условий

@alspav · Регистрация: 29.06.2015

Студворк — интернет-сервис помощи студентам

Здравствуйте
Опыт в программировании небольшой, код получается достаточно раздутый, написал конструкцию достаточно простую, но кажется что её можно оформить более лаконично подскажите пожалуйста как это сделать:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 private double[] td = new double[14];
        public int TDOUT (ref double[] IN,string pair)
        {
            td = IN; //записывааем исходные значения
            if (pair=="pw")
            {
                int temp = PW();
                if (temp == 0)
                {
                    IN = td;
                    return temp;
                }
                else return temp;
            }
            if (pair == "pst")
            {
                int temp = PST();
                if (temp == 0)
                {
                    IN = td;
                    return temp;
                }
                else return temp;
            }
            if (pair == "tw")
            {
                int temp = TW();
                if (temp == 0)
                {
                    IN = td;
                    return temp;
                }
                else return temp;
            }
            if (pair == "tst")
            {
                int temp = TST();
                if (temp == 0)
                {
                    IN = td;
                    return temp;
                }
                else return temp;
            }
            if (pair == "pt")
            {
                int temp = PT();
                if (temp == 0)
                {
                    IN = td;
                    return temp;
                }
                else return temp;
            }
            if (pair == "ph")
            {
                int temp = PH();
                if (temp == 0)
                {
                    IN = td;
                    return temp;
                }
                else return temp;
            }
            return 2;

@Ev_Hyper · 17.07.2015, 14:37

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

Подскажи пожалуйста с какой целью к полю структуры добавляется { get; set; }, ведь насколько я понимаю в них и так можно писать если стоит public?

alspav, публичные поля это не хорошо, поэтому обычно используют свойства. Из msdn: Свойство — это член, предоставляющий гибкий механизм для чтения, записи или вычисления значения частного (private) поля.
По ссылкам более подробно:
https://msdn.microsoft.com/ru-... s7x04.aspx
https://msdn.microsoft.com/ru-... 84054.aspx

@alspav · 17.07.2015, 15:01 **[ТС]**

Идея понятна, в полном смысле ошибкой не является, но неправильно для безопасности кода. Спасибо.

Добавлено через 18 минут
Может если не затруднит подскажите тогда как правильно сделать конструктор для структуры ТТ, так как если только делаю так

C#
1
2
3
default:
                    temp.error = 1;
                    return temp;

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

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
struct TT
    {
        public int error;
        public double P { get; set; }//значение давления
        public double T { get; set; }//значение температуры
        public double h { get; set; }//значение энтальпии
        public string PasheState { get; set; }//фазовое состояние
        public double Po { get; set; } //плотность
        public double S { get; set; } //энтропия
        public double Cp { get; set; }//изобарная теплоемкость
        public double Cv { get; set; }//изохорная теплоемкость
        public double u { get; set; }//внутренняя энергия
        public double Wz { get; set; }//скорость звука в среде
        public double mu { get; set; }//динамическая вязкость
        public double v { get; set; }//кинематическая вязкость
        public double lamda { get; set; }//теплопроводность
        public string ExcessAbsoluteP { get; set; }//избыточное или абсолютное давление используется в расчетах
        public string KelCelT { get; set; }//значение заданное в измерении температуры
        public override string ToString()
        {
            return string.Format("{0};{1}\n", ExcessAbsoluteP,
                string.Join(";", P, T, KelCelT, h, PasheState, Po, S, Cp, Cv, u, Wz, mu, v, lamda));
        }
    }

@Ev_Hyper · 17.07.2015, 15:09

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

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

alspav, подходит, но его тоже нужно "вызвать":

C#
1
TT temp = new TT();

Можно переписать метод следующим образом:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 public TT TDOUT(string pair)
        {
            switch (pair)
            {
                case "pw":
                    return PW();
                case "pst":
                    return PST();
                case "tw":
                    return TW();
                case "tst":
                    return TST();
                case "pt":
                    return PT();
                case "ph":
                    return PH();
                default:
                    return new TT { error = 1 };
            }
        }

Psilon · 17.07.2015, 15:38

alspav, только в ООП стоит исключения бросать, а не возвращать коды ошибок.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 public TT TDOUT(string pair)
        {
            switch (pair)
            {
                case "pw":
                    return PW();
                case "pst":
                    return PST();
                case "tw":
                    return TW();
                case "tst":
                    return TST();
                case "pt":
                    return PT();
                case "ph":
                    return PH();
                default:
                    throw new ArgumentException();
            }
        }

@alspav · 17.07.2015, 15:42 **[ТС]**

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

Psilon · 17.07.2015, 16:09

alspav, ну например:
https://msdn.microsoft.com/ru-... 73160.aspx

Добавлено через 4 минуты
Слева есть менюшка (ну или снизу) со связанными разделами, для более полной информации можно их почитать.

Но на самом деле много слов по простой теме, где ошибка, кидаем исключение (как в примере выше), а в коде ловим

C#
1
2
3
4
5
6
7
8
try
{
   TDOUT("asgas");
}
catch (ArgumentException ex)
{
   MessageBox.Show("Error: " + ex.Message);
}

Новые блоги и статьи Все статьи Все блоги /
Транскрипция 55-минутного видео через Whisper: WhisperDesktop облажался, спас Google Colab[ anaschu 01.06.2026 Понадобилось получить текст из свежезагруженного видео на YouTube. Казалось бы, задача на пять минут. Заняла полтора часа. Делюсь опытом — может кому пригодится последовательность решений. . . .	21 мат мед. Планы на развитие модели здравоСохранения anaschu 01.06.2026 AnyLogic: план развития симуляционной модели рабочего коллектива — динамический абсентеизм, реальные данные, три сценария сравнения Продолжаю серию постов о дискретно-событийной модели рабочего. . .	20. Мат мед. Абсентеизм как отдельный тип простоя anaschu 29.05.2026 Апдейт модели: исправленные баги, абсентеизм и новые механизмы Продолжаю развивать ранее описанную модель рабочего коллектива на AnyLogic. За последние несколько дней был проведён серьёзный. . .	19. здоровье, усталость и психотип работника влияют на производительность предприятия, и наоборот, производительность на здоровье, усталось и психотип anaschu 28.05.2026 Дискретно-событийная модель рабочего коллектива на AnyLogic: здоровье, выгорание, психотипы и микростимуляция Привет, коллеги. Хочу поделиться итогами нескольких недель работы над симуляционной. . .
"Прокси" для последовательного порта Eddy_Em 28.05.2026 Эту штуку написал я достаточно давно. Но сейчас вот понадобилось настроить датчик грозы, но при этом не отключать его от "метеодемона". Соответственно, надо запустить этот "прокси": метеодемон будет. . .	Рефакторинг программы уравнивания. Massaraksh7 26.05.2026 Пример по предыдущей записи в блоге. Но, надо заметить, что, во-первых, там оптимизация не только математики, но и работы с базой данных, и с графами, а во-вторых, это ещё не всё.	Использование TThread в Lazarus для математических вычислений. Massaraksh7 25.05.2026 Производя рефакторинг своих программ на предмет ускорения их работы, обратил внимание на такой аспект, как сокращение времени матвычислений. Дело в том, что приходится работать с большими матрицами. . .	Модель здравосохранения 18. Чем здоровее работник, тем быстрее выгорает anaschu 24.05.2026 Имитационная модель корпоративного здравоохранения: что показывает математика Сегодня в модели рабочего коллектива на AnyLogic появились три новые механики — выгорание через накопленную усталость,. . .

@alspav 1 / 1 / 0 Регистрация: 29.06.2015 Сообщений: 41
	17.07.2015, 15:42 [ТС]
	С обработкой исключений я пока не знаком, согласен что так наверно правильнее, но из за одного локального случая разбираться неохото, если даш ссылку буду признателен обязательно почитаю. 0

Реорганизовать код: вызовы методов в зависимости от условий

Решение