在参数中声明一个数组是否是不好的做法？

Question

validationError([elem1,elem2],type,shiftNo); 

或

var arr = [elem1,elem2]; 
validationError(arr,type,shiftNo); 

我的意思要问的是呼叫认为是坏的功能（也它有任何性能后果）的方法1。并且就此而言，在参数内部声明字符串，对象和函数是一种糟糕的方法。

Answer 1

性能不是问题，不是像JS，Ruby或whatnot这样的语言。所以我们所能做的就是考虑代码的可读性。这个案例与JS并没有很强的关联，所以这将是我的例子。

move = ["E2", "E4"]; 
if chessboard.valid(move, player) { 
    ... 
} 

此明确指出：“如果移动（E2 E4）的有效期为这个棋盘，然后......”，你甚至不需要看文档就知道。如果我们写了不分配我们的阵列的名称，结果看起来有点神秘（仍然很容易猜到，但更难这样一个小例子）：

if chessboard.valid(["E2", "E4"], player) { 
    ... 
} 

这是什么意思？ valid代表什么？也许，这是问这些单元格是否包含有效玩家的棋子？这是一个设计缺陷，更确切地说是错误的命名。它对如何使用棋盘代码做出了大胆的假设。我们可以明显地表明这阵代表通过重命名chessboard的方法一招：

if chessboard.valid_move(["E2", "E4"], player) { 
    ... 
} 

这是更好的，但你可能没有一个API，使你的代码留下脍炙人口没有一些额外的命名。

所以，我建议一个经验法则：

1. 如果阵列将被使用超过一次，将其命名。
2. 如果数组的含义并不明显（函数名），请将其命名。
3. 不要命名它，除非点1或2适用。

Answer 2

确实没有什么区别。无论哪种方式，你创建一个JavaScript数组，它基本上是一个对象并获得一个参考作为回报（你传递给你的方法）。如果您不需要在代码中稍后访问该数组（或其他数据），则第二种方法是完全正确的。

Answer 3

这是主观的，但在我看来，它是更好地使用第二种方法。

由于@jAndy表示在代码执行方面没有区别，您的代码的性能也不差，但它更易于调试并且更易于阅读和理解第二种方法。

Answer 4

arr的内容是否会再次被使用？如果是这样，那么选项2肯定是要走的路。如果不是......这很简单，可能只是个人意见。

就个人而言，我不得不说，选项2是更好的做法，即使有时我有罪使用选项1选项2更容易阅读的，它更容易跟踪和它不太可能有人会必须重新阅读它，因为在阅读代码时（尤其是较新的程序员），他们暂时感到困惑或迷失于思想流动中。由于这些原因，维护起来会更容易，您以及将来可能会使用您的代码的开发人员可能会节省使用它的时间。

我看到的唯一的负面因素是产生的开销绝对很小，现在你有2行代码，而不是1个。但我认为这是无关紧要的，选项2的微小潜在好处超过了微小的负面因素选项1.