Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.64/11: Рейтинг темы: голосов - 11, средняя оценка - 4.64
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454

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

14.07.2019, 23:25. Показов 2218. Ответов 26
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Для решения задачи :
"Given an array a that contains only numbers in the range from 1 to a.length, find the first duplicate number for which the second occurrence has the minimal index. In other words, if there are more than 1 duplicated numbers, return the number for which the second occurrence has a smaller index than the second occurrence of the other number does. If there are no such elements, return -1.

Example

For a = [2, 1, 3, 5, 3, 2], the output should be firstDuplicate(a) = 3.

There are 2 duplicates: numbers 2 and 3. The second occurrence of 3 has a smaller index than the second occurrence of 2 does, so the answer is 3.

For a = [2, 2], the output should be firstDuplicate(a) = 2;

For a = [2, 4, 3, 5, 1], the output should be firstDuplicate(a) = -1.

Input/Output

[execution time limit] 0.5 seconds (cpp)

[input] array.integer a

Guaranteed constraints:
1 ≤ a.length ≤ 105,
1 ≤ a[i] ≤ a.length.

[output] integer

The element in a that occurs in the array more than once and has the minimal index for its second occurrence. If there are no such elements, return -1."
Я написал такой код :
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
int firstDuplicate(std::vector<int> a) 
{
int temp2 = -1,j3=a.size() ,j2=j3 ;
for(int i = 0; i < j3;++i)
{
for(int j = i+1; j < j3;++j)
{
if(a[i] == a[j] && j < j2)
{
j2 = j;
temp2=a[i];
break;
}
}
}
return temp2;
}
Но данный код не проходит тест за максимальное допустимое время (0.5 секунды).
Пожалуйста помогите оптимизировать данный код или придумать другой алгоритм.
0
Лучшие ответы (1)
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
14.07.2019, 23:25
Ответы с готовыми решениями:

Оптимизировать код
Доброго времени суток, как можно оптимизировать код что бы он быстрее работал ? Дана последовательность из n чисел a1, a2,..., an. C...

Оптимизировать код
Первое число входного потока - количество чисел Дальше идут те самые числа Надо найти кол-во пар чисел, для которых выполняется nums...

Оптимизировать код
Для решения задачи : &quot;Note: Write a solution that only iterates over the string once and uses O(1) additional memory, since this is what...

26
863 / 513 / 215
Регистрация: 19.01.2019
Сообщений: 1,216
14.07.2019, 23:57
C++
1
2
3
4
5
6
7
8
9
int firstDuplicate(std::vector<int>& a) {
    short cnt[106] = { 0 };
    for (auto& it : a) {
        if (++cnt[it] != 1) {
            return it;
        }
    }
    return -1;
}
?
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 00:12  [ТС]
Цитата Сообщение от nalbe666 Посмотреть сообщение
?
Вроде правильно скопипастил.
Миниатюры
Оптимизировать код  
0
863 / 513 / 215
Регистрация: 19.01.2019
Сообщений: 1,216
15.07.2019, 00:17
Возможно, вектор нужно передавать в функцию как в вашем примере по значению, не ссылке. std::vector<int> a
Вижу, ретурн не там, где у меня.
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 00:23  [ТС]
Цитата Сообщение от nalbe666 Посмотреть сообщение
.
///
Миниатюры
Оптимизировать код  
0
863 / 513 / 215
Регистрация: 19.01.2019
Сообщений: 1,216
15.07.2019, 01:12
Лучший ответ Сообщение было отмечено plzvtl как решение

Решение

C++
1
2
3
4
5
6
7
8
9
10
11
#include <set>
 
int firstDuplicate(std::vector<int>& a) {
    std::set<int> bset;
    for (auto& it : a) {
        if (!bset.insert(it).second) {
            return it;
        }
    }
    return -1;
}
Тоже самое, но с сетом?
1
 Аватар для zayats80888
6352 / 3523 / 1428
Регистрация: 07.02.2019
Сообщений: 8,995
15.07.2019, 01:20
del
0
863 / 513 / 215
Регистрация: 19.01.2019
Сообщений: 1,216
15.07.2019, 01:31
Полагаю, второй вариант их устроил. Вот ерунда в том, что
Цитата Сообщение от plzvtl Посмотреть сообщение
Guaranteed constraints:
1 ≤ a.length ≤ 105,
1 ≤ a[i] ≤ a.length.
тут они гарантируют размер самих чисел в массиве от 1 до 105 включительно. Соответственно, в первом варианте нужен массив размером 106. А по факту, в тесте они засунули туда другие числа, от чего выход за пределы массива и ошибки.
Зато резиновый сет справился.
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 01:37  [ТС]
Цитата Сообщение от nalbe666 Посмотреть сообщение
Вот ерунда в том, что
Посыпаю голову пеплом.
Тут исключительно моя вина, я в тупую скопировал условие и недоглядел за корректностью скопированого.
Там 10^5.
0
15.07.2019, 01:39

