Question

我想知道你是否能够帮助我减少我编写的一小段代码的运行时间。感觉非常低效，但作为一个菜鸟，我真的不确定如何让它变得更好。任何建议表示赞赏！

前提是一个非常简单的随叫随到的工具。插槽1到4。

用户可以将他们的开始时间，姓名和结束时间放在4个可用插槽中的任何一个中。

当用户在不同时间完成时，它可能会导致它看起来有点乱 - 例如插槽1和4有数据，但插槽2-3是空的。

为了使它看起来有点整洁，我做了一些代码来检查单元格值是否为“无”（我有另一个脚本将空白值更改为“无”，为了连续性），并且如果是这样的话，它就会向下移动（传送带式系统）。

在同一功能中共运行4次，以确保它捕获填充的任何可能的插槽组合。

出于某种原因，我在这里输入代码时遇到了问题（它搞砸了格式化），所以我已经包含了截图和代码的pastebin：代码截图1 代码截图2

Pastebin full copy of code with correct formatting


function onEdit(e) {
  var s = SpreadsheetApp.getActiveSpreadsheet();
 var User1Name = s.getRange("K4");
 var User2Name = s.getRange("AK4");
 var User3Name = s.getRange("BK4");
 var User4Name = s.getRange("CK4");
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4')

          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}
 }

function conveyor() {
    var s = SpreadsheetApp.getActiveSpreadsheet();
    var User1Name = s.getRange("K4").getDisplayValue();
    var User2Name = s.getRange("AK4").getDisplayValue();
    var User3Name = s.getRange("BK4").getDisplayValue();
    var User4Name = s.getRange("CK4").getDisplayValue();
    var U1 = s.getRange('K4')
    var U2 = s.getRange('AK4')
    var U3 = s.getRange('BK4')
    var U4 = s.getRange('CK4')

      if (User3Name == 'None'){
      FourtoThree();
      }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


      if (User2Name == 'None'){
      ThreetoTwo();
      }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


      if (User1Name == 'None'){
      TwotoOne();
      }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

 var User1Name = s.getRange("K4").getDisplayValue();
 var User2Name = s.getRange("AK4").getDisplayValue();
 var User3Name = s.getRange("BK4").getDisplayValue();
 var User4Name = s.getRange("CK4").getDisplayValue();
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4')

  if (User3Name == 'None'){
  FourtoThree();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User2Name == 'None'){
  ThreetoTwo();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User1Name == 'None'){
  TwotoOne();
  }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

 var User1Name = s.getRange("K4").getDisplayValue();
 var User2Name = s.getRange("AK4").getDisplayValue();
 var User3Name = s.getRange("BK4").getDisplayValue();
 var User4Name = s.getRange("CK4").getDisplayValue();
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4')

  if (User3Name == 'None'){
  FourtoThree();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User2Name == 'None'){
  ThreetoTwo();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User1Name == 'None'){
  TwotoOne();
  }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

 var User1Name = s.getRange("K4").getDisplayValue();
 var User2Name = s.getRange("AK4").getDisplayValue();
 var User3Name = s.getRange("BK4").getDisplayValue();
 var User4Name = s.getRange("CK4").getDisplayValue();
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4')

  if (User3Name == 'None'){
  FourtoThree();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User2Name == 'None'){
  ThreetoTwo();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

  if (User1Name == 'None'){
  TwotoOne();
  }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}
  }


// ----------------------------------------------------------------------------------------------------------
// MOVE SCRIPTS //

