0

所以我与谷歌的脚本在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(); 
} 

任何帮助/解释,将不胜感激。

+3

非但没有单独的单元格值 - 使用[的GetValues()](https://developers.google.com/apps-script/reference/spreadsheet/range#getValues()),以获得自然网格和访问数据使用[数组索引]的值(https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Indexed_collections#Multi-dimensional_arrays)。另外一些[最佳实践](https://developers.google.com/apps-script/guides/support/best-practices) –

回答

1

我不认为这个答案是正确的,但我想我只是给你一个什么样我们许多人会推荐给你举个例子。试图限制getValue()的使用并尝试使用返回二维数组(有点)的getValues。

其中一个复杂的是,标从零开始和行和列的一个开始。因此,有时候要保持一切顺利可能有点困难,但是一旦你完成了,你会发现代码运行速度可以提高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. 
} 

我很确定这不起作用。但希望它能让你了解如何使代码更快。