Оптимизировать код

@onlwork · Регистрация: 10.11.2013

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

Доброго времени суток, как можно оптимизировать код что бы он быстрее работал ?

Дана последовательность из n чисел a1, a2,..., an. C ней последовательно производят k операций: на i-ом шаге из этой последовательности одновременно вычеркивают все числа, стоящие на позициях с номерами, кратными bi (позиции нумеруются с 1).

Ваша задача — промоделировать это процесс и вывести массив после выполнения всех операций.

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
#include <iostream>
#include <algorithm>
#include <vector>
using namespace std;
 
int main()
{
 
    freopen("input.txt", "r", stdin);
    freopen("output.txt", "w", stdout);
    int n,w;
    cin>>n>>w;
    vector<int>l;
    //list<int>l;
    int val;
    for(int i(0); i<n; i++)
    {
        scanf("%d",&val);
    
        l.push_back(val);
    }
    
 
        for(int i(0); i<w; i++)
    { int j = 1;
        cin>>val;
        for (vector<int>::iterator itr = l.begin(); itr != l.end();  j++){
            if(j % val == 0){
                itr = l.erase(itr);
            }
            else
                ++itr;
            
        }
 
        
    }
 
 for (vector<int>::iterator itr = l.begin(); itr != l.end();  ++itr)
    cout<<*itr<<" ";
    return 0;
}

Добавлено через 39 секунд
Код работает правильно, но нужно что бы работал быстрее

@Andrej · 23.09.2014, 18:38

onlwork, вы заполняете вектор, а потом удаляете не нужное. Может, легче сразу заполнять вектор нужным? Если правильно понял.

@castaway · 23.09.2014, 18:39

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

@onlwork · 23.09.2014, 18:42 **[ТС]**

входные данные:
2 числа, 1 количество элементов в последовательности, 2 количество операций
2 строка - сама последовательность
3 строка - числа операции
выходные данные
последовательно из которой будут удалены все элементы, чья позиция кратна 2 последовательности

Добавлено через 1 минуту
Точная формулировка задачи

Дана последовательность из n чисел a1, a2,..., an. C ней последовательно производят k операций: на i-ом шаге из этой последовательности одновременно вычеркивают все числа, стоящие на позициях с номерами, кратными bi (позиции нумеруются с 1).

Ваша задача — промоделировать это процесс и вывести массив после выполения всех операций.

@castaway · 23.09.2014, 18:51

Ок. Раз на то пошло...
Не используй контейнеры из STL. Они медленнее чем динамически выделенная память.
Это ускорит твой код. Переписывай, показывай что получилось.

@Andrej · 23.09.2014, 18:53

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

Не используй контейнеры из STL. Они медленнее чем динамически выделенная память.

castaway, как-то холиварно прозвучало =)
Бенчмарки.
Кстати, onlwork, вы как замеряете скорость выполнения? Может вы чего-нибудь сейчас наделаете, а как узнаете что это ускорило выполнение?

MrGluck · 23.09.2014, 18:53

Избавиться от поточного ввода/вывода и использовать printf + scanf

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

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

castaway, как-то холиварно прозвучало =)
Бенчмарки.

Это очевидно всем, мало мальски шарящим в теме.

@onlwork · 23.09.2014, 19:08 **[ТС]**

Andrej, на контесте проверяю и на одном из тесте выдает ошибку, так как программа не укладывается в отведенное ей время для выполнения

Добавлено через 4 минуты
MrGluck, заменил на scanf и printf - не помогло

@Andrej · 23.09.2014, 19:10

onlwork, ок.
1. Ты удаляешь из вектора, а это времязатратная операция.
Можно не удалять из вектора, а отмечать где-нибудь, что элемент не нужен. vector<bool> какой-нить. По типу решета Эрастофена.
2. Вряд ли поможет с временем, но можно сначала узнать длину файла и уже потом захватывать под него память. Т. к. если даже запасённого места в векторе не хватит, он может переаллоцироваться. Или сразу выставить размер вектора.

@onlwork · 23.09.2014, 19:12 **[ТС]**

Заменил vector на list , теперь все нормально, и все хорошо

