javascript – 我的算法是错误的还是正确的,只是需要调整?

<html><head></head>
<body><script>
  function GCD(a, b) {
    if (a == 0) {
      return b;
    }
    return GCD(b % a, a);
  }

  function difference(array) {
    for (var i = Math.min(...array) + 1; i < Math.max(...array); i++) {
      array.push(i);
    }
    array.sort((a, b) => a - b);
  }

  function smallestCommons(arr) {
    difference(arr);
    console.log(arr);
    a = arr[arr.length - 1];
    b = arr[arr.length - 2];
    var LCM = a * b / GCD(a, b);
    while (true) {
      var index = arr.findIndex(element => LCM % element !== 0);
      if (index === -1) {
        return LCM;
      }
      LCM *= arr[index];
      console.log(LCM);
    }
  }
  smallestCommons([1, 5]) // right 
  smallestCommons([2, 10]) // right
  smallestCommons([1, 13]) // wrong
  smallestCommons([23, 18]) // wrong
</script></body>
</html>

这是这个challlenge的代码:
https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/intermediate-algorithm-scripting/smallest-common-multiple/

我使用的算法:

1 – 计算两个最大数字的最小公倍数.

2-在所有数组上尝试LCM%编号,如果找到一个不返回0的数字,则将其分配给var索引并将其乘以* LCM然后继续这样做,直到它找不到任何内容并返回-1相反,当发生这种情况时,最后返回LCM.

代码在前两个阵列工作,但它最后两个不起作用导致我思考,我的算法是错误的,我只是浪费时间试图调整它或它是正确的,它只需要一些调整?

请注意,有3个不同的函数,第一个函数计算GCD以便稍后计算LCM,然后第二个函数推送数组中两个值之间的值然后对其进行排序,第三个函数计算LCM(最低公倍数)

问题是最后一个数组:

smallestCommons([1,13])返回4324320而不是360360

smallestCommons([23,18])返回72681840而不是6056820

所以我想知道,请关注这个:

我的整个方式(算法)是错误的,我需要重写一个完整的或需要一些调整才能工作?

请不要给我准备好的代码.告诉我我想知道什么,谢谢你(:

最佳答案 你错了乘以arr [index].

为了清楚起见,我们重新标记变量:

N =数组[索引]
p = GCD(LCM,N)

现在我们可以将N和LCD重写为:

N = p×q
LCM = p×r

要使最小的数字是当前LCM和N的最低公倍数,您需要将其计算为:

LCM:= p×q×r

但是如果你用代码计算它

LCM *= arr[index];

你实际上在计算

LCM:=(p×r)×(p×q)= p×p×q×r

即一个p因子太多.要根据需要进行计算,您应该使用以下代码:

LCM *= arr[index] / GCD(LCM, arr[index]);
点赞