2つのGASを1つにまとめたい

Question

実現したいこと

2つのGASを組み合わせたい
1つ目のコピペ作業が完了したら、2つ目で値をクリアにしたい

1つ目
https://teratail.com/questions/00pywgdd75r13y
上記の作業

2つ目

繰越残更新用シートのI,K,M,O,Q,AC列に”８”があればそのセルをクリアしたい
・”８”は完全一致のみ反応したい
・繰越残更新用シートはこれからどんどん下に増えていく
・I,K,M,O,Q,AC列はINDIRECT関数で別シートから表示されている

発生している問題・分からないこと

2つのGASの組み合わせ方がわからないです

該当のソースコード

function get1() {
  // 履歴データの基準となるセル
  const startCellName = 'B9';

  // 品名（ダンボール）のセル
  const itemCellName = 'A4';

  // 品名Noの行
  const itemNumberRow = 4;

  // 履歴データの列数
  const numColumns = 5;

  // 送信履歴データの記入開始列
  const logStartColumn = 4;

  // それぞれのシートを取得
  const itemSheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName('ﾀﾞﾝﾎﾞｰﾙ');
  const logSheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName('送信履歴');

  // ダンボールシートの基準セルの行番号と列番号とダンボールNo
  const startCell = itemSheet.getRange(startCellName);
  const itemStartRow = startCell.getRow(); //行
  const itemStartColumn = startCell.getColumn();//列

  // 履歴見出しを作成
  const date = Utilities.formatDate(new Date(), 'JST', 'yyyy/MM/dd');
  const item = itemSheet.getRange(itemCellName).getValue();
  const itemNumber = startCell.offset(itemNumberRow - itemStartRow, 0).getValue();
  const titles = [date, item, itemNumber];

  // ダンボールシートの基準セルからデータが連続する最下行までの行数
  const itemLastRow = startCell.getNextDataCell(SpreadsheetApp.Direction.DOWN).getRow();
  const numRows = itemLastRow - itemStartRow + 1;

  // 送信履歴シートの最終行
  const logLastRow = logSheet.getLastRow();

  // 履歴データを取得
  const values = itemSheet
    .getRange(itemStartRow, itemStartColumn, numRows, numColumns)
    .getValues();

  // 履歴見出しを記入
  logSheet
    .getRange(logLastRow + 1, 1, 1, titles.length)
    .setValues([titles]);

  // 履歴データを記入
  logSheet
    .getRange(logLastRow + 1, logStartColumn, values.length, values[0].length)
    .setValues(values);

  // 結果をログに表示
  Logger.log(values);
}

function RIREKI8() {
  // 初期設定
  var rowData = 3; // データの開始行
  var columnH = 8; // 複数の列の開始列(H列)
  var columnP = 17; // 複数の列の終了列(Q列)
  var columnAR =29; // 単独の列(AC列)
  var clearValue = '8'; // クリア対象の値
  
  var spreadSheet = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = spreadSheet.getSheetByName('繰越残更新用');
  var values = sheet.getDataRange().getValues();
  
  // 行、列の値を確認しながら該当するセルをクリア
  // ※ 配列valuesの要素は行(row)、列(col)ともシートの行番号、列番号より 1 小さい
  for (var row = rowData - 1; row < values.length; row++ ) {
    // 複数の列の範囲の処理
    for (var col = columnH - 1; col <= columnP - 1; col++) {
      if (values[row][col] == clearValue) {
        sheet.getRange(row + 1, col + 1).setValue('');
      }
    }
    // 単独の列の範囲の処理
    if (values[row][columnAR - 1] == clearValue) {
      sheet.getRange(row + 1, columnAR).setValue('');
    }
  }
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

初心者すぎてどこをどう変更したらいいのかわからなかった

補足

特になし

Comments

[YellowGreen]

2つ目の関数がご希望どおりの動作をしているのであれば、
新たに

// ほかの関数を実行する関数
function exec01() {
// 最初に実行したい関数
get1();
// 次に実行したい関数
RIREKI8();
}

のような関数をつくって
この関数(exec01)を実行すれば、
２つの関数が記載された順に実行されます

[aneig]

できました泣
また教えていただいてありがとうございます

RIREKI8の処理時間が8498msなのですが、これ以上時間短縮するのは難しいですかね、、？

Answer 1

早くなっているかどうかわかりませんがお試しください

function rireki8() {
  // 初期設定
console.time('データの取得');
  const rowData = 3; // データの開始行
  const colStart = 8; // 複数の列の開始列(H列)
  const colEnd = 17; // 複数の列の終了列(Q列)
  const colAdd = 29; // 単独の列(AC列)
  const clearValue = '8'; // クリア対象の値
  const colNames = ['A', 'B', 'C', 'D', 'E', 'F', 'G',
    'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q'];
  const colNameAdd = 'AC';

  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheet = ss.getSheetByName('繰越残更新用');
  // const values = sheet.getDataRange().getValues();
  const lastRow = sheet.getLastRow();
  const lastCol = sheet.getLastColumn();
  const values = sheet.getRange(1, 1, lastRow, lastCol).getValues();
console.timeEnd('データの取得');

  // 指定の列の中で値が8となっているセルを全て空白にする
  // ※ 配列valuesの要素は行(row)、列(col)ともシートの行番号、列番号より 1 小さい
console.time('データの消去');
  const rangeNames = [];
  for (let row = rowData - 1; row < values.length; row++) {
    // 複数の列の範囲の処理
    for (let col = colStart - 1; col <= colEnd - 1; col++) {
      if (values[row][col] == clearValue) {
        rangeNames.push(colNames[col] + (row + 1))
      }
    }
    // 単独の列の範囲の処理
    if (values[row][colAdd - 1] == clearValue) {
      rangeNames.push(colNameAdd + (row + 1))
    }
  }
  const rangeList = sheet.getRangeList(rangeNames);
  rangeList.setValue('');
console.timeEnd('データの消去');
}

Comments

[aneig]

400msくらいはやくなりました！！ありがとうございます、、泣

[YellowGreen]

どの部分の処理に時間を要しているのかを
表示できるようにしました
加えて、
getDataRangeをgetRangeに変えてみてもあります

10,000行程度のテストデータで試した結果です
----------
18:47:05 お知らせ 実行開始
18:47:10 デバッグ データの取得: 5024ms
18:47:11 デバッグ データの消去: 1236ms
18:47:12 お知らせ 実行完了

[aneig]

はやくなりました！！
最後まで教えていただき本当にありがとうございました。
勉強になりました