Question

所以我在VBA中做了大量工作后第一次使用谷歌脚本。运行代码时，我收到一条消息，说脚本运行时间太长。某些部分，如.getValue（）和.setValue（）导致问题，但我不知道如何重新编写代码以更好地优化。这是代码：

function getPredictions() {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var Carter = ss.getSheetByName("Carter");
  var back = ss.getSheetByName("Back");
  var monthsLeft= Carter.getRange(13,15)
  var months = monthsLeft.getValue();
  var total = 0;
  Carter.getRange("d2").setValue(months);
  for(var i = 13;Carter.getLastRow(); i++){
    if(Carter.getRange(i, 2).getValue() != ""){
      total = 0;
      var k = 17;
      monthsLeft = Carter.getRange(i,15).getValue();
      for(var j = 2; j<29; j++){
        if(back.getRange(j, 2).getValue() <= monthsLeft){
          total = total + back.getRange(j, 3).getValue();
          Carter.getRange(i,k).setValue((back.getRange(j, 3).getValue() / Carter.getRange(i,16).getValue()) * Carter.getRange(i,11).getValue());
          k++;
        }
      }
    }
  }
}

function getMonthsL () {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var Carter = ss.getSheetByName("Carter");
  for(var i = 13; i < Carter.getLastRow(); i++){
     if(Carter.getRange(i, 2).getValue() != ""){
       Carter.getRange(i,15).setFormula('=if(day($B$5)>20,DATEDIF($B$5,I' +i + ',"M"),DATEDIF($B$5,I' +i+',"M")+1)');
     }
   }
}

function getMonthP(){
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var Carter = ss.getSheetByName("Carter");
  var back = ss.getSheetByName("Back");
  var total = 0;
  for(var i = 13; i < Carter.getLastRow(); i++){
    total = 0;
    if(Carter.getRange(i, 2).getValue() != ""){
      for(var j = 2; Carter.getRange(i,15).getValue() >= back.getRange(j,2).getValue(); j++){
        total = total + back.getRange(j,3).getValue(); 
      }
    Carter.getRange(i,16).setValue(total);
    }
  }
}

function runAll(){
getMonthsL();
getMonthP();
getPredictions();
}

任何有关重构此方法的最佳方式的帮助/解释都将受到赞赏。

Answer 1

我不希望这个答案是正确的，但我想我只是举个例子说明我们很多人会推荐给你的。试图限制getValue（）的使用，并尝试使用getValues返回一个二维数组（排序）。

其中一个复杂因素是索引从零开始，行和列从1开始。因此，有时候保持一切顺利可能有点挑战，但一旦你这样做，你就会发现代码的运行速度要快10倍。

function getPredictions() 
{
  var ss=SpreadsheetApp.getActiveSpreadsheet();
  var Carter=ss.getSheetByName("Carter");
  var crg=Carter.getDataRange();
  var carterA=crg.getValues();
  var back=ss.getSheetByName("Back");
  var backA=back.getDataRange().getValues();
  var months=carterA[12][14];
  var total=0;
  carterA[3][1]=months;
  for(var i = 12;i<carterA.length; i++)
  {
    if(carterA[i][1]!= "")
    {
      total = 0;
      var k = 16;
      var monthsLeft = carterA[i][14];
      for(var j=0;j<28;j++)//I probably got this indexing incorrect
      {
        if(backA[j][1] <= monthsLeft)
        {
          total = total + backA[j][2];
          carterA[i][k]=backA[j][2];
          k++;
        }
      }
    }
  }
  crg.setValues(carterA);//This save all the changes
  //Didn't do backA because I don't think you changed it.
}

我很确定这不会奏效。但希望它能让您了解如何使代码更快。

Google脚本运行时优化

1 个答案: