Форум программистов, компьютерный форум, киберфорум
C++
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.56/18: Рейтинг темы: голосов - 18, средняя оценка - 4.56
0 / 0 / 0
Регистрация: 10.07.2023
Сообщений: 14

Что не так с кодом для задачи "Построение аквариума"? Слишком долго работает

06.07.2024, 17:23. Показов 3658. Ответов 16

Студворк — интернет-сервис помощи студентам
Решал задачу: https://codeforces.com/contest/1873/problem/E.
Подскажите, пожалуйста, что не так в моём решении, так как слишком долго оно работает для ограничений задачи.
Код:
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
#include <iostream>
#include <vector>
 
int main()
{
    std::ios_base::sync_with_stdio(false);
    std::cin.tie();
 
    int t;
    std::cin >> t;
 
    while (t--)
    {
        int n, x;
        std::cin >> n >> x;
 
        std::vector<int> corals(n);
        for (int i = 0; i < n; ++i)
        {
            std::cin >> corals[i];
        }
 
        int maxHeightCoral = -1000000005;
        for (int i = 0; i < n; ++i)
        {
            if (corals[i] > maxHeightCoral) maxHeightCoral = corals[i];
        }
 
        int leftBorder = 1, rightBorder = maxHeightCoral + x / n ;
        while (leftBorder <= rightBorder)
        {
            int middleHeight = (leftBorder + rightBorder) / 2, waterQuantity = 0;
            for (int i = 0; i < n; ++i)
            {
                if (middleHeight > corals[i]) waterQuantity += middleHeight - corals[i];
            }
 
            if (waterQuantity > x) rightBorder = middleHeight - 1;
            else leftBorder = middleHeight + 1;
        }
        
        std::cout << leftBorder - 1 << std::endl;
    }
 
    return 0;
}
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
06.07.2024, 17:23
Ответы с готовыми решениями:

