Разрыв, если значение из диапазона на листе 1 найдено в диапазоне на листе 2, не работает

Question

В основном пользователь будет вводить данные о новом элементе на листе NovoItem. При нажатии кнопки «Сохранить» (которая будет добавлена) код должен проверять на листе 2 (ArquivoItens), существует ли уже элемент. Если так, то оно должно сломаться. Код не нарушается, если на листе 2 (ArquivoItens) найдено значение:

function copyrange() { 
  var sourceSheet = 'Novo Item';
  var destinationSheet = 'ArquivoItens';
  var ss = SpreadsheetApp.getActiveSpreadsheet(); 
  var sheet = ss.getSheetByName(sourceSheet);
  var ActiveUser = Session.getActiveUser();

  //it add current user and a timestamp to the last 2 columns on sheet1.
  var val = sheet.getRange("Q12:Q")
      .getValues();
  for (var i = 0; i < val.length; i++) {
      if (val[i] > 0) {
        sheet.getRange(12, 34, i + 1, 1)
               .setValue(new Date());
       sheet.getRange(12, 35, i + 1, 1)
               .setValue(ActiveUser)
      }
    }

  var LastRowSource = sheet.getLastRow();
  var LastColumnSource = sheet.getLastColumn();
  var values = sheet.getRange(11,1,LastRowSource,LastColumnSource).getValues();
  var csh = ss.getSheetByName(destinationSheet);
  var data = []; 

  for (var i = 1; i < values.length; i++) {    
    if ( values[i][0] != '') { 
      data.push(values[i]);      
      //sheet.deleteRow(i+1) 
    } 
  }

  var dataNovoItem = sheet.getRange("B12:B").getValues(); // gets data (item number) in the sheet where data is input
  var dataArquivoItens = csh.getRange("B2:B").getValues(); //gets the item number on the datawarehouse sheet 

  for(var n=1; n < dataNovoItem.length ; n++){
    for(var j=1; j< dataArquivoItens.length ; j++){
      if (dataNovoItem[n] != 0 && dataArquivoItens[j] != 0) {
        if(dataNovoItem[n] == dataArquivoItens[j]) {
        break;
        }
      }
  }
    }

  Logger.log("Novo Item" + dataNovoItem);
  Logger.log("ArquivoItens" + dataArquivoItens);

  //Copy data array to destination sheet  
 csh.getRange(csh.getLastRow()+1,1,data.length,data[0].length).setValues(data);

}

Вот журнал, который я получаю:

Любая помощь appreiated.

Answer 1

Вы не используете итератор для доступа к индексу, по которому вы перебираете.

Это должно быть

  for(var n=1; n < dataNovoItem.length ; n++){
    for(var j=1; j< dataArquivoItens.length ; j++){
      if (dataNovoItem[n] != 0 && dataArquivoItens[j] != 0) { // this is where I try to eliminate blank rows in both ranges. I've tried != '', but it gets me the same result on the print below.
        if(dataNovoItem[n] == dataArquivoItens[j]) {
        break; //not working
        }
      }
  }
}
  Logger.log("Novo Item" + dataNovoItem);
  Logger.log("ArquivoItens" + dataArquivoItens);

Answer 2

Я решил эту проблему, перебирая только существующий список и сравнивая каждую строку со значением, которое я установил на текущем листе. Это, вероятно, слишком логично c для ваших программистов, но для начинающего, как я, это может быть болезненно.

Вот как сейчас работает:

var newItemID = sheet.getRange("W5").getValue(); //get the destination cell, where the item will be set
  var itemIDArquivoItens = csh.getRange(2, 1, csh.getLastRow(),1).getValues(); //gets the data range where the existing ID's to be compared against are

  var message = "Item já cadastrado!"; //Message to be displayed in case the ID already exists in the list;
  for(j = 1; j < itemIDArquivoItens.length; j++) { //loops through existing ID
    if(itemIDArquivoItens[j][0] == newItemID) { //if the ID already exists, pop up a message and stop the code from running
      Browser.msgBox(message)
      return;
    }
  }