Функция скрытия строки при одновременном соблюдении двух условий. Где ошибка?

Question

[Ty39]

Нужна функция скрытия строки при одновременном соблюдении двух условий. Вот что я накалякал.

function onEdit(e){
  if ((e.range.columnStart != 10 || e.value != "выполнено") && (e.range.columnStart != 13 || e.value != "312")) return;
  SpreadsheetApp.getActiveSheet().hideRows(e.range.rowStart);
};

В моем случае скрытие происходит при соблюдении только одного условия, будто работает "ИЛИ" а не "И". Где ошибка?

Comments

[Вадим]

С ходу одним глазом: ошибка в скобках.
Заключите условия в общие скобки начиная после if и заканчивая перед return

[Сергей Сергей]

Нужно человеческое словесное описание условий. Код работает, и, значит, для машины безошибочен.

[Ty39]

Вадим, спасибо подправил, но ошибка осталась

Answer 1

[Semantic.Moscow]

упростите и проверьте

let a = 9;
let b = 10;
if ((a  != 10 || b != 9) && (a  != 10 || b != 9)) alert(true);
alert(false);

Comments

[Ty39]

Мое почтение

[Semantic.Moscow]

Ty39, я винишко пю и фигню полнейшую написал.

но, надеюсь, что смысл вы поняли :D

Answer 2

[cssfish]

Ну так при выполнении одного условия "И" не выполняется, return не происходит. вот и все.

Answer 3

[Григорий Боев]

(Колонка не 10) <b>ИЛИ</b> (значение не "выполнено")
при этом
(колонка не 13)<b> ИЛИ </b>(значение не "312")

Странное условие. Сгруппируйте по-другому, это должно помочь.
Я часто пользуюсь такой универсальной конструкцией:

// Файл config.gs
const config = {};

config.onEditTrigger = {
  isAllowSheetName: sheetName=>(["1", "2", "3"].includes(sheetName)),  // Лист - "1", "2" или "3"
  isAllowColumn: column=>([11,12,13].includes(column)), // колонка - 11,12 или 13
  isAllowRow: row=>(row>1), // строка - больше 1
  isAllowNewValue: value=>(value==="OK"), // новое значение - "ОК"
  isAllowOldValue: value=>true, // старое значение - любое
};

// Файл onEdit.gs
function onEditTrigger(event) { // переименовать onEditTrigger в onEdit, если хватит simple триггера. Если нужен полноценный триггер - установить триггер на редактирование на эу функцию
  let conf = config.onEditTrigger;
  let ss = SpreadsheetApp.getActiveSpreadsheet();

  let sourceSheet = event.source.getActiveSheet();
  let row = event.range.getRow();
  let col = event.range.getColumn();
  let newValue = event.value;
  let oldValue = event.oldValue;
  
  if(
     conf.isAllowColumn(col) &&
     conf.isAllowRow(row) && 
     conf.isAllowNewValue(newValue) && 
     conf.isAllowOldValue(oldValue) && 
     conf.isAllowSheetName(sourceSheet.getName())
    ){
      // что-то делаем
  };
};

Меняются условия работы в конфиге - для строки/столбца/имени листа/старого значения/нового значения.