Форум программистов, компьютерный форум, киберфорум
C# для начинающих
Войти
Регистрация
Восстановить пароль
Карта форума Темы раздела Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.75/8: Рейтинг темы: голосов - 8, средняя оценка - 4.75
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
1

Оптимизация большого количества if else

19.09.2022, 17:50. Показов 1629. Ответов 25

Author24 — интернет-сервис помощи студентам
В приере только 3 в игре будет 26. и как видите (p1=0 и p2=1) тоже самое что (p1=1 и p2=0) получаетс 2 фразы повторяются. Как это отимезировать?

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
void fight()
{
    // 0 - камень 1- ножницы 2-бумага
    int player_1;
    int player_2;
    string text;
    
    if (s1 == s2)
    {
        text = "ничья";
    }
    else if (s1 == 0)
    {
        if (s2 == 1) 
        {
            text = "Ножницы затупились о камень";
        } 
        else if (s2 == 2) 
        {
            text = "Бумага завернула камень";
        }
    }
    else if (s1 == 1)
    {
        if (s2 == 0) 
        {
            text = "Ножницы затупились о камень";
        } 
        else if (s2 == 2) 
        {
            text = "Ножницы разрезали бумагу";
        }
    }
    else if (s1 == 2)
    {
        if (s2 == 0) 
        {
            text = "Бумага завернула камень";
        } 
        else if (s2 == 1) 
        {
            text = "Ножницы разрезали бумаг";
        }
    }
   
}
0
Лучшие ответы (1)
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
19.09.2022, 17:50
Ответы с готовыми решениями:

Оптимизация большого количества NPC на карте
Пытаюсь сделать Терарию на C#(без Unity и т.д).В этой игре есть довольно большая карта, на которой...

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

Оптимизация хранения большого количества объектов в Dictionary
Добрый день. Имеется некий класс Quest и список public static Dictionary<Subject, Dictionary<byte,...

Оптимизация вывода большого (более 7000) количества записей на странице
Здравствуйте. Прошу совета, как сделать более оптимальным решение. Имеется справочник КБК....

Оптимизация обработки большого количества объектов
Доброго времени суток, господа. У меня есть графическая программа на основе SFML. В программе...

25
2638 / 1566 / 853
Регистрация: 23.02.2019
Сообщений: 3,876
19.09.2022, 18:22 2
Bright4092b, Можно так попробовать
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
void fight()
{
    // 0 - камень 1- ножницы 2-бумага
 
    int stoneScissors = 1; // 0 + 1
    int stonePaper = 2; // 0 + 2
    int scissorsPaper = 3; // 1 + 2
 
    int result = s1 + s2;
 
    if (result == stoneScissors)
    {
        text = "Ножницы затупились о камень";
    }
    else if (result == stonePaper)
    {
        text = "Бумага завернула камень";
    }
    else if (result == scissorsPaper)
    {
        text = "Ножницы разрезали бумагу";
    }
    else
    {
        text = "ничья";
    }
 
}
0
Администратор
Эксперт .NET
9610 / 4752 / 761
Регистрация: 17.04.2012
Сообщений: 9,606
Записей в блоге: 14
19.09.2022, 18:30 3
samana, некоторые состояния этой машины пересекаются
Например, ножницы + ножницы == камень+бумага
1
2638 / 1566 / 853
Регистрация: 23.02.2019
Сообщений: 3,876
19.09.2022, 18:33 4
Лучший ответ Сообщение было отмечено Bright4092b как решение

Решение

