Уменьшить/оптимизировать код

@Zilagar · Регистрация: 06.06.2021

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

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
private string GetUnicodeOfPiece(ChessPieceType type, int team)
    {
        string unicode;
 
        switch (type)
        {
            case ChessPieceType.None:
                unicode = "";
                break;
            case ChessPieceType.Pawn:
                unicode = "";
                break;
            case ChessPieceType.Rook:
                if(team == 0)
                   unicode = "\u265C";
                else
                    unicode = "\u2656";
                break;
            case ChessPieceType.Knight:
                if (team == 0)
                    unicode = "\u265E";
                else
                    unicode = "\u2658";
                break;
            case ChessPieceType.Bishop:
                if (team == 0)
                    unicode = "\u265D";
                else
                    unicode = "\u2657";
                break;
            case ChessPieceType.Queen:
                if (team == 0)
                    unicode = "\u265B";
                else
                    unicode = "\u2655";
                break;
            case ChessPieceType.King:
                if (team == 0)
                    unicode = "\u265A";
                else
                    unicode = "\u2654";
                break;
            default:
                unicode = "";
                break;
        }
        return unicode;
    }

@VladisSVostok · 14.08.2021, 22:02

C#
1
2
3
4
5
6
7
8
private static string GetUnicodeOfPiece(ChessPieceType type, int team) => type switch {
    ChessPieceType.Rook => team == 0? unicode = "\u265C" : unicode = "\u2656",
    ChessPieceType.Knight => team == 0?  unicode = "\u265E" : unicode = "\u2658",
    ChessPieceType.Bishop => team == 0?  unicode = "\u265D" :  unicode = "\u2657",
    ChessPieceType.Queen => team == 0?  unicode = "\u265B" : unicode = "\u2655",
    ChessPieceType.King => team == 0? unicode = "\u265A" : unicode = "\u2654";
    _ => ""
};

@samana · 15.08.2021, 10:03

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

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

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
// словарь для команды по-умолчанию
private readonly static Dictionary<ChessPieceType, string> _unicodeTeam_default = new Dictionary<ChessPieceType, string>
{
    {ChessPieceType.None, "" },
    {ChessPieceType.Pawn, "" },
    {ChessPieceType.Rook, "\u2656" },
    {ChessPieceType.Knight, "\u2658" },
    {ChessPieceType.Bishop, "\u2657" },
    {ChessPieceType.Queen, "\u2655" },
    {ChessPieceType.King, "\u2654" },
};
 
// словарь для команды номер 0
private readonly static Dictionary<ChessPieceType, string> _unicodeTeam_0 = new Dictionary<ChessPieceType, string>
{
    {ChessPieceType.None, "" },
    {ChessPieceType.Pawn, "" },
    {ChessPieceType.Rook, "\u265C" },
    {ChessPieceType.Knight, "\u265E" },
    {ChessPieceType.Bishop, "\u265D" },
    {ChessPieceType.Queen, "\u265B" },
    {ChessPieceType.King, "\u265A" },
};
 
 
// получаем значение юникода конкретной команды
private static string GetUnicodeOfPiece(ChessPieceType type, int team)
{
    string result = default;
 
    switch (team)
    {
        // попытка взятия юникода для указанной команды 0
        case 0: _unicodeTeam_0.TryGetValue(type, out result); break;
    }
 
    // если такой команды нет, или юникод не определён, то возвращаем значение юникода по умолчанию,
    // либо удачный результат, который получился выше.
    return result == default ? _unicodeTeam_default[type] : result;
}

@OwenGlendower · 15.08.2021, 20:54

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

можно создать для каждой свой словарь с юникодами,

Проще обычный массив. При условии что в enum значения идут по порядку как обычно.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
static readonly string[,] _pieces = new[,]
{
    // Белые
    {"", "", "\u2656", "\u2658", "\u2657", "\u2655", "\u2654"}
    // Черные
    {"", "", "\u265C", "\u265E", "\u265D", "\u265B", "\u265A"}
}
 
// получаем значение юникода конкретной команды
private static string GetUnicodeOfPiece(ChessPieceType type, int team)
{
    return _pieces[team, (int)type];
}

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

Если у вас будет больше чем две команды,

Для шахмат это редкость.

@Zilagar · 16.08.2021, 22:39 **[ТС]**

спасибо

Новые блоги и статьи Все статьи Все блоги /
Вывод данных через динамический список в справочнике Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Функция заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .
Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем	10 пpимет, которые всегда сбываются Maks 31.03.2026 1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .	Перемещение выделенных строк ТЧ из одного документа в другой Maks 31.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .

@Zilagar 0 / 0 / 0 Регистрация: 06.06.2021 Сообщений: 17
	16.08.2021, 22:39 [ТС]
	спасибо 0