减少以下程序的执行时间

Question

我需要在不超过时间限制的情况下优化我对以下问题的回答 (0.5s)。这不是作业，也不是比赛，也不是我的面试问题。我目前正在自学编程为了好玩，我在网上找到了这个。

给定一个数组 a，它只包含从 1 到 a.length 范围内的数字，找到第二次出现的具有最小索引的第一个重复数字。换句话说，如果有超过 1 个重复的数字，则返回第二次出现的索引比另一个数字的第二次出现的索引更小的数字。如果没有这样的元素，返回-1。

这是我的回答：

int firstDuplicate(std::vector<int> a) 
{
    int arrLen = a.size();
    int dupIndex = -1;
    int seen[arrLen];

    for (int i = 0; i < arrLen; i++)
        seen[i] = 0;

    for (int i = 0; i < arrLen && dupIndex == -1; i++)
    {
        for (int j = 0; j < arrLen && dupIndex == -1 && seen[j] != 0; j++)
            if (a[i] == seen[j])
                dupIndex = seen[j];
        seen[i] = a[i];
    }

    return dupIndex;
}

[输入] array.integer a
保证约束：

1 ≤ a.length ≤ 105,
1 ≤ a[i] ≤ a.length

我应该如何优化我的代码以及程序优化的因素是什么？ （是的，我是编程新手）

Answer 1

不需要额外的答案。我想通了。

int firstDuplicate(std::vector<int> a) 
{
    int arrLen = a.size();
    int dupIndex = -1;
    int seen[arrLen];

    for (int i = 0; i < arrLen; i++)
        seen[i] = 0;

    for (int i = 0; i < arrLen && dupIndex == -1; i++)
    {
        if (seen[a[i]] == a[i] * -1)
            dupIndex = a[i];
        else
            seen[a[i]] = a[i] * -1;
    }

    return dupIndex;
}

Answer 2

第一个（天真的）优化：在i+1 开始内循环。

第二个（也很幼稚，因为它使用了很多内存）优化：分配一个长度为a.length的数组，初始化为-1。然后，遍历您的数据一次，一旦找到一个数字，就存储它的索引。如果已经找到索引，您可以通过检查 -1 来检测它，并且您有您的副本。