javascript - JS 逻辑算法不工作，不进入 else 分支

Question

我编写了这个操作两个数组的算法。 letter为主，fake为副本。

基本上，我使用循环来遇到 letter 和 fake，如果它们具有相同的 'd' 值，则 tmp 数组是使用 letter[i] 值推送。但只有一次!我不希望 tmp 在其“d”值中包含双值。

我的算法正在运行，但它似乎从未进入我的“else”内部以在“tmp”内部推送一个空的 {d : ""}。

此外，我正在使用 localeCompare()，这是一种用于检查两个字符串是否相等的字符串方法，但如果我使用的是:=== 我得到了相同的结果，所以我不确定它是否有用，是吗？

问题

我没有得到预期的结果...我的算法永远不会在第二次推送时进入 else。所以加倍，留在最后的tmp数组中。

// 'd' letter values : a, b, b, c, d, d

var letter = [
{a : 'one', d : 'a'}, 
{a : 'two', d : 'b'}, 
{a : 'three', d : 'b'}, 
{a : 'four', d : 'c'},
{a : 'five', d : 'd'},
{a : 'six', d : 'd'}];

// 'd' fake values : a, b, b, c, d, d

var fake = [
{d : 'a'}, 
{d : 'b'}, 
{d : 'b'}, 
{d : 'c'}, 
{d : 'd'}, 
{d : 'd'}];

/* tmp output i would like : 
{a : 'one', d : a}, 
{a : 'two', d : b}, 
{a : 'three', d : ""}, 
{a : 'four', d : c}, 
{a : 'five', d : d}, 
{a : 'six', d : ""} 
*/

function sortMyArrays(letter, fake)
{

var tmp = [];

var k = 0;
var i = 0;
var l = 0;
var size = letter.length;

do
{
    if(l !== 0)
    {
        k = l;
    }

    if(letter[k].d !== undefined)
    {   
        var lettString = letter[k].d;
        var fakeString = fake[i].d;
     
        // localeCompare() === 0 if strings are equals
        if(lettString.localeCompare(fakeString) === 0)
        {
            do
            {
                    if(i >= k)
                    {
                        // Always enter in this if
                        if(k === i)
                        {
                            tmp.push({a : letter[i].a , d : letter[i].d});
                      
                            l += 1;
                        }
                        // Never enter in the else...
                        else
                        {
                            tmp.push({a : letter[i].a , d : ""});
                     
                            l += 1; 
                        }
                    }
                    i += 1;
            }
            while(letter[k].d === fake[i].d);
         
        }
    }
}
while(k < size);

console.log("Result into tmp array is : ", tmp);

}

Answer 1

我还没有真正弄清楚你在用这个算法寻找什么，但我实现了你想要的输出。

function sortMyArrays(letter, fake) {

  var tmp = [];

  var index = 0;
  var usedDValues = [];
  var size = letter.length;

  do {
    if (letter[index].d !== undefined) {
      var lettString = letter[index].d;
      var fakeString = fake[index].d;

      // localeCompare() === 0 if strings are equals
      if (lettString === fakeString) {
        // if usedDValues.indexOf(lettString) returns -1, that d value has not been used yet.
        if (usedDValues.indexOf(lettString) === -1) {
          tmp.push({
            a: letter[index].a,
            d: letter[index].d
          });
        } else {
          tmp.push({
            a: letter[index].a,
            d: ""
          });
        }

        usedDValues.push(lettString);
      }
    }

    index += 1;
  }
  while (index < size);

  console.log("Result into tmp array is : ", tmp);
}

我在这里所做的主要更改是更改您独特的“d”比较。将数组与您以前使用过的所有“d”值一起使用，您只需检查该数组以查看您是否使用过它。这不是最简洁的内存方式，但它易于阅读。您还可以检查 temp 数组中以前的 d 值，看看您是否已经使用过该 d 值。

坚持 === 而不是 localeCompare 进行字符串相等性检查。 localCompare 旨在为您提供按字母顺序排在第一位的字符串的顺序。