google-apps-script - 根据行中单元格的内容删除谷歌电子表格中的一行

Question

如果所选列中的所选单元格具有提到的内容，但我不明白它在哪里失败，我有这个脚本来删除某些行

    function DeleteRowByKeyword() {

  var value_to_check = Browser.inputBox("Enter the keyword to trigger delete Row","", Browser.Buttons.OK);

  //  prendo quello attivo
  var DATA_SHEET = SpreadsheetApp.getActiveSpreadsheet().getActiveSheet();
  var FIRST_COLUMN = Browser.inputBox("Number of Column to look at (eg: for column A enter 1)","", Browser.Buttons.OK);

  ss.toast("removing duplicates...","",-1);

  var dataCopy1 = DATA_SHEET.getDataRange().getValues();
  var deleted_rows = 0;
  var rangeToCopy = '';

  if (dataCopy1.length > 0) {
    var i = 1;

    while (i < DATA_SHEET.getMaxRows() - deleted_rows) {
  if ((dataCopy1[i][FIRST_COLUMN]).search(value_to_check) != -1) {

    ss.deleteRow(i);
    deleted_rows++;
  } 
  i++;
    }
  }


  ss.toast("Done! " + deleted_rows + ' rows removed',"",5);
    }

提前感谢您的帮助

Answer 1

还有一些需要改进的地方:

• 请记住，电子表格的行和列的编号从1、对于SpreadsheetApp中的所有方法，而javascript数组从 0 开始编号。您需要在这些数字之间进行调整与两者一起工作时的基础。

• String.search() 方法在这里可能是不合适的选择，因为有两个原因。 .search() 将匹配子字符串，因此('Testing').search('i') 找到匹配项；您可能想寻找确切的相反。此外，.search() 还包括对正则表达式的支持匹配，因此用户可能会惊讶地发现他们的输入被解释作为正则表达式； .indexOf() 可能是更好的选择。

• 要将操作限制为包含数据的行，请使用 .getLastRow()而不是 .getMaxRows()。

• 删除行时，电子表格dataRange的大小将得到较小；你确实考虑到了这一点，但是因为你正在循环直到最大尺寸，代码因这个要求而变得复杂。您可以通过从最大值向下循环来简化事情。

• 列号的输入容易出错，所以让用户输入信;您可以将其转换为数字。

• 您尚未定义 ss 的值(在此函数内)。

这是更新后的代码:

function DeleteRowByKeyword() {

  var value_to_check = Browser.inputBox("Enter the keyword to trigger delete Row", "", Browser.Buttons.OK);

  var matchCol = Browser.inputBox("Column Letter to look at", "", Browser.Buttons.OK);
  var FIRST_COLUMN = (matchCol.toUpperCase().charCodeAt(0) - 'A'.charCodeAt(0) + 1);  // Convert, e.g. "A" -> 1

  //  prendo quello attivo (get active sheet)
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var DATA_SHEET = SpreadsheetApp.getActiveSpreadsheet().getActiveSheet();

  ss.toast("removing duplicates...", "", -1);

  var dataCopy1 = DATA_SHEET.getDataRange().getValues();
  var deleted_rows = 0;

  if (dataCopy1.length > 0) {
    var i = DATA_SHEET.getLastRow();  // start at bottom

    while (i > 0) {
      if (dataCopy1[i-1][FIRST_COLUMN-1] === value_to_check) {

        ss.deleteRow(i);
        deleted_rows++;
      }
      i--;
    }
  }

  ss.toast("Done! " + deleted_rows + ' rows removed', "", 5);
}