为什么我的返回数组函数在返回数组值后不会使用或结合“数学”方法？

Question

猜猜...我正在尝试编写一个函数，该函数返回一个可以传递为字符串格式的数组。就像我将在这里提到的那样：我有 randomColor(customColorsArrays,takingColorsArray) 函数依赖于类已填充 如果它们都是空返回color 的值是通过 Math 方法计算出来的，如果 customColorsArrays 的类已经被用户自己填充了来放置自己的 Array 主题，如果这两个类都是用户自己写的返回数组的 var 已设置在该函数内部的较早位置。为了清楚地看到这些代码的下方：

function randomColor(customColorsArray, takenColorsArray) {
 var text = "",
     colors = ["orange", "yellow", "red", "maroon"];

   if (customColorsArray && takenColorsArray) {
      var text = "["+colors+"]";
   }
     else if (!customColorsArray && !takenColorsArray) {
      text += colors[Math.floor(Math.random() * colors.length)];
   }
     else {
      text += customColorsArray[Math.floor(Math.random() * customColorsArray.length)];
  };

   return text;
}

function personalRandomColor(e) {
 var text = "";

   if (e == "orange") {text += "almond";}
    else if (e == "yellow") {text += "melrose";}
    else if (e == "red") {text += "danube";}
    else if (e == "maroon") {text += "magenta";};

  return text;
}

这是实现上述功能的 HTML 代码。

bla... bla... bla...
  var customColorsArrays = randomColor('passingClass', 'takenColor'),
      randomFirstColor = randomColor(),
      skipFirstColors = customColorsArrays.replace('\[', '\[\"').replace('\]', '\"\]').replace(/[\,]/g, '\"\, \"').replace('\"'+randomColor()+'\"\,', ''),
      randomSecondColor = personalRandomColor(randomColor(toString(skipFirstColors))),
bla... bla... bla...

Answer 1

修复代码的一种简单直接的方法是替换该行

randomSecondColor = personalRandomColor(randomColor(toString(skipFirstColors)))

为

randomSecondColor = personalRandomColor(randomColor(JSON.parse(skipFirstColors)))

原因是randomColor 接受元素数组，而您正在向它传递一个字符串。参数skipFirstColors 是一个字符串，它实际上是字符串列表的 JSON 表示。你希望它是一个实际的列表，而不是一个代表列表的字符串。

JSON.parse 函数将评估由 skipFirstColors 返回的字符串并返回它所代表的列表。

此外，我建议您重新访问并可能重构您的代码。您应该尝试使用内置类型（例如列表）作为函数返回，而不是使用对象的 JSON 表示（序列化为字符串），因为这样可以避免此类错误，并使代码更简洁。