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

@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, не знаю, мне друг задачу дал

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .	SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .	SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .

@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