GASでチェックボックスから該当の行の値を削除する

Question

実現したいこと

チェックボックスでTrueにした行から100行下のデータを行ごと削除する
例)A3のチェックボックスを押すと、103行目の値を削除する。

現在の状態

今は3行目のみ実行できるよう無理やり実装したのですが、
最終的にはチェックボックスの実装を複数行したいので、この内容をもっと効率良く書ける方法があれば知りたいです。

スプレッドシートの状態としては、

	A	B
3	☑️	りんご(=’F103’)
4		みかん(=’F104’)

...

	E	F	G	H
103		りんご	100
104		みかん	200

A3にチェックボックス
B3にデータの名前を参照したもの(='F7')
100行下の103行目にデータがあります。
3行目のチェックボックスを押すと103行目の値が行単位で削除されます。

現在のコード

function onEdit(e) { 
    const inspection_seet = ['Main'];
    const canged_seet = e.source.getSheetName();
    const changed_cell = e.range.getA1Notation();
 
    const cellA3 = ['A3'];

    const ss = SpreadsheetApp.getActiveSpreadsheet();
    const sheet = ss.getActiveSheet();
    const rangeA3 = sheet.getRange('A3');
  
    const checkvalA3 = rangeA3.getValues();
  
    for (var i = 0; i < inspection_cell.length; i++) {
      if (canged_seet == inspection_seet[i]) {
        if (changed_cell == cellA3[i]) {
          if (checkvalA3[i][0] === true) {
            rangeA3.setValue('FALSE')
            Browser.msgBox('A3を削除しました',);
            sheet.getRange('Main!A103:CV103').clearContent();
          }
        }
      }
    }
  }

試したこと

ネットで色々調べて書いてみたのですが初心者なもので...
deleteOneを実行するまではできたのですが、その中のif文がうまく動かないです。

function onEdit(e) { //リセットボタン
    const inspection_seet = ['Main'];//対象のシートを指定
    const canged_seet = e.source.getSheetName();
    const changed_cell = e.range.getA1Notation();

    var colIdx = 1;
    var col = e.range.getColumn();

    // スプレッドシートを取得
    const ss = SpreadsheetApp.getActiveSpreadsheet();
    // シートを取得
    const sheet = ss.getActiveSheet();

    // 値を取得
    const checkval = range.getValues();
    const checkvalA3 = rangeA3.getValues();

    for (var i = 0; i < inspection_cell.length; i++) {
        if (canged_seet == inspection_seet[i]) {//編集されたシートが指定シートだったら実行
            if (col === colIdx) { //A列が編集されたら実行
                deleteOne();
            }
        }
    }
}
function deleteOne() {
    const ss = SpreadsheetApp.getActiveSpreadsheet();
    const sheet = ss.getActiveSheet();
    const range = sheet.getRange('A3:B36');
    const values = range.getValues();

    const cell = ['A3:A13']; //この範囲にあるセル1つが変更されたら働くようにしたいが書き方がわからないです。
    const changed_cell = e.range.getA1Notation();

    var deleCell = changed_cell.offset(100, 0).getRange();
    const deleRow = deleCell.getRow();

    if (changed_cell == cell[i]) {
        if (values[i][0] === true) {
            sheet.getRange(deleRow).clearContent();
            values.setValue('FALSE')
            Browser.msgBox('実行されました');
        }
    }
}

ご教授いただけると幸いです。

Comments

[k.a_teratail]

> for (var i = 0; i < inspection_cell.length; i++) {
ここ、間違ってないですか？
inspection_cell ではなく inspection_seet ではないでしょうか？
inspection_cell は定義されてないので、forループが行われないのではないでしょうか？

[k.a_teratail]

> 試したこと
こちらのコードでは、エラーが発生しています。

エラー
ReferenceError: range is not defined
at onEdit(on_edit:15:22)

エラーを解消してからコード掲載をお願いします。

[Aradbranding]

https://aradbranding.com/fa/

Answer 1

deleteOneを実行するまではできたのですが、その中のif文がうまく動かないです。
const changed_cell = e.range.getA1Notation();

deleteOneメソッド内に変数e の記述がないので、エラーになっていましたので
if文まで到達せずでした。

なので、
deleteOneメソッドで eの変数が必要なので引数として追加する必要があります。

この範囲にあるセル1つが変更されたら働くようにしたい

変更されたセル範囲を「getA1Notation」で取得しているので、文字列として、「A3~A13」の比較を行う必要があります。

下記は参考コードです。

function onEdit(e) { //リセットボタン
    const inspection_seet = ['Main'];//対象のシートを指定
    const canged_seet = e.source.getSheetName();

    var colIdx = 1;
    var col = e.range.getColumn();

    for (var i = 0; i < inspection_seet.length; i++) {
        if (canged_seet == inspection_seet[i]) {//編集されたシートが指定シートだったら実行
            if (col === colIdx) { //A列が編集されたら実行
                deleteOne(e);
            }
        }
    }
}
function deleteOne(e) {
    const ss = SpreadsheetApp.getActiveSpreadsheet();
    const sheet = ss.getActiveSheet();

    // A3~A13の文字列配列を作成
    const cell = [...Array(11)].map((v, index) => `A${index + 3}` ); 
    const changed_cell = e.range.getA1Notation();

    // 変更されたセルから100行下の行数
    var deleCell = sheet.getRange(changed_cell).offset(100, 0).getRow()

    if (cell.indexOf(changed_cell) != -1) { // A3~A13セルが変更されたか
        if (sheet.getRange(changed_cell).getValue()) { // A3~13セルの値がTrueか
            sheet.getRange(`A${deleCell}:ZZ${deleCell}`).clearContent(); // A~ZZセルの内容をクリア
            sheet.getRange(changed_cell).setValue('FALSE') // Aセルのチェックを外す
            Browser.msgBox('実行されました');
        }
    }
}

Comments

[ss2020]

そちらのコードで解決できました。
解説もして頂きありがとうございます。勉強になります。
A3~A13の文字列配列を作成するコードの書き方はまだ理解できていない部分なので勉強してきます。

親切にご回答頂き大変助かりました。ありがとうございました。

[k.a_teratail]

> A3~A13の文字列配列を作成するコードの書き方

1行で書けそうだったので、1行で書けるコードにしました。
やっていることは、下記と同じになります。
let a = []
for (let i = 0; i < 11; i++) {
a.push(`A${i + 3}`)
}
console.log(a) // [ 'A3', 'A4', 'A5', 'A6', 'A7', 'A8', 'A9', 'A10', 'A11', 'A12', 'A13' ]