Что с моим кодом не так?Он работает
Из несколько примеров собрал свой) Я в JS ноль)! Я чувствую что-то с ним не так) $(document).ready(function(){ ...

Что не так с кодом? Выглядит круто,но работает плохо
Вот задание:. Создать класс матрица Данный класс содержит указатель на long, размер строк и столбцов и состояние ошибки. Определить...

Что не так с кодом? Компилируется, но не работает (связные списки)
#include &quot;stdio.h&quot; #include &quot;stdlib.h&quot; #include &lt;string.h&gt; #include &quot;iostream&quot; using namespace std; class Elem { ...

16
фрилансер
 Аватар для Алексей1153
6465 / 5678 / 1131
Регистрация: 11.10.2019
Сообщений: 15,119
06.07.2024, 17:38
Herzig00, в саму задачу я не вникал. Но если там в тестах много строк во входном файле, то получается многократное выделение памяти для вектора.

поэтому я бы вектор создал до цикла
C++
1
std::vector<int> corals;
а в 17 строке вот так
C++
1
corals.resize(n);
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
13177 / 6813 / 1821
Регистрация: 18.10.2014
Сообщений: 17,238
09.07.2024, 20:19
Цитата Сообщение от Herzig00 Посмотреть сообщение
C++
1
2
    std::ios_base::sync_with_stdio(false);
    std::cin.tie();
К чему это здесь?

Цитата Сообщение от Herzig00 Посмотреть сообщение
Подскажите, пожалуйста, что не так в моём решении,
Вот это:

Цитата Сообщение от Herzig00 Посмотреть сообщение
C++
33
34
35
36
            for (int i = 0; i < n; ++i)
            {
                if (middleHeight > corals[i]) waterQuantity += middleHeight - corals[i];
            }
Вы на каждой итерации цикла вычисляете количество воды тупо "по условию", то есть пробеганием по всему кораллу. Разумеется, все будет дико тормозить.

И это задача не на бинарный поиск. От тут нинафиг не нужен.
1
 Аватар для andrey_f
884 / 537 / 228
Регистрация: 21.02.2011
Сообщений: 5,705
10.07.2024, 16:26
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
И это задача не на бинарный поиск. От тут нинафиг не нужен.
Почему? Его можно использовать для определения минимальной высоты воды, при которой количество воды будет не меньше заданного значения x, или я что-то путаю?
Цитата Сообщение от Herzig00 Посмотреть сообщение
Подскажите, пожалуйста, что не так в моём решении, так как слишком долго оно работает для ограничений задачи.
Вы перебираете все возможные высоты для уровня воды от минимальной до максимальной высоты с учетом среднего уровня (middleHeight), т.е. делаете лишние итерации.
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
13177 / 6813 / 1821
Регистрация: 18.10.2014
Сообщений: 17,238
10.07.2024, 16:43
Цитата Сообщение от andreyfreelans Посмотреть сообщение
Почему? Его можно использовать для определения минимальной высоты воды
Конечно, можно. Но бинарный поиск (по крайней мере такой, как у ТС) предполагает, что придется снова и снова вычислять объем воды для разных, никак не связанных друг с другом значений высоты. Это вычисление приходится делать в лоб, по определению, что приводит к дикой неэффективности. Если вы найдете способ "мгновенно" определять объем воды по высоте, то тогда, возможно, и такой бинарный поиск будет работать приемлемо.

Но на самом деле надо было заметить очевидное: ответ задачи не зависит от порядка кораллов в последовательности. Достаточно один раз отсортировать кораллы, и задача решится в один-единственный линейный проход по кораллам.

Цитата Сообщение от andreyfreelans Посмотреть сообщение
Вы перебираете все возможные высоты для уровня воды от минимальной до максимальной высоты с учетом среднего уровня (middleHeight), т.е. делаете лишние итерации.
Не понял. О каком "перебираете все возможные высоты" вы ведете речь, если у ТС в коде - бинарный поиск. Бинарный поиск не перебирает "все возможные".
1
 Аватар для andrey_f
884 / 537 / 228
Регистрация: 21.02.2011
Сообщений: 5,705
10.07.2024, 17:36
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
О каком "перебираете все возможные высоты" вы ведете речь
я имел в виду тоже самое, что вы и написали по поводу реализации у ТС.
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
Вы на каждой итерации цикла вычисляете количество воды тупо "по условию", то есть пробеганием по всему кораллу.
0
 Аватар для SmallEvil
4086 / 2975 / 813
Регистрация: 29.06.2020
Сообщений: 11,000
10.07.2024, 22:49
На отсортированных с отловлей блох (переполнения) :
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
#include <iostream>
#include <map>
 
int process(int n, int x){
    std::map<int, int> nis;
    int ni;
    for(int i = 0; i!= n; ++i){
        std::cin >> ni; 
        ++nis[ni];
    }
    auto first = nis.begin(), second = first;
    int h = first->first;
    int level_width = 0;
    do{
        second = std::next(first);
        if (second == nis.end())
            break;
 
        level_width += first->second;
        if (x >= level_width){
            if ( x >= (second->first - first->first) * (unsigned long long)level_width){
                h += second->first - first->first;
                x -= (second->first - first->first) * level_width;
            }else{
                h += x / level_width;
                return h;
            }
        }else{
            return h;
        }
        first = second;
    }while(true);
    return h + x / n;   
}
 
int main(){
    int k;
    std::cin >> k;
    while(k--){
        int n, x;
        std::cin >> n >> x;
        std::cout << process(n, x) << std::endl;
    }
 
}
Убрал артефакты со старого кода.
1
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
13177 / 6813 / 1821
Регистрация: 18.10.2014
Сообщений: 17,238
10.07.2024, 23:26
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
#include <cassert>
#include <vector>
#include <algorithm>
#include <iostream>
 
int main()
{
  using Corals = std::vector<unsigned>;
 
  unsigned t = 0;
  std::cin >> t;
 
  for (; t > 0; --t)
  {
    unsigned n = 0, x = 0;
    std::cin >> n >> x;
    assert(n > 0 && x > 0);
 
    Corals corals(n);
    for (unsigned &c : corals)
      std::cin >> c;
 
    std::sort(corals.begin(), corals.end());
    corals.push_back(-1u);
 
    unsigned wall = 0;
    for (Corals::const_iterator it = corals.begin(), it_next; ; it = it_next)
    {
      it_next = std::find_if(it, corals.cend(), [it](unsigned c) { return c != *it; });
      assert(it_next != corals.end() && *it_next > *it);
 
      unsigned w = it_next - corals.begin(), h = *it_next - *it;
      unsigned long long a = (unsigned long long) w * h;
 
      if (a >= x)
      {
        wall = *it + x / w;
        break;
      }
 
      x -= a;
    } 
 
    std::cout << wall << std::endl;
  }
}
https://coliru.stacked-crooked... dc3ce3c7d7
1
place status here
 Аватар для gunslinger
3190 / 2227 / 640
Регистрация: 20.07.2013
Сообщений: 6,023
10.07.2024, 23:29
Сумма n по всем наборам входных данных не превышает 2⋅105.
А зачем это условие? Не совсем понятно.


Цитата Сообщение от TheCalligrapher Посмотреть сообщение
Но на самом деле надо было заметить очевидное: ответ задачи не зависит от порядка кораллов в последовательности. Достаточно один раз отсортировать кораллы, и задача решится в один-единственный линейный проход по кораллам.
Можно подробности? Нужно считать суммарную высоту кораллов и исходя из этого вычислять объем воды при некоторой высоте стенок или сразу, с учетом каждого коралла, пошагово (итеративно) "уточняем" высоту стенки и получающийся объем воды?
0
 Аватар для SmallEvil
4086 / 2975 / 813
Регистрация: 29.06.2020
Сообщений: 11,000
10.07.2024, 23:38
на каждом уровне пытаемся заполнить весь объем до след уровня, при этом ширина уровня повышается на уже заполненную.
на просто примере :
Входные данные одного тестового примера :
9 10
1 9 2 2 9 1 3 1 3
=================
w - "ширина" пласта
два числа, высота коралла и сколько их есть такой высоты, в скобках остаток воды :
w = 3
1 - 3 | (7)
w = 5
2 - 2 | (2)
w = 7
3 - 2 |
w = 9
9 - 2
1
 Аватар для SmallEvil
4086 / 2975 / 813
Регистрация: 29.06.2020
Сообщений: 11,000
11.07.2024, 06:54
Цитата Сообщение от gunslinger Посмотреть сообщение
Сумма n по всем наборам входных данных не превышает 2⋅105.
А зачем это условие? Не совсем понятно.
Неудивительно. Я тоже в него не понял. Только теперь.
Авторы из 10минутной задачи сделали цирк.
Это ограничение на количество входных данных.
Если бы оно было доступно описано, то и сортировка не вызывала бы никаких сомнений в её использовании.
То есть, весь цирк сводится к :
У вас есть аквариум из n кораллов (1<=n<=9*104) и x воды (1<=x<=109) ...
И сказочки конец.
0
place status here
 Аватар для gunslinger
3190 / 2227 / 640
Регистрация: 20.07.2013
Сообщений: 6,023
11.07.2024, 10:42
При этом отдельное (максимальное) значение n тоже не может превышать величины 2⋅105.
Может это какая-то попытка запутать или вовсе не стоит особо на данный момент обращать внимание.
Короче, как всегда.
0
 Аватар для SmallEvil
4086 / 2975 / 813
Регистрация: 29.06.2020
Сообщений: 11,000
11.07.2024, 14:11
Цитата Сообщение от gunslinger Посмотреть сообщение
Может это какая-то попытка запутать или вовсе не стоит особо на данный момент обращать внимание.
Стоит обращать. Если не обращать, как я, то сортировка кажется слишком затратной.
Цитата Сообщение от SmallEvil Посмотреть сообщение
(1<=n<=9*104)
Это значение n для одного теста я взял не спроста, а из того же ограничения.
У нас есть t тестов (1<=t<=104).
Рассмотрим случай при максимальном t и максимальном n для всех тестов.
Средний случай : n / t = 105/104 = 10
То есть, 10 кораллов в каждом тесте.
Теперь рассмотрим предельный случай, где в о всех тестам (кроме одного) ровно по 1 кораллу.
В оставшемся получим : 105 - (104 - 1) = 90001 ~ 9*104

Можно было просто сделать задачу на тест где n <= 105 или 9*104 и не клепать ученикам мозг.

Добавлено через 44 секунды
Цитата Сообщение от gunslinger Посмотреть сообщение
Короче, как всегда.
Как всегда на подобных ресурсах.
Мне уже стало интересно кто такую шизу им пишет, не ИИ ли ?
0
place status here
 Аватар для gunslinger
3190 / 2227 / 640
Регистрация: 20.07.2013
Сообщений: 6,023
11.07.2024, 17:21
Цитата Сообщение от SmallEvil Посмотреть сообщение
Мне уже стало интересно кто такую шизу им пишет, не ИИ ли ?
А хз.

Ну и если тест (набор данных) один (t = 1), то nmax может быть равно тому же наиб. значению 2⋅105.
0
 Аватар для SmallEvil
4086 / 2975 / 813
Регистрация: 29.06.2020
Сообщений: 11,000
11.07.2024, 22:05
Цитата Сообщение от gunslinger Посмотреть сообщение
Ну и если тест (набор данных) один (t = 1), то nmax может быть равно тому же наиб. значению 2⋅105.
Да.
Ну так вот, вот и кот там облезлый, вместо внятного задания ),
0
14.07.2024, 06:51

Не по теме:

Меня что-то от жары пробило на ностальгию (старею, ёшкин кот; детство-юность вспоминается, етить её мать):

0
0 / 0 / 0
Регистрация: 10.07.2023
Сообщений: 14
11.08.2024, 13:53  [ТС]
Прошёл месяц, у меня к сожалению не было возможности выйти в интернет. Так вот. при ограничениях задачи бинарный поиск очень даже заходит. Оказалось, это сайт тупил, выдавал ошибку по времени, хотя ошибка была в типе данных, напихал везде ll, и зашло. Но мне интересно не это. Как бы я не пытался понять код TheCalligrapher, SmallEvil, не смог. Дело в том, что я ещё не настолько силён в плюсах, как вы, и разобрать смесь цикла через итераторы и вещей по типу assert'а не представляется возможным. Могли бы вы объяснить, пожалуйста, свой код, так как даже этот линейный проход для меня не очень понятный. Мне идея даже не совсем понятна, ну отсортировал я, а дальше что, не понятно.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
11.08.2024, 13:53
Помогаю со студенческими работами здесь

Написал heap sort. Работает верно, но почему-то долго. Скажите, что я сделал не так
def heap_sort(nums): def max_num(i): # сравниваем числа и большее делаем родительским l, r, par = i - 1, i, (i-1)//2 ...

А это правда, что HDD Regenerator так долго работает на ноутбуках с жёсткими дисками
правда ли что он так долго сканит диски

Usb hdd слишком долго определяется. но работает
Доброго дня всем. В один прекрасный день диск (seagate srd00f1) стал определяться слишком долго, около 15-20 минут точно. Обратил...

Слишком сложно для моего неотесанного ума. Подскажите что же я делаю не так?
https://www.cyberforum.ru/php-beginners/thread2157992.html Столкнулся с точно такой же ошибкой как данный человек, после того как...

Не могу понять что не так с кодом (сжатие кодом Хэмминга)
Написали код по заданию &quot;Считать картинку bmp по пикселям: R 8 бит, G 8 бит, B 8 бит. Старший бит в каждом байте оставить без...


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

Или воспользуйтесь поиском по форуму:
17
Ответ Создать тему
Новые блоги и статьи
1С: Контроль уникальности заводского номера
Maks 23.03.2026
Алгоритм контроля уникальности заводского (или серийного) номера на примере документа выдачи шин для спецтехники с табличной частью. Данные берутся из регистра сведений, по которому настроено. . .
Хочу заставить корпорации вкладываться в здоровье сотрудников: делаю мат модель здравосохранения
anaschu 22.03.2026
e7EYtONaj8Y Z4Tv2zpXVVo https:/ / github. com/ shumilovas/ med2. git
1С: Программный отбор элементов справочника по группе
Maks 22.03.2026
Установка программного отбора элементов справочника "Номенклатура" из модуля формы документа. В качестве фильтра для отбора справочника служит группа номенклатуры. Отбор по наименованию группы. . .
Как я обхитрил таблицу Word
Alexander-7 21.03.2026
Когда мигает курсор у внешнего края таблицы, и нам надо перейти на новую строку, а при нажатии Enter создается новый ряд таблицы с ячейками, то мы вместо нервных нажатий Энтеров мы пишем любые буквы. . .
Krabik - рыболовный бот для WoW 3.3.5a
AmbA 21.03.2026
без регистрации и смс. Это не торговля, приложение не содержит рекламы. Выполняет свою непосредственную задачу - автоматизацию рыбалки в WoW - и ничего более. Однако если админы будут против -. . .
1С: Программный отбор элементов справочника по значению перечисления
Maks 21.03.2026
Установка программного отбора элементов справочника "Сотрудники" из модуля формы документа. В качестве фильтра для отбора служит значение перечислений. / / Событие "НачалоВыбора" реквизита на форме. . .
Переходник USB-CAN-GPIO
Eddy_Em 20.03.2026
Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .
Оттенки серого
Argus19 18.03.2026
Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru