在參數中聲明一個數組是否是不好的做法？

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.