Не по теме:

Загадка разгадана, можно спокойно идти спать :).

0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 01:41  [ТС]

Не по теме:

Цитата Сообщение от nalbe666 Посмотреть сообщение
можно спокойно идти спать :).
может еще 1 оптимизацию?


0
7438 / 5030 / 2892
Регистрация: 18.12.2017
Сообщений: 15,692
15.07.2019, 02:05
plzvtl, а такой вариант проходит тест ?:
C++
1
2
3
4
5
6
7
8
9
10
double firstDuplicate(vector<double> &a) 
{
    double value=-1.;
    size_t n=a.size();
    
    for (size_t i = 0; i < n; i++)           
        for (size_t j = 0; j < n; j++)
        if (i!=j && a[i]==a[j]) {n=j;value=a[i];break;}
    return value;
}
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 02:24  [ТС]
Цитата Сообщение от Yetty Посмотреть сообщение
а такой вариант
вот:
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 02:25  [ТС]
Цитата Сообщение от Yetty Посмотреть сообщение
а такой вариант
вот:
Миниатюры
Оптимизировать код  
0
7438 / 5030 / 2892
Регистрация: 18.12.2017
Сообщений: 15,692
15.07.2019, 02:44
plzvtl, возможно подаётся int - вектор, проверьте с типом int:
C++
1
2
3
4
5
6
7
8
9
10
int firstDuplicate(vector<int> &a) 
{
    int value=-1;
    size_t n=a.size();
    
    for (size_t i = 0; i < n; i++)           
        for (size_t j = 0; j < n; j++)
        if (i!=j && a[i]==a[j]) {n=j;value=a[i];break;}
    return value;
}
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 02:49  [ТС]
Цитата Сообщение от Yetty Посмотреть сообщение
проверьте с типом int:
вот:
Миниатюры
Оптимизировать код  
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 03:22  [ТС]
на 1 тест больше чем 1 вариант (мой)
0
7438 / 5030 / 2892
Регистрация: 18.12.2017
Сообщений: 15,692
15.07.2019, 03:26
plzvtl, size_t не укладывался в значения, ещё разок проверьте пожалста
C++
1
2
3
4
5
6
7
8
9
int firstDuplicate(vector<int> &a) 
{    
    int n=a.size(), value=-1;
    
    for (int i = 0; i < n; i++)           
        for (int j = 0; j < n; j++)
        if (i!=j && a[i]==a[j]) {n=j;value=a[i];break;}
    return value;
}
0
57 / 43 / 12
Регистрация: 27.10.2018
Сообщений: 454
15.07.2019, 03:30  [ТС]
Цитата Сообщение от Yetty Посмотреть сообщение
/
тоже самое
Миниатюры
Оптимизировать код  
0
7438 / 5030 / 2892
Регистрация: 18.12.2017
Сообщений: 15,692
15.07.2019, 03:43
plzvtl, по времени ограничение. возможно (не проверял) быстрее будет если j начинать c i:
C++
1
2
3
4
5
6
7
8
9
int firstDuplicate(vector<int> &a) 
{    
    int n=a.size(), value=-1;
    
    for (int i = 0; i < n; i++)           
        for (int j = i; j < n; j++)
        if (i!=j && a[i]==a[j]) {n=j;value=a[i];break;}
    return value;
}
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
15.07.2019, 03:43
Помогаю со студенческими работами здесь

Как оптимизировать код
Доброй ночи господа у меня к вам такая просьба как можно упростить данный код? #include &lt;iostream&gt; #include &lt;stdlib.h&gt; ...

Оптимизировать и минимализировать код
Cделал легкую прогу. Понимаю логики 0 в коде. Можете помочь оптимизировать код? А заодно и сделать код более минималистичным. #include...

Нужно оптимизировать код
Гистограмма является многоугольником, сформированным из последовательности прямоугольников, выровненных на общей базовой линии....

Как оптимизировать код?
Вот такой код, написанный для микроконтроллера импульсного блока питания. Просто интересно мнение, что можно поменять и изменить для...

Нужно оптимизировать код
Нужно максимально сократить код #include &lt;iostream&gt; using namespace std; int main(int argc, char** argv) { int a, i,...


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

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
Новые блоги и статьи
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
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru