Оптимизация скорости кода

@ITSSS · Регистрация: 05.10.2018

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

Необходимо(дать совет по оптимизации) оптимизировать код, чтоб меньше времени занимало на обработку данных.
Подробнее о задаче можно узнать в прикрепленной пикче ниже:
Код:

C#

using System;
 
public class Program
{
    public static void Main(string[] args)
    {
        long i = long.Parse(Console.ReadLine());
        long min = 0;
        long max = 0;
        long count = 1;
        long first = 0;
        long second = 0;
        long result = i;
        if (i <= 6)
        {
            switch (i)
            {
                case 6:
                    min = 1;
                    max = 6 * 6;
                    break;
                case 5:
                    min = 2;
                    max = 5 * 6;
                    break;
                case 4:
                    min = 3;
                    max = 4 * 6;
                    break;
                case 3:
                    min = 4;
                    max = 3 * 6;
                    break;
                case 2:
                    min = 5;
                    max = 2 * 6;
                    break;
                case 1:
                    min = 6;
                    max = 6;
                    break;
            }
            if (i <= 6)
            {
                Console.WriteLine(min + " " + max);
            }
        }
        else if (i % 1 == 0)
        {
            count = 0;
            while (i >= 6)
            {
                i -= 6;
                count++;
            }
            switch (i)
            {
                case 5:
                    first = count + 2;
                    second = result * 6;
                    break;
                case 4:
                    first = count + 3;
                    second = result * 6;
                    break;
                case 3:
                    first = count + 4;
                    second = result * 6;
                    break;
                case 2:
                    first = count + 5;
                    second = result * 6;
                    break;
                case 1:
                    first = count + 6;
                    second = result * 6;
                    break;
                case 0:
                    first = count;
                    second = result * 6;
                    break;
            }
            Console.WriteLine(first + " " + second);
        }
    }
}

@Почтальон · 31.10.2018, 14:47

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

Подробнее о задаче можно узнать в прикрепленной пикче ниже:

потрудитесь перепечатать текст с этой кикчи

@kolorotur · 31.10.2018, 14:47

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

Минимальное количество кубиков для результата N первого игрока должно быть $https://www.cyberforum.ru/cgi-bin/latex.cgi?\lceil{\frac{N}{6}}\rceil$ , а максимальное количество кубиков — N (все выпали на единицу).

Тогда получается, что минимальный результат, который мог получить второй игрок — $https://www.cyberforum.ru/cgi-bin/latex.cgi?\lceil{\frac{N}{6}}\rceil$ (минимально возможное количество кубиков, которые все выпали на единицу), а максимальный результат — N * 6 (максимально возможное количество кубиков, которые первому игроку все выпали на 1, а второму игроку могли бы выпасть на 6).

@Wolfdp · 31.10.2018, 14:59

либо я не вкурил задачу, либо всеже все нямного проще...

C#

var i = long.Parse(Console.ReadLine());
var max = i * 6;
var min = i / 6 + (7 - i % 6);
Console.WriteLine($"max = {max}; min = {min}")

Добавлено через 50 секунд

Не по теме:

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

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

о, значит таки не показалось...

@ITSSS · 31.10.2018, 15:20 **[ТС]**

либо я не вкурил задачу, либо всеже все нямного проще...

Не, она сложнее, при вводе 36, получается 13 и 216

@Wolfdp · 31.10.2018, 16:25

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

получается 13

ай, не учел ноль...

C#

var i = long.Parse(Console.ReadLine());
var max = i * 6;
var a = i % 6;
var min = i / 6 + (a != 0 ? 7 - a : 0);
Console.WriteLine($"max = {max}; min = {min}");

@ITSSS · 31.10.2018, 16:47 **[ТС]**

Спасибо, у вас хороший вариант, но проблема заключается в том, что числа большие, и при решении задачи существует лимит обработки равной 1-ой секунде. И если такой код использовать, то он занимает полторы секунды(что больше положенного).
Я оптимизировал свой код, который успешно прошел условия по времени:

C#

using System;
 
