JavaScript 中的 Codility Leader 算法

Question

我正在尝试 JavaScript 中的 Codility 'Leader' 问题。

问题如下：

给出了一个由N 整数组成的数组A。数组A的支配者是出现在A的一半以上元素中的值。

例如，考虑数组A 这样

A[0] = 3    A[1] = 4    A[2] =  3
A[3] = 2    A[4] = 3    A[5] = -1
A[6] = 3    A[7] = 3

A 的支配者是 3，因为它出现在 A 的 8 个元素中的 5 个中（即在索引为 0、2、4、6 和 7 的 > 中），并且 5 超过一半8 个。

编写一个函数function solution(A);，给定一个由N 整数组成的数组A，返回数组A 中出现A 的支配者的任何元素的索引。如果数组 A 没有支配者，该函数应该返回 -1。

例如，给定数组A，这样

A[0] = 3    A[1] = 4    A[2] =  3
A[3] = 2    A[4] = 3    A[5] = -1
A[6] = 3    A[7] = 3

该函数可能返回 0、2、4、6 或 7，如上所述。

为以下假设编写一个有效的算法：

N 是[0..100,000]; 范围内的整数 并且数组A的每个元素都是[−2,147,483,648..2,147,483,647]范围内的整数。

我的答案如下：

function solution(A) {
    const length = A.length
    if (length > 100000) return -1
    const counters = new Array(length).fill(0)
    const negativeCounters = new Array(length).fill(0)
    for (i=0; i < length; i++){
        if (A[i] < -2147483648 || A[i] > 2147483647) return -1
        if (A[i] > -1){
            counters[A[i]] = counters[A[i]] + 1
            if (counters[A[i]] > (length / 2)) return i
        } else {
            negativeCounters[A[i] * -1]  =  negativeCounters[A[i] * -1] + 1
            if (negativeCounters[A[i] * -1] > (length / 2)) return i
        }
    }
    return -1
}

尽管我尝试了各种成功的输入，但它未能通过正确性测试。 Codility 评估没有列出测试输入，所以我找不到破坏算法的输入。

谁能发现问题？

Answer 1

与某些 cmets 和一个答案相反，实际上在 JavaScript 中使用一组计数器可以正常工作。这是因为 JavaScript 中的数组是关联对象。您的代码的一个问题是它无法将键/索引大于初始计数器数组长度的计数器初始化为零。

以下代码在 Codility 的所有测量中均获得 100% 的分数：

function solution(A) {
    const arr = []
    for (let i=0; i<A.length; i++){
      if (!arr[A[i]])
        arr[A[i]] = 1
      else
        arr[A[i]]++
      if (arr[A[i]] > A.length/2)
        return i
    }
    return -1
}

Answer 2

您不应该使用具有如此大范围值的数组（正值数组的长度需要为 2,147,483,648）。您应该使用地图或在 js 中您可以使用对象。我一起破解了一些东西，不是很优雅，但它通过了所有测试：

function solution(A) {
    const map = {}
    for (let i = 0; i < A.length; i++) {
        const key = '' + A[i]
        map[key] = key in map ? map[key] + 1 : 1
        if (map[key] > A.length / 2)
            return i
    }
    return -1
}