function TwotoOne() {
  var spreadsheet = SpreadsheetApp.getActive()
          .getSheetByName('Sheet1');
  spreadsheet.getRange('C4:I6').activate();
  spreadsheet.getRange('AC4:AI6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('K4:Q6').activate();
  spreadsheet.getRange('AK4:AQ6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('S4:Y6').activate();
  spreadsheet.getRange('AS4:AY6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('AC4:AI6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('AK4:AQ6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('AS4:AY6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
};

function ThreetoTwo() {
  var spreadsheet = SpreadsheetApp.getActive();
  spreadsheet.getRange('AC4:AI6').activate();
  spreadsheet.getRange('BC4:BI6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('AK4:AQ6').activate();
  spreadsheet.getRange('BK4:BQ6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('AS4:AY6').activate();
  spreadsheet.getRange('BS4:BY6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('BC4:BI6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('BK4:BQ6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('BS4:BY6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
};

function FourtoThree() {
  var spreadsheet = SpreadsheetApp.getActive();
  spreadsheet.getRange('BC4:BI6').activate();
  spreadsheet.getRange('CC4:CI6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('BK4:BQ6').activate();
  spreadsheet.getRange('CK4:CQ6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('BS4:BY6').activate();
  spreadsheet.getRange('CS4:CY6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('CC4:CI6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('CK4:CQ6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('CS4:CY6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});

    };

RAW Paste Data
function onEdit(e) {
  var s = SpreadsheetApp.getActiveSpreadsheet();
 var User1Name = s.getRange("K4");
 var User2Name = s.getRange("AK4");
 var User3Name = s.getRange("BK4");
 var User4Name = s.getRange("CK4"); 
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4') 

          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');} 
 }

function conveyor() {
    var s = SpreadsheetApp.getActiveSpreadsheet();
    var User1Name = s.getRange("K4").getDisplayValue();
    var User2Name = s.getRange("AK4").getDisplayValue();
    var User3Name = s.getRange("BK4").getDisplayValue();
    var User4Name = s.getRange("CK4").getDisplayValue(); 
    var U1 = s.getRange('K4')
    var U2 = s.getRange('AK4')
    var U3 = s.getRange('BK4')
    var U4 = s.getRange('CK4') 

      if (User3Name == 'None'){
      FourtoThree();
      }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


      if (User2Name == 'None'){
      ThreetoTwo();
      }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


      if (User1Name == 'None'){
      TwotoOne();
      }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

 var User1Name = s.getRange("K4").getDisplayValue();
 var User2Name = s.getRange("AK4").getDisplayValue();
 var User3Name = s.getRange("BK4").getDisplayValue();
 var User4Name = s.getRange("CK4").getDisplayValue(); 
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4') 

  if (User3Name == 'None'){
  FourtoThree();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User2Name == 'None'){
  ThreetoTwo();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User1Name == 'None'){
  TwotoOne();
  }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

 var User1Name = s.getRange("K4").getDisplayValue();
 var User2Name = s.getRange("AK4").getDisplayValue();
 var User3Name = s.getRange("BK4").getDisplayValue();
 var User4Name = s.getRange("CK4").getDisplayValue(); 
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4') 

  if (User3Name == 'None'){
  FourtoThree();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User2Name == 'None'){
  ThreetoTwo();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User1Name == 'None'){
  TwotoOne();
  }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

 var User1Name = s.getRange("K4").getDisplayValue();
 var User2Name = s.getRange("AK4").getDisplayValue();
 var User3Name = s.getRange("BK4").getDisplayValue();
 var User4Name = s.getRange("CK4").getDisplayValue(); 
 var U1 = s.getRange('K4')
 var U2 = s.getRange('AK4')
 var U3 = s.getRange('BK4')
 var U4 = s.getRange('CK4') 

  if (User3Name == 'None'){
  FourtoThree();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}


  if (User2Name == 'None'){
  ThreetoTwo();
  }
          if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}

  if (User1Name == 'None'){
  TwotoOne();
  }
        if (U1.isBlank()){
        U1.setValue('None');}

        if (U2.isBlank()){
        U2.setValue('None');}

        if (U3.isBlank()){
        U3.setValue('None');}

        if (U4.isBlank()){
        U4.setValue('None');}
  }


// ----------------------------------------------------------------------------------------------------------
// MOVE SCRIPTS //

function TwotoOne() {
  var spreadsheet = SpreadsheetApp.getActive()
          .getSheetByName('Sheet1');
  spreadsheet.getRange('C4:I6').activate();
  spreadsheet.getRange('AC4:AI6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('K4:Q6').activate();
  spreadsheet.getRange('AK4:AQ6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('S4:Y6').activate();
  spreadsheet.getRange('AS4:AY6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('AC4:AI6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('AK4:AQ6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('AS4:AY6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
};

function ThreetoTwo() {
  var spreadsheet = SpreadsheetApp.getActive();
  spreadsheet.getRange('AC4:AI6').activate();
  spreadsheet.getRange('BC4:BI6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('AK4:AQ6').activate();
  spreadsheet.getRange('BK4:BQ6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('AS4:AY6').activate();
  spreadsheet.getRange('BS4:BY6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('BC4:BI6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('BK4:BQ6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('BS4:BY6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
};

function FourtoThree() {
  var spreadsheet = SpreadsheetApp.getActive();
  spreadsheet.getRange('BC4:BI6').activate();
  spreadsheet.getRange('CC4:CI6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('BK4:BQ6').activate();
  spreadsheet.getRange('CK4:CQ6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('BS4:BY6').activate();
  spreadsheet.getRange('CS4:CY6').copyTo(spreadsheet.getActiveRange(), SpreadsheetApp.CopyPasteType.PASTE_VALUES, false);
  spreadsheet.getRange('CC4:CI6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('CK4:CQ6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
  spreadsheet.getRange('CS4:CY6').activate();
  spreadsheet.getActiveRangeList().clear({contentsOnly: true, skipFilteredRows: true});
};

传送带代码位于function conveyor()下方，移动代码位于function TwotoOne()，function ThreetoTwo()和function FourtoThree()下。

代码的问题（除了它是超级基本的，但这是作为菜鸟的乐趣）是function conveyor()的执行是运行时大约15秒。

有没有什么可以建议你减少运行时间？

如果您希望查看代码，或者查看代码，请继续查看Google Spreadsheet Doc。在每个插槽中放置任何信息组合，点击“RUN”，它将始终将它们向下移动到顺序（因此2和3将变为1和2,1和4将变为1和2,1,3和4将成为1,2和3等）。

（这是我的电子表格的完整副本（非原创），没有保密信息，只有运行所需的基本代码。可以根据需要随意修补或销毁它。有一个完整的备份。，一旦找到解决方案，我将恢复原始版本，以便未来的askers将能够看到初始代码）并查看我的问题。

再次感谢您的时间。如果您有任何其他问题，请告诉我，我会尽力解释！

Answer 1

/**
* Moves Filled Slots to occupy the left empty slots
* @OnlyCurrentDoc
*/
function conveyorBelt() {
  var rangList = SpreadsheetApp.getActiveSheet().getRangeList(['K4','AK4','BK4','CK4']); // Four slot entries
  var rngArr = rangList.getRanges(); //getEachSlotRange
  for(var i=0;i<rngArr.length;i++) { //Loop over each Slot
    if (rngArr[i].isBlank()) { //if current slot is empty,
      for (var j= i+1;j<rngArr.length;j++) { //Loop over other slots to the right to find the  next non empty slot
        if (!rngArr[j].isBlank()) { //next non empty slot
          var targetRange = rngArr[i].offset(0,-8,3,23);//Expand range to full slot(23Columns and 3rows)
          var sourceRange = rngArr[j].offset(0,-8,3,23);//Expand range to full slot(23Columns and 3rows)
          sourceRange.copyTo(targetRange,SpreadsheetApp.CopyPasteType.PASTE_NORMAL,false); //Paste Normal Ignoring Color Formatting
          sourceRange.clearContent()
          break;

        }
      }
    }
  }
}

关于减少Google Apps脚本数据移动操作的运行时间的建议

1 个答案: