質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Google スプレッドシート

Google スプレッドシートは、フリーで利用できる表計算ソフト。Webアプリのためインターネットに接続することで利用できます。チャートやグラフの作成のほか、シートを他のユーザーと共有したり、同時に作業を進めることも可能です。

Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

Q&A

解決済

1回答

1825閲覧

GASでセル内の削除をしたい

Mitsui0408

総合スコア1

Google スプレッドシート

Google スプレッドシートは、フリーで利用できる表計算ソフト。Webアプリのためインターネットに接続することで利用できます。チャートやグラフの作成のほか、シートを他のユーザーと共有したり、同時に作業を進めることも可能です。

Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

0グッド

0クリップ

投稿2021/07/08 04:42

前提・実現したいこと

スプレッドシートで在庫管理をしています。
在庫数が規定数を下回った場合にchatworkに通知が行くようになっています。
参考にしたサイト(https://sterfield.co.jp/programmer/google%E3%82%B9%E3%83%97%E3%83%AC%E3%83%83%E3%83%89%E3%82%B7%E3%83%BC%E3%83%88%E3%81%AE%E6%9B%B4%E6%96%B0%E3%82%92chatwork%E3%81%B8%E9%80%9A%E7%9F%A5%E3%81%97%E3%81%A6%E3%81%BF%E3%81%9F/)
初心者なのでほとんど丸写しです。

これに加えて在庫を補充し、規定数を上回った場合に送信判別をしているセルの状態を空白にしたいです。

現在のソースコード

rb

1function onUpdateBatchCheck() { 2 // スプレッドシートを特定 3 var spreadsheet = SpreadsheetApp.openById(SHEET_ID); 4 var sheet = spreadsheet.getSheetByName(SHEET_NAME); 5 6 // getvaluesの実行速度が遅いのでシート内容を一括で取得 7 // A1行目が[0][0]になることに注意する 8 var data = sheet.getDataRange().getValues(); 9 10 // チャット送信状態を保持する列のインデックス 11 var stateColumnIndex = 3;// D列でチャット送信状態を保持する 12 var targetColumnIndex = 4;// E列の更新を監視する 13 var targetRowIndexes = []; // チャット通知対象の行リスト 14 15 // チャット通知対象の行リストを生成する 16 // ヘッダが0行目なので読み飛ばす 17 for (var i = 1; i < data.length; i++) { 18 // チャット未送信状態 かつ E列に何らかの入力がある 19 if ((!data[i][stateColumnIndex]) && (data[i][targetColumnIndex])) { 20 targetRowIndexes.push(i); 21 } 22 } 23 24 // 通知対象行が0なら離脱 25 if (targetRowIndexes.length == 0) return; 26 27 // シートの行ごとにメッセージブロックを生成する 28 var msg= ''; 29 for (var i = 0; i < targetRowIndexes.length; i++) { 30 var row = targetRowIndexes[i];// シート上はi+1行目 31 var cell = data[row][targetColumnIndex];// i+1行目E列のセルの値 32 msg += Utilities.formatString('[info]\n内容:%s[/info]', cell);//%d行目が更新されました , row + 1 33 sheet.getRange(row + 1, stateColumnIndex + 1).setValue(true); 34 } 35 36 var message = Utilities.formatString('%sが更新されました\n%s', SHEET_NAME, msg); 37 38 // ChatWorkへメッセージを送信 39 postChatwork(message); 40 41}

試したこと

rb

1 for (var i = 0; i < targetRowIndexes.length; i++) { 2 // チャット送信状態 かつ E列に入力がなし 3 if ((data[i][stateColumnIndex]) && (!data[i][targetColumnIndex])) { 4 targetRowIndexes.push(i); 5 sheet.getRange(row + 1, stateColumnIndex + 1).clearContent(i); 6 } 7 }

初心者ができないなりに考えてみたのですが
The starting row of the range is too small.
と、エラーが出てしまいお教えいただければと思います。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

sheet.getRange(row + 1, stateColumnIndex + 1).setValue(true);

これはできているんですよね?
それなら同じループの中での

sheet.getRange(row + 1, stateColumnIndex + 1).clearContent(i);

もできそうに思いますが、ただ、clearContentには引数は不要です。引数としてiを渡してしまっているのが原因かもしれません。それにしては腑に落ちないエラーメッセージではありますが。とにかくclearContent(i)は誤りだと思いますのでclearContent()に直して実行してみてください。

投稿2021/07/08 10:51

itagagaki

総合スコア8402

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Mitsui0408

2021/07/08 23:54

ありがとうございます。 エラーはスプレッドシートの更新がない状態でデバッグしていたため取る情報がないエラーではないかと思います。 iを抜いてスプレッドシートの更新をすると思い通りに動きました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問