高效的 Javascript 代码：最好将变量作为参数传递给函数，还是复制变量？

Question

我想在函数 A 之后运行函数 B，但在运行时使用函数 A 中的一些变量。假设为了这个练习，我不想声明全局变量。

在两个函数中编写这些变量会是“更好”的代码（场景 A），还是在函数 A 中包含局部变量并将它们作为参数传递给函数 B（场景B)?

以下是这两种情况的表示：

场景 A

function A() {
    var ss = SpreadsheetApp.getActiveSpreadsheet();
    var sheet = ss.getActiveSheet();
    var cell = sheet.getCurrentCell();

    cell.setValue("Hello World");

}

function B(cell) {
    var ss = SpreadsheetApp.getActiveSpreadsheet();
    var sheet = ss.getActiveSheet();
    var cell = sheet.getCurrentCell();
    var txt = "New Text";

    cell.setValue(txt);
}

场景 B

function A() {
    var ss = SpreadsheetApp.getActiveSpreadsheet();
    var sheet = ss.getActiveSheet();
    var cell = sheet.getCurrentCell();
    cell.setValue("Hello World");

    B(cell);
}

function B(cell) {
    var txt = "New Text";
    cell.setValue(txt);
}

在我最近从事的一个项目中，我有几十个变量，场景 B 对我来说似乎“更干净”，但我真的不知道我是否应该这样做。

我知道这是基本的，但我对 JavaScript 编码还是很陌生，我无法理解这一点。任何帮助将不胜感激！

上面的代码在 Google Apps 脚本中。

谢谢你:)

Answer 1

A 和 B 不是同一个函数。 B 是parameterized function。所以，逻辑应该是，如果我们已经有了单元格值，则调用 B，如果我们没有单元格，则调用 A。

现在，对于您的问题，是的，option B is more appealable 因为它可以让您避免代码重复。但我看不出在这种情况下从 A 调用 B 的目的。

In my opinion，如果你写一个单独的函数来获取单元格会更好，代码遵守one function one purpose(SRP)。

下面是工作的sn-p：

 //a function that return default cell value if called
 function GetCell(){
    let ss = SpreadsheetApp.getActiveSpreadsheet();
    let sheet = ss.getActiveSheet();
    return sheet.getCurrentCell();    
  }

 function A() {
    let cell=GetCell(); //get cell value by calling function
    cell.setValue("Hello World");
 } 


//function B execute successfully with parameter or without parameter. 
 function B(cell) {

   if(typeof cell == "undefined") {
    cell=GetCell();
   }
   let txt = "New Text";
   cell.setValue(txt);
 }

Answer 2

这个怎么样：

function A() {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getActiveSheet();
  var cell = sheet.getCurrentCell();
  cell.setValue("Hello World");
  return cell;
}

function B(cell) {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getActiveSheet();
  var cell = sheet.getCurrentCell();
  var txt = "New Text";
  cell.setValue(txt);
}

function runMe() {
 B(A);
}