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

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

06.07.2024, 17:23. Показов 3630. Ответов 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
6444 / 5639 / 1128
Регистрация: 11.10.2019
Сообщений: 15,003
06.07.2024, 17:38
Herzig00, в саму задачу я не вникал. Но если там в тестах много строк во входном файле, то получается многократное выделение памяти для вектора.

поэтому я бы вектор создал до цикла
C++
1
std::vector<int> corals;
а в 17 строке вот так
C++
1
corals.resize(n);
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12930 / 6798 / 1819
Регистрация: 18.10.2014
Сообщений: 17,198
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
883 / 536 / 228
Регистрация: 21.02.2011
Сообщений: 5,706
10.07.2024, 16:26
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
И это задача не на бинарный поиск. От тут нинафиг не нужен.
Почему? Его можно использовать для определения минимальной высоты воды, при которой количество воды будет не меньше заданного значения x, или я что-то путаю?
Цитата Сообщение от Herzig00 Посмотреть сообщение
Подскажите, пожалуйста, что не так в моём решении, так как слишком долго оно работает для ограничений задачи.
Вы перебираете все возможные высоты для уровня воды от минимальной до максимальной высоты с учетом среднего уровня (middleHeight), т.е. делаете лишние итерации.
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12930 / 6798 / 1819
Регистрация: 18.10.2014
Сообщений: 17,198
10.07.2024, 16:43
Цитата Сообщение от andreyfreelans Посмотреть сообщение
Почему? Его можно использовать для определения минимальной высоты воды
Конечно, можно. Но бинарный поиск (по крайней мере такой, как у ТС) предполагает, что придется снова и снова вычислять объем воды для разных, никак не связанных друг с другом значений высоты. Это вычисление приходится делать в лоб, по определению, что приводит к дикой неэффективности. Если вы найдете способ "мгновенно" определять объем воды по высоте, то тогда, возможно, и такой бинарный поиск будет работать приемлемо.

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

Цитата Сообщение от andreyfreelans Посмотреть сообщение
Вы перебираете все возможные высоты для уровня воды от минимальной до максимальной высоты с учетом среднего уровня (middleHeight), т.е. делаете лишние итерации.
Не понял. О каком "перебираете все возможные высоты" вы ведете речь, если у ТС в коде - бинарный поиск. Бинарный поиск не перебирает "все возможные".
1
 Аватар для andrey_f
883 / 536 / 228
Регистрация: 21.02.2011
Сообщений: 5,706
10.07.2024, 17:36
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
О каком "перебираете все возможные высоты" вы ведете речь
я имел в виду тоже самое, что вы и написали по поводу реализации у ТС.
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
Вы на каждой итерации цикла вычисляете количество воды тупо "по условию", то есть пробеганием по всему кораллу.
0
Заблокирован
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
12930 / 6798 / 1819
Регистрация: 18.10.2014
Сообщений: 17,198
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
3186 / 2220 / 640
Регистрация: 20.07.2013
Сообщений: 6,010
10.07.2024, 23:29
Сумма n по всем наборам входных данных не превышает 2⋅105.
А зачем это условие? Не совсем понятно.


Цитата Сообщение от TheCalligrapher Посмотреть сообщение
Но на самом деле надо было заметить очевидное: ответ задачи не зависит от порядка кораллов в последовательности. Достаточно один раз отсортировать кораллы, и задача решится в один-единственный линейный проход по кораллам.
Можно подробности? Нужно считать суммарную высоту кораллов и исходя из этого вычислять объем воды при некоторой высоте стенок или сразу, с учетом каждого коралла, пошагово (итеративно) "уточняем" высоту стенки и получающийся объем воды?
0
Заблокирован
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
Заблокирован
11.07.2024, 06:54
Цитата Сообщение от gunslinger Посмотреть сообщение
Сумма n по всем наборам входных данных не превышает 2⋅105.
А зачем это условие? Не совсем понятно.
Неудивительно. Я тоже в него не понял. Только теперь.
Авторы из 10минутной задачи сделали цирк.
Это ограничение на количество входных данных.
Если бы оно было доступно описано, то и сортировка не вызывала бы никаких сомнений в её использовании.
То есть, весь цирк сводится к :
У вас есть аквариум из n кораллов (1<=n<=9*104) и x воды (1<=x<=109) ...
И сказочки конец.
0
place status here
 Аватар для gunslinger
3186 / 2220 / 640
Регистрация: 20.07.2013
Сообщений: 6,010
11.07.2024, 10:42
При этом отдельное (максимальное) значение n тоже не может превышать величины 2⋅105.
Может это какая-то попытка запутать или вовсе не стоит особо на данный момент обращать внимание.
Короче, как всегда.
0
Заблокирован
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
3186 / 2220 / 640
Регистрация: 20.07.2013
Сообщений: 6,010
11.07.2024, 17:21
Цитата Сообщение от SmallEvil Посмотреть сообщение
Мне уже стало интересно кто такую шизу им пишет, не ИИ ли ?
А хз.

Ну и если тест (набор данных) один (t = 1), то nmax может быть равно тому же наиб. значению 2⋅105.
0
Заблокирован
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
Ответ Создать тему
Новые блоги и статьи
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru