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

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

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

Author24 — интернет-сервис помощи студентам
Решал задачу: 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
6327 / 5458 / 1108
Регистрация: 11.10.2019
Сообщений: 14,518
06.07.2024, 17:38 2
Herzig00, в саму задачу я не вникал. Но если там в тестах много строк во входном файле, то получается многократное выделение памяти для вектора.

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

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

Цитата Сообщение от andreyfreelans Посмотреть сообщение
Вы перебираете все возможные высоты для уровня воды от минимальной до максимальной высоты с учетом среднего уровня (middleHeight), т.е. делаете лишние итерации.
Не понял. О каком "перебираете все возможные высоты" вы ведете речь, если у ТС в коде - бинарный поиск. Бинарный поиск не перебирает "все возможные".
1
 Аватар для andrey_f
596 / 379 / 184
Регистрация: 21.02.2011
Сообщений: 5,130
10.07.2024, 17:36 6
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
О каком "перебираете все возможные высоты" вы ведете речь
я имел в виду тоже самое, что вы и написали по поводу реализации у ТС.
Цитата Сообщение от TheCalligrapher Посмотреть сообщение
Вы на каждой итерации цикла вычисляете количество воды тупо "по условию", то есть пробеганием по всему кораллу.
0
 Аватар для SmallEvil
3782 / 2708 / 765
Регистрация: 29.06.2020
Сообщений: 9,954
10.07.2024, 22:49 7
На отсортированных с отловлей блох (переполнения) :
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
12856 / 6722 / 1806
Регистрация: 18.10.2014
Сообщений: 17,020
10.07.2024, 23:26 8
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
случайный прохожий
 Аватар для gunslinger
3163 / 2167 / 636
Регистрация: 20.07.2013
Сообщений: 5,815
10.07.2024, 23:29 9
Сумма n по всем наборам входных данных не превышает 2⋅105.
А зачем это условие? Не совсем понятно.


Цитата Сообщение от TheCalligrapher Посмотреть сообщение
Но на самом деле надо было заметить очевидное: ответ задачи не зависит от порядка кораллов в последовательности. Достаточно один раз отсортировать кораллы, и задача решится в один-единственный линейный проход по кораллам.
Можно подробности? Нужно считать суммарную высоту кораллов и исходя из этого вычислять объем воды при некоторой высоте стенок или сразу, с учетом каждого коралла, пошагово (итеративно) "уточняем" высоту стенки и получающийся объем воды?
0
 Аватар для SmallEvil
3782 / 2708 / 765
Регистрация: 29.06.2020
Сообщений: 9,954
10.07.2024, 23:38 10
на каждом уровне пытаемся заполнить весь объем до след уровня, при этом ширина уровня повышается на уже заполненную.
на просто примере :
Входные данные одного тестового примера :
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
3782 / 2708 / 765
Регистрация: 29.06.2020
Сообщений: 9,954
11.07.2024, 06:54 11
Цитата Сообщение от gunslinger Посмотреть сообщение
Сумма n по всем наборам входных данных не превышает 2⋅105.
А зачем это условие? Не совсем понятно.
Неудивительно. Я тоже в него не понял. Только теперь.
Авторы из 10минутной задачи сделали цирк.
Это ограничение на количество входных данных.
Если бы оно было доступно описано, то и сортировка не вызывала бы никаких сомнений в её использовании.
То есть, весь цирк сводится к :
У вас есть аквариум из n кораллов (1<=n<=9*104) и x воды (1<=x<=109) ...
И сказочки конец.
0
случайный прохожий
 Аватар для gunslinger
3163 / 2167 / 636
Регистрация: 20.07.2013
Сообщений: 5,815
11.07.2024, 10:42 12
При этом отдельное (максимальное) значение n тоже не может превышать величины 2⋅105.
Может это какая-то попытка запутать или вовсе не стоит особо на данный момент обращать внимание.
Короче, как всегда.
0
 Аватар для SmallEvil