public class Program
{
    public static void Main(string[] args)
    {
        long i = long.Parse(Console.ReadLine());
        long max = i * 6;
        long min = 0;
        long k = i;
        long count = 0;
        if(i <= 6)
        {
            switch (i)
            {
                case 6:
                    min = 1;
                    break;
                case 5:
                    min = 2;
                    break;
                case 4:
                    min = 3;
                    break;
                case 3:
                    min = 4;
                    break;
                case 2:
                    min = 5;
                    break;
                case 1:
                    min = 6;
                    break;
            }
        } else
        {
            for(; i > 6; i = i - 6)
            {
                if(i > 6000)
                {
                    i -= 6000;
                    count += 1000;
                }
                count++;
            }
            if (i <= 6)
            {
                switch (i)
                {
                    case 6:
                        Console.WriteLine((count + 1) + " " + max);
                        break;
                    case 5:
                        Console.WriteLine((count + 2) + " " + max);
                        break;
                    case 4:
                        Console.WriteLine((count + 3) + " " + max);
                        break;
                    case 3:
                        Console.WriteLine((count + 4) + " " + max);
                        break;
                    case 2:
                        Console.WriteLine((count + 5) + " " + max);
                        break;
                    case 1:
                        Console.WriteLine((count + 6) + " " + max);
                        break;
                    case 0:
                        Console.WriteLine((count + i) + " " + max);
                        break;
                }
            }
        }
        if(k <= 6)
        {
            Console.WriteLine(min + " " + max);
        }
    }
}

@Wolfdp · 31.10.2018, 17:21

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

И если такой код использовать, то он занимает полторы секунды(что больше положенного).

Такое впечатление что вы мой код еще зачемто как-то "расширяли". Замерял свой (вверху) и ваш пример

Код

max = 12884901882; min = 357913947
00:00:00.0008771
357913947 12884901882
00:00:00.0020140
Press any key to continue . . .

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

@kolorotur · 01.11.2018, 09:21

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

при вводе 36, получается 13 и 216

13-то откуда?
Это в примерах дано или вы сами как-то рассчитали?
Чтобы первый игрок получил 36, ему нужно бросить как минимум 6 кубиков, да так, чтобы они упали на 6.
Это значит, что минимум второго игрока будет 6, но никак не 13.

@ITSSS · 02.11.2018, 11:41 **[ТС]**

Ошибка в коде была

Оптимизация скорости кода

Решение

Решение

Решение

@Почтальон управление сложностью 1693 / 1306 / 259 Регистрация: 22.03.2015 Сообщений: 7,545 Записей в блоге: 5
	31.10.2018, 14:47	2
	Сообщение от ITSSS Подробнее о задаче можно узнать в прикрепленной пикче ниже: потрудитесь перепечатать текст с этой кикчи 0

@kolorotur 17791 / 12942 / 3381 Регистрация: 17.09.2011 Сообщений: 21,219
	31.10.2018, 14:47	3
	Сообщение было отмечено ITSSS как решение Решение Может, я как-то не так понял задание, но если количество бросаемых кубиков неизвестно, то отталкиваться нужно от этого. Минимальное количество кубиков для результата N первого игрока должно быть $https://www.cyberforum.ru/cgi-bin/latex.cgi?\lceil{\frac{N}{6}}\rceil$ , а максимальное количество кубиков — N (все выпали на единицу). Тогда получается, что минимальный результат, который мог получить второй игрок — $https://www.cyberforum.ru/cgi-bin/latex.cgi?\lceil{\frac{N}{6}}\rceil$ (минимально возможное количество кубиков, которые все выпали на единицу), а максимальный результат — N * 6 (максимально возможное количество кубиков, которые первому игроку все выпали на 1, а второму игроку могли бы выпасть на 6). 1

@ITSSS 0 / 0 / 0 Регистрация: 05.10.2018 Сообщений: 27
	31.10.2018, 15:20 [ТС]	5
	либо я не вкурил задачу, либо всеже все нямного проще... Не, она сложнее, при вводе 36, получается 13 и 216 0

@Wolfdp 1968 / 1427 / 337 Регистрация: 15.06.2012 Сообщений: 5,357 Записей в блоге: 3
	31.10.2018, 17:21	8
	Сообщение было отмечено ITSSS как решение Решение Сообщение от ITSSS И если такой код использовать, то он занимает полторы секунды(что больше положенного). Такое впечатление что вы мой код еще зачемто как-то "расширяли". Замерял свой (вверху) и ваш пример Код max = 12884901882; min = 357913947 00:00:00.0008771 357913947 12884901882 00:00:00.0020140 Press any key to continue . . . учитывая что ваш код базируется на проходе циклом, он априори будет медленее для больших чисел 1

@kolorotur 17791 / 12942 / 3381 Регистрация: 17.09.2011 Сообщений: 21,219
	01.11.2018, 09:21	9
	Сообщение от ITSSS при вводе 36, получается 13 и 216 13-то откуда? Это в примерах дано или вы сами как-то рассчитали? Чтобы первый игрок получил 36, ему нужно бросить как минимум 6 кубиков, да так, чтобы они упали на 6. Это значит, что минимум второго игрока будет 6, но никак не 13. 1

@ITSSS 0 / 0 / 0 Регистрация: 05.10.2018 Сообщений: 27
	02.11.2018, 11:41 [ТС]	10
	Ошибка в коде была 0