Google Scriptsランタイムの最適化

Question

私はVBAで多くの作業をして初めてGoogleスクリプトで作業しています。コードを実行すると、スクリプトの実行に時間がかかりすぎるというメッセージが表示されます。 .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（）の使用を制限し、2次元配列（sort of）を返すgetValuesを使用しようとしています。

複雑さの1つは、インデックスがゼロから始まり、行と列が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. 
} 

これはうまくいきません。しかし、うまくいけば、コードをどのように速くすることができるかというアイデアがあなたに与えられます。