3782 / 2708 / 765
Регистрация: 29.06.2020
Сообщений: 9,954
11.07.2024, 14:11 13
Цитата Сообщение от 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
случайный прохожий
 Аватар для gunslinger
3163 / 2167 / 636
Регистрация: 20.07.2013
Сообщений: 5,815
11.07.2024, 17:21 14
Цитата Сообщение от SmallEvil Посмотреть сообщение
Мне уже стало интересно кто такую шизу им пишет, не ИИ ли ?
А хз.

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

Не по теме:

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

0
0 / 0 / 0
Регистрация: 10.07.2023
Сообщений: 14
11.08.2024, 13:53  [ТС] 17
Прошёл месяц, у меня к сожалению не было возможности выйти в интернет. Так вот. при ограничениях задачи бинарный поиск очень даже заходит. Оказалось, это сайт тупил, выдавал ошибку по времени, хотя ошибка была в типе данных, напихал везде 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
Ответ Создать тему

Редактор формул (кликните на картинку в правом углу, чтобы закрыть)
Новые блоги и статьи
Что нового в C# 14
UnmanagedCoder 10.03.2025
Предстоящая версия C# 14 обещает принести изменения, которые сделают разработку еще более приятной и эффективной. Что стоит отметить, так это влияние сообщества разработчиков на формирование новых. . .
Формулы поворота
Igor3D 10.03.2025
Добрый день Тема Эти формулы приводятся во множестве тьюториалов, часто под видом "матрица вращения на плоскости". x' = x * cos(a) - y * sin(a) y' = y * cos(a) + x * sin(a) Как бы Вы их. . .
Что нового в .NET 10
UnmanagedCoder 10.03.2025
. NET 10 выходит как релиз с длительной поддержкой (LTS), включающей три года обновлений. В этом обновлении Microsoft сфокусировались на нескольких направлениях: производительность, оптимизация. . .
Отложенное высвобождение, RCU и Hazard Pointer в C++26
NullReferenced 09.03.2025
Многопоточное программирование стало важной частью современной разработки. Когда несколько потоков одновременно работают с общими данными, возникает целый ряд проблем, связанных с синхронизацией и. . .
Неблокирующийся стек на C++26
NullReferenced 09.03.2025
Традиционные способы синхронизации в многопоточном программировании — мьютексы, семафоры, условные переменные — часто превращаются в узкое место в плане производительности. При этом неблокирующиеся. . .
Обработка строк в C++26: Новые возможности string и string_view
NullReferenced 09.03.2025
Новый стандарт C++26 предлагает много улучшений для работы с привычными string и относительно новыми string_view. string_view - это невладеющая ссылка на последовательность символов, появившаяся в. . .
Мой первый аддон для Blender 3D, с помощью нейронки (не зная даже азов пайтона, но это не значит что так и с остальным).
Hrethgir 09.03.2025
Потратил весь день. Пол-дня мне хватило, чтобы понять что с версией с 14B мне не одолеть написание функционального кода, на языке с которым я вообще никак не знаком - пайтон. Версия 22B от другого. . .
Einstein@Home сегодня исполняется двадцать лет!
Programma_Boinc 09.03.2025
Einstein@Home сегодня исполняется двадцать лет! Отправлено 19 февраля 2025 года в 17:20:21 UTC Я хочу поздравить всех наших волонтеров, разработчиков и ученых из Einstein@Home. Мы официально. . .
Заполнители и расширенный набор символов в C++26
NullReferenced 09.03.2025
C++26 представляет два важных обновления: заполнители и расширенный набор символов. Заполнители (placeholders) решают давнюю проблему лаконичности кода в шаблонных выражениях и лямбда-функциях. Они. . .
Контракты в C++26
NullReferenced 09.03.2025
Контракты – это механизм, позволяющий указывать предусловия, постусловия и инварианты для функций в коде. Эта функциональность должна была стать частью C++20, но была исключена на встрече комитета. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru