Форум программистов, компьютерный форум, киберфорум
C# для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 5.00/3: Рейтинг темы: голосов - 3, средняя оценка - 5.00
bumbaram
47 / 47 / 9
Регистрация: 13.02.2012
Сообщений: 176
1

Покритиковать код, объяснить, где здесь есть потенциальные возможности для ускорения

21.03.2012, 09:15. Просмотров 592. Ответов 2
Метки нет (Все метки)

Добрый день.
Прошу вас покритиковать код, объяснить где здесь есть потенциальные возможности для ускорения.
Просто решение не укладывается на 0.046 от установленного лимита по времени.
задача
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
using System;
 
namespace ConsoleApplication1
{
    class Program
    {
        static int Main()
        {
            int i,j, n = int.Parse(Console.ReadLine());
            int[] tmp = new int[n];
            for (i = 0; i < n; i++)
            {
                tmp[i] = int.Parse(Console.ReadLine()) - 1;
            }
            for (i = 0; i < n; i++)
            {
                j = 1;
                while (tmp[i] > 0)
                {
                    tmp[i] -= 1 * j;
                    j++;
                }
                if (tmp[i] < 0)
                {
                    Console.Write("0 ");
                }
                else
                {
                    Console.Write("1 ");
                }
            }
            //Console.Read();
            return 0;
        }
    }
}
0
QA
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
21.03.2012, 09:15
Ответы с готовыми решениями:

Есть ли в Word возможности для ускорения набора наподобие Т9?
Доброго времени суток! Подскажите, есть ли возможность включить в Word словарь типа Т9. Может есть...

Ищу игру, где есть меч, лук, магия, разнообразная система прокачки и квестов, всяческие возможности для лута
Скучно стало. Хочу поиграть в какую нибудь MMORPG. Что нибудь где есть меч, лук, магия,...

JS код для сбора статистики о посетителях на сайте - есть ли готовый где-то?
Пишу систему сбора статистики о посетителях сайта (эдакий продвинутый счетчик посещений), но весьма...

Здесь есть какие нибудь ошибки ? и как здесь получается в ответе 6.25?
program r2; var b:real; begin b:=100; repeat b:=b/2; until b&lt;10; writeln (b:0:2); end. ...

2
SharpDeveloper
137 / 134 / 22
Регистрация: 16.02.2012
Сообщений: 453
21.03.2012, 11:56 2
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
            int k;
            for (i = 0; i < n; i++)
            {
                j = 1;
                k = tmp[i];
                while (k > 0)
                {
                    k -= /*1 * */j;
                    j++;
                }
                tmp[i] = k;
                //вывод в консоль лучше вынести из этого цикла в другой цикл, 
                //который будет отвечать за вывод
                if (k < 0)
                {
                    Console.Write("0 ");
                }
                else
                {
                    Console.Write("1 ");
                }
            }
0
kolorotur
Эксперт .NET
13263 / 10293 / 2675
Регистрация: 17.09.2011
Сообщений: 17,482
Завершенные тесты: 1
21.03.2012, 19:39 3
bumbaram, с вашего позволения, покритикую.

Во-первых, у вас в коде слишком много циклов:
1. Считывание всех элементов из консоли.
2. Обработка каждого элемента.

Работа с вводом/выводом - самая тормозящая приложение часть, так что объедините лучше первые два цикла в один: считали строчку, обработали, - читаем следующую.

То же самое касается вывода. Если на каждой итерации выводить результат, то получится жутко тормознуто. Лучше выстраивайте строку по мере обработки данных, а уже после обработки выведите ее в консоль одним вызовом Console.WriteLine.

Сама часть, где определяется, располагается ли на данной позиции единица - тоже далеко не идеальна. В частности, скорость выполнения этого куска зависит от величины значения.

А вот если вы присмотритесь по-внимательнее к условию, то увидите там вполне конкретную формулу, по которой высчитывается наличие единицы или нуля на данной позиции.

Исправьте приведенные замечания и скорость обработки списка из 65535-и элементов, где каждый элемент равен https://www.cyberforum.ru/cgi-bin/latex.cgi?{2}^{31}-1 (худший случай по условию задачи), упадет с секунды до ~130-150мсек (это на моем стареньком P4).
0
21.03.2012, 19:39
Answers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
21.03.2012, 19:39

Заказываю контрольные, курсовые, дипломные и любые другие студенческие работы здесь.

У меня есть код , кто может объяснить?
Всем доброго времени суток! У меня есть код можете его проверить ? и объяснить каждую строку ...

Есть следующий код как он работает, кто может объяснить подробнее
Сам я дельфист. Знатоки С++ есть массив следующий: const uint8_t Font = { { 0x00, 0x00,...

МОЖНА ЛИ ЗДЕСЬ ПРОСИТ НАПИСАТЬ КОД ЗА ДЕНЬГИ ? ИЛИ ГДЕ МОЖНА СДЕЛАТЬ ЭТО?
МОЖНА ЛИ ЗДЕСЬ ПРОСИТ НАПИСАТЬ КОД ЗА ДЕНЬГИ ? ИЛИ ГДЕ МОЖНА СДЕЛАТЬ ЭТО? Добавлено через 1...

Функция пика в последовательности - оптимизировать код для ускорения его работы
Я написала функцию для поиска пика, но программа выполняется примерно 14 сек. Как возможно...


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

Или воспользуйтесь поиском по форуму:
3
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2020, vBulletin Solutions, Inc.