Цитата Сообщение от tezaurismosis Посмотреть сообщение
samana, некоторые состояния этой машины пересекаются
Например, ножницы + ножницы == камень+бумага
Справедливое замечание! Спасибо. Тогда лучше сразу проверить на совпадение одинаковых результатов
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
void fight()
{
    // 0 - камень 1- ножницы 2-бумага
 
    int stoneScissors = 1; // 0 + 1
    int stonePaper = 2; // 0 + 2
    int scissorsPaper = 3; // 1 + 2
 
    if (s1 == s2)
    {
        text = "ничья";
        return;
    }
 
    int result = s1 + s2;
 
    if (result == stoneScissors)
    {
        text = "Ножницы затупились о камень";
    }
    else if (result == stonePaper)
    {
        text = "Бумага завернула камень";
    }
    else if (result == scissorsPaper)
    {
        text = "Ножницы разрезали бумагу";
    }
 
}
2
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
19.09.2022, 19:32  [ТС] 5
Цитата Сообщение от samana Посмотреть сообщение
Справедливое замечание! Спасибо. Тогда лучше сразу проверить на совпадение одинаковых результатов
[/CSHARP]
а что если я сделаю пожоже sting камень-ножниы, камень-бумага и тд. по моему идеально подяойдет и не придется по 2 раза одно и тоже писать
0
2638 / 1566 / 853
Регистрация: 23.02.2019
Сообщений: 3,876
19.09.2022, 19:47 6
Bright4092b, если делать через string, тогда придется создавать строки по два варианта: камень-ножницы и ножницы-камень например.
0
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
19.09.2022, 20:39  [ТС] 7
Цитата Сообщение от samana Посмотреть сообщение
Bright4092b, если делать через string, тогда придется создавать строки по два варианта: камень-ножницы и ножницы-камень например.
нет не придется я проверю что больше s1 или s2 и у меня будет названия грубо говоря 0_1 и 0_2 а если например 1 ножницы а 2й камень. я просто запишу s2_s1 просто услови что больше и в строке меньшее-большее
0
2638 / 1566 / 853
Регистрация: 23.02.2019
Сообщений: 3,876
19.09.2022, 20:51 8
Bright4092b, наверно лучше покажите вашу идею в виде реализованного кода, потому что я не совсем понимаю каким образом вы сократите if-ы (которых всего 4, что в принципе не критично).
0
Эксперт по электронике
3073 / 2146 / 548
Регистрация: 04.09.2018
Сообщений: 7,368
Записей в блоге: 4
19.09.2022, 21:19 9
Если немного модифицировать решение samana:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
Dictionary<int, string> Phrases = new()
{
    { 1, "Ножницы затупились о камень" },
    { 2, "Бумага завернула камень" },
    { 3, "Ножницы разрезали бумагу" }
};
 
int s1 = 1, s2 = 2;
 
if ( s1 == s2 )
    Console.WriteLine( "Ничья!" );
else
    Console.WriteLine( Phrases[ s1 + s2 ] );
2
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
19.09.2022, 21:25  [ТС] 10
Цитата Сообщение от samana Посмотреть сообщение
Bright4092b, наверно лучше покажите вашу идею в виде реализованного кода, потому что я не совсем понимаю каким образом вы сократите if-ы (которых всего 4, что в принципе не критично).
примерно так. помоему идельное решение

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
    string[] symbol = new string[3];
    int p1, p2;
    string text = "text";
    
    void start_func()
    {
        if (p1==p2) 
        {
            text = "text"
        }
        else if (p1 < p2)
        {
            text = symbol[p1]+"_"+symbol[p2];
        }
        else
        {
            text = symbol[p2]+"_"+symbol[p1];
        }
        
        control(text);
    }
    void control(string str)
    {
        switch (str)
        {
            case "text":
                    text = "ничья";
                break;
            case "stone_seas":
                    text = "Ножницы затупились о камень";
                break;  
            case "stone_peap":
                    text = "Бумага завернула камень";
                break;
            case "peap_seas":
                    text = "Ножницы разрезали бумаг";
                break;
        }
}
0
Эксперт по электронике
3073 / 2146 / 548
Регистрация: 04.09.2018
Сообщений: 7,368
Записей в блоге: 4
19.09.2022, 21:29 11
Цитата Сообщение от Bright4092b Посмотреть сообщение
помоему идельное решение
Даже больше. Гениальное!
Цитата Сообщение от Bright4092b Посмотреть сообщение
C#
1
2
3
4
if (p1==p2) 
{
      text = "text"
}
Цитата Сообщение от Bright4092b Посмотреть сообщение
C#
1
2
case "text":
        text = "ничья";
Это вообще радует глаз!
1
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
19.09.2022, 21:34  [ТС] 12
если про то что милион слов text) это просто чтобы в качеству показа набросал за 5 минут, плюс я планшета

Добавлено через 1 минуту
конечно в программе будет более красиво)) но идею вы мне подали с int а я уже придумал вот так сделать
0
Эксперт по электронике
3073 / 2146 / 548
Регистрация: 04.09.2018
Сообщений: 7,368
Записей в блоге: 4
19.09.2022, 21:36 13
Bright4092b, осталось совсем малость: известить участников, какой именно игрок выиграл...
1
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
19.09.2022, 21:41  [ТС] 14
а если их 2) первый подскзкал и ваш коммент тоже. с миру по нитке короче. Но тот наверно больше так как первый был. Но я бось кого то обидеь)

