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

@plzvtl · Регистрация: 27.10.2018

Студворк — интернет-сервис помощи студентам

Для решения задачи :
"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 секунды).
Пожалуйста помогите оптимизировать данный код или придумать другой алгоритм.

@plzvtl · 15.07.2019, 03:47 **[ТС]**

Yetty, мне не трудно пробовать , но я все таки спрошу : зачем? Помогли уже ж.

@Yetty · 15.07.2019, 03:54

plzvtl, мне показалось, что Вы хотите разобраться, почему именно Ваш вариант не проходит тесты, а код
nalbe666 с другим алгоритмом, и пытался сбросить наиболее близкий к Вашему алгоритму вариант.

@plzvtl · 15.07.2019, 04:01 **[ТС]**

Сообщение от Yetty

разобраться

Да , это имеет смысл.
По поводу начинать с i , так в моем примере я начинаю с i+1.

Добавлено через 2 минуты
после этого так то и "i !=j" стало лишним потому что мы с +1 начинаем

@Yetty · 15.07.2019, 04:03

Сообщение от plzvtl

в моем примере я начинаю с i+1

скорее всего именно так и нужно. но так как Вы не сообщили результат моего последнего варианта, наверное для Вас это уже не актуально.

@plzvtl · 15.07.2019, 04:08 **[ТС]**

Yetty, результат тот же, могу кинуть скрин , но я не понимаю зачем начинать с i а не i+1 если сразу проверка в if не проходит при j=i

@Yetty · 15.07.2019, 04:15

Сообщение от plzvtl

могу кинуть скрин

зачем скрин, если результат тот же ? там 2 теста вроде не проходит, проверьте ещё с Вашей поправкой:

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+1; j < n; j++)
        if (a[i]==a[j]) {n=j;value=a[i];break;}
    return value;
}

@plzvtl · 15.07.2019, 04:17 **[ТС]**

Сообщение от Yetty

проверьте

Проверил , все еще нет.

Новые блоги и статьи Все статьи Все блоги /
Море в объективе kumehtar 06.07.2026 Хотел написать много проникновенных слов, но передумал. Оно само - говорит. Кто слышит тот слышит.	Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii. dcc0 05.07.2026 Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .	Установка статуса документа по условию Maks 05.07.2026 Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .	Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .
В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .	интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .	Сам себя обучал rest api anaschu 02.07.2026 Педагогический лайфхак: Почему чистый REST API для ученика намного круче, чем готовые библиотеки Когда мы отказались от капризного JAR-файла AnyLogic и переписали код на стандартный HttpClient,. . .

@plzvtl 57 / 43 / 12 Регистрация: 27.10.2018 Сообщений: 454
	15.07.2019, 03:47 [ТС]
	Yetty, мне не трудно пробовать , но я все таки спрошу : зачем? Помогли уже ж. 0

@Yetty 7438 / 5030 / 2892 Регистрация: 18.12.2017 Сообщений: 15,692
	15.07.2019, 03:54
	plzvtl, мне показалось, что Вы хотите разобраться, почему именно Ваш вариант не проходит тесты, а код nalbe666 с другим алгоритмом, и пытался сбросить наиболее близкий к Вашему алгоритму вариант. 0

@plzvtl 57 / 43 / 12 Регистрация: 27.10.2018 Сообщений: 454
	15.07.2019, 04:08 [ТС]
	Yetty, результат тот же, могу кинуть скрин , но я не понимаю зачем начинать с i а не i+1 если сразу проверка в if не проходит при j=i 0