@SlavaSSU · 23.09.2014, 19:12

onlwork, n и m до 10^5?

@onlwork · 23.09.2014, 19:13 **[ТС]**

неSlavaSSU, не знаю, мне друг задачу дал

Новые блоги и статьи Все статьи Все блоги /
Учебный мини "цифровой двойник" = (Сервер = Java + Flask_питон )+(3D = Гугловскаая 4T nvideo + google colab) +( WMS = Odoo на Docker) anaschu 07.07.2026 Увидел в требованиях к вакансии создателя имитационных моделей связку (REst + Api + WMS + Anylogic + питон + 3D Nvideo), и решил посмотреть, что это такое. Из этого всего я работал более менее. . .	Море в объективе kumehtar 06.07.2026 Хотел написать много проникновенных слов, но передумал. Оно само - говорит. Кто слышит тот слышит.	Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii. dcc0 05.07.2026 Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .	Установка статуса документа по условию Maks 05.07.2026 Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .
Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .	В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .	интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .

@Andrej И целого heap'а мало 96 / 57 / 17 Регистрация: 31.07.2014 Сообщений: 291
	23.09.2014, 18:38
	onlwork, вы заполняете вектор, а потом удаляете не нужное. Может, легче сразу заполнять вектор нужным? Если правильно понял. 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	23.09.2014, 18:39
	Смысл того, что делает код был бы очень кстати. Просто мне, например, вдумываться в него лень, даже в такой простой. Чтобы нам не было лень в него вдумываться, опиши хотя бы задачу. Не поленись, будь так любезен. 0

@onlwork 1 / 1 / 0 Регистрация: 10.11.2013 Сообщений: 53
	23.09.2014, 18:42 [ТС]
	входные данные: 2 числа, 1 количество элементов в последовательности, 2 количество операций 2 строка - сама последовательность 3 строка - числа операции выходные данные последовательно из которой будут удалены все элементы, чья позиция кратна 2 последовательности Добавлено через 1 минуту Точная формулировка задачи Дана последовательность из n чисел a1, a2,..., an. C ней последовательно производят k операций: на i-ом шаге из этой последовательности одновременно вычеркивают все числа, стоящие на позициях с номерами, кратными bi (позиции нумеруются с 1). Ваша задача — промоделировать это процесс и вывести массив после выполения всех операций. 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	23.09.2014, 18:51
	Ок. Раз на то пошло... Не используй контейнеры из STL. Они медленнее чем динамически выделенная память. Это ускорит твой код. Переписывай, показывай что получилось. 0

@onlwork 1 / 1 / 0 Регистрация: 10.11.2013 Сообщений: 53
	23.09.2014, 19:08 [ТС]
	Andrej, на контесте проверяю и на одном из тесте выдает ошибку, так как программа не укладывается в отведенное ей время для выполнения Добавлено через 4 минуты MrGluck, заменил на scanf и printf - не помогло 0

@Andrej И целого heap'а мало 96 / 57 / 17 Регистрация: 31.07.2014 Сообщений: 291
	23.09.2014, 19:10
	onlwork, ок. 1. Ты удаляешь из вектора, а это времязатратная операция. Можно не удалять из вектора, а отмечать где-нибудь, что элемент не нужен. vector<bool> какой-нить. По типу решета Эрастофена. 2. Вряд ли поможет с временем, но можно сначала узнать длину файла и уже потом захватывать под него память. Т. к. если даже запасённого места в векторе не хватит, он может переаллоцироваться. Или сразу выставить размер вектора. 0

@onlwork 1 / 1 / 0 Регистрация: 10.11.2013 Сообщений: 53
	23.09.2014, 19:12 [ТС]
	Заменил vector на list , теперь все нормально, и все хорошо 0

@SlavaSSU 221 / 166 / 47 Регистрация: 17.07.2012 Сообщений: 587
	23.09.2014, 19:12
	onlwork, n и m до 10^5? 0

@onlwork 1 / 1 / 0 Регистрация: 10.11.2013 Сообщений: 53
	23.09.2014, 19:13 [ТС]
	неSlavaSSU, не знаю, мне друг задачу дал 0

Опции темы