Добавлено через 47 секунд
я из 2х ответов собрал свой вот в чем прикол
0
Эксперт по электронике
3073 / 2146 / 548
Регистрация: 04.09.2018
Сообщений: 7,368
Записей в блоге: 4
19.09.2022, 21:43 15
Цитата Сообщение от Bright4092b Посмотреть сообщение
а если их 2)
Это ничья.
Цитата Сообщение от Bright4092b Посмотреть сообщение
Но я бось кого то обидеь)
Тут не обидчивые сидят.
0
2638 / 1566 / 853
Регистрация: 23.02.2019
Сообщений: 3,876
19.09.2022, 21:44 16
Bright4092b, вы из двух нормальных вариантов собрали Франкенштейна. Но ничего, всё приходит с опытом, надо же с чего-то начинать.
Здесь не принято обижаться, ведь мы собрались чтобы помогать друг другу, а не участвовать с соревнованиях кто быстрее и лучше.
Вариант от wizard41 аккуратный и понятный (с одним if).
1
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
19.09.2022, 21:44  [ТС] 17
тогда тот, кто был самый первый
0
Эксперт по электронике
3073 / 2146 / 548
Регистрация: 04.09.2018
Сообщений: 7,368
Записей в блоге: 4
19.09.2022, 21:45 18
Bright4092b, общая идея в том, что если у тебя начинает собираться целая туча ИФов, то стоит пересмотреть свою реализацию и применить что-нибудь другое.
1
0 / 0 / 0
Регистрация: 27.05.2018
Сообщений: 51
19.09.2022, 21:52  [ТС] 19
Цитата Сообщение от samana Посмотреть сообщение
Справедливое замечание! Спасибо. Тогда лучше сразу проверить на совпадение одинаковых результатов
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
void fight()
{
    // 0 - камень 1- ножницы 2-бумага
 
    int stoneScissors = 1; // 0 + 1
    int stonePaper = 2; // 0 + 2
    int scissorsPaper = 3; // 1 + 2
 
    if (s1 == s2)
    {
        text = "ничья";
        return;
    }
 
    int result = s1 + s2;
 
    if (result == stoneScissors)
    {
        text = "Ножницы затупились о камень";
    }
    else if (result == stonePaper)
    {
        text = "Бумага завернула камень";
    }
    else if (result == scissorsPaper)
    {
        text = "Ножницы разрезали бумагу";
    }
 
}
но код, я немного поменял на основе этого

Добавлено через 1 минуту
у меня их 300 этих фраз ровно 300. У меня есть другие варианты?

Добавлено через 2 минуты
если можно сделать проще если фраз 300 штук. то скажите у меня в планах выкладывать это в гугл, ибо потом менять код будет тяжеело. 3 это только пример был

Добавлено через 2 минуты
+ еще они вс в разнобой, я не смогу так просто взять прибвить. + будет некорректо отображаться ибо при сложении числа будут повторяться очень часто. поэтому по моему мнение это единственно правильное решение
0
Эксперт по электронике
3073 / 2146 / 548
Регистрация: 04.09.2018
Сообщений: 7,368
Записей в блоге: 4
19.09.2022, 21:56 20
Цитата Сообщение от Bright4092b Посмотреть сообщение
у меня их 300 этих фраз ровно 300. У меня есть другие варианты?
Что показали, то и решили. Что это за 300 фраз? Огласите само задание толком.
Цитата Сообщение от Bright4092b Посмотреть сообщение
по моему мнение это единственно правильное решение
А по моему мнению - это не правильное мнение.
0
19.09.2022, 21:56
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
19.09.2022, 21:56
Помогаю со студенческими работами здесь

Передача большого объема данных (большого количества фотографий)
Есть веб-сервис ASP.NET, который работает по http протоколу. Однако есть необходимость передавать...

Оптимизация большого потока запросов на сервер
Тут такая проблема, клиент не справляется с большим потоком запросов и либо долго думает, либо...

Оптимизация выборки из List большого объема
Всем добрый день! У меня есть список координат, где хранятся больше 100к записей. Я прохожу этот...

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

Оптимизация insert в таблицу большого числа строк (100000 и более)
Добрый день! Задача такая - используя задачу по расписанию (vbs-скрипт) перелить из одной таблицы...

Замена большого количества if
Всем добрый день. Возникла задача разбора текстового файла следующего вида: Советская 23, 23 т...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru