Замена switch полиморфизмом

Psilon · Регистрация: 10.07.2011

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

Здравствуйте. Читаю в данный момент книгу по рефакторингу, решил посмотреть свой старый код и переделать. Скажите, как можно заменить это switch? А то что-то в голову ничего не идет (может это потому, что не спал пару дней, не знаю):

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
        private static ILevel GetLevel(int lev)
        {
            ILevel level;
            switch (lev)
            {
                case 2:
                    level = new Level2();
                    break;
                case 3:
                    level = new Level3();
                    break;
                case 4:
                    level = new Level4();
                    break;
                default:
                    level = new Level1();
                    break;
            }
            return level;
        }

Весь код региона:

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
private static ILevel GetLevel(int lev)
        {
            ILevel level;
            switch (lev)
            {
                case 2:
                    level = new Level2();
                    break;
                case 3:
                    level = new Level3();
                    break;
                case 4:
                    level = new Level4();
                    break;
                default:
                    level = new Level1();
                    break;
            }
            return level;
        }
 
        //Содержит код генерации барьеров для различных уровней
 
        #region Levels
 
        private interface ILevel
        {
            HashSet<Point> SetBarriers();
        }
 
        private class Level1 : ILevel
        {
            public HashSet<Point> SetBarriers()
            {
                return new HashSet<Point>();
            }
        }
 
        private class Level2 : ILevel
        {
            public HashSet<Point> SetBarriers()
            {
                var barriers = new HashSet<Point>();
                for (int i = 0; i < _size.Width; i++)
                {
                    barriers.Add(new Point(i, 0));
                    barriers.Add(new Point(i, _size.Height - 1));
                }
                for (int i = 0; i < _size.Height; i++)
                {
                    barriers.Add(new Point(0, i));
                    barriers.Add(new Point(_size.Width - 1, i));
                }
                return barriers;
            }
        }
 
        private class Level3 : ILevel
        {
            public HashSet<Point> SetBarriers()
            {
                var barriers = new HashSet<Point>();
                for (int i = 0; i < _size.Width/3; i++)
                {
                    barriers.Add(new Point(i, _size.Height/3));
                    barriers.Add(new Point(_size.Width - i - 1, _size.Height/3));
                    barriers.Add(new Point(i, (_size.Height*2)/3));
                    barriers.Add(new Point(_size.Width - i - 1, (_size.Height*2)/3));
                }
                return barriers;
            }
        }
 
        private class Level4 : ILevel
        {
            public HashSet<Point> SetBarriers()
            {
                var barriers = new HashSet<Point>();
                for (int i = 0; i < (_size.Width*2)/3; i++)
                {
                    barriers.Add(new Point(i, _size.Height/4));
                    barriers.Add(new Point(_size.Width - i - 1, (_size.Height*3)/4));
                }
                for (int i = 0; i < _size.Height/4; i++)
                {
                    barriers.Add(new Point((_size.Width*2)/3, i));
                    barriers.Add(new Point(_size.Width/3, _size.Height - i - 1));
                }
                barriers.Add(new Point((_size.Width*2)/3, _size.Height/4));
                return barriers;
            }
        }
 
        #endregion

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

@kolorotur · 29.08.2012, 08:04

Нормальная фабрика - где-то все равно должна проверка происходить, а свитч в данном случае эффективнее словаря будет.

@freeba · 29.08.2012, 11:54

Эмм. А не проще было написать метод для генерации барьеров, а информацию как генерировать вынести в data-файлы. В этом случае код был бы проще.

Уровни можно было-бы хранить в словаре типа <№, Имя_Файла_Уровня>, в метод генерации блоков передаем имя файла, и он все генерит. Т.е. мы заменяем весь код региона на словарь и один метод, явный PROFIT.

Psilon · 29.08.2012, 18:54 **[ТС]**

В общем тупанул с методом, вот гораздо более простой вариант:

C#
1
2
3
4
5
6
7
8
9
10
private static ILevel GetLevel(int level)
        {
            switch (level)
            {
                case 2: return new Level2();
                case 3: return new Level3();
                case 4: return new Level4();
                default: return new Level1();
            }
        }

