Избавиться от повторения вызова одного и того же метода с одним меняющимся параметром

@Злобный Зайц · Регистрация: 21.10.2010

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

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
        struct Marker
        {
            public static string chipValue= '\"' + "Value" + '\"';
            public static string chipElementInfo = '\"' + "ElementInfo" + '\"';
            public static string chipDescription = '\"' + "Description" + '\"';
        }
 
        private void ParseCadStarNetList(List<string> netList)
        {
            foreach (string line in netList)
            {
                if (line.Contains(Marker.chipValue))
                    UpdateChipNameAndValue(ChipValues, line, Marker.chipValue);
                if (line.Contains(Marker.chipElementInfo))
                    UpdateChipNameAndValue(ChipValues, line, Marker.chipElementInfo);
                if (line.Contains(Marker.chipDescription))
                    UpdateChipNameAndValue(ChipValues, line, Marker.chipDescription);
            }
            ...........................................

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

@OwenGlendower · 15.02.2015, 16:38

Злобный Зайц, например так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
private static string[] Markers = new[] {
    '\"' + "Value" + '\"',
    '\"' + "ElementInfo" + '\"',
    '\"' + "Description" + '\"'
};
 
private void ParseCadStarNetList(List<string> netList)
{
    foreach (string line in netList)
    {
        foreach (string marker in Markers)
        {
            if (line.Contains(marker))
            {
                UpdateChipNameAndValue(ChipValues, line, marker);
            }
        }
    }
}

@Metall_Version · 15.02.2015, 16:44

Сообщение от Злобный Зайц

Как сделать более прилично?

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

если же это константы то нужно создать статический класс.

можно рефлексией пройтись по полям
или делать авто заполняемый шаблон в самой ide (это к псилону обращаться)

вот рефлексией

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
        private static void ParseCadStarNetList(List<string> netList)
        {
            foreach (string line in netList)
            {
                foreach (var field in typeof(Marker).GetFields())
                {
                    if(field.FieldType != typeof(String)) continue;
                    var value = (String)field.GetValue(null);
                    if (line.Contains(value))
                        UpdateChipNameAndValue(ChipValues, line, value);
                }
            }
        }
 
    public static class  Marker
    {
        public static string ChipValue = "\"Value\"";
        public static string ChipElementInfo = "\"ElementInfo\"";
        public static string ChipDescription = "\"Description\"";
    }

@Евгений В · 15.02.2015, 22:23

Злобный Зайц,
Также, как Metall_Version предложу рефлексию. Такой вариант:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
        struct Marker
        {
            public static string chipValue = '\"' + "Value" + '\"';
            public static string chipElementInfo = '\"' + "ElementInfo" + '\"';
            public static string chipDescription = '\"' + "Description" + '\"';
 
            public static List<string> ChipList()
            {
                List<string> list = new List<string>();
                foreach (var field in typeof(Marker).GetFields())
                    list .Add((String)field.GetValue(null));            
              return list;
            }           
        }
 
        private void ParseCadStarNetList(List<string> netList)
        {
            foreach (string line in netList)           
                foreach (string marker in Marker.ChipList())
                    if (line.Contains(marker))
                        UpdateChipNameAndValue(marker); 
        }

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .