Несколько сценариев onEdit, но ошибка в строке 2 - PullRequest
0 голосов
/ 07 сентября 2018

Я не могу понять, что я делаю неправильно. Я все еще новичок в этом, поэтому есть много мест для ошибок. Я пытаюсь запустить две версии одного и того же скрипта. Когда флажок установлен, если он находится в столбце F, я хочу, чтобы функция 7R выполнялась, но если флажок установлен в столбце K, я хочу, чтобы функция 8R выполнялась. Я использовал потоки с похожими вопросами, чтобы попытаться объединить два сценария, но получаю сообщение об ошибке в строке 2.

Ошибка, которую я получаю, отсутствует; до выписки: строка 2

Вот что у меня есть:

function onEdit(e) {
8R();
7R();
}

function 8R() {
//Get the sheet you want to work with. 
var editrange = {
  top : 2,
  bottom : 260,
  left : 11,
  right : 11};
//getRow() and not getrow()
var thisrow = e.range.getRow();
if (thisrow < editrange.top || thisrow > editrange.bottom) return;
//getColumn() and not getcolumn()
var thiscolumn = e.range.getColumn();
if (thiscolumn < editrange.left || thiscolumn > editrange.right) return;
//Line that replaces the erroneous 'var ss = e.range.getSheet()';
var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getSheetByName("Responsible");
//Grab the entire Range, and grab whatever values you need from it. EX: 
rangevalues
var range8 = sheet.getRange("K3:K90");
var range28 = sheet.getRange("M3:M90");
var range2values8 = range28.getValues();
var rangevalues8 = range8.getValues();
//Loops through range results
for (var i in rangevalues8) {
  Logger.log("rangevalues8["+i+"]["+0+"] is:"+rangevalues8[i][0]);//Added
  //Set the rules logic
  if (rangevalues8[i][0] == true) { //Modified
  //Set the cell 
  range2values8[i][0] += 1; //Directly add 1 to range2values
  Logger.log(range2values8);//Added
  }
}
//copy new information
var destination = ss.getSheetByName('Compiled Data');//whatever page
var destCell8 = destination.getRange("I3:I90");
destCell8.setValues(range2values8);
//clear checkboxes
var cleaning = ss.getSheetByName('Asset Bank');
var cleaningcell8 = cleaning.getRange("A3:A90").getValues();
range8.setValues(cleaningcell8);
}

function 7R() {
//Get the sheet you want to work with. 
var editrange = {
  top : 2,
  bottom : 260,
  left : 6,
  right : 6};
//getRow() and not getrow()
var thisrow7 = e.range.getRow();
if (thisrow7 < editrange.top || thisrow7 > editrange.bottom) return;
//getColumn() and not getcolumn()
var thiscolumn7 = e.range.getColumn();
if (thiscolumn7 < editrange.left || thiscolumn7 > editrange.right) return;
//Line that replaces the erroneous 'var ss = e.range.getSheet()';
var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getSheetByName("Responsible");
//Grab the entire Range, and grab whatever values you need from it. EX: 
rangevalues
var range7 = sheet.getRange("F3:F90");
var range27 = sheet.getRange("H3:H90");
var range2values7 = range27.getValues();
var rangevalues7 = range7.getValues();
//Loops through range results
for (var i in rangevalues7) {
  Logger.log("rangevalues7["+i+"]["+0+"] is:"+rangevalues7[i][0]);//Added
  //Set the rules logic
  if (rangevalues7[i][0] == true) { //Modified
    //Set the cell 
    range2values7[i][0] += 1; //Directly add 1 to range2values
    Logger.log(range2values7);//Added
  }
}
//copy new information
var destination = ss.getSheetByName('Compiled Data');//whatever page
var destCell7 = destination.getRange("I93:I180");
destCell7.setValues(range2values7);
//clear checkboxes
var cleaning = ss.getSheetByName('Asset Bank');
var cleaningcell7 = cleaning.getRange("A3:A90").getValues();
range7.setValues(cleaningcell7);
}

Вот ссылка на лист реплик того, над чем я работаю, со всей соответствующей информацией: https://docs.google.com/spreadsheets/d/1PWaWm7AryljOMd5Aq1O2RSADyRFNVZbUDqKO__SzW2w/edit?usp=sharing

Ответы [ 2 ]

0 голосов
/ 11 сентября 2018

Наконец-то удалось заставить скрипт работать. Сначала я сменил имена, как предложил Энрике. Затем разобрались с блоком onEdit. Спасибо всем за помощь и советы. Я бы все еще злился на свою машину без вас всех.