Добавлено через 13 минут
Вопрос такой. Может все это загнать как методы одного класса, например у класса Level сделать методы GetLevel1, GetLevel2... и потом как-то с помощью рефлексии вызывать метод

C#
1
2
3
4
5
HashSet<Point> SetBarriers(int level)
{
   ...
   return Invoke("GetLevel" + level);
}

я наверняка написал бредятину, потому что с рефлексией не работал, но мне кажется, что что-то такое можно провернуть.

@turbanoff · 31.08.2012, 00:01

Если можно без рефлексии, то лучше ее не применять.
Чем вас просто не устраивает например массив с делегатами?

Psilon · 31.08.2012, 00:04 **[ТС]**

turbanoff, да я думаю, может так оставить, или все же некрасиво? Вроде, добавление нового уровня сведется к написанию еще одного класса и добавление в этот метод еще одного return, и все.

@Noob.net · 31.08.2012, 00:31

Можно объединить методы по создание объектов в массив и вызывать по имени.Будет без свитч.

Ну или медленная рефлексия)

C#
1
2
Type T = Type.GetType("Namespace.And.Your.TypeName");
object Obj = Activator.CreateInstance(T);

@m0nax · 31.08.2012, 13:43

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

Добавлено через 13 минут
Вопрос такой. Может все это загнать как методы одного класса, например у класса Level сделать методы GetLevel1, GetLevel2... и потом как-то с помощью рефлексии вызывать метод

C#
1
2
3
4
5
HashSet<Point> SetBarriers(int level)
{
   ...
   return Invoke("GetLevel" + level);
}

я наверняка написал бредятину, потому что с рефлексией не работал, но мне кажется, что что-то такое можно провернуть.

а если рука дрогнет и получится метод GetLebel4() для нового уровня?
компилятор это радостно скушает, все будет отлично запускаться и работать, но вместо 4 уровня будет непонятно что без намека на причину ошибки

@iliabest · 31.08.2012, 15:53

Я бы решил эту проблему так. каждый класс помечаешь атрибутом с номером. К примеру DisplayName("1").
В начале работы программе или где-то в другой момент формируешь Dictionary<int, ILevel> с помощью рефлексии. В момент вызова метода делаешь

C#
1
return Dictionary<int, ILevel>[level]

@Andre_Annrey · 01.06.2015, 13:52

Избавиться от свитча, можно, с помощью делегатов:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
Private delegate double LevelDelegate(ILevel level);
Private Dictionary<int, LevelDelegate > _levels;
_levels = new Dictionary<int, LevelDelegate >
{
{2, this.Level2},
{3, this.Level3},
{4, this.Level3}
};
public double PerformLevel (int op, ILevel level)
{
    if (!_levels.ContainsKey(op))
        throw new ArgumentException(string.Format("Level {0} is invalid", op), "op");
    return _ levels[op](level);
}

Новые блоги и статьи Все статьи Все блоги /
Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: показать затраченные материалы за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В качестве. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	29.08.2012, 08:04
	Нормальная фабрика - где-то все равно должна проверка происходить, а свитч в данном случае эффективнее словаря будет. 1

@freeba Неадекват 1501 / 1237 / 248 Регистрация: 02.04.2010 Сообщений: 2,807
	29.08.2012, 11:54
	Эмм. А не проще было написать метод для генерации барьеров, а информацию как генерировать вынести в data-файлы. В этом случае код был бы проще. Уровни можно было-бы хранить в словаре типа <№, Имя_Файла_Уровня>, в метод генерации блоков передаем имя файла, и он все генерит. Т.е. мы заменяем весь код региона на словарь и один метод, явный PROFIT. 1

@turbanoff 4094 / 3828 / 745 Регистрация: 18.05.2010 Сообщений: 9,331 Записей в блоге: 12
	31.08.2012, 00:01
	Если можно без рефлексии, то лучше ее не применять. Чем вас просто не устраивает например массив с делегатами? 0

Psilon Master of Orion 6102 / 4958 / 905 Регистрация: 10.07.2011 Сообщений: 14,522 Записей в блоге: 5
	31.08.2012, 00:04 [ТС]
	turbanoff, да я думаю, может так оставить, или все же некрасиво? Вроде, добавление нового уровня сведется к написанию еще одного класса и добавление в этот метод еще одного return, и все. 0