function onEdit(e) {
  //don't need entire range, just the column that was modified to figure out which function to call
  var editColumn = e.range.getColumn();

  //confirm edit was a box being checked before running the code (as this seems to run on ANY sheet edit)
  if (e.oldValue === "false" && e.value === "TRUE") {
    if (editColumn === 6)
      func7R(e);
    else if (editColumn === 11)
      func8R(e);
  }
}

function func8R(e) {
  //Get the sheet you want to work with. 
  var editRange = {
    top : 2,
    bottom : 260,
    left : 11,
    right : 11};
  //getRow() and not getrow()
  var thisRow = e.range.getRow();
  if (thisRow < editRange.top || thisRow > editRange.bottom) return;
  //getColumn() and not getcolumn()
  var thisColumn = e.range.getColumn();
  if (thisColumn < editRange.left || thisColumn > editRange.right) return;
  //Line that replaces the erroneous 'var ss = e.range.getSheet()';
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getSheetByName("Responsible");
  //Grab the entire Range, and grab whatever values you need from it. EX: rangevalues
  var range8 = sheet.getRange("K3:K90");
  var range28 = sheet.getRange("M3:M90");
  var range2values8 = range28.getValues();
  var rangevalues8 = range8.getValues();
  //Loops through range results
  for (var i in rangevalues8) {
    Logger.log("rangevalues8["+i+"]["+0+"] is:"+rangevalues8[i][0]);//Added
    //Set the rules logic
    if (rangevalues8[i][0] == true) { //Modified
      //Set the cell 
      range2values8[i][0] += 1; //Directly add 1 to range2values
      Logger.log(range2values8);//Added
    }
  }
  //copy new information
  var destination = ss.getSheetByName('Compiled Data');//whatever page
  var destCell8 = destination.getRange("I3:I90");
  destCell8.setValues(range2values8);
  //clear checkboxes
  var cleaning = ss.getSheetByName('Asset Bank');
  var cleaningcell8 = cleaning.getRange("A3:A90").getValues();
  range8.setValues(cleaningcell8);
}

function func7R(e) {
  //Get the sheet you want to work with. 
  var editRange = {
    top : 2,
    bottom : 260,
    left : 6,
    right : 6};
  //getRow() and not getrow()
  var thisrow7 = e.range.getRow();
  if (thisrow7 < editRange.top || thisrow7 > editRange.bottom) return;
  //getColumn() and not getcolumn()
  var thiscolumn7 = e.range.getColumn();
  if (thiscolumn7 < editRange.left || thiscolumn7 > editRange.right) return;
  //Line that replaces the erroneous 'var ss = e.range.getSheet()';
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getSheetByName("Responsible");
  //Grab the entire Range, and grab whatever values you need from it. EX: rangevalues
  var range7 = sheet.getRange("F3:F90");
  var range27 = sheet.getRange("H3:H90");
  var range2values7 = range27.getValues();
  var rangevalues7 = range7.getValues();
  //Loops through range results
  for (var i in rangevalues7) {
    Logger.log("rangevalues7["+i+"]["+0+"] is:"+rangevalues7[i][0]);//Added
    //Set the rules logic
    if (rangevalues7[i][0] == true) { //Modified
      //Set the cell 
      range2values7[i][0] += 1; //Directly add 1 to range2values
      Logger.log(range2values7);//Added
    }
  }
  //copy new information
  var destination = ss.getSheetByName('Compiled Data');//whatever page
  var destCell7 = destination.getRange("I93:I180");
  destCell7.setValues(range2values7);
  //clear checkboxes
  var cleaning = ss.getSheetByName('Asset Bank');
  var cleaningcell7 = cleaning.getRange("A3:A90").getValues();
  range7.setValues(cleaningcell7);
}
0 голосов
/ 07 сентября 2018

Имена переменных и функций JavaScript не могут начинаться с цифр. Замените 8R и 7R действительным идентификатором , например func8R и func7R.

Кроме того, функции 8R и 7R пытаются сразу получить доступ к переменной e.range, но она нигде не определена. Я предполагаю, что вы пытаетесь прочитать параметр события onEdit, но вы должны передать его в свои функции, а также определить параметр их, например,

function onEdit(e) {
  f8R(e);
  f7R(e);
}

function f8R(e) {
  //... continue (remember to also declare on f7R)

Если вы хотите вручную протестировать эту функцию, вы должны заполнить этот параметр e, как это будет происходить в среде, когда вы действительно что-то изменяете в электронной таблице. Мне нравится делать это, написав другую функцию "вызывающего", например:

function testOnEdit() {
  onEdit({range: SpreadsheetApp.getActiveSheet().getRange('F2')});
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...